From: Thierry Reding <thierry.reding@gmail.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>,
linux-mm@kvack.org, Matt Mackall <mpm@selenic.com>,
Cliff Wickman <cpw@sgi.com>,
KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>,
Johannes Weiner <hannes@cmpxchg.org>,
KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
Michal Hocko <mhocko@suse.cz>,
"Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com>,
Pavel Emelyanov <xemul@parallels.com>,
linux-kernel@vger.kernel.org, Mark Brown <broonie@kernel.org>
Subject: Re: [PATCH 0/11] update page table walker
Date: Wed, 16 Oct 2013 11:33:09 +0200 [thread overview]
Message-ID: <20131016093308.GD21963@ulmo.nvidia.com> (raw)
In-Reply-To: <20131015134317.02d819f6905f790007ba1842@linux-foundation.org>
[-- Attachment #1: Type: text/plain, Size: 2164 bytes --]
On Tue, Oct 15, 2013 at 01:43:17PM -0700, Andrew Morton wrote:
> On Mon, 14 Oct 2013 13:36:59 -0400 Naoya Horiguchi <n-horiguchi@ah.jp.nec.com> wrote:
>
> > Page table walker is widely used when you want to traverse page table
> > tree and do some work for the entries (and pages pointed to by them.)
> > This is a common operation, and keep the code clean and maintainable
> > is important. Moreover this patchset introduces caller-specific walk
> > control function which is helpful for us to newly introduce page table
> > walker to some other users. Core change comes from patch 1, so please
> > see it for how it's supposed to work.
> >
> > This patchset changes core code in mm/pagewalk.c at first in patch 1 and 2,
> > and then updates all of current users to make the code cleaner in patch
> > 3-9. Patch 10 changes the interface of hugetlb_entry(), I put it here to
> > keep bisectability of the whole patchset. Patch 11 applies page table walker
> > to a new user queue_pages_range().
>
> Unfortunately this is very incompatible with pending changes in
> fs/proc/task_mmu.c. Especially Kirill's "mm, thp: change
> pmd_trans_huge_lock() to return taken lock".
>
> Stephen will be away for a couple more weeks so I'll get an mmotm
> released and hopefully Thierry and Mark will scoop it up(?).
> Alternatively, http://git.cmpxchg.org/?p=linux-mmots.git;a=summary is
> up to date.
>
> Please take a look, decide what you think we should do?
Hi Andrew,
I haven't had the time to look at writing up the scripts to import the
mmotm into the linux-next trees that I create. From what I understand,
it might be unwise to just pull linux-mmots into linux-next because it
isn't very well tested. Then again, increasing test coverage is one of
the goals of linux-next. If you think it's safe to include linux-mmots
in linux-next I can easily do that.
Otherwise I'll see if I can resume work on the scripts I started to
import the mmotm. Stephen has also provided the scripts that he used,
but I haven't had much time to look at them in detail yet because of
other things that have been keeping me busy.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
prev parent reply other threads:[~2013-10-16 9:35 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-14 17:36 [PATCH 0/11] update page table walker Naoya Horiguchi
2013-10-14 17:36 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 01/11] pagewalk: update page table walker core Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 02/11] pagewalk: add walk_page_vma() Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 03/11] smaps: redefine callback functions for page table walker Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 04/11] clear_refs: " Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 05/11] pagemap: " Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 06/11] numa_maps: " Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 07/11] memcg: " Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 08/11] madvise: " Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 09/11] arch/powerpc/mm/subpage-prot.c: use walk_page_vma() instead of walk_page_range() Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 10/11] pagewalk: remove argument hmask from hugetlb_entry() Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-14 17:37 ` [PATCH 11/11] mempolicy: apply page table walker on queue_pages_range() Naoya Horiguchi
2013-10-14 17:37 ` Naoya Horiguchi
2013-10-15 20:43 ` [PATCH 0/11] update page table walker Andrew Morton
2013-10-15 20:43 ` Andrew Morton
2013-10-15 21:03 ` Naoya Horiguchi
2013-10-15 21:03 ` Naoya Horiguchi
2013-10-16 9:33 ` Thierry Reding [this message]
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=20131016093308.GD21963@ulmo.nvidia.com \
--to=thierry.reding@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=aneesh.kumar@linux.vnet.ibm.com \
--cc=broonie@kernel.org \
--cc=cpw@sgi.com \
--cc=hannes@cmpxchg.org \
--cc=kamezawa.hiroyu@jp.fujitsu.com \
--cc=kosaki.motohiro@jp.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=mhocko@suse.cz \
--cc=mpm@selenic.com \
--cc=n-horiguchi@ah.jp.nec.com \
--cc=xemul@parallels.com \
/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.