Re: [coreboot] [PATCH] Fix infinite loop in pnp_get_ioresource()

2011-01-29 Thread Alex G.
> > Unfortunately the patch was lost. Please send it, want to fix all > bugs like this! :) > You're a bit late :) . I resent the patch last night, and Stefan wrote a better one based on it. Alex -- coreboot mailing list: coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] [PATCH] Fix infinite loop in pnp_get_ioresource()

2011-01-29 Thread Peter Stuge
Alex G. wrote: > Fix an infinite loop in pnp_get_ioresource(), which freezes > coreboot if a rare condition arises. > > Signed-off-by: Alexandru Gagniuc Unfortunately the patch was lost. Please send it, want to fix all bugs like this! :) //Peter -- coreboot mailing list: coreboot@coreboot.or

Re: [coreboot] [PATCH] Fix infinite loop in pnp_get_ioresource()

2011-01-28 Thread Alex G.
Hi Stefan, Thanks for the heads-up. In my case it's the MIDI device from the W83627EHG that has the broken mask. I see no difference when not creating the resource, so I doubt this breaks anything. Alex Acked-by: Alexandru Gagniuc On 01/29/2011 01:03 AM, Stefan Reinauer wrote: > Hi Alexandru,

Re: [coreboot] [PATCH] Fix infinite loop in pnp_get_ioresource()

2011-01-28 Thread Stefan Reinauer
Hi Alexandru, thanks for finding this issue. I think I ran into this before. We should not silently continue in this case, and not create a resource either. The problem is most likely caused by a faulty devicetree.cb or superio driver. Hence, I suggest to print an error instead. See patch. Can you

[coreboot] [PATCH] Fix infinite loop in pnp_get_ioresource()

2011-01-28 Thread Alex G.
Fix an infinite loop in pnp_get_ioresource(), which freezes coreboot if a rare condition arises. Signed-off-by: Alexandru Gagniuc --- Index: src/devices/pnp_device.c === --- src/devices/pnp_device.c (revision 6315) +++ src/devices/p

[coreboot] [PATCH] Fix infinite loop in pnp_get_ioresource()

2011-01-28 Thread Alex G.
Fix an infinite loop in pnp_get_ioresource(), which freezes coreboot if a rare condition arises. Signed-off-by: Alexandru Gagniuc --- -- coreboot mailing list: coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot