Re: [Qemu-devel] [PATCHv2 0/3] Better 'Force Unit Access' handling

2016-05-11 Thread Stefan Hajnoczi
On Tue, May 10, 2016 at 02:38:29PM +0200, Kevin Wolf wrote:
> Am 04.05.2016 um 00:39 hat Eric Blake geschrieben:
> > I noticed some inconsistencies in FUA handling while working
> > with NBD, then Kevin pointed out that my initial attempt wasn't
> > quite right for iscsi which also had problems, so this has
> > expanded into a series rather than a single patch.
> > 
> > I'm not sure if this is qemu-stable material at this point.
> > 
> > Depends on Kevin's block-next branch.
> > 
> > Also available as a tag at this location:
> > git fetch git://repo.or.cz/qemu/ericb.git nbd-fua-v2
> 
> Reviewed-by: Kevin Wolf 
> 
> Stefan, if you're happy with the series, I can merge it into block-next.
> 
> Kevin

Acked-by: Stefan Hajnoczi 


signature.asc
Description: PGP signature


Re: [Qemu-devel] [PATCHv2 0/3] Better 'Force Unit Access' handling

2016-05-11 Thread Kevin Wolf
Am 11.05.2016 um 03:34 hat Fam Zheng geschrieben:
> On Tue, 05/03 16:39, Eric Blake wrote:
> > I noticed some inconsistencies in FUA handling while working
> > with NBD, then Kevin pointed out that my initial attempt wasn't
> > quite right for iscsi which also had problems, so this has
> > expanded into a series rather than a single patch.
> > 
> > I'm not sure if this is qemu-stable material at this point.
> > 
> > Depends on Kevin's block-next branch.
> > 
> > Also available as a tag at this location:
> > git fetch git://repo.or.cz/qemu/ericb.git nbd-fua-v2
> > 
> > v1 was here:
> > https://lists.gnu.org/archive/html/qemu-devel/2016-04/msg04674.html
> > 
> > diffstat not worth posting, as the series is basically rewritten
> > 
> > Eric Blake (3):
> >   block: Make supported_write_flags a per-bds property
> >   block: Honor BDRV_REQ_FUA during write_zeroes
> >   nbd: Simplify client FUA handling
> 
> Reviewed-by: Fam Zheng 

Ah, sorry, forgot that you co-maintain this code now. So this is good
enough to merge the series. :-)

Thanks, applied to block-next.

Kevin



Re: [Qemu-devel] [PATCHv2 0/3] Better 'Force Unit Access' handling

2016-05-10 Thread Fam Zheng
On Tue, 05/03 16:39, Eric Blake wrote:
> I noticed some inconsistencies in FUA handling while working
> with NBD, then Kevin pointed out that my initial attempt wasn't
> quite right for iscsi which also had problems, so this has
> expanded into a series rather than a single patch.
> 
> I'm not sure if this is qemu-stable material at this point.
> 
> Depends on Kevin's block-next branch.
> 
> Also available as a tag at this location:
> git fetch git://repo.or.cz/qemu/ericb.git nbd-fua-v2
> 
> v1 was here:
> https://lists.gnu.org/archive/html/qemu-devel/2016-04/msg04674.html
> 
> diffstat not worth posting, as the series is basically rewritten
> 
> Eric Blake (3):
>   block: Make supported_write_flags a per-bds property
>   block: Honor BDRV_REQ_FUA during write_zeroes
>   nbd: Simplify client FUA handling

Reviewed-by: Fam Zheng 

> 
>  block/nbd-client.h|  2 +-
>  include/block/block_int.h | 11 +++
>  block/io.c| 39 ++-
>  block/iscsi.c | 11 ---
>  block/nbd-client.c| 11 +++
>  block/nbd.c   | 28 +++-
>  block/raw-posix.c |  1 +
>  block/raw_bsd.c   |  3 ++-
>  8 files changed, 59 insertions(+), 47 deletions(-)
> 
> -- 
> 2.5.5
> 
> 



Re: [Qemu-devel] [PATCHv2 0/3] Better 'Force Unit Access' handling

2016-05-10 Thread Kevin Wolf
Am 04.05.2016 um 00:39 hat Eric Blake geschrieben:
> I noticed some inconsistencies in FUA handling while working
> with NBD, then Kevin pointed out that my initial attempt wasn't
> quite right for iscsi which also had problems, so this has
> expanded into a series rather than a single patch.
> 
> I'm not sure if this is qemu-stable material at this point.
> 
> Depends on Kevin's block-next branch.
> 
> Also available as a tag at this location:
> git fetch git://repo.or.cz/qemu/ericb.git nbd-fua-v2

Reviewed-by: Kevin Wolf 

Stefan, if you're happy with the series, I can merge it into block-next.

Kevin



[Qemu-devel] [PATCHv2 0/3] Better 'Force Unit Access' handling

2016-05-03 Thread Eric Blake
I noticed some inconsistencies in FUA handling while working
with NBD, then Kevin pointed out that my initial attempt wasn't
quite right for iscsi which also had problems, so this has
expanded into a series rather than a single patch.

I'm not sure if this is qemu-stable material at this point.

Depends on Kevin's block-next branch.

Also available as a tag at this location:
git fetch git://repo.or.cz/qemu/ericb.git nbd-fua-v2

v1 was here:
https://lists.gnu.org/archive/html/qemu-devel/2016-04/msg04674.html

diffstat not worth posting, as the series is basically rewritten

Eric Blake (3):
  block: Make supported_write_flags a per-bds property
  block: Honor BDRV_REQ_FUA during write_zeroes
  nbd: Simplify client FUA handling

 block/nbd-client.h|  2 +-
 include/block/block_int.h | 11 +++
 block/io.c| 39 ++-
 block/iscsi.c | 11 ---
 block/nbd-client.c| 11 +++
 block/nbd.c   | 28 +++-
 block/raw-posix.c |  1 +
 block/raw_bsd.c   |  3 ++-
 8 files changed, 59 insertions(+), 47 deletions(-)

-- 
2.5.5