[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 2/7] hw/pci: Determine if rombar is explicitly enabled
From: |
Michael S. Tsirkin |
Subject: |
Re: [PATCH v3 2/7] hw/pci: Determine if rombar is explicitly enabled |
Date: |
Tue, 13 Feb 2024 05:52:11 -0500 |
On Mon, Feb 12, 2024 at 07:20:30PM +0900, Akihiko Odaki wrote:
> vfio determines if rombar is explicitly enabled by inspecting QDict.
> Inspecting QDict is not nice because QDict is untyped and depends on the
> details on the external interface. Add an infrastructure to determine if
> rombar is explicitly enabled to hw/pci.
>
> Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
> ---
> include/hw/pci/pci_device.h | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/include/hw/pci/pci_device.h b/include/hw/pci/pci_device.h
> index d3dd0f64b273..7564e9536dbd 100644
> --- a/include/hw/pci/pci_device.h
> +++ b/include/hw/pci/pci_device.h
> @@ -205,6 +205,11 @@ static inline uint16_t pci_get_bdf(PCIDevice *dev)
> return PCI_BUILD_BDF(pci_bus_num(pci_get_bus(dev)), dev->devfn);
> }
>
> +static inline bool pci_rom_bar_explicitly_enabled(PCIDevice *dev)
> +{
> + return dev->rom_bar > 0;
> +}
> +
I don't get it. rom_bar is uint32_t if it's set to "-1" is is still >0.
How was this patchset tested?
> uint16_t pci_requester_id(PCIDevice *dev);
>
> /* DMA access functions */
>
> --
> 2.43.0
- [PATCH v3 0/7] hw/pci: SR-IOV related fixes and improvements, Akihiko Odaki, 2024/02/12
- [PATCH v3 1/7] hw/pci: Use -1 as a default value for rombar, Akihiko Odaki, 2024/02/12
- [PATCH v3 2/7] hw/pci: Determine if rombar is explicitly enabled, Akihiko Odaki, 2024/02/12
- Re: [PATCH v3 2/7] hw/pci: Determine if rombar is explicitly enabled,
Michael S. Tsirkin <=
- [PATCH v3 3/7] vfio: Avoid inspecting option QDict for rombar, Akihiko Odaki, 2024/02/12
- [PATCH v3 4/7] hw/qdev: Remove opts member, Akihiko Odaki, 2024/02/12
- [PATCH v3 5/7] pcie_sriov: Validate NumVFs, Akihiko Odaki, 2024/02/12
- [PATCH v3 6/7] pcie_sriov: Reuse SR-IOV VF device instances, Akihiko Odaki, 2024/02/12
- [PATCH v3 7/7] pcie_sriov: Release VFs failed to realize, Akihiko Odaki, 2024/02/12