Re: [PATCH 18/20] objtool: Add UACCESS validation

From: Linus Torvalds
Date: Thu Mar 07 2019 - 13:01:33 EST


On Thu, Mar 7, 2019 at 9:41 AM Peter Zijlstra <peterz@xxxxxxxxxxxxx> 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