[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] pci: fix requester id with PCI bridges
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH] pci: fix requester id with PCI bridges |
Date: |
Mon, 16 May 2016 15:23:30 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Thu, May 12, 2016 at 10:51:43AM +0300, Michael S. Tsirkin wrote:
> On Thu, May 12, 2016 at 03:32:03PM +0800, Peter Xu wrote:
[...]
> > > Are there cases where we need the original requester id value?
> > > I am guessing not, and if I'm right we should just change the
> > > implementation
> > > of pci_requester_id to DTRT.
> >
> > There are two other callers for it (besides the MSI one):
> >
> > - assigned_device_pci_cap_init()
This should use the new interface.
> > - do_pcie_aer_inject_error()
This should use the old one, which is to get BDF only.
> >
> > For both the cases, I am not sure whether we can do the replacement.
> >
> > -- peterx
>
> Read the specs please. We can't just pile on APIs.
IIUC, we may still need to add a new API here, because not all of
them are looking for requester IDs. I just posted v2 for this patch.
Instead of renaming the new function, I renamed the old one to
pci_get_bdf(). Hope that works.
Thanks,
-- peterx
Re: [Qemu-devel] [PATCH] pci: fix requester id with PCI bridges, Peter Xu, 2016/05/11