> > On Wed, Jun 27, 2018 at 04:55:20PM +0530, Pankaj Gupta wrote: > > +static void virtio_rng_set_status(VirtIODevice *vdev, uint8_t status) > > +{ > > + VirtIORNG *vrng = VIRTIO_RNG(vdev); > > + > > + if (!vdev->vm_running) { > > + return; > > + } > > + vdev->status = status; > > Please add a comment to explain this, otherwise someone might think this > is unnecessary since virtio.c also does it.
Sure. Will add a comment and send v3. Thanks, Pankaj