![]() | |||||||||||
Messages in this thread Patch in this message |
On Saturday 07 June 2008 3:50:59 pm Frans Pop wrote: > Jesper Krogh wrote: > > Not that they seem critical to the system but I do get alot of these. I > > cant remember having seen that before. > > > > [ 2.904467] system 00:06: iomem range 0x0-0x0 could not be reserved > > [ 2.904469] system 00:06: iomem range 0x0-0x0 could not be reserved > > [ 2.904471] system 00:06: iomem range 0x0-0x0 could not be reserved > > [ 2.904473] system 00:06: iomem range 0x0-0x0 could not be reserved > > [...] > > I'm getting these too. Not present in the last -rc4 kernel I built. > > Reverting this commit (the only recent one to the file the message > originates from), gets rid of the extra zero-range messages: > > commit 4b34fe156455d26ee6ed67b61539f136bf4e439c > Author: Bjorn Helgaas <bjorn.helgaas@hp.com> > Date: Mon Jun 2 16:42:49 2008 -0600 > > PNP: mark resources that conflict with PCI devices "disabled" > > Both the PNP/PCI conflict detection quirk and the PNP system > driver must use the same mechanism to mark resources as disabled. > > I think it's best to keep the resource and to keep the type bit > (IORESOURCE_MEM, etc), so that we match the list from firmware > as closely as possible. > > Fixes this regression from 2.6.25: http://lkml.org/lkml/2008/6/1/82 > > Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> > Tested-by: Avuton Olrich <avuton@gmail.com> > Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> > > Relevant CCs added. The patch below should fix this and is already in Linus' tree. Can you give it a whirl to confirm? Thanks! PNP: skip UNSET MEM resources as well as DISABLED ones We don't need to reserve "unset" resources. Trying to reserve them results in messages like this, which are ugly but harmless: system 00:08: iomem range 0x0-0x0 could not be reserved Future PNP patches will remove use of IORESOURCE_UNSET, but we still need it for now. Signed-off-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Index: work11/drivers/pnp/system.c =================================================================== --- work11.orig/drivers/pnp/system.c 2008-06-05 09:46:33.000000000 -0600 +++ work11/drivers/pnp/system.c 2008-06-05 10:29:10.000000000 -0600 @@ -81,7 +81,7 @@ static void reserve_resources_of_dev(str } for (i = 0; (res = pnp_get_resource(dev, IORESOURCE_MEM, i)); i++) { - if (res->flags & IORESOURCE_DISABLED) + if (res->flags & (IORESOURCE_UNSET | IORESOURCE_DISABLED)) continue; reserve_range(dev, res->start, res->end, 0); | ||||||||||
| Last update: 2008-06-09 05:43 [from the cache] ©2003-2008 | |||||||||||