[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 2/5] virtio-blk: add "discard-wzeroes" boolea
From: |
Stefano Garzarella |
Subject: |
Re: [Qemu-block] [PATCH v2 2/5] virtio-blk: add "discard-wzeroes" boolean property |
Date: |
Fri, 1 Feb 2019 18:18:52 +0100 |
On Fri, Feb 1, 2019 at 4:17 PM Michael S. Tsirkin <address@hidden> wrote:
>
> On Thu, Jan 31, 2019 at 04:19:11PM +0100, Stefano Garzarella wrote:
> > In order to avoid migration issues, we enable DISCARD and
> > WRITE ZEROES features only for machine type >= 4.0
> >
> > Suggested-by: Dr. David Alan Gilbert <address@hidden>
> > Signed-off-by: Stefano Garzarella <address@hidden>
> > ---
> > hw/block/virtio-blk.c | 2 ++
> > hw/core/machine.c | 1 +
> > include/hw/virtio/virtio-blk.h | 1 +
> > 3 files changed, 4 insertions(+)
> >
> > diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c
> > index 8a6754d9a2..542ec52536 100644
> > --- a/hw/block/virtio-blk.c
> > +++ b/hw/block/virtio-blk.c
> > @@ -1026,6 +1026,8 @@ static Property virtio_blk_properties[] = {
> > DEFINE_PROP_UINT16("queue-size", VirtIOBlock, conf.queue_size, 128),
> > DEFINE_PROP_LINK("iothread", VirtIOBlock, conf.iothread, TYPE_IOTHREAD,
> > IOThread *),
> > + DEFINE_PROP_BIT("discard-wzeroes", VirtIOBlock, conf.discard_wzeroes,
> > 0,
> > + true),
> > DEFINE_PROP_END_OF_LIST(),
> > };
> >
>
> Thinking about it, are there security implications for discard?
> Should we make it default false?
Hi Michael,
I'm not completely sure but if the guest can write on a specific sector,
discard or write_zeroes operations should not have a security implication.
Do I miss something?
Thanks,
Stefano
Re: [Qemu-block] [PATCH v2 2/5] virtio-blk: add "discard-wzeroes" boolean property, Michael S. Tsirkin, 2019/02/05