Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h

[Jan Kiszka]

On 2011-06-08 22:56, Michael S. Tsirkin wrote:
> On Wed, Jun 08, 2011 at 10:44:58PM +0200, Jan Kiszka wrote:
>> On 2011-06-08 21:48, Michael S. Tsirkin wrote:
>>> On Wed, Jun 08, 2011 at 06:21:52PM +0200, Jan Kiszka wrote:
>>>> Signed-off-by: Jan Kiszka <address@hidden>
>>>
>>> pci_regs.h from linux does not have these
>>> this is why we keep them in msi.c
>>>
>>> address@hidden linux-2.6]$ grep PCI_MSI_ include/linux/pci_regs.h 
>>> #define PCI_MSI_FLAGS           2       /* Various flags */
>>> #define  PCI_MSI_FLAGS_64BIT    0x80    /* 64-bit addresses allowed */
>>> #define  PCI_MSI_FLAGS_QSIZE    0x70    /* Message queue size configured
>>> */
>>> #define  PCI_MSI_FLAGS_QMASK    0x0e    /* Maximum queue size available
>>> */
>>> #define  PCI_MSI_FLAGS_ENABLE   0x01    /* MSI feature enabled */
>>> #define  PCI_MSI_FLAGS_MASKBIT  0x100   /* 64-bit mask bits allowed */
>>> #define PCI_MSI_RFU             3       /* Rest of capability flags */
>>> #define PCI_MSI_ADDRESS_LO      4       /* Lower 32 bits */
>>> #define PCI_MSI_ADDRESS_HI      8       /* Upper 32 bits (if
>>> PCI_MSI_FLAGS_64BIT set) */
>>> #define PCI_MSI_DATA_32         8       /* 16 bits of data for 32-bit
>>> devices */
>>> #define PCI_MSI_MASK_32         12      /* Mask bits register for 32-bit
>>> devices */
>>> #define PCI_MSI_DATA_64         12      /* 16 bits of data for 64-bit
>>> devices */
>>> #define PCI_MSI_MASK_64         16      /* Mask bits register for 64-bit
>>> devices */
>>>
>>>
>>> If you want to move them, please send them upstream we'll merge when
>>> they are there.
>>
>> In fact, both defines are already in libpci. Since 3.0.0. Released 5
>> years ago. OK, I'll send a header resync patch against a more recent
>> release.
>>
>> Then we should just lack something like PCI_MSIX_CTRL. I will have a look.
> 
> PCI_MSIX_FLAGS is the same.

That does not exist in libpci's header.

Jan

signature.asc
Description: OpenPGP digital signature