From mboxrd@z Thu Jan 1 00:00:00 1970 From: Joe Perches Subject: Re: [PATCH 8/8] arch/x86: use printk_ratelimited instead of printk_ratelimit Date: Mon, 06 Jun 2011 20:37:25 -0700 Message-ID: <1307417845.4994.57.camel@Joe-Laptop> References: <874o43ii1m.fsf@rustcorp.com.au> <87mxhugwu1.fsf@rustcorp.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: richard -rw- weinberger , Christian Dietrich , "Venkatesh Pallipadi (Venki)" , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Avi Kivity , Marcelo Tosatti , Kevin Tian , Fengzhe Zhang , John Stultz , Lucas De Marchi , linux-kernel@vger.kernel.org, kvm@vger.kernel.org, lguest@lists.ozlabs.org, trivial@kernel.org To: Rusty Russell Return-path: In-Reply-To: <87mxhugwu1.fsf@rustcorp.com.au> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Tue, 2011-06-07 at 12:41 +0930, Rusty Russell wrote: > On Mon, 6 Jun 2011 10:11:53 +0200, richard -rw- weinberger wrote: > > printk_ratelimited() needs DEFINE_RATELIMIT_STATE() which is defined > > in ratelimit.h. > Yech. I'm assuming that making printk.h include ratelimit.h makes a > nasty mess? Yup. I proposed moving the _ratelimited dclarations to ratelimited.h without any comment. http://lkml.org/lkml/2010/2/18/377