On 04/06/2020 15.42, Thomas Huth wrote: > On 30/05/2020 19.14, Coiby Xu wrote: >> This test case has the same tests as tests/virtio-blk-test.c except for >> tests have block_resize. Since vhost-user server can only server one >> client one time, two instances of qemu-storage-daemon are launched >> for the hotplug test. >> >> In order to not block scripts/tap-driver.pl, vhost-user-blk-server will >> send "quit" command to qemu-storage-daemon's QMP monitor. So a function >> is added to libqtest.c to establish socket connection with socket >> server. >> >> Signed-off-by: Coiby Xu <coiby...@gmail.com> >> --- >> tests/Makefile.include | 3 +- >> tests/qtest/Makefile.include | 2 + >> tests/qtest/libqos/vhost-user-blk.c | 126 +++++ >> tests/qtest/libqos/vhost-user-blk.h | 44 ++ >> tests/qtest/libqtest.c | 44 +- >> tests/qtest/libqtest.h | 38 ++ >> tests/qtest/vhost-user-blk-test.c | 741 ++++++++++++++++++++++++++++ >> 7 files changed, 966 insertions(+), 32 deletions(-) >> create mode 100644 tests/qtest/libqos/vhost-user-blk.c >> create mode 100644 tests/qtest/libqos/vhost-user-blk.h >> create mode 100644 tests/qtest/vhost-user-blk-test.c > [...] >> qos-test-obj-y += tests/qtest/virtio-scsi-test.o >> diff --git a/tests/qtest/libqos/vhost-user-blk.c >> b/tests/qtest/libqos/vhost-user-blk.c >> new file mode 100644 >> index 0000000000..ec46b7ddb4 >> --- /dev/null >> +++ b/tests/qtest/libqos/vhost-user-blk.c >> @@ -0,0 +1,126 @@ >> +/* >> + * libqos driver framework >> + * >> + * Copyright (c) 2018 Emanuele Giuseppe Esposito >> <e.emanuelegiuse...@gmail.com> > > Don't you want to add a remark here for you, too? > >> + * This library is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU Lesser General Public >> + * License version 2 as published by the Free Software Foundation. > > Could you please change "version 2" into "verion 2.1" ? There never was > a "version 2" of the LGPL, only version 2.1 (or 3.x).
I meant, there never was a version 2 of the "Lesser GPL". Version 2.0 of the license was still called "Library GPL" instead. Thomas
