Re: [PATCH] serial8250: ratelimit "too much work" error

From: Andrew Morton
Date: Mon Oct 04 2010 - 18:03:23 EST


On Mon, 4 Oct 2010 14:51:01 -0700
Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:

> someone stuck a
> DEFINE_RATELIMIT_STATE() in include/linux/kernel.h and it ain't defined
> anywhere. Let me fix that up...

Well that's a PITA. Can't include ratelimit.h into kernel.h because a)
it'll slow everyone's compiels down and b) ratelimit.h needs spinlock.h
which surely needs kernel.h. Fixable by adding a new
printk_ratelimit.h and including that from 135 source files, blah.

I'll give up and pronounce that users of printk_ratelimited() need to
include ratelimit.h as well.

--
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/