|
From: | Marcel Apfelbaum |
Subject: | Re: [Qemu-devel] [PATCH v4 1/3] pci: Make use of the devfn property when registering new devices |
Date: | Thu, 17 Sep 2015 15:41:53 +0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 |
On 09/17/2015 03:12 PM, Knut Omang wrote:
On Thu, 2015-09-17 at 14:11 +0300, Marcel Apfelbaum wrote:On 09/12/2015 03:36 PM, Knut Omang wrote:Without this, the devfn argument to pci_create_*() does not affect the assigned devfn. Needed to support (VF_STRIDE,VF_OFFSET) values other than (1,1) for SR/IOV. Signed-off-by: Knut Omang <address@hidden> --- hw/pci/pci.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index ccea628..a5cc015 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c @@ -1840,7 +1840,7 @@ static void pci_qdev_realize(DeviceState *qdev, Error **errp) bus = PCI_BUS(qdev_get_parent_bus(qdev)); pci_dev = do_pci_register_device(pci_dev, bus, object_get_typename(OBJECT(qdev)), - pci_dev->devfn, errp); + object_property_get_int(OBJECT(qdev), "addr", NULL), errp);Hi, I don't get this, using object_property_get_int on "addr" should return the value of pci_dev->devfn, can you please tell what exactly is not working?The problem is that at that point pci_dev->devfn has not been set yet - have commented on this before somewhere..
But "addr" property has the right value? Is indeed strange because it should get the value from pci_dev->devfn. Don't get me wrong, this patch is OK. I just want to understand if we have a hidden bug somewhere. Anyway, Reviewed-by: Marcel Apfelbaum <address@hidden>
KnutThanks, Marcelif (pci_dev == NULL) return;
[Prev in Thread] | Current Thread | [Next in Thread] |