From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53928) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Umrjs-0000bO-Ez for qemu-devel@nongnu.org; Wed, 12 Jun 2013 16:33:53 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Umrjq-0000CC-GY for qemu-devel@nongnu.org; Wed, 12 Jun 2013 16:33:52 -0400 Sender: fluxion Date: Wed, 12 Jun 2013 15:33:34 -0500 From: mdroth Message-ID: <20130612203334.GG12585@vm> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH v2] virtio-rng: Fix crash with non-default backend List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cole Robinson Cc: amit.shah@redhat.com, qemu-stable@nongnu.org, mst@redhat.com, qemu-devel@nongnu.org, fred.konrad@greensocs.com On Fri, May 31, 2013 at 02:12:48PM -0400, Cole Robinson wrote: > 'default_backend' isn't always set, but 'rng' is, so use that. > > $ ./x86_64-softmmu/qemu-system-x86_64 -object rng-random,id=rng0,filename=/dev/random -device virtio-rng-pci,rng=rng0 > Segmentation fault (core dumped) > > Regressed with virtio refactoring in 59ccd20a9ac719cff82180429458728f03ec612f > > CC: qemu-stable@nongnu.org > Signed-off-by: Cole Robinson Tested-by: Michael Roth Looking to pull this in for 1.5.1 > --- > > Notes: > v2: Fix other instances in s390x code, pointed out by Frederic > > hw/s390x/s390-virtio-bus.c | 2 +- > hw/s390x/virtio-ccw.c | 2 +- > hw/virtio/virtio-pci.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/hw/s390x/s390-virtio-bus.c b/hw/s390x/s390-virtio-bus.c > index a1cdfb0..207eb82 100644 > --- a/hw/s390x/s390-virtio-bus.c > +++ b/hw/s390x/s390-virtio-bus.c > @@ -300,7 +300,7 @@ static int s390_virtio_rng_init(VirtIOS390Device *s390_dev) > } > > object_property_set_link(OBJECT(dev), > - OBJECT(dev->vdev.conf.default_backend), "rng", > + OBJECT(dev->vdev.conf.rng), "rng", > NULL); > > return s390_virtio_device_init(s390_dev, VIRTIO_DEVICE(vdev)); > diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c > index 5f5e267..7dcd98d 100644 > --- a/hw/s390x/virtio-ccw.c > +++ b/hw/s390x/virtio-ccw.c > @@ -744,7 +744,7 @@ static int virtio_ccw_rng_init(VirtioCcwDevice *ccw_dev) > } > > object_property_set_link(OBJECT(dev), > - OBJECT(dev->vdev.conf.default_backend), "rng", > + OBJECT(dev->vdev.conf.rng), "rng", > NULL); > > return virtio_ccw_device_init(ccw_dev, VIRTIO_DEVICE(vdev)); > diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c > index 444b71a..b070b64 100644 > --- a/hw/virtio/virtio-pci.c > +++ b/hw/virtio/virtio-pci.c > @@ -1455,7 +1455,7 @@ static int virtio_rng_pci_init(VirtIOPCIProxy *vpci_dev) > } > > object_property_set_link(OBJECT(vrng), > - OBJECT(vrng->vdev.conf.default_backend), "rng", > + OBJECT(vrng->vdev.conf.rng), "rng", > NULL); > > return 0; > -- > 1.8.2.1 > >