Re: [PATCH 2/4] mm, rmap: allocate anon_vma_chain before starting to link anon_vma_chain

2013-08-07 Thread Joonsoo Kim
On Wed, Aug 07, 2013 at 02:08:03AM -0400, Johannes Weiner wrote: > > > > list_for_each_entry_reverse(pavc, >anon_vma_chain, same_vma) { > > struct anon_vma *anon_vma; > > > > - avc = anon_vma_chain_alloc(GFP_NOWAIT | __GFP_NOWARN); > > - if (unlikely(!avc))

Re: [PATCH 2/4] mm, rmap: allocate anon_vma_chain before starting to link anon_vma_chain

2013-08-07 Thread Johannes Weiner
On Tue, Aug 06, 2013 at 05:43:38PM +0900, Joonsoo Kim wrote: > If we allocate anon_vma_chain before starting to link, we can reduce > the lock hold time. This patch implement it. > > Signed-off-by: Joonsoo Kim > > diff --git a/mm/rmap.c b/mm/rmap.c > index c2f51cb..1603f64 100644 > ---

Re: [PATCH 2/4] mm, rmap: allocate anon_vma_chain before starting to link anon_vma_chain

2013-08-07 Thread Johannes Weiner
On Tue, Aug 06, 2013 at 05:43:38PM +0900, Joonsoo Kim wrote: If we allocate anon_vma_chain before starting to link, we can reduce the lock hold time. This patch implement it. Signed-off-by: Joonsoo Kim iamjoonsoo@lge.com diff --git a/mm/rmap.c b/mm/rmap.c index c2f51cb..1603f64 100644

Re: [PATCH 2/4] mm, rmap: allocate anon_vma_chain before starting to link anon_vma_chain

2013-08-07 Thread Joonsoo Kim
On Wed, Aug 07, 2013 at 02:08:03AM -0400, Johannes Weiner wrote: list_for_each_entry_reverse(pavc, src-anon_vma_chain, same_vma) { struct anon_vma *anon_vma; - avc = anon_vma_chain_alloc(GFP_NOWAIT | __GFP_NOWARN); - if (unlikely(!avc)) { -

[PATCH 2/4] mm, rmap: allocate anon_vma_chain before starting to link anon_vma_chain

2013-08-06 Thread Joonsoo Kim
If we allocate anon_vma_chain before starting to link, we can reduce the lock hold time. This patch implement it. Signed-off-by: Joonsoo Kim diff --git a/mm/rmap.c b/mm/rmap.c index c2f51cb..1603f64 100644 --- a/mm/rmap.c +++ b/mm/rmap.c @@ -240,18 +240,21 @@ int anon_vma_clone(struct

[PATCH 2/4] mm, rmap: allocate anon_vma_chain before starting to link anon_vma_chain

2013-08-06 Thread Joonsoo Kim
If we allocate anon_vma_chain before starting to link, we can reduce the lock hold time. This patch implement it. Signed-off-by: Joonsoo Kim iamjoonsoo@lge.com diff --git a/mm/rmap.c b/mm/rmap.c index c2f51cb..1603f64 100644 --- a/mm/rmap.c +++ b/mm/rmap.c @@ -240,18 +240,21 @@ int