> 
> 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

Reply via email to