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, 12 Sep 2013 14:13:48 +0000 From: "Myklebust, Trond" <Trond.Myklebust@...app.com> To: Geert Uytterhoeven <geert@...ux-m68k.org> CC: Linux/m68k <linux-m68k@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org> Subject: Re: Kernel size increase of +256 KiB (was: Re: RPCSEC_GSS: Share all credential caches on a per-transport basis) On Thu, 2013-09-12 at 15:24 +0200, Geert Uytterhoeven wrote: > On Mon, Sep 9, 2013 at 6:57 PM, Linux Kernel Mailing List > <linux-kernel@...r.kernel.org> wrote: > > diff --git a/net/sunrpc/auth_gss/auth_gss.c b/net/sunrpc/auth_gss/auth_gss.c > > index 5ec15bb..dc4b449 100644 > > --- a/net/sunrpc/auth_gss/auth_gss.c > > +++ b/net/sunrpc/auth_gss/auth_gss.c > > @@ -51,6 +51,7 @@ > > #include <linux/sunrpc/rpc_pipe_fs.h> > > #include <linux/sunrpc/gss_api.h> > > #include <asm/uaccess.h> > > +#include <linux/hashtable.h> > > > > #include "../netns.h" > > > > @@ -71,6 +72,9 @@ static unsigned int gss_expired_cred_retry_delay = GSS_RETRY_EXPIRED; > > * using integrity (two 4-byte integers): */ > > #define GSS_VERF_SLACK 100 > > > > +static DEFINE_HASHTABLE(gss_auth_hash_table, 16); > > +static DEFINE_SPINLOCK(gss_auth_hash_lock); > > Today's m68k/atari-defconfig kernel no longer boots, as it became larger than > 4 MiB. > > bloat-o-meter tells me: > > function old new delta > gss_auth_hash_table - 262144 +262144 > > Woops... Whoops indeed. The above should have declared 16 buckets, and not 1<<16. I fell for Sasha's subtle trap... > Are you trying to game Tim's survey? ;-) > (question 13 at http://www.embeddedlinuxconference.com/cgi-bin/survey.cgi) > > Can this memory be allocated dynamically / only when it's used? :-) It's declared inside a module, so that should already be the case, however I'll send in a patch to change the above to the intended: DEFINE_HASHTABLE(gss_auth_hash_table, 4); Thanks Geert! -- Trond Myklebust Linux NFS client maintainer NetApp Trond.Myklebust@...app.com www.netapp.com
Powered by blists - more mailing lists