Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 20 May 2015 19:57:38 +0300
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Eric van Gyzen <eric@vangyzen.net>
Cc:        Adrian Chadd <adrian@freebsd.org>, "freebsd-arch@freebsd.org" <freebsd-arch@freebsd.org>, "freebsd-embedded@freebsd.org" <freebsd-embedded@freebsd.org>
Subject:   Re: RFC: make UMTX hash table size configurable
Message-ID:  <20150520165738.GL2499@kib.kiev.ua>
In-Reply-To: <555CB7F7.5070409@vangyzen.net>
References:  <CAJ-VmonJ2LuU9kEk4h70JvF3C-uObS0FEY3G0Bemfuq3zYKdeQ@mail.gmail.com> <555CB7F7.5070409@vangyzen.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, May 20, 2015 at 12:36:07PM -0400, Eric van Gyzen wrote:
> Adrian,
> 
> Good idea.  Looks good to me.
> 
> Eric
> 
> On 05/20/2015 12:28, Adrian Chadd wrote:
> > Hi,
> >
> > This patch makes the umtx hash table size configurable. It saves ~40k
> > of footprint from embedded images.
> >
> > (I can't seem to get phabricator to create a diff based on a set of
> > files, rather than the whole tree. Sigh.)
> >
> > adrian@lucy-11i386:~/work/freebsd/head-embedded/src % svn diff
> > sys/kern/ sys/conf/options
> > Index: sys/kern/kern_umtx.c
> > ===================================================================
> > --- sys/kern/kern_umtx.c    (revision 282934)
> > +++ sys/kern/kern_umtx.c    (working copy)
> > @@ -29,7 +29,7 @@
> >  __FBSDID("$FreeBSD$");
> >
> >  #include "opt_compat.h"
> > -#include "opt_umtx_profiling.h"
> > +#include "opt_umtx.h"
> >
> >  #include <sys/param.h>
> >  #include <sys/kernel.h>
> > @@ -184,7 +184,11 @@
> >               PRI_MAX_TIMESHARE : (td)->td_user_pri)
> >
> >  #define    GOLDEN_RATIO_PRIME    2654404609U
> > +#ifdef    UMTX_NUM_CHAINS
> > +#define    UMTX_CHAINS        UMTX_NUM_CHAINS
> > +#else
> >  #define    UMTX_CHAINS        512
> > +#endif    /* UMTX_NUM_CHAINS */
> >  #define    UMTX_SHIFTS        (__WORD_BIT - 9)
> >
> >  #define    GET_SHARE(flags)    \
> > Index: sys/conf/options
> > ===================================================================
> > --- sys/conf/options    (revision 282934)
> > +++ sys/conf/options    (working copy)
> > @@ -209,7 +209,8 @@
> >  SYSVSHM        opt_sysvipc.h
> >  SW_WATCHDOG    opt_watchdog.h
> >  TURNSTILE_PROFILING
> > -UMTX_PROFILING
> > +UMTX_PROFILING    opt_umtx.h
> > +UMTX_NUM_CHAINS    opt_umtx.h
> >  VFS_AIO
> >  VERBOSE_SYSINIT
> >  WLCACHE        opt_wavelan.h

It makes more sense to make the hash size a tunable.  The hash mask
would also need to be a variable.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150520165738.GL2499>