[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h |
Date: |
Thu, 9 Jun 2011 00:01:49 +0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Jun 08, 2011 at 10:57:13PM +0200, Jan Kiszka wrote:
> 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
>
The upstream I meant is pci_Regs.h in Linux.
--
MST
- Re: [Qemu-devel] [PATCH v2 8/9] msix: Align MSI-X constants to libpci definitions and extend them, (continued)
[Qemu-devel] [PATCH v2 5/9] msi: Invoke msi/msix_reset from PCI core, Jan Kiszka, 2011/06/08
[Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Jan Kiszka, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Michael S. Tsirkin, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Jan Kiszka, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Michael S. Tsirkin, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Jan Kiszka, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Jan Kiszka, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Michael S. Tsirkin, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Jan Kiszka, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Jan Kiszka, 2011/06/08
- Re: [Qemu-devel] [PATCH v2 9/9] msi: Move PCI_MSI_PENDING_32/64 into pci_regs.h, Michael S. Tsirkin, 2011/06/08
[Qemu-devel] [PATCH v2 1/9] msi: Fix copy&paste mistake in msi_uninit, Jan Kiszka, 2011/06/08
[Qemu-devel] [PATCH v2 2/9] msi: Guard msi/msix_write_config with msi_present, Jan Kiszka, 2011/06/08
[Qemu-devel] [PATCH v2 6/9] msi: Invoke msi/msix_write_config from PCI core, Jan Kiszka, 2011/06/08