On Wed, May 23, 2018 at 11:52 AM, John Baldwin <j...@freebsd.org> wrote:
> On Wednesday, May 23, 2018 05:00:05 PM Matt Macy wrote:
>> Author: mmacy
>> Date: Wed May 23 17:00:05 2018
>> New Revision: 334104
>> URL: https://svnweb.freebsd.org/changeset/base/334104
>>
>> Log:
>>   epoch: allow for conditionally asserting that the epoch context fields
>>   are unused by zeroing on INVARIANTS builds
>
> Is M_ZERO really so bad that you need to make it conditional?

In this case not at all. It's only exercised by sysctl handlers. I'm
mostly responding to an inquiry by jtl. However, gratuitous M_ZERO
usage does have a cumulative adverse performance impact.

> I would probably have preferred something like 'M_ZERO_INVARIANTS'
> instead perhaps (or M_ZERO_EPOCH) that only controls M_ZERO and is
> still or'd with M_WAITOK or M_NOWAIT.

Yes. I like that better too. Thanks.

-M
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to