r324870 breaks boot on amd64 with WITNESS (was "svn commit: r324870 - in head/sys: amd64/include kern")

From: Ngie Cooper (yaneurabeya) <"Ngie> Date: Sun, 22 Oct 2017 17:21:59 -0700 · This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:41:13 UTC

All,
	I highly advise not upgrading to this revision if you use WITNESS. Please see the attached message for more details and reply to the commit log.
Cheers,
-Ngie

> Begin forwarded message:
> 
> From: "Ngie Cooper (yaneurabeya)" <yaneurabeya_at_gmail.com>
> Subject: Re: svn commit: r324870 - in head/sys: amd64/include kern
> Date: October 22, 2017 at 17:19:32 PDT
> To: Mateusz Guzik <mjg_at_FreeBSD.org>
> Cc: src-committers <src-committers_at_freebsd.org>, svn-src-all_at_freebsd.org, svn-src-head_at_freebsd.org
> 
> 
>> On Oct 22, 2017, at 13:43, Mateusz Guzik <mjg_at_FreeBSD.org> wrote:
>> 
>> Author: mjg
>> Date: Sun Oct 22 20:43:50 2017
>> New Revision: 324870
>> URL: https://svnweb.freebsd.org/changeset/base/324870
>> 
>> Log:
>> Make the sleepq chain hash size configurable per-arch and bump on amd64.
>> 
>> While here cache-align chains.
>> 
>> This shortens longest found chain during poudriere -j 80 from 32 to 16.
>> 
>> Pushing this higher up will probably require allocation on boot.
> 
> Hi Mateusz,
> 	This change causes the Jenkins VMs to panic at boot with "panic: witness_init: pending locks list is too small, increase WITNESS_PENDLIST” when WITNESS is enabled: https://ci.freebsd.org/job/FreeBSD-head-amd64-test/4781/console .
> 	Please fix or revert.
> Thanks,
> -Ngie