On 18/05/2023 11:57 am, Roger Pau Monne wrote:
> When a domain parameter is provided to pci_get_pdev() the search
> function would match against the bdf, without taking the segment into
> account.
>
> Fix this and also account for the passed segment.
>
> Fixes: 8cf6e0738906 ('PCI: simplify (and thus correct)
> pci_get_pdev{,_by_domain}()')
> Signed-off-by: Roger Pau Monné <roger....@citrix.com>
> ---
> There's no mention in 8cf6e0738906 that avoiding the segment check is
> fine, and hence I assume it's an oversight, as it should be possible
> to have devices from multiple segments assigned to the same domain.
Oh, absolutely - skipping the segment check is very much not fine.
Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>
