[RFC PATCH v4 0/3] Sheepdog: distributed storage system for QEMU

2010-05-27 Thread MORITA Kazutaka
Hi all,

This patch adds a block driver for Sheepdog distributed storage
system.  Please consider for inclusion.

I applied comments for the 2nd patch (thanks Kevin!).  The rest
patches are not changed from the previous version.


Changes from v3 to v4 are:
 - fix error handling in bdrv_snapshot_goto.

Changes from v2 to v3 are:

 - add drv->bdrv_close() and drv->bdrv_open() before and after
   bdrv_snapshot_goto() call of the protocol.
 - address the review comments on the sheepdog driver code.

Changes from v1 to v2 are:

 - rebase onto git://repo.or.cz/qemu/kevin.git block
 - modify the sheepdog driver as a protocol driver
 - add new patch to call the snapshot handler of the protocol

Thanks,

Kazutaka


MORITA Kazutaka (3):
  close all the block drivers before the qemu process exits
  block: call the snapshot handlers of the protocol drivers
  block: add sheepdog driver for distributed storage support

 Makefile.objs|2 +-
 block.c  |   70 ++-
 block.h  |1 +
 block/sheepdog.c | 1835 ++
 vl.c |1 +
 5 files changed, 1890 insertions(+), 19 deletions(-)
 create mode 100644 block/sheepdog.c

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC PATCH v4 0/3] Sheepdog: distributed storage system for QEMU

2010-06-02 Thread Kevin Wolf
Am 28.05.2010 04:44, schrieb MORITA Kazutaka:
> Hi all,
> 
> This patch adds a block driver for Sheepdog distributed storage
> system.  Please consider for inclusion.

Hint for next time: You should remove the RFC from the subject line if
you think the patch is ready for inclusion. Otherwise I might miss this
and think you only want comments on it.

> MORITA Kazutaka (3):
>   close all the block drivers before the qemu process exits
>   block: call the snapshot handlers of the protocol drivers
>   block: add sheepdog driver for distributed storage support

Thanks, I have applied the first two patches to the block branch, they
look good to me. I'll send some comments for the third one (though it's
only coding style until now).

Kevin
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [RFC PATCH v4 0/3] Sheepdog: distributed storage system for QEMU

2010-06-03 Thread MORITA Kazutaka
At Wed, 02 Jun 2010 12:49:02 +0200,
Kevin Wolf wrote:
> 
> Am 28.05.2010 04:44, schrieb MORITA Kazutaka:
> > Hi all,
> > 
> > This patch adds a block driver for Sheepdog distributed storage
> > system.  Please consider for inclusion.
> 
> Hint for next time: You should remove the RFC from the subject line if
> you think the patch is ready for inclusion. Otherwise I might miss this
> and think you only want comments on it.
> 

Thanks for the advice. I'll do so the next time.

> > MORITA Kazutaka (3):
> >   close all the block drivers before the qemu process exits
> >   block: call the snapshot handlers of the protocol drivers
> >   block: add sheepdog driver for distributed storage support
> 
> Thanks, I have applied the first two patches to the block branch, they
> look good to me. I'll send some comments for the third one (though it's
> only coding style until now).
> 

Thanks a lot.

Kazutaka
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html