From: Rohit Seth <rohitseth@google.com>
To: Nick Piggin <nickpiggin@yahoo.com.au>
Cc: Andrew Morton <akpm@osdl.org>,
Linux-mm@kvack.org, Linux-kernel@vger.kernel.org
Subject: Re: [PATCH]: Adding a counter in vma to indicate the number of physical pages backing it
Date: Mon, 12 Jun 2006 10:36:34 -0700 [thread overview]
Message-ID: <1150133795.9576.19.camel@galaxy.corp.google.com> (raw)
In-Reply-To: <448A762F.7000105@yahoo.com.au>
On Sat, 2006-06-10 at 17:35 +1000, Nick Piggin wrote:
> Rohit Seth wrote:
> > Below is a patch that adds number of physical pages that each vma is
> > using in a process. Exporting this information to user space
> > using /proc/<pid>/maps interface.
> >
> > There is currently /proc/<pid>/smaps that prints the detailed
> > information about the usage of physical pages but that is a very
> > expensive operation as it traverses all the PTs (for some one who is
> > just interested in getting that data for each vma).
>
> Yet more cacheline footprint in the page fault and unmap paths...
>
Not necessarily. If I'm doing calculation right then vm_struct is
currently 176 bytes (without the addition of nphys) on my x86_64 box. So
in this case addition would not result in bigger cache foot print of
page fulats. Also currently two adjacent vmas share a cache line. So
there is already that much of cache line ping pong going on.
Though I agree that we should try to not extend this size beyond
absolutely necessary.
> What is this used for and why do we want it? Could you do some
> smaps-like interface that can work on ranges of memory, and
> continue to walk pagetables instead?
>
It is just the price of those walks that makes smaps not an attractive
solution for monitoring purposes.
I'm thinking if it is possible to extend current interfaces (possibly
having a new system call) in such a way that a user land process can
give some hints/preferences to kernel in terms of <pid, virtual_range>
to remove/inactivate. This can help in keeping the current kernel
behavior for vmscans but at the same time provide little bit of
non-symmetry for user land applications. Thoughts?
-rohit
WARNING: multiple messages have this Message-ID (diff)
From: Rohit Seth <rohitseth@google.com>
To: Nick Piggin <nickpiggin@yahoo.com.au>
Cc: Andrew Morton <akpm@osdl.org>,
Linux-mm@kvack.org, Linux-kernel@vger.kernel.org
Subject: Re: [PATCH]: Adding a counter in vma to indicate the number of physical pages backing it
Date: Mon, 12 Jun 2006 10:36:34 -0700 [thread overview]
Message-ID: <1150133795.9576.19.camel@galaxy.corp.google.com> (raw)
In-Reply-To: <448A762F.7000105@yahoo.com.au>
On Sat, 2006-06-10 at 17:35 +1000, Nick Piggin wrote:
> Rohit Seth wrote:
> > Below is a patch that adds number of physical pages that each vma is
> > using in a process. Exporting this information to user space
> > using /proc/<pid>/maps interface.
> >
> > There is currently /proc/<pid>/smaps that prints the detailed
> > information about the usage of physical pages but that is a very
> > expensive operation as it traverses all the PTs (for some one who is
> > just interested in getting that data for each vma).
>
> Yet more cacheline footprint in the page fault and unmap paths...
>
Not necessarily. If I'm doing calculation right then vm_struct is
currently 176 bytes (without the addition of nphys) on my x86_64 box. So
in this case addition would not result in bigger cache foot print of
page fulats. Also currently two adjacent vmas share a cache line. So
there is already that much of cache line ping pong going on.
Though I agree that we should try to not extend this size beyond
absolutely necessary.
> What is this used for and why do we want it? Could you do some
> smaps-like interface that can work on ranges of memory, and
> continue to walk pagetables instead?
>
It is just the price of those walks that makes smaps not an attractive
solution for monitoring purposes.
I'm thinking if it is possible to extend current interfaces (possibly
having a new system call) in such a way that a user land process can
give some hints/preferences to kernel in terms of <pid, virtual_range>
to remove/inactivate. This can help in keeping the current kernel
behavior for vmscans but at the same time provide little bit of
non-symmetry for user land applications. Thoughts?
-rohit
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2006-06-12 17:36 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-10 1:33 [PATCH]: Adding a counter in vma to indicate the number of physical pages backing it Rohit Seth
2006-06-10 1:33 ` Rohit Seth
2006-06-10 2:42 ` Andrew Morton
2006-06-10 2:42 ` Andrew Morton
2006-06-12 17:49 ` Rohit Seth
2006-06-12 17:49 ` Rohit Seth
2006-06-10 7:35 ` Nick Piggin
2006-06-10 7:35 ` Nick Piggin
2006-06-11 10:15 ` Jan Engelhardt
2006-06-11 10:15 ` Jan Engelhardt
2006-06-12 17:36 ` Rohit Seth [this message]
2006-06-12 17:36 ` Rohit Seth
2006-06-12 17:58 ` Andi Kleen
2006-06-12 17:58 ` Andi Kleen
2006-06-12 19:42 ` Rohit Seth
2006-06-12 19:42 ` Rohit Seth
2006-06-13 3:51 ` Andi Kleen
2006-06-13 3:51 ` Andi Kleen
2006-06-13 4:27 ` Nick Piggin
2006-06-13 4:27 ` Nick Piggin
2006-06-13 16:59 ` Rohit Seth
2006-06-13 16:59 ` Rohit Seth
2006-06-13 17:28 ` Hugh Dickins
2006-06-13 17:28 ` Hugh Dickins
2006-06-13 18:09 ` Rohit Seth
2006-06-13 18:09 ` Rohit Seth
2006-06-13 17:31 ` Andi Kleen
2006-06-13 17:31 ` Andi Kleen
2006-06-11 16:09 ` Arjan van de Ven
2006-06-11 16:09 ` Arjan van de Ven
2006-06-12 11:17 ` Andi Kleen
2006-06-12 11:17 ` Andi Kleen
2006-06-12 12:49 ` Jan Engelhardt
2006-06-12 12:49 ` Jan Engelhardt
2006-06-12 12:54 ` Andi Kleen
2006-06-12 12:54 ` Andi Kleen
2006-06-12 16:43 ` Christoph Lameter
2006-06-12 16:43 ` Christoph Lameter
-- strict thread matches above, loose matches on Subject: below --
2006-06-13 5:53 [PATCH]: Adding a counter in vma to indicate the number of physical_pages_backing it Albert Cahalan
2006-06-13 5:53 ` Albert Cahalan
2006-06-13 5:56 ` Andi Kleen
2006-06-13 5:56 ` Andi Kleen
2006-06-13 17:10 ` Rohit Seth
2006-06-13 17:10 ` Rohit Seth
2006-06-13 17:18 ` Andi Kleen
2006-06-13 17:18 ` Andi Kleen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1150133795.9576.19.camel@galaxy.corp.google.com \
--to=rohitseth@google.com \
--cc=Linux-kernel@vger.kernel.org \
--cc=Linux-mm@kvack.org \
--cc=akpm@osdl.org \
--cc=nickpiggin@yahoo.com.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.