Date: Wed, 28 Mar 2007 16:24:38 +0200 From: "Attilio Rao" <attilio@freebsd.org> To: "John Baldwin" <jhb@freebsd.org> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/kern init_main.c kern_thread.c Message-ID: <3bbf2fe10703280724h3d59f3d8t159c2997d84cb950@mail.gmail.com> In-Reply-To: <3bbf2fe10703271120p4c33230dpbcfc2d72164113e9@mail.gmail.com> References: <200703271651.l2RGpZfv012442@repoman.freebsd.org> <3bbf2fe10703271120p4c33230dpbcfc2d72164113e9@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
2007/3/27, Attilio Rao <attilio@freebsd.org>: > 2007/3/27, John Baldwin <jhb@freebsd.org>: > > jhb 2007-03-27 16:51:34 UTC > > > > FreeBSD src repository > > > > Modified files: > > sys/kern init_main.c kern_thread.c > > Log: > > Align 'struct thread' on 16 byte boundaries so that the lower 4 bits are > > always 0. Previously we aligned threads on a minimum of 8-byte boundaries. > > > > Note: This changes the uma zone to no longer cache align threads. We > > really want the uma zone to do align threads to MAX(16, cache line size) > > but there currently isn't a good way to express that to uma. > > We can workaround it using a function like: > int uma_get_align(void) > (which would be the counter part of uma_set_align). > Its implementations is trivial. I have a diff for that: http://users.gufi.org/~rookie/works/patches/uma2.diff I had no time to sync my branches with last HEAD changes, so I can't fix uma_zcreate() call... just adding the uma_get_align() function. Thanks, Attilio -- Peace can only be achieved by understanding - A. Einstein
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3bbf2fe10703280724h3d59f3d8t159c2997d84cb950>