Messages in this thread | | | From | Linus Torvalds <> | Date | Thu, 7 Mar 2019 09:54:14 -0800 | Subject | Re: [PATCH 18/20] objtool: Add UACCESS validation |
| |
On Thu, Mar 7, 2019 at 9:41 AM Peter Zijlstra <peterz@infradead.org> 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
| |