Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965067AbXASLZk (ORCPT ); Fri, 19 Jan 2007 06:25:40 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965066AbXASLZk (ORCPT ); Fri, 19 Jan 2007 06:25:40 -0500 Received: from mailhub.sw.ru ([195.214.233.200]:8265 "EHLO relay.sw.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965068AbXASLZj (ORCPT ); Fri, 19 Jan 2007 06:25:39 -0500 Message-ID: <45B0AD28.9030607@sw.ru> Date: Fri, 19 Jan 2007 14:36:08 +0300 From: Kirill Korotaev User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.13) Gecko/20060417 X-Accept-Language: en-us, en, ru MIME-Version: 1.0 To: devel@openvz.org CC: ak@suse.de, linux-kernel@vger.kernel.org Subject: Re: [Devel] [PATCH] Print number of oopses in Sysrq-P output References: <20070118170522.GA23679@localhost.sw.ru> In-Reply-To: <20070118170522.GA23679@localhost.sw.ru> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3489 Lines: 105 please fix the comment as well. oops number is very helpful in dealing with people reports. Very often the first Oops is required to get understanding of the real problem, so further oops can be ignored and the first one requested if the problem is reproducable. Kirill > From: Pavel Emelianov > > Useful in deciding whether said output should be ignored > in absence of other info. :) > > Signed-off-by: Pavel Emelianov > Signed-off-by: Alexey Dobriyan > --- > > arch/i386/kernel/process.c | 4 +++- > arch/i386/kernel/traps.c | 2 +- > arch/x86_64/kernel/process.c | 6 ++++-- > arch/x86_64/kernel/traps.c | 3 ++- > 4 files changed, 10 insertions(+), 5 deletions(-) > > --- a/arch/i386/kernel/process.c > +++ b/arch/i386/kernel/process.c > @@ -292,9 +292,11 @@ __setup("idle=", idle_setup); > void show_regs(struct pt_regs * regs) > { > unsigned long cr0 = 0L, cr2 = 0L, cr3 = 0L, cr4 = 0L; > + extern int die_counter; > > printk("\n"); > - printk("Pid: %d, comm: %20s\n", current->pid, current->comm); > + printk("Pid: %d, comm: %20s, oopses: %d\n", > + current->pid, current->comm, die_counter); > printk("EIP: %04x:[<%08lx>] CPU: %d\n",0xffff & regs->xcs,regs->eip, smp_processor_id()); > print_symbol("EIP is at %s\n", regs->eip); > > --- a/arch/i386/kernel/traps.c > +++ b/arch/i386/kernel/traps.c > @@ -366,6 +366,7 @@ int is_valid_bugaddr(unsigned long eip) > return ud2 == 0x0b0f; > } > > +int die_counter = 0; > /* > * This is gone through when something in the kernel has done something bad and > * is about to be terminated. > @@ -381,7 +382,6 @@ void die(const char * str, struct pt_reg > .lock_owner = -1, > .lock_owner_depth = 0 > }; > - static int die_counter; > unsigned long flags; > > oops_enter(); > --- a/arch/x86_64/kernel/process.c > +++ b/arch/x86_64/kernel/process.c > @@ -305,14 +305,16 @@ void __show_regs(struct pt_regs * regs) > unsigned long cr0 = 0L, cr2 = 0L, cr3 = 0L, cr4 = 0L, fs, gs, shadowgs; > unsigned int fsindex,gsindex; > unsigned int ds,cs,es; > + extern int die_counter; > > printk("\n"); > print_modules(); > - printk("Pid: %d, comm: %.20s %s %s %.*s\n", > + printk("Pid: %d, comm: %.20s %s %s %.*s oopses: %d\n", > current->pid, current->comm, print_tainted(), > init_utsname()->release, > (int)strcspn(init_utsname()->version, " "), > - init_utsname()->version); > + init_utsname()->version, > + die_counter); > printk("RIP: %04lx:[<%016lx>] ", regs->cs & 0xffff, regs->rip); > printk_address(regs->rip); > printk("RSP: %04lx:%016lx EFLAGS: %08lx\n", regs->ss, regs->rsp, > --- a/arch/x86_64/kernel/traps.c > +++ b/arch/x86_64/kernel/traps.c > @@ -519,9 +519,10 @@ void __kprobes oops_end(unsigned long fl > oops_exit(); > } > > +int die_counter = 0; > + > void __kprobes __die(const char * str, struct pt_regs * regs, long err) > { > - static int die_counter; > printk(KERN_EMERG "%s: %04lx [%u] ", str, err & 0xffff,++die_counter); > #ifdef CONFIG_PREEMPT > printk("PREEMPT "); > > _______________________________________________ > Devel mailing list > Devel@openvz.org > https://openvz.org/mailman/listinfo/devel > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/