At Fri, 8 Aug 2014 15:49:37 +0800,
Liu Yuan wrote:
>
> On Fri, Aug 08, 2014 at 03:12:17PM +0900, Hitoshi Mitake wrote:
> > At Fri, 8 Aug 2014 13:20:39 +0800,
> > Liu Yuan wrote:
> > >
> > > On Thu, Aug 07, 2014 at 04:28:39PM +0900, Hitoshi Mitake wrote
At Fri, 8 Aug 2014 14:43:16 +0800,
Liu Yuan wrote:
>
> On Fri, Aug 08, 2014 at 03:17:59PM +0900, Hitoshi Mitake wrote:
> > At Fri, 8 Aug 2014 13:31:39 +0800,
> > Liu Yuan wrote:
> > >
> > > On Thu, Aug 07, 2014 at 04:28:40PM +0900, Hitoshi Mitake wrote:
> &
Recently, sheepdog revived VDI locking functionality. This patchset
updates sheepdog driver of QEMU for this feature.
v2:
- don't handle SD_RES_VDI_NOT_LOCKED as a special case
Hitoshi Mitake (2):
sheepdog: adopting protocol update for VDI locking
sheepdog: improve error handling for a
Kazutaka
Signed-off-by: Hitoshi Mitake
---
v2:
- don't handle SD_RES_VDI_NOT_LOCKED as a special case
block/sheepdog.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/block/sheepdog.c b/block/sheepdog.c
index 36f76f0..f08182d 100644
--- a/block/sheepdog.c
+++ b/block/sheepdog.c
@@ -1
The update is required for supporting iSCSI multipath. It doesn't
affect behavior of QEMU driver but adding a new field to vdi request
struct is required.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 8 ++
Recently, sheepdog revived VDI locking functionality. This patchset
updates sheepdog driver of QEMU for this feature.
v3:
- keep backward compatibility
v2:
- don't handle SD_RES_VDI_NOT_LOCKED as a special case
Hitoshi Mitake (2):
sheepdog: adopting protocol update for VDI lo
The update is required for supporting iSCSI multipath. It doesn't
affect behavior of QEMU driver but adding a new field to vdi request
struct is required.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 8 ++
Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 2 ++
1 file changed, 2 insertions(+)
v2:
- don't handle SD_RES_VDI_NOT_LOCKED as a special case
diff --git a/block/sheepdog.c b/block/sheepdog.c
index 3da5ea2..53f9d90 100644
--- a/block/sheepdog.c
+++ b/block/sheepdog.c
@@ -1
Calling error_get_progname() in the context of qemu-io can cause
segmentation fault because qemu-io doesn't initialize its progname
with error_set_progname(). This patch adds the initialization.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Signed-off-by: Hitoshi Mitake
---
qemu-io.c | 1 +
1
At Mon, 11 Aug 2014 13:20:38 +0200,
Markus Armbruster wrote:
>
> Hitoshi Mitake writes:
>
> > Calling error_get_progname() in the context of qemu-io can cause
> > segmentation fault because qemu-io doesn't initialize its progname
> > with error_set_progname(). Th
At Mon, 11 Aug 2014 15:38:21 +0100,
Stefan Hajnoczi wrote:
>
> [1 ]
> On Mon, Aug 11, 2014 at 03:50:03PM +0900, Hitoshi Mitake wrote:
> > Calling error_get_progname() in the context of qemu-io can cause
> > segmentation fault because qemu-io doesn't initia
ot;removable": true,
"tray_open": false, "type": "unknown"}]}
{"execute": "set-block-fault-state", "arguments": {"id": "ide0-hd0",
"state": "error"}} # <- inject error to /dev/sda
{"retur
At Wed, 13 Aug 2014 08:20:53 -0600,
Eric Blake wrote:
>
> [1 ]
> On 08/12/2014 10:48 PM, Hitoshi Mitake wrote:
> > This patch makes the fault injection functionality of blkdebug
> > callable from QMP. Motivation of this change is for testing and
> > debugging d
At Wed, 13 Aug 2014 15:33:38 +0100,
Stefan Hajnoczi wrote:
>
> [1 ]
> On Wed, Aug 13, 2014 at 01:48:41PM +0900, Hitoshi Mitake wrote:
> > @@ -485,6 +493,10 @@ static BlockDriverAIOCB
> > *blkdebug_aio_readv(BlockDriverState *bs,
> > BDRVBlkdebugState *s = bs->
. In addition, a
testing framework for injecting faults at critical timing (which
requires solid understanding of target systems) is in progress.
[1] http://ucare.cs.uchicago.edu/pdf/socc13-limplock.pdf
[2] http://docs.openstack.org/developer/swift/howto_installmultinode.html
[3] http://www.am
At Tue, 26 Aug 2014 20:54:57 -0600,
Eric Blake wrote:
>
> On 08/26/2014 07:59 PM, Hitoshi Mitake wrote:
> > This patch makes the fault injection functionality of blkdebug
> > callable from QMP. Motivation of this change is for testing and
> > debugging distributed syst
At Wed, 27 Aug 2014 06:43:49 -0600,
Eric Blake wrote:
>
> [1 ]
> On 08/26/2014 11:34 PM, Hitoshi Mitake wrote:
>
> >>> {"execute": "blkdebug-set-rules", "arguments": {"device": "ide0-hd0",
> >>> "rul
Hi QEMU list,
I have a question about life cycle of QEMU's stable branches. Is there
an explicit support period for the stable branches?
We are maintaining stable branches of sheepdog, distributed block
storage system for QEMU. The stable branches of sheepdog corresponds
to the stable branches
Hi Stefan, Eric, Sorry for my late reply.
At Tue, 2 Sep 2014 16:29:14 +0100,
Stefan Hajnoczi wrote:
>
> [1 ]
> On Thu, Aug 28, 2014 at 03:18:48PM +0900, Hitoshi Mitake wrote:
> > At Wed, 27 Aug 2014 06:43:49 -0600,
> > Eric Blake wrote:
> > >
> > > [1
zi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 4
1 file changed, 4 insertions(+)
diff --git a/block/sheepdog.c b/block/sheepdog.c
index 36f76f0..0b3f86d 100644
--- a/block/sheepdog.c
+++ b/block/sheepdog.c
@@ -1112,9 +1112,13 @@ stat
Recently, sheepdog revived VDI locking functionality. This patch
updates sheepdog driver of QEMU for this feature.
Hitoshi Mitake (2):
sheepdog: adopting protocol update for VDI locking
sheepdog: improve error handling for a case of failed lock
block/sheepdog.c | 12 +++-
1 file
The update is required for supporting iSCSI multipath. It doesn't
affect behavior of QEMU driver but adding a new field to vdi request
struct is required.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 8 ++
At Fri, 8 Aug 2014 13:20:39 +0800,
Liu Yuan wrote:
>
> On Thu, Aug 07, 2014 at 04:28:39PM +0900, Hitoshi Mitake wrote:
> > The update is required for supporting iSCSI multipath. It doesn't
> > affect behavior of QEMU driver but adding a new field to vdi request
> > st
At Fri, 8 Aug 2014 13:31:39 +0800,
Liu Yuan wrote:
>
> On Thu, Aug 07, 2014 at 04:28:40PM +0900, Hitoshi Mitake wrote:
> > Recently, sheepdog revived its VDI locking functionality. This patch
> > updates sheepdog driver of QEMU for this feature:
> >
> > 1. Improve
At Tue, 10 Feb 2015 18:35:58 +0800,
Liu Yuan wrote:
>
> On Tue, Feb 10, 2015 at 06:56:33PM +0900, Teruaki Ishizaki wrote:
> > (2015/02/10 17:58), Liu Yuan wrote:
> > >On Tue, Feb 10, 2015 at 05:22:02PM +0900, Teruaki Ishizaki wrote:
> > >>(2015/02/10 12:10), Liu Yuan wrote:
> > >>>On Tue, Jan 27,
At Thu, 12 Feb 2015 15:00:49 +0800,
Liu Yuan wrote:
>
> On Thu, Feb 12, 2015 at 03:19:21PM +0900, Hitoshi Mitake wrote:
> > At Tue, 10 Feb 2015 18:35:58 +0800,
> > Liu Yuan wrote:
> > >
> > > On Tue, Feb 10, 2015 at 06:56:33PM +0900, Teruaki Ishizaki wrote:
At Thu, 12 Feb 2015 15:42:17 +0800,
Liu Yuan wrote:
>
> On Thu, Feb 12, 2015 at 04:28:01PM +0900, Hitoshi Mitake wrote:
> > At Thu, 12 Feb 2015 15:00:49 +0800,
> > Liu Yuan wrote:
> > >
> > > On Thu, Feb 12, 2015 at 03:19:21PM +0900, Hitoshi Mitake wrote:
- Fix coding style problem (white space).
> - Add members, store_policy and block_size_shift to struct SheepdogVdiReq.
> - Initialize request header to use block_size_shift specified by user.
> ---
> block/sheepdog.c | 155
> ++++++++++---
pace).
> - Add members, store_policy and block_size_shift to struct SheepdogVdiReq
> - Initialize request header to use block_size_shift specified by user.
> ---
> block/sheepdog.c | 140
> ++---
> include/block/block_int.h |1 +
> 2 files changed, 119 insertions(+), 22 deletions(-)
Looks good to me.
Acked-by: Hitoshi Mitake
Thanks,
Hitoshi
Hi Peter,
Sorry for my late reply.
At Thu, 22 Jan 2015 10:07:27 +,
Peter Maydell wrote:
>
> On 22 January 2015 at 09:08, Hitoshi Mitake
> wrote:
> > Calling error_get_progname() in the context of qemu-x86_64 can cause
> > segmentation fault because linux-user/main.c d
At Thu, 22 Jan 2015 18:08:11 +0900,
Hitoshi Mitake wrote:
>
> Calling error_get_progname() in the context of qemu-io can cause
> segmentation fault because qemu-io doesn't initialize its progname
> with error_set_progname(). This patch adds the initialization.
>
> Current
At Mon, 2 Feb 2015 12:36:00 +0100,
Kevin Wolf wrote:
>
> Am 02.02.2015 um 10:51 hat Hitoshi Mitake geschrieben:
> > At Thu, 22 Jan 2015 18:08:11 +0900,
> > Hitoshi Mitake wrote:
> > >
> > > Calling error_get_progname() in the context of qemu-io can cause
>
At Mon, 22 Dec 2014 17:41:33 +0800,
$B=y>.$AAz(B wrote:
>
> [1 ]
> hi,all,
> when i use `qemu-img snapshot -c test-s sheepdog:test` to create a
> snapshot of sheepdog vdi,the snapshot's lock not be released.
>
> root@hty-compute1:~/qemu# ./qemu-img create sheepdog:test 1G
> Formatting 's
At Tue, 13 Jan 2015 17:41:12 +0900,
Teruaki Ishizaki wrote:
>
> Previously, qemu block driver of sheepdog used hard-coded VDI object size.
> This patch enables users to handle "block_size_shift" value for
> calculating VDI object size.
>
> When you start qemu, you don't need to specify additional
At Tue, 20 Jan 2015 16:14:28 +0900,
Teruaki Ishizaki wrote:
>
> Previously, qemu block driver of sheepdog used hard-coded VDI object size.
> This patch enables users to handle "block_size_shift" value for
> calculating VDI object size.
>
> When you start qemu, you don't need to specify additional
e qemu-nbd doesn't use error_get_progname(). This patch
is a proactive action.
Cc: Paolo Bonzini
Cc: Markus Armbruster
Signed-off-by: Hitoshi Mitake
---
qemu-nbd.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/qemu-nbd.c b/qemu-nbd.c
index d222512..fad5634 100644
--- a/qemu-nbd.c
+
g driver will use it soon).
v2:
- pass argv[0] directly to error_set_progname() without calling basename
- add initialization to commands other than qemu-io
The thread of v1 can be found here:
http://lists.gnu.org/archive/html/qemu-devel/2014-08/msg01502.html
Hitoshi Mitake (4):
qemu-io: init
io doesn't use error_get_progname(). This patch
is a proactive action.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Markus Armbruster
Signed-off-by: Hitoshi Mitake
---
qemu-io.c | 1 +
1 file changed, 1 insertion(+)
v2:
- pass argv[0] directly to error_set_progname() without calling basenam
oesn't use error_get_progname(). This patch
is a proactive action.
Cc: Michael Roth
Cc: Markus Armbruster
Signed-off-by: Hitoshi Mitake
---
qga/main.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/qga/main.c b/qga/main.c
index 9939a2b..0885fc6 100644
--- a/qga/main.c
+++ b/qga/main.c
@
ms because qemu-x86_64 doesn't use error_get_progname(). This
patch is a proactive action.
Cc: Riku Voipio
Cc: Markus Armbruster
Signed-off-by: Hitoshi Mitake
---
linux-user/main.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/linux-user/main.c b/linux-user/main.c
index 8c70be4.
Current sheepdog driver has two problems in a mechanism of inode
object reloading for live snapshot. It causes inconsistent state of
snapshot volumes. A new GC algorithm implemented in sheepdog exposes
the problems. This patchset contains bugfixes for them.
Hitoshi Mitake (2):
sheepdog: fix vdi
sheepdog driver should decide a write request is COW or not based on
inode object which is active when the write request is issued.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 40
sheepdog dirver doesn't need to read data_vdi_id[] when a live
snapshot is created.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/
On Tue, Jun 3, 2014 at 12:48 AM, Eric Blake wrote:
> On 06/02/2014 09:06 AM, Hitoshi Mitake wrote:
>> sheepdog dirver doesn't need to read data_vdi_id[] when a live
>
> s/dirver/driver/
Thanks for your comment. I'll send v2 later.
Thanks,
Hitoshi
>
>> sna
sheepdog driver should decide a write request is COW or not based on
inode object which is active when the write request is issued.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 40
commit log of 2nd patch
Hitoshi Mitake (2):
sheepdog: fix vdi object update after live snapshot
sheepdog: reload only header in a case of live snapshot
block/sheepdog.c | 49 +
1 files changed, 29 insertions(+), 20 deletions(-)
sheepdog driver doesn't need to read data_vdi_id[] when a live
snapshot is created.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
v2:
- correct wrong spelling in the commit log
block/sheepdog.c |9 ++---
1 files chang
On Tue, Jun 3, 2014 at 9:41 PM, Liu Yuan wrote:
> On Tue, Jun 03, 2014 at 01:54:21PM +0900, Hitoshi Mitake wrote:
>> sheepdog driver should decide a write request is COW or not based on
>> inode object which is active when the write request is issued.
>>
>> Cc: Kevin W
:
- corrrect wrong spelling in the commit log of 2nd patch
Hitoshi Mitake (2):
sheepdog: fix vdi object update after live snapshot
sheepdog: reload only header in a case of live snapshot
block/sheepdog.c | 49 +
1 files changed, 29 insertions(+), 20
sheepdog driver doesn't need to read data_vdi_id[] when a live snapshot is
created.
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
v2:
- correct wrong spelling in the commit log
block/sheepdog.c |9 ++---
1 files chang
ve VDI, it results object
leaking
Cc: Kevin Wolf
Cc: Stefan Hajnoczi
Cc: Liu Yuan
Cc: MORITA Kazutaka
Signed-off-by: Hitoshi Mitake
---
v3:
- update commit log
block/sheepdog.c | 40 +++-
1 files changed, 23 insertions(+), 17 deletions(-)
diff --git a
On Wed, Dec 2, 2015 at 1:12 AM, Vasiliy Tolstov wrote:
> Signed-off-by: Vasiliy Tolstov
> ---
> block/sheepdog.c | 59
> ++--
> 1 file changed, 57 insertions(+), 2 deletions(-)
>
Seems that your patch violates the coding style of qemu. You ca
On Wed, Dec 23, 2015 at 9:22 PM, Hitoshi Mitake <
mitake.hito...@lab.ntt.co.jp> wrote:
> From: Vasiliy Tolstov
>
> This patch implements a blockdriver function bdrv_snapshot_delete() in
> the sheepdog driver. With the new function, snapshots of sheepdog can
> be deleted
From: Vasiliy Tolstov
This patch implements a blockdriver function bdrv_snapshot_delete() in
the sheepdog driver. With the new function, snapshots of sheepdog can
be deleted from libvirt.
Cc: Jeff Cody
Signed-off-by: Hitoshi Mitake
Signed-off-by: Vasiliy Tolstov
---
block/sheepdog.c | 125
e
overhead seems to be large. It has an impact especially in a case of
disk I/O heavy workload.
I don't have a comparison of benchmark result, but it is not obvious
that the approach of this patch is better.
BTW, sheepdog project was already forked, why don't you fork the bl
On Fri, Sep 18, 2015 at 6:49 PM, Kevin Wolf wrote:
> Am 18.09.2015 um 11:03 hat Vasiliy Tolstov geschrieben:
>> 2015-09-18 12:02 GMT+03:00 Kevin Wolf :
>> > Doesn't sheepdog already support storing snapshots in the same image?
>> > I thought it would just work; at least, there's some code there fo
On Wed, Sep 2, 2015 at 9:36 PM, Vasiliy Tolstov wrote:
> 2015-09-01 6:03 GMT+03:00 Hitoshi Mitake :
>> n the commit 96b14ff85acf, requests for overlapping areas are
>> serialized. However, it cannot handle a case of non overlapping
>> requests. In such a case, m
On Wed, Sep 2, 2015 at 9:36 PM, Vasiliy Tolstov wrote:
> 2015-09-01 6:03 GMT+03:00 Hitoshi Mitake :
>> This patch refines discard support of the sheepdog driver. The
>> existing discard mechanism was implemented on SD_OP_DISCARD_OBJ, which
>> was introduced before fine grain
On Fri, Sep 4, 2015 at 5:51 PM, Hitoshi Mitake wrote:
> On Wed, Sep 2, 2015 at 9:36 PM, Vasiliy Tolstov wrote:
>> 2015-09-01 6:03 GMT+03:00 Hitoshi Mitake :
>>> This patch refines discard support of the sheepdog driver. The
>>> existing discard mechanism was imple
eff Cody
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 63 +++-
1 file changed, 35 insertions(+), 28 deletions(-)
diff --git a/block/sheepdog.c b/block/sheepdog.c
index 9585beb..24341ea 100644
--- a/block/sheepdog.c
+++ b/block/sheepdo
patch solves the problem.
Hitoshi Mitake (2):
sheepdog: use per AIOCB dirty indexes for non overlapping requests
sheepdog: refine discard support
block/sheepdog.c | 92 +---
1 file changed, 55 insertions(+), 37 deletions(-)
--
1.9.1
jects unconditionally.
With this patch, the driver just updates an inode object for updating
reference. Removing the object is done in sheep process side.
Cc: Teruaki Ishizaki
Cc: Vasiliy Tolstov
Cc: Jeff Cody
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 29 -
1 file ch
f
Cc: Stefan Hajnoczi
Cc: Teruaki Ishizaki
Cc: Vasiliy Tolstov
Signed-off-by: Hitoshi Mitake
---
block/sheepdog.c | 152 ++-
1 file changed, 71 insertions(+), 81 deletions(-)
diff --git a/block/sheepdog.c b/block/sheepdog.c
index bd7cbed..95
strtoul().
>
> This patch also adds more meaningful error handling to the
> qemu_strtoul() call, and subsequent results.
>
> Reported-by: Paolo Bonzini
> Signed-off-by: Jeff Cody
> ---
> block/sheepdog.c | 11 +++
> 1 file changed, 7 insertions(+), 4 deletions(-)
On Thu, Mar 3, 2016 at 1:36 AM, Jeff Cody wrote:
> On Wed, Mar 02, 2016 at 05:32:11PM +0100, Max Reitz wrote:
> > On 02.03.2016 17:24, Jeff Cody wrote:
> > > The function qemu_strtoul() reads 'unsigned long' sized data,
> > > which is larger than uint32_t on 64-bit machines.
> > >
> > > Even thou
On Mon, Apr 4, 2016 at 10:04 PM, Paolo Bonzini wrote:
> I am curious about why overlapping_queue is required for sheepdog.
> Overlapping requests have unspecified outcome so the CoQueue is not
> necessary as long as the server doesn't crash or return an error.
>
> Hitoshi, could you clarify this
On Tue, Mar 22, 2016 at 1:33 PM, Takashi Menjo
wrote:
> Errors have been ignored or not propagated in some code paths
> in sd_snapshot_delete(). This patch adds error handling.
>
> Cc: Hitoshi Mitake
> Cc: Jeff Cody
> Cc: Vasiliy Tolstov
> Cc: sheep...@lists.wpkg.
At Wed, 15 Jun 2016 18:14:37 +0200,
Laurent Vivier wrote:
>
> This patch is the result of coccinelle script
> scripts/coccinelle/typecast.cocci
>
> CC: Hitoshi Mitake
> CC: qemu-bl...@nongnu.org
> Signed-off-by: Laurent Vivier
> ---
> block/sheepdog.c | 4 ++--
>
68 matches
Mail list logo
