[Qemu-devel] bonito: PCI_STATUS questions

[Michael S. Tsirkin]

I see code in bonito.c that clears bits:
PCI_STATUS_REC_MASTER_ABORT | PCI_STATUS_REC_TARGET_ABORT
on each read and write.

However
1. I don't see anything in code that would set these bits
2. The PCI spec says this about the status register:

        Reads to this register behave normally. Writes are slightly different in
        that bits can be reset, but not set. A one bit is reset (if it is not
        read-only) whenever the register is written, and the write data in the
        corresponding bit location is a 1. For instance, to clear bit 14 and not
        affect any other bits, write the value 0100_0000_0000_0000b to the
        register.

while the code in bonito.c resets the bits to 0 on each write.

Comments?

-- 
MST