* [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
@ 2016-09-09 8:04 Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 1/3] tests: virtio-9p: introduce start/stop functions Greg Kurz
` (5 more replies)
0 siblings, 6 replies; 11+ messages in thread
From: Greg Kurz @ 2016-09-09 8:04 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Greg Kurz, Aneesh Kumar K.V
As with other virtio-* qtests, PC platform is assumed.
---
Greg Kurz (3):
tests: virtio-9p: introduce start/stop functions
tests: virtio-9p: add basic configuration test
tests: virtio-9p: add basic transaction test
tests/Makefile.include | 2 -
tests/virtio-9p-test.c | 179 ++++++++++++++++++++++++++++++++++++++++++++----
2 files changed, 164 insertions(+), 17 deletions(-)
--
Greg
^ permalink raw reply [flat|nested] 11+ messages in thread
* [Qemu-devel] [PATCH 1/3] tests: virtio-9p: introduce start/stop functions
2016-09-09 8:04 [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p Greg Kurz
@ 2016-09-09 8:04 ` Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test Greg Kurz
` (4 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Greg Kurz @ 2016-09-09 8:04 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Greg Kurz, Aneesh Kumar K.V
First step to be able to run several functional steps.
Signed-off-by: Greg Kurz <groug@kaod.org>
---
tests/virtio-9p-test.c | 37 ++++++++++++++++++++++---------------
1 file changed, 22 insertions(+), 15 deletions(-)
diff --git a/tests/virtio-9p-test.c b/tests/virtio-9p-test.c
index 1e39335a7945..ed53f21bc31f 100644
--- a/tests/virtio-9p-test.c
+++ b/tests/virtio-9p-test.c
@@ -11,33 +11,40 @@
#include "libqtest.h"
#include "qemu-common.h"
-/* Tests only initialization so far. TODO: Replace with functional tests */
-static void pci_nop(void)
-{
-}
+static char *test_share;
-static char test_share[] = "/tmp/qtest.XXXXXX";
-
-int main(int argc, char **argv)
+static void qvirtio_9p_start(void)
{
char *args;
- int ret;
- g_test_init(&argc, &argv, NULL);
- qtest_add_func("/virtio/9p/pci/nop", pci_nop);
-
- g_assert(mkdtemp(test_share));
+ test_share = g_strdup("/tmp/qtest.XXXXXX");
+ g_assert_nonnull(mkdtemp(test_share));
args = g_strdup_printf("-fsdev local,id=fsdev0,security_model=none,path=%s "
"-device virtio-9p-pci,fsdev=fsdev0,mount_tag=qtest",
test_share);
+
qtest_start(args);
g_free(args);
+}
- ret = g_test_run();
-
+static void qvirtio_9p_stop(void)
+{
qtest_end();
rmdir(test_share);
+ g_free(test_share);
+}
+
+static void pci_nop(void)
+{
+ qvirtio_9p_start();
+ qvirtio_9p_stop();
+}
+
+int main(int argc, char **argv)
+{
+ g_test_init(&argc, &argv, NULL);
+ qtest_add_func("/virtio/9p/pci/nop", pci_nop);
- return ret;
+ return g_test_run();
}
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test
2016-09-09 8:04 [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 1/3] tests: virtio-9p: introduce start/stop functions Greg Kurz
@ 2016-09-09 8:04 ` Greg Kurz
2016-09-15 8:40 ` Cornelia Huck
2016-09-09 8:04 ` [Qemu-devel] [PATCH 3/3] tests: virtio-9p: add basic transaction test Greg Kurz
` (3 subsequent siblings)
5 siblings, 1 reply; 11+ messages in thread
From: Greg Kurz @ 2016-09-09 8:04 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Greg Kurz, Aneesh Kumar K.V
This adds PCI init code and a basic test that checks the device config
matches what is passed on the command line.
Signed-off-by: Greg Kurz <groug@kaod.org>
---
tests/Makefile.include | 2 +
tests/virtio-9p-test.c | 82 +++++++++++++++++++++++++++++++++++++++++++++++-
2 files changed, 81 insertions(+), 3 deletions(-)
diff --git a/tests/Makefile.include b/tests/Makefile.include
index 03382b5fe7b8..a9dce206fbf6 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -600,7 +600,7 @@ tests/virtio-blk-test$(EXESUF): tests/virtio-blk-test.o $(libqos-virtio-obj-y)
tests/virtio-net-test$(EXESUF): tests/virtio-net-test.o $(libqos-pc-obj-y) $(libqos-virtio-obj-y)
tests/virtio-rng-test$(EXESUF): tests/virtio-rng-test.o $(libqos-pc-obj-y)
tests/virtio-scsi-test$(EXESUF): tests/virtio-scsi-test.o $(libqos-virtio-obj-y)
-tests/virtio-9p-test$(EXESUF): tests/virtio-9p-test.o
+tests/virtio-9p-test$(EXESUF): tests/virtio-9p-test.o $(libqos-virtio-obj-y)
tests/virtio-serial-test$(EXESUF): tests/virtio-serial-test.o
tests/virtio-console-test$(EXESUF): tests/virtio-console-test.o
tests/tpci200-test$(EXESUF): tests/tpci200-test.o
diff --git a/tests/virtio-9p-test.c b/tests/virtio-9p-test.c
index ed53f21bc31f..d7d508481bc0 100644
--- a/tests/virtio-9p-test.c
+++ b/tests/virtio-9p-test.c
@@ -10,7 +10,15 @@
#include "qemu/osdep.h"
#include "libqtest.h"
#include "qemu-common.h"
+#include "libqos/pci-pc.h"
+#include "libqos/virtio.h"
+#include "libqos/virtio-pci.h"
+#include "libqos/malloc.h"
+#include "libqos/malloc-pc.h"
+#include "standard-headers/linux/virtio_ids.h"
+#include "standard-headers/linux/virtio_pci.h"
+static const char mount_tag[] = "qtest";
static char *test_share;
static void qvirtio_9p_start(void)
@@ -21,8 +29,8 @@ static void qvirtio_9p_start(void)
g_assert_nonnull(mkdtemp(test_share));
args = g_strdup_printf("-fsdev local,id=fsdev0,security_model=none,path=%s "
- "-device virtio-9p-pci,fsdev=fsdev0,mount_tag=qtest",
- test_share);
+ "-device virtio-9p-pci,fsdev=fsdev0,mount_tag=%s",
+ test_share, mount_tag);
qtest_start(args);
g_free(args);
@@ -41,10 +49,80 @@ static void pci_nop(void)
qvirtio_9p_stop();
}
+typedef struct {
+ QVirtioDevice *dev;
+ QGuestAllocator *alloc;
+ QPCIBus *bus;
+ QVirtQueue *vq;
+} QVirtIO9P;
+
+static QVirtIO9P *qvirtio_9p_pci_init(void)
+{
+ QVirtIO9P *v9p;
+ QVirtioPCIDevice *dev;
+
+ v9p = g_new0(QVirtIO9P, 1);
+ v9p->alloc = pc_alloc_init();
+ v9p->bus = qpci_init_pc();
+
+ dev = qvirtio_pci_device_find(v9p->bus, VIRTIO_ID_9P);
+ g_assert_nonnull(dev);
+ g_assert_cmphex(dev->vdev.device_type, ==, VIRTIO_ID_9P);
+ v9p->dev = (QVirtioDevice *) dev;
+
+ qvirtio_pci_device_enable(dev);
+ qvirtio_reset(&qvirtio_pci, v9p->dev);
+ qvirtio_set_acknowledge(&qvirtio_pci, v9p->dev);
+ qvirtio_set_driver(&qvirtio_pci, v9p->dev);
+
+ v9p->vq = qvirtqueue_setup(&qvirtio_pci, v9p->dev, v9p->alloc, 0);
+ return v9p;
+}
+
+static void qvirtio_9p_pci_free(QVirtIO9P *v9p)
+{
+ qvirtqueue_cleanup(&qvirtio_pci, v9p->vq, v9p->alloc);
+ pc_alloc_uninit(v9p->alloc);
+ qvirtio_pci_device_disable(container_of(v9p->dev, QVirtioPCIDevice, vdev));
+ g_free(v9p->dev);
+ qpci_free_pc(v9p->bus);
+ g_free(v9p);
+}
+
+static void pci_basic_config(void)
+{
+ QVirtIO9P *v9p;
+ void *addr;
+ size_t tag_len;
+ char* tag;
+ int i;
+
+ qvirtio_9p_start();
+ v9p = qvirtio_9p_pci_init();
+
+ addr = ((QVirtioPCIDevice *) v9p->dev)->addr + VIRTIO_PCI_CONFIG_OFF(false);
+ tag_len = qvirtio_config_readw(&qvirtio_pci, v9p->dev,
+ (uint64_t)(uintptr_t)addr);
+ g_assert_cmpint(tag_len, ==, strlen(mount_tag));
+ addr += sizeof(uint16_t);
+
+ tag = g_malloc(tag_len);
+ for (i = 0; i < tag_len; i++) {
+ tag[i] = qvirtio_config_readb(&qvirtio_pci, v9p->dev,
+ (uint64_t)(uintptr_t)addr + i);
+ }
+ g_assert_cmpmem(tag, tag_len, mount_tag, tag_len);
+ g_free(tag);
+
+ qvirtio_9p_pci_free(v9p);
+ qvirtio_9p_stop();
+}
+
int main(int argc, char **argv)
{
g_test_init(&argc, &argv, NULL);
qtest_add_func("/virtio/9p/pci/nop", pci_nop);
+ qtest_add_func("/virtio/9p/pci/basic/configuration", pci_basic_config);
return g_test_run();
}
^ permalink raw reply related [flat|nested] 11+ messages in thread
* [Qemu-devel] [PATCH 3/3] tests: virtio-9p: add basic transaction test
2016-09-09 8:04 [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 1/3] tests: virtio-9p: introduce start/stop functions Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test Greg Kurz
@ 2016-09-09 8:04 ` Greg Kurz
2016-09-09 8:10 ` [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p no-reply
` (2 subsequent siblings)
5 siblings, 0 replies; 11+ messages in thread
From: Greg Kurz @ 2016-09-09 8:04 UTC (permalink / raw)
To: qemu-devel; +Cc: Michael S. Tsirkin, Greg Kurz, Aneesh Kumar K.V
This adds a simple test to validate the device is functional: it transmits
a request with type Terror, which is not used by the 9P protocol [1], and
expects QEMU to return a reply with type Rerror and the "Operation not
supported" error string.
[1] http://lxr.free-electrons.com/source/include/net/9p/9p.h#L121
Signed-off-by: Greg Kurz <groug@kaod.org>
---
tests/virtio-9p-test.c | 62 ++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 62 insertions(+)
diff --git a/tests/virtio-9p-test.c b/tests/virtio-9p-test.c
index d7d508481bc0..d13a30cf0dee 100644
--- a/tests/virtio-9p-test.c
+++ b/tests/virtio-9p-test.c
@@ -8,6 +8,7 @@
*/
#include "qemu/osdep.h"
+#include <glib/gprintf.h>
#include "libqtest.h"
#include "qemu-common.h"
#include "libqos/pci-pc.h"
@@ -17,6 +18,9 @@
#include "libqos/malloc-pc.h"
#include "standard-headers/linux/virtio_ids.h"
#include "standard-headers/linux/virtio_pci.h"
+#include "hw/9pfs/9p.h"
+
+#define QVIRTIO_9P_TIMEOUT_US (1 * 1000 * 1000)
static const char mount_tag[] = "qtest";
static char *test_share;
@@ -118,11 +122,69 @@ static void pci_basic_config(void)
qvirtio_9p_stop();
}
+typedef struct VirtIO9PHdr {
+ uint32_t size;
+ uint8_t id;
+ uint16_t tag;
+} QEMU_PACKED VirtIO9PHdr;
+
+typedef struct VirtIO9PMsgRError {
+ uint16_t error_len;
+ char error[0];
+} QEMU_PACKED VirtIO9PMsgRError;
+
+#define P9_MAX_SIZE 8192
+
+static void pci_basic_transaction(void)
+{
+ QVirtIO9P *v9p;
+ VirtIO9PHdr hdr;
+ VirtIO9PMsgRError *resp;
+ uint64_t req_addr, resp_addr;
+ uint32_t free_head;
+ char *expected_error = strerror(ENOTSUP);
+
+ qvirtio_9p_start();
+ v9p = qvirtio_9p_pci_init();
+
+ hdr.size = sizeof(hdr);
+ hdr.id = P9_TERROR;
+ hdr.tag = 12345;
+
+ req_addr = guest_alloc(v9p->alloc, hdr.size);
+ memwrite(req_addr, &hdr, sizeof(hdr));
+ free_head = qvirtqueue_add(v9p->vq, req_addr, hdr.size, false, true);
+
+ resp_addr = guest_alloc(v9p->alloc, P9_MAX_SIZE);
+ qvirtqueue_add(v9p->vq, resp_addr, P9_MAX_SIZE, true, false);
+
+ qvirtqueue_kick(&qvirtio_pci, v9p->dev, v9p->vq, free_head);
+ guest_free(v9p->alloc, req_addr);
+ qvirtio_wait_queue_isr(&qvirtio_pci, v9p->dev, v9p->vq,
+ QVIRTIO_9P_TIMEOUT_US);
+
+ memread(resp_addr, &hdr, sizeof(hdr));
+ g_assert_cmpint(hdr.size, <, (uint32_t) P9_MAX_SIZE);
+ g_assert_cmpint(hdr.id, ==, (uint8_t) P9_RERROR);
+ g_assert_cmpint(hdr.tag, ==, (uint16_t) 12345);
+
+ resp = g_malloc(hdr.size);
+ memread(resp_addr + sizeof(hdr), resp, hdr.size - sizeof(hdr));
+ guest_free(v9p->alloc, resp_addr);
+ g_assert_cmpmem(resp->error, resp->error_len, expected_error,
+ strlen(expected_error));
+ g_free(resp);
+
+ qvirtio_9p_pci_free(v9p);
+ qvirtio_9p_stop();
+}
+
int main(int argc, char **argv)
{
g_test_init(&argc, &argv, NULL);
qtest_add_func("/virtio/9p/pci/nop", pci_nop);
qtest_add_func("/virtio/9p/pci/basic/configuration", pci_basic_config);
+ qtest_add_func("/virtio/9p/pci/basic/transaction", pci_basic_transaction);
return g_test_run();
}
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
2016-09-09 8:04 [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p Greg Kurz
` (2 preceding siblings ...)
2016-09-09 8:04 ` [Qemu-devel] [PATCH 3/3] tests: virtio-9p: add basic transaction test Greg Kurz
@ 2016-09-09 8:10 ` no-reply
2016-09-15 7:45 ` Greg Kurz
2016-09-15 8:41 ` Cornelia Huck
5 siblings, 0 replies; 11+ messages in thread
From: no-reply @ 2016-09-09 8:10 UTC (permalink / raw)
To: groug; +Cc: famz, qemu-devel, aneesh.kumar, mst
Hi,
Your series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 147340827736.6462.8546871953640244651.stgit@bahia
Subject: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0
# Useful git options
git config --local diff.renamelimit 0
git config --local diff.renames True
commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
echo "Checking PATCH $n/$total: $(git show --no-patch --format=%s $c)..."
if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
failed=1
echo
fi
n=$((n+1))
done
exit $failed
=== TEST SCRIPT END ===
Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
* [new tag] patchew/147340827736.6462.8546871953640244651.stgit@bahia -> patchew/147340827736.6462.8546871953640244651.stgit@bahia
Switched to a new branch 'test'
04c0f1e tests: virtio-9p: add basic transaction test
5194780 tests: virtio-9p: add basic configuration test
e762ebb tests: virtio-9p: introduce start/stop functions
=== OUTPUT BEGIN ===
Checking PATCH 1/3: tests: virtio-9p: introduce start/stop functions...
Checking PATCH 2/3: tests: virtio-9p: add basic configuration test...
ERROR: "foo* bar" should be "foo *bar"
#105: FILE: tests/virtio-9p-test.c:97:
+ char* tag;
total: 1 errors, 0 warnings, 113 lines checked
Your patch has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 3/3: tests: virtio-9p: add basic transaction test...
=== OUTPUT END ===
Test command exited with code: 1
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@freelists.org
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
2016-09-09 8:04 [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p Greg Kurz
` (3 preceding siblings ...)
2016-09-09 8:10 ` [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p no-reply
@ 2016-09-15 7:45 ` Greg Kurz
2016-09-15 8:41 ` Cornelia Huck
5 siblings, 0 replies; 11+ messages in thread
From: Greg Kurz @ 2016-09-15 7:45 UTC (permalink / raw)
To: qemu-devel
Cc: Aneesh Kumar K.V, Michael S. Tsirkin, Stefan Hajnoczi, Cornelia Huck
Hi,
Even if I am maintainer for 9P stuff, I'm not sure I can send a pull request
for patches without at least an Acked-by... :-\
I plan to add true 9P functional tests in the future, but these are more
about the virtio device actually.
Any chances some virtio people can have a look and ack or nack ?
Cheers.
--
Greg
On Fri, 09 Sep 2016 10:04:37 +0200
Greg Kurz <groug@kaod.org> wrote:
> As with other virtio-* qtests, PC platform is assumed.
>
> ---
>
> Greg Kurz (3):
> tests: virtio-9p: introduce start/stop functions
> tests: virtio-9p: add basic configuration test
> tests: virtio-9p: add basic transaction test
>
>
> tests/Makefile.include | 2 -
> tests/virtio-9p-test.c | 179 ++++++++++++++++++++++++++++++++++++++++++++----
> 2 files changed, 164 insertions(+), 17 deletions(-)
>
> --
> Greg
>
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test
2016-09-09 8:04 ` [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test Greg Kurz
@ 2016-09-15 8:40 ` Cornelia Huck
2016-09-15 12:09 ` Greg Kurz
0 siblings, 1 reply; 11+ messages in thread
From: Cornelia Huck @ 2016-09-15 8:40 UTC (permalink / raw)
To: Greg Kurz; +Cc: qemu-devel, Aneesh Kumar K.V, Michael S. Tsirkin
On Fri, 09 Sep 2016 10:04:51 +0200
Greg Kurz <groug@kaod.org> wrote:
> This adds PCI init code and a basic test that checks the device config
> matches what is passed on the command line.
>
> Signed-off-by: Greg Kurz <groug@kaod.org>
> ---
> tests/Makefile.include | 2 +
> tests/virtio-9p-test.c | 82 +++++++++++++++++++++++++++++++++++++++++++++++-
> 2 files changed, 81 insertions(+), 3 deletions(-)
> +static const char mount_tag[] = "qtest";
> static char *test_share;
>
> static void qvirtio_9p_start(void)
> @@ -21,8 +29,8 @@ static void qvirtio_9p_start(void)
> g_assert_nonnull(mkdtemp(test_share));
>
> args = g_strdup_printf("-fsdev local,id=fsdev0,security_model=none,path=%s "
> - "-device virtio-9p-pci,fsdev=fsdev0,mount_tag=qtest",
> - test_share);
> + "-device virtio-9p-pci,fsdev=fsdev0,mount_tag=%s",
> + test_share, mount_tag);
>
> qtest_start(args);
> g_free(args);
Maybe move this change to patch 1?
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
2016-09-09 8:04 [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p Greg Kurz
` (4 preceding siblings ...)
2016-09-15 7:45 ` Greg Kurz
@ 2016-09-15 8:41 ` Cornelia Huck
2016-09-15 12:45 ` Greg Kurz
5 siblings, 1 reply; 11+ messages in thread
From: Cornelia Huck @ 2016-09-15 8:41 UTC (permalink / raw)
To: Greg Kurz; +Cc: qemu-devel, Aneesh Kumar K.V, Michael S. Tsirkin
On Fri, 09 Sep 2016 10:04:37 +0200
Greg Kurz <groug@kaod.org> wrote:
> As with other virtio-* qtests, PC platform is assumed.
>
> ---
>
> Greg Kurz (3):
> tests: virtio-9p: introduce start/stop functions
> tests: virtio-9p: add basic configuration test
> tests: virtio-9p: add basic transaction test
>
>
> tests/Makefile.include | 2 -
> tests/virtio-9p-test.c | 179 ++++++++++++++++++++++++++++++++++++++++++++----
> 2 files changed, 164 insertions(+), 17 deletions(-)
>
I took a quick look and nothing bad jumped out at me, but that is
probably not saying much :/
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test
2016-09-15 8:40 ` Cornelia Huck
@ 2016-09-15 12:09 ` Greg Kurz
0 siblings, 0 replies; 11+ messages in thread
From: Greg Kurz @ 2016-09-15 12:09 UTC (permalink / raw)
To: Cornelia Huck; +Cc: qemu-devel, Aneesh Kumar K.V, Michael S. Tsirkin
On Thu, 15 Sep 2016 10:40:45 +0200
Cornelia Huck <cornelia.huck@de.ibm.com> wrote:
> On Fri, 09 Sep 2016 10:04:51 +0200
> Greg Kurz <groug@kaod.org> wrote:
>
> > This adds PCI init code and a basic test that checks the device config
> > matches what is passed on the command line.
> >
> > Signed-off-by: Greg Kurz <groug@kaod.org>
> > ---
> > tests/Makefile.include | 2 +
> > tests/virtio-9p-test.c | 82 +++++++++++++++++++++++++++++++++++++++++++++++-
> > 2 files changed, 81 insertions(+), 3 deletions(-)
>
> > +static const char mount_tag[] = "qtest";
> > static char *test_share;
> >
> > static void qvirtio_9p_start(void)
> > @@ -21,8 +29,8 @@ static void qvirtio_9p_start(void)
> > g_assert_nonnull(mkdtemp(test_share));
> >
> > args = g_strdup_printf("-fsdev local,id=fsdev0,security_model=none,path=%s "
> > - "-device virtio-9p-pci,fsdev=fsdev0,mount_tag=qtest",
> > - test_share);
> > + "-device virtio-9p-pci,fsdev=fsdev0,mount_tag=%s",
> > + test_share, mount_tag);
> >
> > qtest_start(args);
> > g_free(args);
>
> Maybe move this change to patch 1?
>
It isn't strictly needed in patch 1 since the mount tag has only one user there,
but on the other hand, it isn't related to the purpose of patch 2 either. So I
guess your suggestion makes sense: I'll move this change.
Thanks.
--
Greg
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
2016-09-15 8:41 ` Cornelia Huck
@ 2016-09-15 12:45 ` Greg Kurz
2016-09-15 13:04 ` Cornelia Huck
0 siblings, 1 reply; 11+ messages in thread
From: Greg Kurz @ 2016-09-15 12:45 UTC (permalink / raw)
To: Cornelia Huck; +Cc: qemu-devel, Aneesh Kumar K.V, Michael S. Tsirkin
On Thu, 15 Sep 2016 10:41:38 +0200
Cornelia Huck <cornelia.huck@de.ibm.com> wrote:
> On Fri, 09 Sep 2016 10:04:37 +0200
> Greg Kurz <groug@kaod.org> wrote:
>
> > As with other virtio-* qtests, PC platform is assumed.
> >
> > ---
> >
> > Greg Kurz (3):
> > tests: virtio-9p: introduce start/stop functions
> > tests: virtio-9p: add basic configuration test
> > tests: virtio-9p: add basic transaction test
> >
> >
> > tests/Makefile.include | 2 -
> > tests/virtio-9p-test.c | 179 ++++++++++++++++++++++++++++++++++++++++++++----
> > 2 files changed, 164 insertions(+), 17 deletions(-)
> >
>
> I took a quick look and nothing bad jumped out at me, but that is
> probably not saying much :/
>
Not much is a lot more than nothing, as it is usually the case with 9pfs
patches that don't fix security issues or deprecated glibc calls ;)
Can this "nothing bad jumped out" be formalized into an Acked-by ?
Thanks.
--
Greg
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
2016-09-15 12:45 ` Greg Kurz
@ 2016-09-15 13:04 ` Cornelia Huck
0 siblings, 0 replies; 11+ messages in thread
From: Cornelia Huck @ 2016-09-15 13:04 UTC (permalink / raw)
To: Greg Kurz; +Cc: qemu-devel, Aneesh Kumar K.V, Michael S. Tsirkin
On Thu, 15 Sep 2016 14:45:06 +0200
Greg Kurz <groug@kaod.org> wrote:
> On Thu, 15 Sep 2016 10:41:38 +0200
> Cornelia Huck <cornelia.huck@de.ibm.com> wrote:
>
> > On Fri, 09 Sep 2016 10:04:37 +0200
> > Greg Kurz <groug@kaod.org> wrote:
> >
> > > As with other virtio-* qtests, PC platform is assumed.
> > >
> > > ---
> > >
> > > Greg Kurz (3):
> > > tests: virtio-9p: introduce start/stop functions
> > > tests: virtio-9p: add basic configuration test
> > > tests: virtio-9p: add basic transaction test
> > >
> > >
> > > tests/Makefile.include | 2 -
> > > tests/virtio-9p-test.c | 179 ++++++++++++++++++++++++++++++++++++++++++++----
> > > 2 files changed, 164 insertions(+), 17 deletions(-)
> > >
> >
> > I took a quick look and nothing bad jumped out at me, but that is
> > probably not saying much :/
> >
>
> Not much is a lot more than nothing, as it is usually the case with 9pfs
> patches that don't fix security issues or deprecated glibc calls ;)
>
> Can this "nothing bad jumped out" be formalized into an Acked-by ?
Feel free:
Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com>
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2016-09-15 13:04 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-09 8:04 [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 1/3] tests: virtio-9p: introduce start/stop functions Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test Greg Kurz
2016-09-15 8:40 ` Cornelia Huck
2016-09-15 12:09 ` Greg Kurz
2016-09-09 8:04 ` [Qemu-devel] [PATCH 3/3] tests: virtio-9p: add basic transaction test Greg Kurz
2016-09-09 8:10 ` [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p no-reply
2016-09-15 7:45 ` Greg Kurz
2016-09-15 8:41 ` Cornelia Huck
2016-09-15 12:45 ` Greg Kurz
2016-09-15 13:04 ` Cornelia Huck
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.