Re: [PATCH v3 08/13] pci: provide pci_scan_bars()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 14, 2016 at 05:28:06PM -0500, Peter Xu wrote:
> Let's provide a more general way to scan PCI bars, rather than read the
> config registers every time.
> 
> Then let x86/vmexit.c leverage pci_scan_bars()
> 
> Signed-off-by: Peter Xu <peterx@xxxxxxxxxx>
> ---
>  lib/pci.c    | 17 ++++++++++++++++-
>  lib/pci.h    |  5 +++++
>  x86/vmexit.c | 17 ++++++-----------
>  3 files changed, 27 insertions(+), 12 deletions(-)
> 
> diff --git a/lib/pci.c b/lib/pci.c
> index 8f2356d..462c370 100644
> --- a/lib/pci.c
> +++ b/lib/pci.c
> @@ -207,7 +207,7 @@ static void pci_dev_print(pcidevaddr_t dev)
>  	if ((header & PCI_HEADER_TYPE_MASK) != PCI_HEADER_TYPE_NORMAL)
>  		return;
>  
> -	for (i = 0; i < 6; i++) {
> +	for (i = 0; i < PCI_BAR_NUM; i++) {
>  		if (pci_bar_size(&pci_dev, i)) {
>  			printf("\t");
>  			pci_bar_print(&pci_dev, i);
> @@ -227,3 +227,18 @@ void pci_print(void)
>  			pci_dev_print(dev);
>  	}
>  }
> +
> +void pci_scan_bars(struct pci_dev *dev)
> +{
> +	int i;
> +
> +	for (i = 0; i < PCI_BAR_NUM; i++) {
> +		if (!pci_bar_is_valid(dev, i))
> +			continue;
> +		dev->bar[i] = pci_bar_get_addr(dev, i);
> +		if (pci_bar_is64(dev, i)) {
> +			i++;
> +			dev->bar[i] = (phys_addr_t)0;
> +		}
> +	}
> +}
> diff --git a/lib/pci.h b/lib/pci.h
> index d7140dd..338ad51 100644
> --- a/lib/pci.h
> +++ b/lib/pci.h
> @@ -15,13 +15,16 @@ enum {
>  	PCIDEVADDR_INVALID = 0xffff,
>  };
>  
> +#define PCI_BAR_NUM                     6
>  #define PCI_DEVFN_MAX                   256
>  
>  struct pci_dev {
>  	uint16_t bdf;
> +	phys_addr_t bar[PCI_BAR_NUM];
>  };
>  
>  void pci_dev_init(struct pci_dev *dev, pcidevaddr_t bdf);
> +void pci_scan_bars(struct pci_dev *dev);
>  
>  extern bool pci_probe(void);
>  extern void pci_print(void);
> @@ -65,6 +68,8 @@ int pci_testdev(void);
>   * pci-testdev supports at least three types of tests (via mmio and
>   * portio BARs): no-eventfd, wildcard-eventfd and datamatch-eventfd
>   */
> +#define PCI_TESTDEV_BAR_MEM		0
> +#define PCI_TESTDEV_BAR_IO		1
>  #define PCI_TESTDEV_NUM_BARS		2
>  #define PCI_TESTDEV_NUM_TESTS		3
>  
> diff --git a/x86/vmexit.c b/x86/vmexit.c
> index 63fa070..a22f43f 100644
> --- a/x86/vmexit.c
> +++ b/x86/vmexit.c
> @@ -389,17 +389,12 @@ int main(int ac, char **av)
>  	ret = pci_find_dev(PCI_VENDOR_ID_REDHAT, PCI_DEVICE_ID_REDHAT_TEST);
>  	if (ret != PCIDEVADDR_INVALID) {
>  		pci_dev_init(&pcidev, ret);
> -		for (i = 0; i < PCI_TESTDEV_NUM_BARS; i++) {
> -			if (!pci_bar_is_valid(&pcidev, i)) {
> -				continue;
> -			}
> -			if (pci_bar_is_memory(&pcidev, i)) {
> -				membar = pci_bar_get_addr(&pcidev, i);
> -				pci_test.memaddr = ioremap(membar, PAGE_SIZE);
> -			} else {
> -				pci_test.iobar = pci_bar_get_addr(&pcidev, i);
> -			}
> -		}
> +		pci_scan_bars(&pcidev);
> +		assert(pci_bar_is_memory(&pcidev, PCI_TESTDEV_BAR_MEM));
> +		assert(!pci_bar_is_memory(&pcidev, PCI_TESTDEV_BAR_IO));
> +		membar = pcidev.bar[PCI_TESTDEV_BAR_MEM];
> +		pci_test.memaddr = ioremap(membar, PAGE_SIZE);
> +		pci_test.iobar = pcidev.bar[PCI_TESTDEV_BAR_IO];
>  		printf("pci-testdev at 0x%x membar %lx iobar %x\n",
>  		       pcidev.bdf, membar, pci_test.iobar);
>  	}
> -- 
> 2.7.4
> 

You're still missing at least one 256 and one 6 that you wanted to change
to PCI_DEVFN_MAX, PCI_BAR_NUM. See lib/pci-host-generic.c:pci_probe. Also,
Alex's series is merged now so you can rebase on master to get it.

Otherwise
Reviewed-by: Andrew Jones <drjones@xxxxxxxxxx>

Thanks,
drew

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux