From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59764) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aDWK7-0000kp-Ar for qemu-devel@nongnu.org; Mon, 28 Dec 2015 06:50:48 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aDWK6-0006Hj-DI for qemu-devel@nongnu.org; Mon, 28 Dec 2015 06:50:47 -0500 Received: from mx1.redhat.com ([209.132.183.28]:37894) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aDWK6-0006Hf-7j for qemu-devel@nongnu.org; Mon, 28 Dec 2015 06:50:46 -0500 Date: Mon, 28 Dec 2015 13:50:42 +0200 From: "Michael S. Tsirkin" Message-ID: <20151228115042.GB18063@redhat.com> References: <1451271263-8275-1-git-send-email-caoj.fnst@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1451271263-8275-1-git-send-email-caoj.fnst@cn.fujitsu.com> Subject: Re: [Qemu-devel] [PATCH] bugfix: passing reference instead of value List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cao jin Cc: qemu-devel@nongnu.org, stefano.stabellini@eu.citrix.com On Mon, Dec 28, 2015 at 10:54:23AM +0800, Cao jin wrote: > Fix the bug introduced by 595a4f07. Function host_pci_config_read() should be > passed by a reference, not a value, for the later pci_default_write_config(). What's the effect of the bug? Does it break igd assignment? How come it worked for people? If the function is never called, mayber we can get rid of it completely? Stefano? > > Signed-off-by: Cao jin > --- > Separated from previous "igd-passthru convert to realize" patch. Since these > two don`t have dependency, can send it solely. > > Not test since it is easy to find out if reading carefully, just compiled. > > hw/pci-host/piix.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/hw/pci-host/piix.c b/hw/pci-host/piix.c > index 715208b..924f0fa 100644 > --- a/hw/pci-host/piix.c > +++ b/hw/pci-host/piix.c > @@ -761,7 +761,7 @@ static const IGDHostInfo igd_host_bridge_infos[] = { > {0xa8, 4}, /* SNB: base of GTT stolen memory */ > }; > > -static int host_pci_config_read(int pos, int len, uint32_t val) > +static int host_pci_config_read(int pos, int len, uint32_t *val) > { > char path[PATH_MAX]; > int config_fd; > @@ -784,12 +784,14 @@ static int host_pci_config_read(int pos, int len, uint32_t val) > ret = -errno; > goto out; > } > + > do { > - rc = read(config_fd, (uint8_t *)&val, len); > + rc = read(config_fd, (uint8_t *)val, len); > } while (rc < 0 && (errno == EINTR || errno == EAGAIN)); > if (rc != len) { > ret = -errno; > } > + > out: > close(config_fd); > return ret; > @@ -805,7 +807,7 @@ static int igd_pt_i440fx_initfn(struct PCIDevice *pci_dev) > for (i = 0; i < num; i++) { > pos = igd_host_bridge_infos[i].offset; > len = igd_host_bridge_infos[i].len; > - rc = host_pci_config_read(pos, len, val); > + rc = host_pci_config_read(pos, len, &val); > if (rc) { > return -ENODEV; > } > -- > 2.1.0 > >