[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p |
Date: |
Thu, 15 Sep 2016 15:04:30 +0200 |
On Thu, 15 Sep 2016 14:45:06 +0200
Greg Kurz <address@hidden> wrote:
> On Thu, 15 Sep 2016 10:41:38 +0200
> Cornelia Huck <address@hidden> wrote:
>
> > On Fri, 09 Sep 2016 10:04:37 +0200
> > Greg Kurz <address@hidden> 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 <address@hidden>
- [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p, Greg Kurz, 2016/09/09
- [Qemu-devel] [PATCH 1/3] tests: virtio-9p: introduce start/stop functions, Greg Kurz, 2016/09/09
- [Qemu-devel] [PATCH 2/3] tests: virtio-9p: add basic configuration test, Greg Kurz, 2016/09/09
- [Qemu-devel] [PATCH 3/3] tests: virtio-9p: add basic transaction test, Greg Kurz, 2016/09/09
- Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p, no-reply, 2016/09/09
- Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p, Greg Kurz, 2016/09/15
- Re: [Qemu-devel] [PATCH 0/3] tests: more test cases for virtio-9p, Cornelia Huck, 2016/09/15