[RFQ] make witness panic an option

Alfred Perlstein bright at mu.org
Fri Nov 16 18:36:08 UTC 2012


On 11/16/12 10:18 AM, Adrian Chadd wrote:
> On 16 November 2012 00:26, Alfred Perlstein <bright at mu.org> wrote:
>
>> Adding another option to tag asserts so that it was sort of like:
>>
>> KASSERT((cond, section, "string")); would be interesting, then you could
>> turn KASSERTS on based on "vfs" or possibly file by file.
> That's orthogonal to my developer-focused request. I'm also a big fan
> of correctly using asserts/panics - ie, asserts shouldn't replace
> correct error handling.
> (Yes, I'm guilty of this in ath(4), but I have plans soon to rectify this.)
>
>
>
> Adrian
I apologize if you took a wishlist item for me as a request for you to 
take on/augment your patch.

It was not my intention.

Back to your work, I like your patch quite a bit, I am wondering though 
if it can be worked into something under witness_kdb though.

-Alfred


More information about the freebsd-hackers mailing list