svn commit: r346150 - head/sys/vm

Enji Cooper (yaneurabeya) yaneurabeya at gmail.com
Sat Apr 13 01:16:36 UTC 2019


> On Apr 12, 2019, at 13:10, Tycho Nightingale <tychon at freebsd.org> wrote:
> 
>> On Apr 12, 2019, at 12:43 PM, Enji Cooper <yaneurabeya at gmail.com> wrote:
>> 
>> 
>>> On Apr 12, 2019, at 5:46 AM, Tycho Nightingale <tychon at FreeBSD.org> wrote:
>>> 
>>> Author: tychon
>>> Date: Fri Apr 12 12:46:25 2019
>>> New Revision: 346150
>>> URL: https://svnweb.freebsd.org/changeset/base/346150
>>> 
>>> Log:
>>> for a cache-only zone the destructor tries to destroy a non-existent keg
>>> 
>>> Reviewed by:	markj
>>> Sponsored by:	Dell EMC Isilon
>>> Differential Revision:	https://reviews.freebsd.org/D19835
>> 
>> 
>> Hi Tycho!
>> 	Is the goal of this change to prevent a panic (with an INVARIANTS kernel)?
>> Thank you!
>> -Enji
> 
> Hi,
> 
> The goal of this change is to fix a regression introduced in r343026.  It does cause a panic independent of INVARIANTS.  Fortunately there isn’t currently any in-tree code which triggers the panic.  It does however show up when running with the patch under review in D19845.

Hi again!
	Thank you for the explanation! This part was missing from the commit message, hence my question. I wanted to make sure I understood the change in this commit properly.
Cheers!
-Enji


More information about the svn-src-head mailing list