On 06.11.19 10:24, Stefan Hajnoczi wrote: > On Tue, Nov 05, 2019 at 07:11:05PM +0300, Denis Plotnikov wrote: >> It tests proper queue size settings for all available machine types. >> >> Signed-off-by: Denis Plotnikov <dplotni...@virtuozzo.com> >> --- >> tests/qemu-iotests/267 | 154 +++++++++++++++++++++++++++++++++++++ >> tests/qemu-iotests/267.out | 1 + >> tests/qemu-iotests/group | 1 + >> 3 files changed, 156 insertions(+) >> create mode 100755 tests/qemu-iotests/267 >> create mode 100644 tests/qemu-iotests/267.out > > The qemu-iotests maintainers might prefer for this to be at the > top-level in tests/ since it's not really an iotest, but the code itself > looks fine to me: > > Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
Good question. I don’t really mind, but it would be weird if started adding all kinds of “external” qemu tests (i.e. that use QMP) in the iotests directory. What is the alternative? Just putting it in a different directory doesn’t sound that appealing to me either, because it would still depend on the iotests infrastructure, right? (i.e., iotests.py and check) Max
signature.asc
Description: OpenPGP digital signature
