qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 2/3] block: test blk_aio_flush() with blk->root


From: Stefan Hajnoczi
Subject: Re: [Qemu-devel] [PATCH 2/3] block: test blk_aio_flush() with blk->root == NULL
Date: Mon, 12 Feb 2018 14:10:18 +0000
User-agent: Mutt/1.9.2 (2017-12-15)

On Fri, Feb 09, 2018 at 09:23:53AM -0600, Eric Blake wrote:
> On 02/08/2018 11:18 AM, Stefan Hajnoczi wrote:
> > From: Kevin Wolf <address@hidden>
> > 
> > This patch adds test cases for the scenario where blk_aio_flush() is
> > called on a BlockBackend with no root.  Calling drain afterwards should
> > complete the requests with -ENOMEDIUM.
> > 
> > Signed-off-by: Kevin Wolf <address@hidden>
> > Signed-off-by: Stefan Hajnoczi <address@hidden>
> > ---
> >   tests/Makefile.include     |  2 ++
> >   tests/test-block-backend.c | 82 
> > ++++++++++++++++++++++++++++++++++++++++++++++
> >   2 files changed, 84 insertions(+)
> >   create mode 100644 tests/test-block-backend.c
> > 
> 
> > +++ b/tests/test-block-backend.c
> > @@ -0,0 +1,82 @@
> > +/*
> > + * BlockBackend tests
> > + *
> > + * Copyright (c) 2017 Kevin Wolf <address@hidden>
> 
> Want to include 2018?  Using a redhat.com address but a personal copyright?
> (Then again, Red Hat has some pretty cool policies on what it permits for
> code written on an employee's own time).
> 
> As long as Kevin is happy with the final copyright line that goes into the
> commit:
> 
> Reviewed-by: Eric Blake <address@hidden>

This is how I got the test case from Kevin, I didn't change the
copyright line.

Stefan

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]