On Thu, Mar 7, 2019 at 9:41 AM Peter Zijlstra <[email protected]> wrote:
> >
> > What's the call site that made you go "just add __memset() to the list"?
>
> __asan_{,un}poinson_stack_memory()
> kasan_{,un}poison_shadow()
> __memset()
Ugh. I think I almost just agree with your decision to just let that
memset go unchecked.
I'm not saying it's right, but it doesn't seem to be a fight worth fighting.
Again, maybe we could avoid the static checking entirely for the
complex configs, and just make preempt_schedule() not do it for AC
regions.
Because AC vs KASAN in general ends up smelling like "not a fight
worth fighting" to me. You've done a herculean job, but..
Linus