linux-kernel - Re: [PATCH] prom_find_machine_type typo breaks pSeries lpar boot

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 15 Nov 2007 09:13:02 +1100
From:	Benjamin Herrenschmidt <benh@...nel.crashing.org>
To:	"B. N. Poornima" <poornima@...ibm.com>
Cc:	Nathan Lynch <ntl@...ox.com>, Andrew Morton <akpm@...l.org>,
	Linus Torvalds <torvalds@...l.org>,
	ppc64 dev list <linuxppc64-dev@...abs.org>,
	linux-kernel <linux-kernel@...r.kernel.org>,
	"Martin J. Bligh" <mbligh@...igh.org>, roland@...spin.com
Subject: Re: [PATCH] prom_find_machine_type typo breaks pSeries lpar boot


On Wed, 2007-11-14 at 17:57 +0530, B. N. Poornima wrote:
> Benjamin Herrenschmidt wrote:
> > On Fri, 2005-06-03 at 14:25 -0500, Nathan Lynch wrote:
> >   
> >> Typo in prom_find_machine_type from Ben's recent patch "ppc64: Fix
> >> result code handling in prom_init" prevents pSeries LPAR systems from
> >> booting.
> >>
> >> Tested on a pSeries 570 and OpenPower 720 (both Power5 LPAR).
> >>     
> >
> > Damn ! I'm certain I tested it on P5 ! I must have forgotten to "quilt
> > ref" before sending the patch (as I did notice this typo and fixed it
> > just before sending).
> >
> > Sorry!
> >
> > Ben.
> >
> >
> > -
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@...r.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at  http://www.tux.org/lkml/
> >
> >
> >   
> Hello,
> Same typo has creped in again from 2.6.17 kernel onwards and causing 
> open firmware exception in pSeries systems.

Can you forward the patch ?

> =====================================================================================
> 
> Welcome to yaboot version 10.1.21-r910.SuSE
> booted from '/vdevice/v-scsi@...00003/disk@...0000000000000'
> Enter "help" to get some basic usage information
> boot: 2.6.23.1
> Please wait, loading kernel...
> Allocated 03500000 bytes for kernel @ 01c00000
>    Elf64 kernel loaded...
> Loading ramdisk...
> ramdisk loaded 0020d000 @ 06b00000
> DEFAULT CATCH!, exception-handler=fff00300 
> at   %SRR0: c00000000202705c   %SRR1: 8000000000003002 
> Open Firmware exception handler entered from non-OF code
> 
> Client's Fix Pt Regs:
>  00 0223100003080000 00000000018bfac0 0000000004bbeaf8 00000000018bfb48
>  04 00000000018bfc90 0000000000c39a48 0000000000000000 5c3a03080000a0fa
>  08 0000000000000001 0000000000000000 05910000194c0223 000000005cae0000
>  0c 0000000003f1eaf8 0000000000000000 0000000000000000 0000000000000000
>  10 0000000000000000 0000000000c39a48 0000000000000000 00000000018bfc90
>  14 0000000000c00000 000000000020d000 0000000000000000 0000000008b00000
>  18 0000000000280004 00000000018bff04 0000000000000001 0000000000000000
>  1c 0000000008b00000 0000000000c39a48 c000000004d1e530 0000000008b00000
> Special Regs:
>     %IV: 00000300     %CR: 44000028    %XER: 00000000  %DSISR: 08000000 
>   %SRR0: c00000000202705c   %SRR1: 8000000000003002 
>     %LR: c000000002027428    %CTR: c0000000020273f0 
>    %DAR: 05910000194c02eb 
> Virtual PID = 0 
> PFW: Unable to send error log!
>  ofdbg
> 0 > 
> 
> ===========================================================================================
> The patch that is provided solves the problem.
> 
> Regards,
> Poornima.

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