[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: |
Wed, 8 Jun 2011 23:56:22 +0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
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.
>
> Jan
>
- 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 <=
- 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, 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