Hi Wolfram, On Tue, Jan 7, 2020 at 11:26 AM Wolfram Sang <wsa@xxxxxxxxxxxxx> wrote: > > Quoting GregKH: > > | We really do not want WARN_ON() anywhere, as that causes systems with > > | panic-on-warn to reboot. > > > > https://lore.kernel.org/lkml/20191121135743.GA552517@xxxxxxxxx/ > > Huh? This renders WARN completely useless, or? If somebody wants panic > on warn, this person should get it? I also have my doubts... > If we don't add a stack trace, we only know that *someone* registered an > invalid address. Finding out who can be annoying. Kieran spotted this > correctly. What other information will you have in the backtrace, that you don't have available inside the function? Would printing the i2c_driver name be sufficient? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds