Re: [Qemu-devel] [PATCH 4/4] pci: use pci_config_header in pci.c

[Gerd Hoffmann <kraxel@redhat.com>]

Anthony Liguori wrote:
> Gerd Hoffmann wrote:
>> Well.  That assumes the guests always use the correct in{b,w,l}
>> instruction to access the config space values, i.e. never ever try to
>> use two inb reads for a 16bit value for example.  Or a inw for the lower
>> 16 bits of a 32bit value because they know the high bits are zero anyway.
>>
>> I have my doubts that this is a sane expectation.
> 
> The config space seems organized by data size so you could always just
> take care of this in the accessors.  That is, if you get a byte access
> to a 32-bit value, you would do something like:
> 
> (cpu_to_le32(d->config + (addr & ~3)) >> (8 * (addr % 4))) & 0xFF;
> 
> Not the prettiest thing in the world, but it would work.

-ETOO_MUCH_MAGIC.

I want make the code more readable, that would go into the wrong
direction IMHO.  I think it is much more sane to explicitly call the
byteswapping macros in the code.  And add sparse checking support to
qemu so we have some way to check correctness without taking the
struct-trick route.

cheers,
  Gerd