mbox series

[v3,0/2] qcow2: include LUKS payload overhead in qemu-img measure

Message ID 20190218104525.23674-1-stefanha@redhat.com
Headers show
Series qcow2: include LUKS payload overhead in qemu-img measure | expand

Message

Stefan Hajnoczi Feb. 18, 2019, 10:45 a.m. UTC 
v3:
 * Use encrypt.iter-time=10 to speed up qemu-iotests 178 [Max]
v2:
 * Take into account all "encrypt." image creation options since the LUKS
   payload size varies depending on the cipher [Max]

The LUKS payload has a significant size (>1 MB).  It must be included in the
qemu-img measure calculation so we arrive at the correct minimum image size.

Stefan Hajnoczi (2):
  qcow2: include LUKS payload overhead in qemu-img measure
  iotests: add LUKS payload overhead to 178 qemu-img measure test

 block/qcow2.c                    | 72 +++++++++++++++++++++++++++++++-
 tests/qemu-iotests/178           |  8 ++++
 tests/qemu-iotests/178.out.qcow2 | 24 +++++++++++
 3 files changed, 103 insertions(+), 1 deletion(-)

Comments

Max Reitz Feb. 18, 2019, 5:30 p.m. UTC | #1 
On 18.02.19 11:45, Stefan Hajnoczi wrote:
> v3:
>  * Use encrypt.iter-time=10 to speed up qemu-iotests 178 [Max]

Thanks!

> v2:
>  * Take into account all "encrypt." image creation options since the LUKS
>    payload size varies depending on the cipher [Max]
> 
> The LUKS payload has a significant size (>1 MB).  It must be included in the
> qemu-img measure calculation so we arrive at the correct minimum image size.
> 
> Stefan Hajnoczi (2):
>   qcow2: include LUKS payload overhead in qemu-img measure
>   iotests: add LUKS payload overhead to 178 qemu-img measure test
> 
>  block/qcow2.c                    | 72 +++++++++++++++++++++++++++++++-
>  tests/qemu-iotests/178           |  8 ++++
>  tests/qemu-iotests/178.out.qcow2 | 24 +++++++++++
>  3 files changed, 103 insertions(+), 1 deletion(-)

Applied to my block branch:

https://git.xanclic.moe/XanClic/qemu/commits/branch/block

Max