Re: [Qemu-devel] [PATCH v6 RESENT 0/2] sheepdog: add user-defined redundancy option

2013-11-21 Thread Stefan Hajnoczi
On Thu, Nov 21, 2013 at 01:22:53AM +0800, Liu Yuan wrote:
 On Mon, Nov 18, 2013 at 01:58:50PM +0800, Liu Yuan wrote:
  On Thu, Nov 07, 2013 at 10:56:36PM +0800, Liu Yuan wrote:
   v6:
- update comment typo
- remove is_number()
  
  Anyone pick this set up?
 
 Ping again. Without this two patches, qemu-img can't create|convert erasure 
 coded
 images for sheepdog. I hope this two can go with release of v1.7 if possible.

QEMU entered hard freeze on 6th of November:
http://qemu-project.org/Planning/1.7

This means only bug fixes should be merged.  Right now we're up to the
final -rc and the QEMU 1.7 release is scheduled for 22nd of November
(tomorrow).

The best I can do is review and apply it to the block-next branch for
qemu.git merge once QEMU 1.8 development begins next week.

Stefan



Re: [Qemu-devel] [PATCH v6 RESENT 0/2] sheepdog: add user-defined redundancy option

2013-11-21 Thread Stefan Hajnoczi
On Thu, Nov 07, 2013 at 10:56:36PM +0800, Liu Yuan wrote:
 v6:
  - update comment typo
  - remove is_number()
 
 v5:
  - use pstrcpy instead of strncpy
  - fix a segfalt for 'null' string option string
 
 v4:
  - fix do_sd_create that forgot to pass nr_copies
  - fix parse_redundancy dealing with replicated vdi
 
 v3:
  - rework is_numeric
 
 v2:
  - fix a typo in comment and commit log
 
  This patch set add one sheepdog specific option for qemu-img to control
  redundancy.
 
  This patch set is on top of Kevin's block tree.
 
 Liu Yuan (2):
   sheepdog: refactor do_sd_create()
   sheepdog: support user-defined redundancy option
 
  block/sheepdog.c  |  127 
 +
  include/block/block_int.h |1 +
  2 files changed, 105 insertions(+), 23 deletions(-)

Thanks, applied to my block-next tree:
https://github.com/stefanha/qemu/commits/block-next

Stefan



Re: [Qemu-devel] [PATCH v6 RESENT 0/2] sheepdog: add user-defined redundancy option

2013-11-21 Thread Liu Yuan
On Thu, Nov 21, 2013 at 09:23:59AM +0100, Stefan Hajnoczi wrote:
 On Thu, Nov 21, 2013 at 01:22:53AM +0800, Liu Yuan wrote:
  On Mon, Nov 18, 2013 at 01:58:50PM +0800, Liu Yuan wrote:
   On Thu, Nov 07, 2013 at 10:56:36PM +0800, Liu Yuan wrote:
v6:
 - update comment typo
 - remove is_number()
   
   Anyone pick this set up?
  
  Ping again. Without this two patches, qemu-img can't create|convert erasure 
  coded
  images for sheepdog. I hope this two can go with release of v1.7 if 
  possible.
 
 QEMU entered hard freeze on 6th of November:
 http://qemu-project.org/Planning/1.7
 
 This means only bug fixes should be merged.  Right now we're up to the
 final -rc and the QEMU 1.7 release is scheduled for 22nd of November
 (tomorrow).
 
 The best I can do is review and apply it to the block-next branch for
 qemu.git merge once QEMU 1.8 development begins next week.
 

Thanks for your explanation. It is helpful :)

Yuan



Re: [Qemu-devel] [PATCH v6 RESENT 0/2] sheepdog: add user-defined redundancy option

2013-11-20 Thread Liu Yuan
On Mon, Nov 18, 2013 at 01:58:50PM +0800, Liu Yuan wrote:
 On Thu, Nov 07, 2013 at 10:56:36PM +0800, Liu Yuan wrote:
  v6:
   - update comment typo
   - remove is_number()
 
 Anyone pick this set up?

Ping again. Without this two patches, qemu-img can't create|convert erasure 
coded
images for sheepdog. I hope this two can go with release of v1.7 if possible.

Thanks
Yuan



Re: [Qemu-devel] [PATCH v6 RESENT 0/2] sheepdog: add user-defined redundancy option

2013-11-17 Thread Liu Yuan
On Thu, Nov 07, 2013 at 10:56:36PM +0800, Liu Yuan wrote:
 v6:
  - update comment typo
  - remove is_number()

Anyone pick this set up?

Thanks
Yuan