Thanks! Regards, Wang Sheng-Hui ------------------ Original ------------------ From: "Bjorn Helgaas";<helgaas@xxxxxxxxxx>; Date: Sat, Nov 12, 2016 04:51 AM To: "Wang Sheng-Hui"<shhuiw@xxxxxxxxxxx>; Cc: "yu.zhao"<yu.zhao@xxxxxxxxx>; "jbarnes"<jbarnes@xxxxxxxxxxxxxxxx>; "alex.williamson"<alex.williamson@xxxxxxxxxx>; "bhelgaas"<bhelgaas@xxxxxxxxxx>; "linux-pci"<linux-pci@xxxxxxxxxxxxxxx>; "kvm"<kvm@xxxxxxxxxxxxxxx>; "linux-kernel"<linux-kernel@xxxxxxxxxxxxxxx>; Subject: Re: [PATCH] PCI: move cfg space size (256/4096) macros to pci_regs.h On Thu, Sep 22, 2016 at 09:05:46AM +0800, Wang Sheng-Hui wrote: > This is a cleanup that moves PCI configuration space size > macros (PCI_CFG_SPACE_SIZE and PCI_CFG_SPACE_EXP_SIZE) from > drivers/pci/pci.h to include/uapi/linux/pci_regs.h > > So that the macros can be used by more drivers and eliminate > potential duplicate definition. > > Signed-off-by: Wang Sheng-Hui <shhuiw@xxxxxxxxxxx> Applied to pci/misc for v4.10, thanks! I updated the comment to include PCI-X details, since the extended config space applies to PCI-X Mode 2 devices as well as to PCIe devices: + * Conventional PCI and PCI-X Mode 1 devices have 256 bytes of + * configuration space. PCI-X Mode 2 and PCIe devices have 4096 bytes of + * configuration space. > --- > drivers/pci/pci.h | 3 --- > drivers/vfio/pci/vfio_pci_config.c | 2 -- > include/uapi/linux/pci_regs.h | 7 +++++++ > 3 files changed, 7 insertions(+), 5 deletions(-) > > diff --git a/drivers/pci/pci.h b/drivers/pci/pci.h > index 9730c47..12d651a 100644 > --- a/drivers/pci/pci.h > +++ b/drivers/pci/pci.h > @@ -1,9 +1,6 @@ > #ifndef DRIVERS_PCI_H > #define DRIVERS_PCI_H > > -#define PCI_CFG_SPACE_SIZE 256 > -#define PCI_CFG_SPACE_EXP_SIZE 4096 > - > #define PCI_FIND_CAP_TTL 48 > > extern const unsigned char pcie_link_speed[]; > diff --git a/drivers/vfio/pci/vfio_pci_config.c b/drivers/vfio/pci/vfio_pci_config.c > index 688691d..8dc8a5a 100644 > --- a/drivers/vfio/pci/vfio_pci_config.c > +++ b/drivers/vfio/pci/vfio_pci_config.c > @@ -31,8 +31,6 @@ > > #include "vfio_pci_private.h" > > -#define PCI_CFG_SPACE_SIZE 256 > - > /* Fake capability ID for standard config space */ > #define PCI_CAP_ID_BASIC 0 > > diff --git a/include/uapi/linux/pci_regs.h b/include/uapi/linux/pci_regs.h > index 4040951..a07c53c 100644 > --- a/include/uapi/linux/pci_regs.h > +++ b/include/uapi/linux/pci_regs.h > @@ -23,6 +23,13 @@ > #define LINUX_PCI_REGS_H > > /* > + * Under PCI, each device has 256 bytes of configuration address space. > + * Under PCIe, each device has 4096 bytes of configuration address space. > + */ > +#define PCI_CFG_SPACE_SIZE 256 > +#define PCI_CFG_SPACE_EXP_SIZE 4096 > + > +/* > * Under PCI, each device has 256 bytes of configuration address space, > * of which the first 64 bytes are standardized as follows: > */ > -- > 2.7.4 > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-pci" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.htmlÿôèº{.nÇ+?·?®??+%?Ëÿ±éݶ;¥?wÿº{.nÇ+?·¤¾oâ?Ø^n?r¡ö¦zË?ëh?¨èÚ&£ûàz¿äz¹Þ?ú+?Ê+zf£¢·h??§~??Ûiÿÿï?êÿ?êçz_è®æj:+v?¨þ)ߣøm
