Re: [Qemu-block] [PATCH v8 3/3] block: enable testing of LUKS driver with block I/O tests

[Max Reitz]

On 18.04.2016 14:21, Daniel P. Berrange wrote:
> This adds support for testing the LUKS driver with the block
> I/O test framework.
> 
>    cd tests/qemu-io-tests
>    ./check -luks
> 
> A handful of test cases are modified to work with luks
> 
>  - 004 - whitelist luks format
>  - 012 - use TEST_IMG_FILE instead of TEST_IMG for file ops
>  - 048 - use TEST_IMG_FILE instead of TEST_IMG for file ops.
>          don't assume extended image contents is all zeros,
>          explicitly initialize with zeros
>          Make file size smaller to avoid having to decrypt
>          1 GB of data.
>  - 052 - don't assume initial image contents is all zeros,
>          explicitly initialize with zeros
>  - 100 - don't assume initial image contents is all zeros,
>          explicitly initialize with zeros

You're doing that with "write -P 0" in this patch. Would it be possible
to use "write -z" instead?

Max

> 
> With this patch applied, the results are as follows:
> 
>   Passed: 001 002 003 004 005 008 009 010 011 012 021 032 043
>           047 048 049 052 087 100 134 143
>   Failed: 033 120 140 145
>  Skipped: 007 013 014 015 017 018 019 020 022 023 024 025 026
>           027 028 029 030 031 034 035 036 037 038 039 040 041
>           042 043 044 045 046 047 049 050 051 053 054 055 056
>           057 058 059 060 061 062 063 064 065 066 067 068 069
>           070 071 072 073 074 075 076 077 078 079 080 081 082
>           083 084 085 086 087 088 089 090 091 092 093 094 095
>           096 097 098 099 101 102 103 104 105 107 108 109 110
>           111 112 113 114 115 116 117 118 119 121 122 123 124
>           128 129 130 131 132 133 134 135 136 137 138 139 141
>           142 144 146 148
> 
> The reasons for the failed tests are:
> 
>  - 033 - needs adapting to use image opts syntax with blkdebug
>          and test image in order to correctly set align property
>  - 120 - needs adapting to use correct -drive syntax for luks
>  - 140 - needs adapting to use correct -drive syntax for luks
>  - 145 - needs adapting to use correct -drive syntax for luks
> 
> The vast majority of skipped tests are exercising code that is
> qcow2 specific, though a couple could probably be usefully
> enabled for luks too.
> 
> Signed-off-by: Daniel P. Berrange <address@hidden>
> ---
>  tests/qemu-iotests/004       |  2 +-
>  tests/qemu-iotests/012       |  5 ++++-
>  tests/qemu-iotests/048       | 26 +++++++++++++++++++-------
>  tests/qemu-iotests/048.out   |  6 ++++--
>  tests/qemu-iotests/052       |  4 ++++
>  tests/qemu-iotests/052.out   |  4 ++++
>  tests/qemu-iotests/100       |  7 +++++++
>  tests/qemu-iotests/100.out   | 14 ++++++++++++++
>  tests/qemu-iotests/common    |  7 +++++++
>  tests/qemu-iotests/common.rc |  3 +++
>  10 files changed, 67 insertions(+), 11 deletions(-)

signature.asc
Description: OpenPGP digital signature