I was having a similar oops (just got around to having time to decode it
earlier, and was getting ready to send it when I got this :), so I tried
this patch. So far, so good.
Jeremy
-- Jeremy Katz http://linuxpower.orgOn Sat, 7 Aug 1999, Linus Torvalds wrote:
> > > Oops, there's a silly bug in __request_region() that bites you if you have > resource clashes. We don't correctly clean up, so we return a half-assed > "successful" allocation for some cases we definitely should not.. > > It then crashes in __release_region() because the bad success we returned > had not actually been initialized as a successful resource. > > Does this simple patch fix it for you? > > Linus > > ---- > --- linux-pre8/kernel/resource.c Thu Aug 5 18:24:21 1999 > +++ linux/kernel/resource.c Sat Aug 7 11:08:54 1999 > @@ -145,7 +145,7 @@ > > write_lock(&resource_lock); > > - while (!(parent->flags & IORESOURCE_BUSY)) { > + for (;;) { > struct resource *conflict; > > conflict = __request_resource(parent, res); > @@ -153,7 +153,8 @@ > break; > if (conflict != parent) { > parent = conflict; > - continue; > + if (!(conflict->flags & IORESOURCE_BUSY)) > + continue; > } > > /* Uhhuh, that didn't work out.. */ > > > - > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.rutgers.edu > Please read the FAQ at http://www.tux.org/lkml/ >
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/