Re: [tip:x86/urgent] x86: mmiotrace: quieten spurious warning message

[Pekka Paalanen <pq@iki.fi>]

On Wed, 11 Mar 2009 20:45:45 GMT
Stuart Bennett <stuart@freedesktop.org> wrote:

> Commit-ID:  afcfe024aebd74b0984a41af9a34e009cf5badaf
> Gitweb:     http://git.kernel.org/tip/afcfe024aebd74b0984a41af9a34e009cf5badaf
> Author:     "Stuart Bennett" <stuart@freedesktop.org>
> AuthorDate: Wed, 11 Mar 2009 20:29:45 +0000
> Commit:     Ingo Molnar <mingo@elte.hu>
> CommitDate: Wed, 11 Mar 2009 21:41:58 +0100
> 
> x86: mmiotrace: quieten spurious warning message
> 
> This message was being incorrectly emitted when using gdb,
> so compile it out by default for now; there will be a
> better fix in v2.6.30.
> 
> Reported-by: Ingo Molnar <mingo@elte.hu>
> Signed-off-by: Stuart Bennett <stuart@freedesktop.org>
> Acked-by: Pekka Paalanen <pq@iki.fi>
> Signed-off-by: Ingo Molnar <mingo@elte.hu>
> 

Guys,

wasn't this supposed to go into 2.6.29?
I don't see this in Linus' master, was it too late?

> 
> ---
>  arch/x86/mm/kmmio.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/arch/x86/mm/kmmio.c b/arch/x86/mm/kmmio.c
> index 6a518dd..4f115e0 100644
> --- a/arch/x86/mm/kmmio.c
> +++ b/arch/x86/mm/kmmio.c
> @@ -310,7 +310,7 @@ static int post_kmmio_handler(unsigned long condition, struct pt_regs *regs)
>  	struct kmmio_context *ctx = &get_cpu_var(kmmio_ctx);
>  
>  	if (!ctx->active) {
> -		pr_warning("kmmio: spurious debug trap on CPU %d.\n",
> +		pr_debug("kmmio: spurious debug trap on CPU %d.\n",
>  							smp_processor_id());
>  		goto out;
>  	}
> 


-- 
Pekka Paalanen
http://www.iki.fi/pq/