Re: linux-next: Tree for July 30

From: Andrew Morton
Date: Thu Jul 31 2008 - 16:13:46 EST


On Thu, 31 Jul 2008 15:42:07 -0400
Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx> wrote:

> - return bits_to_user(bits, len, _IOC_SIZE(cmd), p, compat_mode);
> +
> + if ((_IOC_NR(cmd) & EV_MAX) == EV_KEY && buf_len == 0x1ff) {
> + printk(KERN_WARNING
> + "evdev.c(EVIOCGBIT): Detected suspicious "
> + "buffer size 0x1ff, limiting output to 64 "
> + "bytes. Make sure you are not using "
> + "EVIOCGBIT(EV_KEY, KEY_MAX)\n");
> + buf_len = 64;
> + }

If that works then great. But I think the printk could be improved.
Please provide sufficient information so that users (not programmers)
can go off and fix things up without needing to email kernel developers.

One suitable approach would be

printk("see http://userweb.kernel.org/~dtor/read-this.txt";)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/