* [Qemu-devel] [PATCH v2] more check for replaced node
@ 2015-07-15 1:41 Wen Congyang
2015-07-15 3:05 ` Fam Zheng
0 siblings, 1 reply; 6+ messages in thread
From: Wen Congyang @ 2015-07-15 1:41 UTC (permalink / raw)
To: qemu-devl, Stefan Hajnoczi, Jeff Cody; +Cc: Fam Zheng
We use mirror+replace to fix quorum's broken child. bs/s->common.bs
is quorum, and to_replace is the broken child. The new child is target_bs.
Without this patch, the replace node can be any node, and it can be
top BDS with BB, or another quorum's child. We just check if the broken
child is part of the quorum BDS in this patch.
Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
---
block.c | 5 +++--
block/mirror.c | 3 ++-
blockdev.c | 2 +-
include/block/block.h | 3 ++-
4 files changed, 8 insertions(+), 5 deletions(-)
diff --git a/block.c b/block.c
index d088ee0..090923c 100644
--- a/block.c
+++ b/block.c
@@ -4077,7 +4077,8 @@ bool bdrv_is_first_non_filter(BlockDriverState *candidate)
return false;
}
-BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
+BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
+ const char *node_name, Error **errp)
{
BlockDriverState *to_replace_bs = bdrv_find_node(node_name);
AioContext *aio_context;
@@ -4100,7 +4101,7 @@ BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
* Another benefit is that this tests exclude backing files which are
* blocked by the backing blockers.
*/
- if (!bdrv_is_first_non_filter(to_replace_bs)) {
+ if (!bdrv_recurse_is_first_non_filter(parent_bs, to_replace_bs)) {
error_setg(errp, "Only top most non filter can be replaced");
to_replace_bs = NULL;
goto out;
diff --git a/block/mirror.c b/block/mirror.c
index 238a070..b81077e 100644
--- a/block/mirror.c
+++ b/block/mirror.c
@@ -626,7 +626,8 @@ static void mirror_complete(BlockJob *job, Error **errp)
if (s->replaces) {
AioContext *replace_aio_context;
- s->to_replace = check_to_replace_node(s->replaces, &local_err);
+ s->to_replace = check_to_replace_node(s->common.bs, s->replaces,
+ &local_err);
if (!s->to_replace) {
error_propagate(errp, local_err);
return;
diff --git a/blockdev.c b/blockdev.c
index c11611d..bf12e2e 100644
--- a/blockdev.c
+++ b/blockdev.c
@@ -2757,7 +2757,7 @@ void qmp_drive_mirror(const char *device, const char *target,
goto out;
}
- to_replace_bs = check_to_replace_node(replaces, &local_err);
+ to_replace_bs = check_to_replace_node(bs, replaces, &local_err);
if (!to_replace_bs) {
error_propagate(errp, local_err);
diff --git a/include/block/block.h b/include/block/block.h
index 37916f7..608cd4e 100644
--- a/include/block/block.h
+++ b/include/block/block.h
@@ -317,7 +317,8 @@ bool bdrv_recurse_is_first_non_filter(BlockDriverState *bs,
bool bdrv_is_first_non_filter(BlockDriverState *candidate);
/* check if a named node can be replaced when doing drive-mirror */
-BlockDriverState *check_to_replace_node(const char *node_name, Error **errp);
+BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
+ const char *node_name, Error **errp);
/* async block I/O */
typedef void BlockDriverDirtyHandler(BlockDriverState *bs, int64_t sector,
--
2.4.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2] more check for replaced node
2015-07-15 1:41 [Qemu-devel] [PATCH v2] more check for replaced node Wen Congyang
@ 2015-07-15 3:05 ` Fam Zheng
2015-07-15 3:22 ` Wen Congyang
2015-07-17 1:37 ` Wen Congyang
0 siblings, 2 replies; 6+ messages in thread
From: Fam Zheng @ 2015-07-15 3:05 UTC (permalink / raw)
To: Wen Congyang; +Cc: Jeff Cody, qemu-devl, Stefan Hajnoczi
On Wed, 07/15 09:41, Wen Congyang wrote:
> We use mirror+replace to fix quorum's broken child. bs/s->common.bs
> is quorum, and to_replace is the broken child. The new child is target_bs.
> Without this patch, the replace node can be any node, and it can be
> top BDS with BB, or another quorum's child. We just check if the broken
> child is part of the quorum BDS in this patch.
>
> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
> ---
> block.c | 5 +++--
> block/mirror.c | 3 ++-
> blockdev.c | 2 +-
> include/block/block.h | 3 ++-
> 4 files changed, 8 insertions(+), 5 deletions(-)
>
> diff --git a/block.c b/block.c
> index d088ee0..090923c 100644
> --- a/block.c
> +++ b/block.c
> @@ -4077,7 +4077,8 @@ bool bdrv_is_first_non_filter(BlockDriverState *candidate)
> return false;
> }
>
> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
> + const char *node_name, Error **errp)
> {
> BlockDriverState *to_replace_bs = bdrv_find_node(node_name);
> AioContext *aio_context;
> @@ -4100,7 +4101,7 @@ BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
> * Another benefit is that this tests exclude backing files which are
> * blocked by the backing blockers.
> */
> - if (!bdrv_is_first_non_filter(to_replace_bs)) {
> + if (!bdrv_recurse_is_first_non_filter(parent_bs, to_replace_bs)) {
> error_setg(errp, "Only top most non filter can be replaced");
> to_replace_bs = NULL;
> goto out;
> diff --git a/block/mirror.c b/block/mirror.c
> index 238a070..b81077e 100644
> --- a/block/mirror.c
> +++ b/block/mirror.c
> @@ -626,7 +626,8 @@ static void mirror_complete(BlockJob *job, Error **errp)
> if (s->replaces) {
> AioContext *replace_aio_context;
>
> - s->to_replace = check_to_replace_node(s->replaces, &local_err);
> + s->to_replace = check_to_replace_node(s->common.bs, s->replaces,
> + &local_err);
Why is the check in qmp_drive_mirror not enough? Isn't this redundant?
Fam
> if (!s->to_replace) {
> error_propagate(errp, local_err);
> return;
> diff --git a/blockdev.c b/blockdev.c
> index c11611d..bf12e2e 100644
> --- a/blockdev.c
> +++ b/blockdev.c
> @@ -2757,7 +2757,7 @@ void qmp_drive_mirror(const char *device, const char *target,
> goto out;
> }
>
> - to_replace_bs = check_to_replace_node(replaces, &local_err);
> + to_replace_bs = check_to_replace_node(bs, replaces, &local_err);
>
> if (!to_replace_bs) {
> error_propagate(errp, local_err);
> diff --git a/include/block/block.h b/include/block/block.h
> index 37916f7..608cd4e 100644
> --- a/include/block/block.h
> +++ b/include/block/block.h
> @@ -317,7 +317,8 @@ bool bdrv_recurse_is_first_non_filter(BlockDriverState *bs,
> bool bdrv_is_first_non_filter(BlockDriverState *candidate);
>
> /* check if a named node can be replaced when doing drive-mirror */
> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp);
> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
> + const char *node_name, Error **errp);
>
> /* async block I/O */
> typedef void BlockDriverDirtyHandler(BlockDriverState *bs, int64_t sector,
> --
> 2.4.3
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2] more check for replaced node
2015-07-15 3:05 ` Fam Zheng
@ 2015-07-15 3:22 ` Wen Congyang
2015-07-16 13:36 ` Stefan Hajnoczi
2015-07-17 1:37 ` Wen Congyang
1 sibling, 1 reply; 6+ messages in thread
From: Wen Congyang @ 2015-07-15 3:22 UTC (permalink / raw)
To: Fam Zheng; +Cc: benoit.canet, Jeff Cody, qemu-devl, Stefan Hajnoczi
On 07/15/2015 11:05 AM, Fam Zheng wrote:
> On Wed, 07/15 09:41, Wen Congyang wrote:
>> We use mirror+replace to fix quorum's broken child. bs/s->common.bs
>> is quorum, and to_replace is the broken child. The new child is target_bs.
>> Without this patch, the replace node can be any node, and it can be
>> top BDS with BB, or another quorum's child. We just check if the broken
>> child is part of the quorum BDS in this patch.
>>
>> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
>> ---
>> block.c | 5 +++--
>> block/mirror.c | 3 ++-
>> blockdev.c | 2 +-
>> include/block/block.h | 3 ++-
>> 4 files changed, 8 insertions(+), 5 deletions(-)
>>
>> diff --git a/block.c b/block.c
>> index d088ee0..090923c 100644
>> --- a/block.c
>> +++ b/block.c
>> @@ -4077,7 +4077,8 @@ bool bdrv_is_first_non_filter(BlockDriverState *candidate)
>> return false;
>> }
>>
>> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
>> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
>> + const char *node_name, Error **errp)
>> {
>> BlockDriverState *to_replace_bs = bdrv_find_node(node_name);
>> AioContext *aio_context;
>> @@ -4100,7 +4101,7 @@ BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
>> * Another benefit is that this tests exclude backing files which are
>> * blocked by the backing blockers.
>> */
>> - if (!bdrv_is_first_non_filter(to_replace_bs)) {
>> + if (!bdrv_recurse_is_first_non_filter(parent_bs, to_replace_bs)) {
>> error_setg(errp, "Only top most non filter can be replaced");
>> to_replace_bs = NULL;
>> goto out;
>> diff --git a/block/mirror.c b/block/mirror.c
>> index 238a070..b81077e 100644
>> --- a/block/mirror.c
>> +++ b/block/mirror.c
>> @@ -626,7 +626,8 @@ static void mirror_complete(BlockJob *job, Error **errp)
>> if (s->replaces) {
>> AioContext *replace_aio_context;
>>
>> - s->to_replace = check_to_replace_node(s->replaces, &local_err);
>> + s->to_replace = check_to_replace_node(s->common.bs, s->replaces,
>> + &local_err);
>
> Why is the check in qmp_drive_mirror not enough? Isn't this redundant?
I don't know why we check it twice. And I think it is redundant too.
Thanks
Wen Congyang
>
> Fam
>
>> if (!s->to_replace) {
>> error_propagate(errp, local_err);
>> return;
>> diff --git a/blockdev.c b/blockdev.c
>> index c11611d..bf12e2e 100644
>> --- a/blockdev.c
>> +++ b/blockdev.c
>> @@ -2757,7 +2757,7 @@ void qmp_drive_mirror(const char *device, const char *target,
>> goto out;
>> }
>>
>> - to_replace_bs = check_to_replace_node(replaces, &local_err);
>> + to_replace_bs = check_to_replace_node(bs, replaces, &local_err);
>>
>> if (!to_replace_bs) {
>> error_propagate(errp, local_err);
>> diff --git a/include/block/block.h b/include/block/block.h
>> index 37916f7..608cd4e 100644
>> --- a/include/block/block.h
>> +++ b/include/block/block.h
>> @@ -317,7 +317,8 @@ bool bdrv_recurse_is_first_non_filter(BlockDriverState *bs,
>> bool bdrv_is_first_non_filter(BlockDriverState *candidate);
>>
>> /* check if a named node can be replaced when doing drive-mirror */
>> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp);
>> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
>> + const char *node_name, Error **errp);
>>
>> /* async block I/O */
>> typedef void BlockDriverDirtyHandler(BlockDriverState *bs, int64_t sector,
>> --
>> 2.4.3
>>
> .
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2] more check for replaced node
2015-07-15 3:22 ` Wen Congyang
@ 2015-07-16 13:36 ` Stefan Hajnoczi
2015-07-17 1:35 ` Wen Congyang
0 siblings, 1 reply; 6+ messages in thread
From: Stefan Hajnoczi @ 2015-07-16 13:36 UTC (permalink / raw)
To: Wen Congyang
Cc: benoit.canet, Jeff Cody, Fam Zheng, qemu-devl, Stefan Hajnoczi
[-- Attachment #1: Type: text/plain, Size: 2674 bytes --]
On Wed, Jul 15, 2015 at 11:22:52AM +0800, Wen Congyang wrote:
> On 07/15/2015 11:05 AM, Fam Zheng wrote:
> > On Wed, 07/15 09:41, Wen Congyang wrote:
> >> We use mirror+replace to fix quorum's broken child. bs/s->common.bs
> >> is quorum, and to_replace is the broken child. The new child is target_bs.
> >> Without this patch, the replace node can be any node, and it can be
> >> top BDS with BB, or another quorum's child. We just check if the broken
> >> child is part of the quorum BDS in this patch.
> >>
> >> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
> >> ---
> >> block.c | 5 +++--
> >> block/mirror.c | 3 ++-
> >> blockdev.c | 2 +-
> >> include/block/block.h | 3 ++-
> >> 4 files changed, 8 insertions(+), 5 deletions(-)
> >>
> >> diff --git a/block.c b/block.c
> >> index d088ee0..090923c 100644
> >> --- a/block.c
> >> +++ b/block.c
> >> @@ -4077,7 +4077,8 @@ bool bdrv_is_first_non_filter(BlockDriverState *candidate)
> >> return false;
> >> }
> >>
> >> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
> >> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
> >> + const char *node_name, Error **errp)
> >> {
> >> BlockDriverState *to_replace_bs = bdrv_find_node(node_name);
> >> AioContext *aio_context;
> >> @@ -4100,7 +4101,7 @@ BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
> >> * Another benefit is that this tests exclude backing files which are
> >> * blocked by the backing blockers.
> >> */
> >> - if (!bdrv_is_first_non_filter(to_replace_bs)) {
> >> + if (!bdrv_recurse_is_first_non_filter(parent_bs, to_replace_bs)) {
> >> error_setg(errp, "Only top most non filter can be replaced");
> >> to_replace_bs = NULL;
> >> goto out;
> >> diff --git a/block/mirror.c b/block/mirror.c
> >> index 238a070..b81077e 100644
> >> --- a/block/mirror.c
> >> +++ b/block/mirror.c
> >> @@ -626,7 +626,8 @@ static void mirror_complete(BlockJob *job, Error **errp)
> >> if (s->replaces) {
> >> AioContext *replace_aio_context;
> >>
> >> - s->to_replace = check_to_replace_node(s->replaces, &local_err);
> >> + s->to_replace = check_to_replace_node(s->common.bs, s->replaces,
> >> + &local_err);
> >
> > Why is the check in qmp_drive_mirror not enough? Isn't this redundant?
>
> I don't know why we check it twice. And I think it is redundant too.
Okay, does that mean you are dropping this patch now?
[-- Attachment #2: Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2] more check for replaced node
2015-07-16 13:36 ` Stefan Hajnoczi
@ 2015-07-17 1:35 ` Wen Congyang
0 siblings, 0 replies; 6+ messages in thread
From: Wen Congyang @ 2015-07-17 1:35 UTC (permalink / raw)
To: Stefan Hajnoczi
Cc: benoit.canet, Jeff Cody, Fam Zheng, qemu-devl, Stefan Hajnoczi
On 07/16/2015 09:36 PM, Stefan Hajnoczi wrote:
> On Wed, Jul 15, 2015 at 11:22:52AM +0800, Wen Congyang wrote:
>> On 07/15/2015 11:05 AM, Fam Zheng wrote:
>>> On Wed, 07/15 09:41, Wen Congyang wrote:
>>>> We use mirror+replace to fix quorum's broken child. bs/s->common.bs
>>>> is quorum, and to_replace is the broken child. The new child is target_bs.
>>>> Without this patch, the replace node can be any node, and it can be
>>>> top BDS with BB, or another quorum's child. We just check if the broken
>>>> child is part of the quorum BDS in this patch.
>>>>
>>>> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
>>>> ---
>>>> block.c | 5 +++--
>>>> block/mirror.c | 3 ++-
>>>> blockdev.c | 2 +-
>>>> include/block/block.h | 3 ++-
>>>> 4 files changed, 8 insertions(+), 5 deletions(-)
>>>>
>>>> diff --git a/block.c b/block.c
>>>> index d088ee0..090923c 100644
>>>> --- a/block.c
>>>> +++ b/block.c
>>>> @@ -4077,7 +4077,8 @@ bool bdrv_is_first_non_filter(BlockDriverState *candidate)
>>>> return false;
>>>> }
>>>>
>>>> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
>>>> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
>>>> + const char *node_name, Error **errp)
>>>> {
>>>> BlockDriverState *to_replace_bs = bdrv_find_node(node_name);
>>>> AioContext *aio_context;
>>>> @@ -4100,7 +4101,7 @@ BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
>>>> * Another benefit is that this tests exclude backing files which are
>>>> * blocked by the backing blockers.
>>>> */
>>>> - if (!bdrv_is_first_non_filter(to_replace_bs)) {
>>>> + if (!bdrv_recurse_is_first_non_filter(parent_bs, to_replace_bs)) {
>>>> error_setg(errp, "Only top most non filter can be replaced");
>>>> to_replace_bs = NULL;
>>>> goto out;
>>>> diff --git a/block/mirror.c b/block/mirror.c
>>>> index 238a070..b81077e 100644
>>>> --- a/block/mirror.c
>>>> +++ b/block/mirror.c
>>>> @@ -626,7 +626,8 @@ static void mirror_complete(BlockJob *job, Error **errp)
>>>> if (s->replaces) {
>>>> AioContext *replace_aio_context;
>>>>
>>>> - s->to_replace = check_to_replace_node(s->replaces, &local_err);
>>>> + s->to_replace = check_to_replace_node(s->common.bs, s->replaces,
>>>> + &local_err);
>>>
>>> Why is the check in qmp_drive_mirror not enough? Isn't this redundant?
>>
>> I don't know why we check it twice. And I think it is redundant too.
>
> Okay, does that mean you are dropping this patch now?
>
No, we call check_to_replace_node() twice. So we can remove it from mirror_complete().
Will update it in the next version.
Thanks
Wen Congyang
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH v2] more check for replaced node
2015-07-15 3:05 ` Fam Zheng
2015-07-15 3:22 ` Wen Congyang
@ 2015-07-17 1:37 ` Wen Congyang
1 sibling, 0 replies; 6+ messages in thread
From: Wen Congyang @ 2015-07-17 1:37 UTC (permalink / raw)
To: Fam Zheng; +Cc: Jeff Cody, qemu-devl, Stefan Hajnoczi
On 07/15/2015 11:05 AM, Fam Zheng wrote:
> On Wed, 07/15 09:41, Wen Congyang wrote:
>> We use mirror+replace to fix quorum's broken child. bs/s->common.bs
>> is quorum, and to_replace is the broken child. The new child is target_bs.
>> Without this patch, the replace node can be any node, and it can be
>> top BDS with BB, or another quorum's child. We just check if the broken
>> child is part of the quorum BDS in this patch.
>>
>> Signed-off-by: Wen Congyang <wency@cn.fujitsu.com>
>> ---
>> block.c | 5 +++--
>> block/mirror.c | 3 ++-
>> blockdev.c | 2 +-
>> include/block/block.h | 3 ++-
>> 4 files changed, 8 insertions(+), 5 deletions(-)
>>
>> diff --git a/block.c b/block.c
>> index d088ee0..090923c 100644
>> --- a/block.c
>> +++ b/block.c
>> @@ -4077,7 +4077,8 @@ bool bdrv_is_first_non_filter(BlockDriverState *candidate)
>> return false;
>> }
>>
>> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
>> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
>> + const char *node_name, Error **errp)
>> {
>> BlockDriverState *to_replace_bs = bdrv_find_node(node_name);
>> AioContext *aio_context;
>> @@ -4100,7 +4101,7 @@ BlockDriverState *check_to_replace_node(const char *node_name, Error **errp)
>> * Another benefit is that this tests exclude backing files which are
>> * blocked by the backing blockers.
>> */
>> - if (!bdrv_is_first_non_filter(to_replace_bs)) {
>> + if (!bdrv_recurse_is_first_non_filter(parent_bs, to_replace_bs)) {
>> error_setg(errp, "Only top most non filter can be replaced");
>> to_replace_bs = NULL;
>> goto out;
>> diff --git a/block/mirror.c b/block/mirror.c
>> index 238a070..b81077e 100644
>> --- a/block/mirror.c
>> +++ b/block/mirror.c
>> @@ -626,7 +626,8 @@ static void mirror_complete(BlockJob *job, Error **errp)
>> if (s->replaces) {
>> AioContext *replace_aio_context;
>>
>> - s->to_replace = check_to_replace_node(s->replaces, &local_err);
>> + s->to_replace = check_to_replace_node(s->common.bs, s->replaces,
>> + &local_err);
>
> Why is the check in qmp_drive_mirror not enough? Isn't this redundant?
I guess the reason is that: we only pass replaced node name to the job, and
we only need to get the BDS here. So we can use bdrv_find_node() to replace
check_to_replace_node() here.
Thanks
Wen Congyang
>
> Fam
>
>> if (!s->to_replace) {
>> error_propagate(errp, local_err);
>> return;
>> diff --git a/blockdev.c b/blockdev.c
>> index c11611d..bf12e2e 100644
>> --- a/blockdev.c
>> +++ b/blockdev.c
>> @@ -2757,7 +2757,7 @@ void qmp_drive_mirror(const char *device, const char *target,
>> goto out;
>> }
>>
>> - to_replace_bs = check_to_replace_node(replaces, &local_err);
>> + to_replace_bs = check_to_replace_node(bs, replaces, &local_err);
>>
>> if (!to_replace_bs) {
>> error_propagate(errp, local_err);
>> diff --git a/include/block/block.h b/include/block/block.h
>> index 37916f7..608cd4e 100644
>> --- a/include/block/block.h
>> +++ b/include/block/block.h
>> @@ -317,7 +317,8 @@ bool bdrv_recurse_is_first_non_filter(BlockDriverState *bs,
>> bool bdrv_is_first_non_filter(BlockDriverState *candidate);
>>
>> /* check if a named node can be replaced when doing drive-mirror */
>> -BlockDriverState *check_to_replace_node(const char *node_name, Error **errp);
>> +BlockDriverState *check_to_replace_node(BlockDriverState *parent_bs,
>> + const char *node_name, Error **errp);
>>
>> /* async block I/O */
>> typedef void BlockDriverDirtyHandler(BlockDriverState *bs, int64_t sector,
>> --
>> 2.4.3
>>
> .
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-07-17 1:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-15 1:41 [Qemu-devel] [PATCH v2] more check for replaced node Wen Congyang
2015-07-15 3:05 ` Fam Zheng
2015-07-15 3:22 ` Wen Congyang
2015-07-16 13:36 ` Stefan Hajnoczi
2015-07-17 1:35 ` Wen Congyang
2015-07-17 1:37 ` Wen Congyang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.