On Wed, Sep 16, 2020 at 01:28:19PM +0200, Dmitry Vyukov wrote: > On Fri, Sep 4, 2020 at 6:05 PM Tetsuo Handa > <[email protected]> wrote: > > > > Hello. Can we apply this patch? > > > > This patch addresses top crashers for syzbot, and applying this patch > > will help utilizing syzbot's resource for finding other bugs. > > Acked-by: Dmitry Vyukov <[email protected]> > > Peter, do you still have concerns with this?
Yeah, I still hate it with a passion; it discourages thinking. A bad annotation that blows up the lockdep storage, no worries, we'll just increase this :/ IIRC the issue with syzbot is that the current sysfs annotation is pretty terrible and generates a gazillion classes, and syzbot likes poking at /sys a lot and thus floods the system. I don't know enough about sysfs to suggest an alternative, and haven't exactly had spare time to look into it either :/ Examples of bad annotations is getting every CPU a separate class, that leads to nr_cpus! chains if CPUs arbitrarily nest (nr_cpus^2 if there's only a single nesting level).
