[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] blockdev: enable discard by default
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] blockdev: enable discard by default |
Date: |
Wed, 20 Feb 2013 15:13:12 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Feb 08, 2013 at 02:06:14PM +0100, Paolo Bonzini wrote:
> Because discard is now a host parameter, we can always fake it as enabled
> in the guest. This is an extension of the current choice to ignore
> "not supported" errors from the host when discard_granularity is set
> to nonzero.
>
> The default granularity is set to the logical block size or 4k, whichever
> is largest, because cluster sizes below 4k are rarely used and 4K is a
> typical block size for files.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
I wonder if it would make sense to default to 64k so that a qcow2 image
with default options can make use of it. On the other hand, it might
just mean that discard requests are already dropped in the guest kernel
instead of qemu, then we don't really win much.
Kevin
- [Qemu-devel] [RFC PATCH 0/5] Make discard visible to the guests, Paolo Bonzini, 2013/02/08
- [Qemu-devel] [PATCH 1/5] block: implement BDRV_O_UNMAP, Paolo Bonzini, 2013/02/08
- [Qemu-devel] [PATCH 2/5] blockdev: add discard suboption to -drive, Paolo Bonzini, 2013/02/08
- [Qemu-devel] [PATCH 3/5] qemu-nbd: add --discard option, Paolo Bonzini, 2013/02/08
- [Qemu-devel] [PATCH 5/5] pc: add compatibility machine types for 1.4, Paolo Bonzini, 2013/02/08
- [Qemu-devel] [PATCH 4/5] blockdev: enable discard by default, Paolo Bonzini, 2013/02/08
- Re: [Qemu-devel] [PATCH 4/5] blockdev: enable discard by default,
Kevin Wolf <=
- Re: [Qemu-devel] [RFC PATCH 0/5] Make discard visible to the guests, Stefan Hajnoczi, 2013/02/13