lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 7 Aug 2008 13:42:24 +0100 (BST) From: Hugh Dickins <hugh@...itas.com> To: Adrian Bunk <bunk@...nel.org> cc: Rik van Riel <riel@...hat.com>, linux-kernel@...r.kernel.org, Andrew Morton <akpm@...ux-foundation.org> Subject: Re: [2.6 patch] make mm/rmap.c:anon_vma_cachep static On Thu, 7 Aug 2008, Adrian Bunk wrote: > This patch makes the needlessly global anon_vma_cachep static. > > Signed-off-by: Adrian Bunk <bunk@...nel.org> > Acked-by: Rik van Riel <riel@...hat.com> Yes, it's much better your way, thanks Adrian: I guess there must have been some reason during development why those went in rmap.h instead of rmap.c, but I can't see any good reason since it hit the tree in 2.6.7. Acked-by: Hugh Dickins <hugh@...itas.com> > > --- > > This patch has been sent on: > - 5 May 2008 > - 22 Apr 2008 > - 14 Apr 2008 > - 31 Mar 2008 > - 25 Feb 2008 > > include/linux/rmap.h | 12 ------------ > mm/rmap.c | 12 +++++++++++- > 2 files changed, 11 insertions(+), 13 deletions(-) > > 8d2d63d71ecbf2db78f31025acb136c9dd4db6a2 diff --git a/include/linux/rmap.h b/include/linux/rmap.h > index 1383692..8a8dbb9 100644 > --- a/include/linux/rmap.h > +++ b/include/linux/rmap.h > @@ -31,18 +31,6 @@ struct anon_vma { > > #ifdef CONFIG_MMU > > -extern struct kmem_cache *anon_vma_cachep; > - > -static inline struct anon_vma *anon_vma_alloc(void) > -{ > - return kmem_cache_alloc(anon_vma_cachep, GFP_KERNEL); > -} > - > -static inline void anon_vma_free(struct anon_vma *anon_vma) > -{ > - kmem_cache_free(anon_vma_cachep, anon_vma); > -} > - > static inline void anon_vma_lock(struct vm_area_struct *vma) > { > struct anon_vma *anon_vma = vma->anon_vma; > diff --git a/mm/rmap.c b/mm/rmap.c > index 8fd527c..ef6bb36 100644 > --- a/mm/rmap.c > +++ b/mm/rmap.c > @@ -52,7 +52,17 @@ > > #include <asm/tlbflush.h> > > -struct kmem_cache *anon_vma_cachep; > +static struct kmem_cache *anon_vma_cachep; > + > +static inline struct anon_vma *anon_vma_alloc(void) > +{ > + return kmem_cache_alloc(anon_vma_cachep, GFP_KERNEL); > +} > + > +static inline void anon_vma_free(struct anon_vma *anon_vma) > +{ > + kmem_cache_free(anon_vma_cachep, anon_vma); > +} > > /* This must be called under the mmap_sem. */ > int anon_vma_prepare(struct vm_area_struct *vma) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists