Date: Fri, 9 Aug 2013 15:56:47 -0400 From: John Baldwin <jhb@freebsd.org> To: "David E. O'Brien" <obrien@freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r254150 - head/sys/vm Message-ID: <201308091556.47535.jhb@freebsd.org> In-Reply-To: <201308091643.r79GhoWx023884@svn.freebsd.org> References: <201308091643.r79GhoWx023884@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Friday, August 09, 2013 12:43:50 pm David E. O'Brien wrote: > Author: obrien > Date: Fri Aug 9 16:43:50 2013 > New Revision: 254150 > URL: http://svnweb.freebsd.org/changeset/base/254150 > > Log: > Add missing 'VPO_BUSY' from r254141 to fix kernel build break. > > Modified: > head/sys/vm/vm_page.h This can't possibly be correct as r254138 just removed this flag. If it isn't obvious how to fix the uses added back in r254141, then r254141 should be reverted instead. Hmm, looking at the relevant bits of r254141, it doesn't look obvious: + /* Detach the old page from the resident tailq. */ + TAILQ_REMOVE(&object->memq, mold, listq); + vm_page_lock(mold); + if (mold->oflags & VPO_BUSY) { + mold->oflags &= ~VPO_BUSY; + vm_page_flash(mold); + } Since nothing is setting this flag, this can't possibly work correctly currently. I wouldn't boot a top-of-tree kernel right now. :( -- John Baldwin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201308091556.47535.jhb>