Re: Print taint info in more places.

Mauricio Mauad Menegaz Filho
2007-12-13T23:43:28Z

2007/12/13, Andi Kleen <andi@firstfloor.org>:
> Dave Jones <davej@redhat.com> writes:
>
> > #define WARN_ON(condition) ({ \
> > int __ret_warn_on = !!(condition); \
> > if (unlikely(__ret_warn_on)) { \
> > - printk("WARNING: at %s:%d %s()\n", __FILE__, \
> > - __LINE__, __FUNCTION__); \
> > + printk(KERN_ERR "WARNING: at %s:%d %s() (%s)\n", \
> > + __FILE__, __LINE__, __FUNCTION__, \
> > + print_tainted()); \
> > dump_stack(); \
>
> Have you checked how this affects code size? It might be worth it
> now to do a out of line helper.
>

Have you checked how does spotting a bug is worth the extra size
sometimes (almost all the time)?