Date: Tue, 12 Mar 2013 13:04:51 +0100 From: Attilio Rao <attilio@freebsd.org> To: Alan Cox <alc@freebsd.org> Cc: src-committers@freebsd.org, svn-src-user@freebsd.org Subject: Re: svn commit: r248185 - user/attilio/vmcontention/sys/vm Message-ID: <CAJ-FndCU5T4K=sLMF0t0vrn_J=nhASgrtGH7%2BqbeBrZoXr1MmQ@mail.gmail.com> In-Reply-To: <201303120614.r2C6EWve058965@svn.freebsd.org> References: <201303120614.r2C6EWve058965@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Mar 12, 2013 at 7:14 AM, Alan Cox <alc@freebsd.org> wrote: > Author: alc > Date: Tue Mar 12 06:14:31 2013 > New Revision: 248185 > URL: http://svnweb.freebsd.org/changeset/base/248185 > > Log: > When transferring the page from one object to another, don't insert the > page into its new object until the page's pindex has been updated. > Otherwise, one code path within vm_radix_insert() may use the wrong > pindex value. This is just a style change really because the code already subtracts offindxstart from current m->pindex when inserting. The pindex on the page is then adjusted when the real subtraction happens. IIRC, I did this way because of less diffs against -CURRENT, but the code looks correct to me already in the older version. 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?CAJ-FndCU5T4K=sLMF0t0vrn_J=nhASgrtGH7%2BqbeBrZoXr1MmQ>