At Mon, 24 May 2010 14:16:32 -0500,
Anthony Liguori wrote:
>
> On 05/24/2010 06:56 AM, Avi Kivity wrote:
> > On 05/24/2010 02:42 PM, MORITA Kazutaka wrote:
> >>
> >>> The server would be local and talk over a unix domain socket, perhaps
> >>> anony
At Tue, 25 May 2010 15:43:17 +0200,
Kevin Wolf wrote:
>
> Am 24.05.2010 08:34, schrieb MORITA Kazutaka:
> > At Fri, 21 May 2010 18:57:36 +0200,
> > Kevin Wolf wrote:
> >>
> >> Am 20.05.2010 07:36, schrieb MORITA Kazutaka:
> >>> +
> >>>
At Tue, 25 May 2010 10:12:53 -0700 (PDT),
Sage Weil wrote:
>
> On Tue, 25 May 2010, Avi Kivity wrote:
> > > What's the reason for not having these drivers upstream? Do we gain
> > > anything by hiding them from our users and requiring them to install the
> > > drivers separately from somewhere els
() call of the protocol. It is
because the contents of the block driver state may need to be changed
after loading vmstate.
Signed-off-by: MORITA Kazutaka
---
block.c | 61 +++--
1 files changed, 43 insertions(+), 18 deletions(-)
diff
fy 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 driv
This patch calls the close handler of the block driver before the qemu
process exits.
This is necessary because the sheepdog block driver releases the lock
of VM images in the close handler.
Signed-off-by: MORITA Kazutaka
---
block.c |9 +
block.h |1 +
vl.c|1 +
3
/sheepdog/
Signed-off-by: MORITA Kazutaka
---
Makefile.objs|2 +-
block/sheepdog.c | 1835 ++
2 files changed, 1836 insertions(+), 1 deletions(-)
create mode 100644 block/sheepdog.c
diff --git a/Makefile.objs b/Makefile.objs
index
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
At Tue, 01 Jun 2010 09:58:04 -0500,
Thanks for your comments!
Chris Krumme wrote:
>
> On 05/27/2010 09:44 PM, MORITA Kazutaka wrote:
> > Sheepdog is a distributed storage system for QEMU. It provides highly
> > +
> > +static int connect_to_sdog(const char *addr)
&g
At Wed, 02 Jun 2010 15:55:42 +0200,
Kevin Wolf wrote:
>
> Am 28.05.2010 04:44, schrieb MORITA Kazutaka:
> > Sheepdog is a distributed storage system for QEMU. It provides highly
> > available block level storage volumes to VMs like Amazon EBS. This
> > patch adds a qemu b
At Fri, 04 Jun 2010 13:04:00 +0200,
Kevin Wolf wrote:
>
> Am 03.06.2010 18:23, schrieb MORITA Kazutaka:
> >>> +static void sd_aio_cancel(BlockDriverAIOCB *blockacb)
> >>> +{
> >>> + SheepdogAIOCB *acb = (SheepdogAIOCB *)blockacb;
> >>> +
&
/sheepdog/
Signed-off-by: MORITA Kazutaka
---
Changes from v4 to v5 are:
- address the comments to the sheepdog driver (Thanks Kevin, Chris!)
-- fix a coding style
-- fix aio_cancel handling
-- fix an overflow bug in coping hostname
-- add comments to the non-trivial functions
- remove
At Fri, 4 Jun 2010 16:35:59 -0300,
Miguel Di Ciurcio Filho wrote:
>
> Both bdrv_can_snapshot() and bdrv_has_snapshot() does not work as advertized.
>
> First issue: Their names implies different porpouses, but they do the same
> thing
> and have exactly the same code. Maybe copied and pasted an
At Tue, 15 Jun 2010 10:24:14 +0200,
Kevin Wolf wrote:
>
> Am 14.06.2010 21:48, schrieb MORITA Kazutaka:
> >> 3) qemu-io aio_read/write doesn't seem to work well with it. I only get
> >> the result of the AIO request when I exit qemu-io. This may be a qemu-io
>
posix-aio-compat sends a signal in aio operations, so we should
consider that fgets() could be interrupted here.
Signed-off-by: MORITA Kazutaka
---
cmd.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/cmd.c b/cmd.c
index 2336334..460df92 100644
--- a/cmd.c
+++ b
command_loop(). Any other
handlers can be invoked when user input is idle.
Signed-off-by: MORITA Kazutaka
---
cmd.c | 53 +++--
1 files changed, 39 insertions(+), 14 deletions(-)
diff --git a/cmd.c b/cmd.c
index 460df92..2b66e24 100644
--- a
a fd
handler to get I/O completion.
Thanks,
Kazutaka
MORITA Kazutaka (2):
qemu-io: retry fgets() when errno is EINTR
qemu-io: check registered fds in command_loop()
cmd.c | 56 ++--
1 files changed, 42 insertions(+), 14 deletions(-)
At Wed, 16 Jun 2010 13:04:47 +0200,
Kevin Wolf wrote:
>
> Am 15.06.2010 19:53, schrieb MORITA Kazutaka:
> > posix-aio-compat sends a signal in aio operations, so we should
> > consider that fgets() could be interrupted here.
> >
> > Signed-off-by: MORITA Kazut
At Thu, 17 Jun 2010 18:18:18 +0100,
Jamie Lokier wrote:
>
> Kevin Wolf wrote:
> > Am 16.06.2010 18:52, schrieb MORITA Kazutaka:
> > > At Wed, 16 Jun 2010 13:04:47 +0200,
> > > Kevin Wolf wrote:
> > >>
> > >> Am 15.06.2010 19:53, schrieb MORIT
ddr)[nr / BITS_PER_LONG])) != 0;
> + return 1UL & (addr[nr / BITS_PER_LONG] >> (nr & (BITS_PER_LONG-1)));
> }
>
> The body is equivalent, but the argument is not: there's
> "volatile" in there. Why it is used for - I'm not sure.
>
> Signed-off-by: Michael Tokarev
Looks good. Thanks!
Acked-by: MORITA Kazutaka
This patchset fixes the Sheepodg AIO problems pointed out in:
http://lists.gnu.org/archive/html/qemu-devel/2011-02/msg02495.html
http://lists.gnu.org/archive/html/qemu-devel/2011-02/msg02474.html
Thanks,
Kazutaka
MORITA Kazutaka (3):
sheepdog: make send/recv operations non-blocking
We cannot access the buffer of the canceled I/O request because its
AIOCB callback is already called and the buffer is not valid.
Signed-off-by: MORITA Kazutaka
---
block/sheepdog.c | 12 ++--
1 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/block/sheepdog.c b/block
We can cancel I/O requests safely if they are not sent to the servers.
Signed-off-by: MORITA Kazutaka
---
block/sheepdog.c | 37 +
1 files changed, 37 insertions(+), 0 deletions(-)
diff --git a/block/sheepdog.c b/block/sheepdog.c
index cedf806..ed98701
This patch avoids retrying send/recv in AIO path when the sheepdog
connection is not ready for the operation.
Signed-off-by: MORITA Kazutaka
---
block/sheepdog.c | 417 +-
1 files changed, 289 insertions(+), 128 deletions(-)
diff --git a
This patch parses the input filename in sd_create(), and enables us
specifying a target server to create sheepdog images.
Signed-off-by: MORITA Kazutaka
---
block/sheepdog.c | 17 ++---
1 files changed, 14 insertions(+), 3 deletions(-)
diff --git a/block/sheepdog.c b/block
Signed-off-by: MORITA Kazutaka
---
qemu-doc.texi | 52
1 files changed, 52 insertions(+), 0 deletions(-)
diff --git a/qemu-doc.texi b/qemu-doc.texi
index 22a8663..86e017c 100644
--- a/qemu-doc.texi
+++ b/qemu-doc.texi
@@ -407,6 +407,7
At Mon, 21 Feb 2011 17:48:49 +0100,
Kevin Wolf wrote:
>
> Am 21.02.2011 17:31, schrieb Nicholas Thomas:
> > Hi again,
> >
> > Thanks for looking through the patches. I'm just going through and
> > making the suggested changes now. I've also got qemu-nbd and block/nbd.c
> > working over IPv6 :) -
calls qemu_aio_wait() in
command_loop(). Any other handlers can be invoked when user input is
idle.
Signed-off-by: MORITA Kazutaka
---
It seems that the QEMU aio implementation doesn't allow to call
qemu_aio_wait() in the aio handler, so the previous patch is broken.
This patch only checks
the close handler.
Signed-off-by: MORITA Kazutaka
---
qemu-img.c | 235 +++-
1 files changed, 184 insertions(+), 51 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index ea091f0..fe8a577 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -39,14
/sheepdog/
Signed-off-by: MORITA Kazutaka
---
I've addressed the comments and tested with qemu-iotests which is
hacked for Sheepdog.
This version changes an inode data format to support a snapshot tag
name, so to test this patch, please pull the latest sheepdog server
codes.
At Fri, 2 Jul 2010 19:14:59 +0200,
Kevin Wolf wrote:
>
> People think that their images are corrupted when in fact there are just some
> leaked clusters. Differentiating several error cases should make the messages
> more comprehensible.
>
> Signed-off-by: Kevin Wolf
> ---
> block.c| 10
WIN32 is not only the system which doesn't have TCP_CORK (e.g. OS X).
Signed-off-by: MORITA Kazutaka
---
Betts, I think this patch fix the compile error. Can you try this
one?
block/sheepdog.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/block/sheepdog.c b/
This patch calls the close handler of the block driver before the qemu
process exits.
This is necessary because the sheepdog block driver releases the lock
of VM images in the close handler.
Signed-off-by: MORITA Kazutaka
---
block.c | 11 +++
block.h |1 +
monitor.c |1
Thanks,
Kazutaka
[1] http://www.osrg.net/sheepdog/
[2] http://lists.nongnu.org/archive/html/qemu-devel/2009-10/msg01773.html
MORITA Kazutaka (2):
close all the block drivers before the qemu process exits
block: add sheepdog driver for distributed storage support
Makefile |2
/sheepdog/
Signed-off-by: MORITA Kazutaka
---
Makefile |2 +-
block.c |3 +-
block/sheepdog.c | 1828 ++
3 files changed, 1831 insertions(+), 2 deletions(-)
create mode 100644 block/sheepdog.c
diff --git a/Makefile b
At Thu, 13 May 2010 05:16:35 +0900,
MORITA Kazutaka wrote:
>
> On 2010/05/12 23:28, Avi Kivity wrote:
> > On 05/12/2010 01:46 PM, MORITA Kazutaka wrote:
> >> This patch calls the close handler of the block driver before the qemu
> >> process exits.
> >>
>
On 2010/05/12 23:28, Avi Kivity wrote:
> On 05/12/2010 01:46 PM, MORITA Kazutaka wrote:
>> This patch calls the close handler of the block driver before the qemu
>> process exits.
>>
>> This is necessary because the sheepdog block driver releases the lock
>>
On 2010/05/12 20:38, Kevin Wolf wrote:
> Am 12.05.2010 12:46, schrieb MORITA Kazutaka:
>> Hi all,
>>
>> This patch adds a block driver for Sheepdog distributed storage
>> system. Please consider for inclusion.
>>
>> Sheepdog is a distributed stora
On 2010/05/12 23:01, Christoph Hellwig wrote:
> On Wed, May 12, 2010 at 07:46:52PM +0900, MORITA Kazutaka wrote:
>> This patch calls the close handler of the block driver before the qemu
>> process exits.
>>
>> This is necessary because the sheepdog block driver releas
At Thu, 13 May 2010 04:46:46 +0900,
MORITA Kazutaka wrote:
>
> On 2010/05/12 20:38, Kevin Wolf wrote:
> > I'll have a closer look at your code later, but one thing I noticed is
> > that the new block driver is something in between a protocol and a
> > format driver (
This patch calls the close handler of the block driver before the qemu
process exits.
This is necessary because the sheepdog block driver releases the lock
of VM images in the close handler.
Signed-off-by: MORITA Kazutaka
---
block.c |9 +
block.h |1 +
vl.c|1 +
3
When snapshot handlers of the format driver is not defined, it is
better to call the ones of the protocol driver.
This enables us to implement snapshot support in the protocol driver.
Signed-off-by: MORITA Kazutaka
---
block.c | 48 ++--
1 files
reate -b sheepdog:vol1:1 -f sheepdog vol2
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 +-
bl
/sheepdog/
Signed-off-by: MORITA Kazutaka
---
Makefile.objs|2 +-
block/sheepdog.c | 1831 ++
2 files changed, 1832 insertions(+), 1 deletions(-)
create mode 100644 block/sheepdog.c
diff --git a/Makefile.objs b/Makefile.objs
index
At Fri, 14 May 2010 10:32:26 +0200,
Kevin Wolf wrote:
>
> Am 13.05.2010 16:03, schrieb MORITA Kazutaka:
> > To support snapshot in a protocol, I'd like to call the hander of the
> > protocol driver in the following functions in block.c:
> >
> > bdrv_snapsho
() call of the protocol. It is
because the contents of the block driver state may need to be changed
after loading vmstate.
Signed-off-by: MORITA Kazutaka
---
block.c | 61 +++--
1 files changed, 43 insertions(+), 18 deletions(-)
diff
This patch calls the close handler of the block driver before the qemu
process exits.
This is necessary because the sheepdog block driver releases the lock
of VM images in the close handler.
Signed-off-by: MORITA Kazutaka
---
block.c |9 +
block.h |1 +
vl.c|1 +
3
creation problem of the
protocol driver.
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|
/sheepdog/
Signed-off-by: MORITA Kazutaka
---
Makefile.objs|2 +-
block/sheepdog.c | 1845 ++
2 files changed, 1846 insertions(+), 1 deletions(-)
create mode 100644 block/sheepdog.c
diff --git a/Makefile.objs b/Makefile.objs
index
Hi,
Thank you very much for the reviewing!
At Fri, 14 May 2010 13:08:06 +0200,
Kevin Wolf wrote:
> > +
> > +struct sd_req {
> > + uint8_t proto_ver;
> > + uint8_t opcode;
> > + uint16_tflags;
> > + uint32_tepoch;
> > + uint32_tid;
> > + uint32_
At Mon, 17 May 2010 13:08:08 +0200,
Kevin Wolf wrote:
>
> Am 17.05.2010 12:19, schrieb MORITA Kazutaka:
> >
> > int bdrv_snapshot_goto(BlockDriverState *bs,
> > const char *snapshot_id)
> > {
> > BlockDriver *drv =
On Mon, May 17, 2010 at 9:20 PM, Kevin Wolf wrote:
> Am 17.05.2010 14:19, schrieb MORITA Kazutaka:
>> At Mon, 17 May 2010 13:08:08 +0200,
>> Kevin Wolf wrote:
>>>
>>> Am 17.05.2010 12:19, schrieb MORITA Kazutaka:
>>>>
>
This patch enables protocol drivers to use their create options which
are not supported by the format. For example, protcol drivers can use
a backing_file option with raw format.
Signed-off-by: MORITA Kazutaka
---
block.c |7 +++
block.h |1 +
qemu-img.c| 49
"
- clone from a snapshot
$ kvm-img create -b sheepdog:"Alice's Disk":0 -f sheepdog "Charlie's Disk"
Thanks.
--
MORITA, Kazutaka
NTT Cyber Space Labs
OSS Computing Project
Kernel Group
E-mail: morita.kazut...@lab.ntt.co.jp
uct flock’
> /usr/include/asm-generic/fcntl.h:140: error: redefinition of ‘struct flock64’
> make[1]: *** [work.o] Error 1
> make[1]: Leaving directory `/home/shiny/Packages/sheepdog-2009102101/sheep'
> make: *** [all] Error 2
>
> I have all the required libs installe
>> -Original Message-
>> From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] On
>> Behalf Of MORITA Kazutaka
>> Sent: Mittwoch, 21. Oktober 2009 07:14
>> To: k...@vger.kernel.org; qemu-devel@nongnu.org; linux-
>> fsde...@vger.kernel.org
>&g
On Fri, Oct 23, 2009 at 12:30 AM, Avi Kivity wrote:
> On 10/21/2009 07:13 AM, MORITA Kazutaka wrote:
>>
>> Hi everyone,
>>
>> Sheepdog is a distributed storage system for KVM/QEMU. It provides
>> highly available block level storage volumes to VMs like Amazon E
On Fri, Oct 23, 2009 at 8:10 PM, Alexander Graf wrote:
>
> On 23.10.2009, at 12:41, MORITA Kazutaka wrote:
>
> On Fri, Oct 23, 2009 at 12:30 AM, Avi Kivity wrote:
>
> How is load balancing implemented? Can you move an image transparently
>
> while a guest is running?
Hi,
Thanks for many comments.
Sheepdog git trees are created.
Sheepdog server
git://sheepdog.git.sourceforge.net/gitroot/sheepdog/sheepdog
Sheepdog client
git://sheepdog.git.sourceforge.net/gitroot/sheepdog/qemu-kvm
Please try!
On Wed, Oct 21, 2009 at 2:13 PM, MORITA Kazutaka
On Sat, Oct 24, 2009 at 4:45 AM, Javier Guerra wrote:
> On Fri, Oct 23, 2009 at 2:39 PM, MORITA Kazutaka
> wrote:
>> Thanks for many comments.
>>
>> Sheepdog git trees are created.
>
> great!
>
> is there any client (no matter how crude) besides the patch
, because Sheepdog locks a VM image when it is opened.
To avoid this problem, locking must be delayed until migration has done.
This is also a TODO item.
--
MORITA Kazutaka
On 2009/10/21 14:13, MORITA Kazutaka wrote:
Hi everyone,
Sheepdog is a distributed storage system for KVM/QEMU. It provides
highly available block level storage volumes to VMs like Amazon EBS.
Sheepdog supports advanced volume management features such as snapshot,
cloning, and thin provisioning
201 - 262 of 262 matches
Mail list logo
