Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 9 Apr 2015 15:52:56 +0800
From:      luke <luke.tw@gmail.com>
To:        "Justin T. Gibbs" <gibbs@scsiguy.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, Dmitry Chagin <dchagin@freebsd.org>
Subject:   Re: svn commit: r281162 - head/sys/vm
Message-ID:  <CAGVycpZ0=VQBC_FM3SZ_dczFYaYVQKNbnMrGpVY344m%2BeSDOiw@mail.gmail.com>
In-Reply-To: <0342E981-09D2-4DA8-B3FC-2EC528762127@scsiguy.com>
References:  <201504061845.t36Ijggp094300@svn.freebsd.org> <0342E981-09D2-4DA8-B3FC-2EC528762127@scsiguy.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Apr 9, 2015 at 1:27 PM, Justin T. Gibbs <gibbs@scsiguy.com> wrote:

> On Apr 6, 2015, at 12:45 PM, Dmitry Chagin <dchagin@FreeBSD.org> wrote:
> >
> > Author: dchagin
> > Date: Mon Apr  6 18:45:41 2015
> > New Revision: 281162
> > URL: https://svnweb.freebsd.org/changeset/base/281162
> >
> > Log:
> >  Properly calculate "UMA Zones" per cpu cache size. Avoid allocating
> >  an extra struct uma_cache since the struct uma_zone already has one.
>
> Isn=E2=80=99t the better fix here to use a C99 style flexible array membe=
r (i.e.
> "struct uma_cache uz_cpu[];=E2=80=9D at the end of struct uma_zone)?
>
>
Agree. It will save one struct uma_cache space for 5 UMA_internal zones.

--
luke



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGVycpZ0=VQBC_FM3SZ_dczFYaYVQKNbnMrGpVY344m%2BeSDOiw>