Bug in physmap driver?

[Lennert Buytenhek]

On Thu, Apr 16, 2009 at 04:09:41PM +0100, David Howells wrote:

> > That would suggest that dev->resource[i].end is off-by-one?
> 
> Actually, that's a good question: should end be one after the last useful
> addres, or should it be that last useful address?

The last useful address.  That's what the resource code expects, as
e.g. two regions with start=00100000,end=00200000 and
start=00200000,end=00300000 will conflict (registration of the second
will fail), whereas if you change the end addresses to 001fffff and
002fffff, they will no longer conflict.

And e.g. /proc/iomem on my x86 box has:

	c4000000-c40fffff : PCI Bus #01
	dff00000-efefffff : PCI Bus #05
	ff500000-ff5fffff : PCI Bus #01
	ff600000-ff6fffff : PCI Bus #02
	ff700000-ff7fffff : PCI Bus #03
	ff800000-ff8fffff : PCI Bus #04
	ff900000-ff9fffff : PCI Bus #05