Re: MMIO ioremap() error with PCI passthrough

[Andy Burns <lists.xensource.com@adslpipe.co.uk>]

On 01/07/2008 14:31, Keir Fraser wrote:

> Well, your analysis is correct, and the size argument to __ioremap() is
> bogus. It shouldn't have been rounded up to 4096 without also rounding down
> the base address. 

Just been having a look and a think, the size isn't being rounded up 
anywhere, the size is actually passed in as 4K from the driver

      dev->lmmio = ioremap(pci_resource_start(pci_dev,0), 0x1000);

Perhaps the rounding up to 8K is happening because the address isn't 
aligned on a page boundary, so is split over two pages?

So I assume it is the driver at fault for mapping a larger resource area 
than the card actually supports?

I'll try changing the size argument to 0x400 and rebuilding the saa7134 
module.

> So, could be a bug specific to the FC8 kernel. 

Actually centos5.2 is the domU, FC8 is the dom0, since it's x86_64, I 
assume it's the Fedora/Redhat forward-port effort, rather than the 
paravirt_ops effort.