Re: [Qemu-devel] [PATCH 4/7] kvm: Add sanity checks to slot management

[Anthony Liguori <anthony@codemonkey.ws>]

Hollis Blanchard wrote:
> On Wed, 2009-04-29 at 12:38 +0200, Jan Kiszka wrote:
>   
>> What is the alignment of those regions then? None? And do regions of
>> different types overlap even on the same page? Maybe the check reveals
>> some deeper conflict /wrt KVM. Can you point me to the involved code files?
>>     
>
> These PCI controllers make separate calls to
> cpu_register_physical_memory() for separate callbacks. Reading
> ppce500_pci_init(), for example:
> 0xe0008000 -> CFGADDR (4 bytes)
> 0xe0008004 -> CFGDATA (4 bytes)
> 0xe0008c00 -> other registers
>   

That's goofy.  If the single device owns the entire region, it should 
region the entire region instead of relying on subpage functionality.

If just requires a switch() on the address to dispatch to the 
appropriate functions.  It should be easy enough to fix.

Regards,

Anthony Liguori