Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps

From	Andrew Morton <akpm@linux-foundation.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps
Date	2017-03-08 02:30 +0100
Message-ID	<tiyAp-7Eu-1@gated-at.bofh.it> (permalink)
References	(4 earlier) <tghDH-77b-9@gated-at.bofh.it> <tgzTY-3s2-11@gated-at.bofh.it> <tgBVM-4ZX-3@gated-at.bofh.it> <th5Au-t5-15@gated-at.bofh.it> <timpz-7rg-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tue, 7 Mar 2017 11:05:45 +0100 Michal Hocko <mhocko@kernel.org> wrote:

> On Fri 03-03-17 16:10:27, Andrew Morton wrote:
> > On Thu, 2 Mar 2017 17:30:54 +0100 Michal Hocko <mhocko@kernel.org> wrote:
> > 
> > > > It's not that I think you're wrong: it *is* an implementation detail.
> > > > But we take a bit of incoherency from batching all over the place, so
> > > > it's a little odd to take a stand over this particular instance of it
> > > > - whether demanding that it'd be fixed, or be documented, which would
> > > > only suggest to users that this is special when it really isn't etc.
> > > 
> > > I am not aware of other counter printed in smaps that would suffer from
> > > the same problem, but I haven't checked too deeply so I might be wrong. 
> > > 
> > > Anyway it seems that I am alone in my position so I will not insist.
> > > If we have any bug report then we can still fix it.
> > 
> > A single lru_add_drain_all() right at the top level (in smaps_show()?)
> > won't kill us
> 
> I do not think we want to put lru_add_drain_all cost to a random
> process reading /proc/<pid>/smaps.

Why not?  It's that process which is calling for the work to be done.

> If anything the one which does the
> madvise should be doing this.

But it would be silly to do extra work in madvise() if nobody will be
reading smaps for the next two months.

How much work is it anyway?  What would be the relative impact upon a
smaps read?

Thread

Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Michal Hocko <mhocko@kernel.org> - 2017-03-01 15:20 +0100
  Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Michal Hocko <mhocko@kernel.org> - 2017-03-01 19:00 +0100
  Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Michal Hocko <mhocko@kernel.org> - 2017-03-01 20:00 +0100
    Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Minchan Kim <minchan@kernel.org> - 2017-03-02 09:00 +0100
    Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Johannes Weiner <hannes@cmpxchg.org> - 2017-03-02 15:30 +0100
      Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Michal Hocko <mhocko@kernel.org> - 2017-03-02 17:40 +0100
        Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Andrew Morton <akpm@linux-foundation.org> - 2017-03-04 01:20 +0100
          Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Michal Hocko <mhocko@kernel.org> - 2017-03-07 13:30 +0100
            Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Andrew Morton <akpm@linux-foundation.org> - 2017-03-08 02:30 +0100
              Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Minchan Kim <minchan@kernel.org> - 2017-03-08 07:00 +0100
  Re: [PATCH V5 6/6] proc: show MADV_FREE pages info in smaps Johannes Weiner <hannes@cmpxchg.org> - 2017-03-01 21:30 +0100

csiph-web