2:26 a.m.
In qemuMigrationRun the jobinfo type won't be updated until
qemuMigrationWaitForCompletion. If migration is failed or cancelled before
that(such as in qemuMigrationDriveMirror), we can't get the right jobinfo
type. The following patches fix it.
Wang Rui (2):
qemu: set jobinfo type to CANCELLED if migration is cancelled in all
conditions
qemu: set jobinfo type to FAILED if job is failed in qemuMigrationRun
src/qemu/qemu_migration.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
--
1.7.12.4
2:26 a.m.
New subject: [libvirt] [PATCH 1/2] qemu: set jobinfo type to CANCELLED if migration is cancelled in all conditions
The migration job status is traced in qemuMigrationUpdateJobStatus which is
called in qemuMigrationRun. But if migration is cancelled before the trace
such as in qemuMigrationDriveMirror, the jobinfo type won't be update to
CANCELLED. After this patch, we can get jobinfo type as CANCELLED if migration
is cancelled during the drive mirror migration.
Signed-off-by: Wang Rui <moon.wangrui(a)huawei.com>
---
src/qemu/qemu_migration.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index a1b1458..c678ba7 100644
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -1504,6 +1504,7 @@ qemuMigrationDriveMirror(virQEMUDriverPtr driver,
* as this is a critical section so we are guaranteed
* priv->job.asyncAbort will not change */
qemuDomainObjExitMonitor(driver, vm);
+ priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
_("canceled by client"));
@@ -3611,6 +3612,7 @@ qemuMigrationRun(virQEMUDriverPtr driver,
* as this is a critical section so we are guaranteed
* priv->job.asyncAbort will not change */
qemuDomainObjExitMonitor(driver, vm);
+ priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
_("canceled by client"));
--
1.7.12.4
4:48 a.m.
New subject: [libvirt] [PATCH 1/2] qemu: set jobinfo type to CANCELLED if migration is cancelled in all conditions
On Thu, Nov 27, 2014 at 04:26:24PM +0800, Wang Rui wrote:
The migration job status is traced in qemuMigrationUpdateJobStatus
which is
called in qemuMigrationRun. But if migration is cancelled before the trace
such as in qemuMigrationDriveMirror, the jobinfo type won't be update to
CANCELLED. After this patch, we can get jobinfo type as CANCELLED if migration
is cancelled during the drive mirror migration.
Signed-off-by: Wang Rui <moon.wangrui(a)huawei.com>
---
src/qemu/qemu_migration.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index a1b1458..c678ba7 100644
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -1504,6 +1504,7 @@ qemuMigrationDriveMirror(virQEMUDriverPtr driver,
* as this is a critical section so we are guaranteed
* priv->job.asyncAbort will not change */
qemuDomainObjExitMonitor(driver, vm);
+ priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
_("canceled by client"));
@@ -3611,6 +3612,7 @@ qemuMigrationRun(virQEMUDriverPtr driver,
* as this is a critical section so we are guaranteed
* priv->job.asyncAbort will not change */
qemuDomainObjExitMonitor(driver, vm);
+ priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
_("canceled by client"));
--
1.7.12.4
Can't we use qemuMigrationUpdateJobStatus() here?
Martin
12:24 a.m.
New subject: [libvirt] [PATCH 1/2] qemu: set jobinfo type to CANCELLED if migration is cancelled in all conditions
On 2014/11/27 18:48, Martin Kletzander wrote:
On Thu, Nov 27, 2014 at 04:26:24PM +0800, Wang Rui wrote:
> The migration job status is traced in qemuMigrationUpdateJobStatus which is
> called in qemuMigrationRun. But if migration is cancelled before the trace
> such as in qemuMigrationDriveMirror, the jobinfo type won't be update to
> CANCELLED. After this patch, we can get jobinfo type as CANCELLED if migration
> is cancelled during the drive mirror migration.
>
> Signed-off-by: Wang Rui <moon.wangrui(a)huawei.com>
> ---
> src/qemu/qemu_migration.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
> index a1b1458..c678ba7 100644
> --- a/src/qemu/qemu_migration.c
> +++ b/src/qemu/qemu_migration.c
> @@ -1504,6 +1504,7 @@ qemuMigrationDriveMirror(virQEMUDriverPtr driver,
> * as this is a critical section so we are guaranteed
> * priv->job.asyncAbort will not change */
> qemuDomainObjExitMonitor(driver, vm);
> + priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
> virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
>
qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
> _("canceled by client"));
> @@ -3611,6 +3612,7 @@ qemuMigrationRun(virQEMUDriverPtr driver,
> * as this is a critical section so we are guaranteed
> * priv->job.asyncAbort will not change */
> qemuDomainObjExitMonitor(driver, vm);
> + priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
> virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
> qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
> _("canceled by client"));
> --
> 1.7.12.4
>
Can't we use qemuMigrationUpdateJobStatus() here?
Martin
Hi, thanks for your reply.
I don't think we can.
I tried this way but couldn't get jobinfo type as CANCELLED if I cancelled it
during drive mirror migration. When libvirt sends qmp "migrate_cancel" to QEMU,
QEMU only sets its state if its old state is MIG_STATE_SETUP or MIG_STATE_ACTIVE.
But in drive mirror migration, QEMU's state is MIG_STATE_NONE.
static void migrate_fd_cancel(MigrationState *s)
{
...
old_state = s->state;
if (old_state != MIG_STATE_SETUP && old_state != MIG_STATE_ACTIVE) {
break;
...
}
So we haven't set QEMU's state to CANCELLED. And then if we query status in
libvirt
by qemuMigrationUpdateJobStatus, QEMU won't tell us the right status.
MigrationInfo *qmp_query_migrate(Error **errp)
{
...
switch (s->state) {
case MIG_STATE_NONE:
/* no migration has happened ever */
break;
...
}
Another reason why I don't use qemuMigrationUpdateJobStatus is considering job failed
in
libvirt(in my patch 2/2). If job is failed in libvirt, it's not accurate to query job
status from QEMU.
2:29 p.m.
New subject: [libvirt] [PATCH 1/2] qemu: set jobinfo type to CANCELLED if migration is cancelled in all conditions
On Fri, Nov 28, 2014 at 02:24:56PM +0800, Wang Rui wrote:
On 2014/11/27 18:48, Martin Kletzander wrote:
> On Thu, Nov 27, 2014 at 04:26:24PM +0800, Wang Rui wrote:
>> The migration job status is traced in qemuMigrationUpdateJobStatus which is
>> called in qemuMigrationRun. But if migration is cancelled before the trace
>> such as in qemuMigrationDriveMirror, the jobinfo type won't be update to
>> CANCELLED. After this patch, we can get jobinfo type as CANCELLED if migration
>> is cancelled during the drive mirror migration.
>>
>> Signed-off-by: Wang Rui <moon.wangrui(a)huawei.com>
>> ---
>> src/qemu/qemu_migration.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
>> index a1b1458..c678ba7 100644
>> --- a/src/qemu/qemu_migration.c
>> +++ b/src/qemu/qemu_migration.c
>> @@ -1504,6 +1504,7 @@ qemuMigrationDriveMirror(virQEMUDriverPtr driver,
>> * as this is a critical section so we are guaranteed
>> * priv->job.asyncAbort will not change */
>> qemuDomainObjExitMonitor(driver, vm);
>> + priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
>> virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
>>
qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
>> _("canceled by client"));
>> @@ -3611,6 +3612,7 @@ qemuMigrationRun(virQEMUDriverPtr driver,
>> * as this is a critical section so we are guaranteed
>> * priv->job.asyncAbort will not change */
>> qemuDomainObjExitMonitor(driver, vm);
>> + priv->job.current->type = VIR_DOMAIN_JOB_CANCELLED;
>> virReportError(VIR_ERR_OPERATION_ABORTED, _("%s: %s"),
>> qemuDomainAsyncJobTypeToString(priv->job.asyncJob),
>> _("canceled by client"));
>> --
>> 1.7.12.4
>>
>
> Can't we use qemuMigrationUpdateJobStatus() here?
>
> Martin
Hi, thanks for your reply.
I don't think we can.
I tried this way but couldn't get jobinfo type as CANCELLED if I cancelled it
during drive mirror migration. When libvirt sends qmp "migrate_cancel" to QEMU,
QEMU only sets its state if its old state is MIG_STATE_SETUP or MIG_STATE_ACTIVE.
But in drive mirror migration, QEMU's state is MIG_STATE_NONE.
static void migrate_fd_cancel(MigrationState *s)
{
...
old_state = s->state;
if (old_state != MIG_STATE_SETUP && old_state != MIG_STATE_ACTIVE) {
break;
...
}
So we haven't set QEMU's state to CANCELLED. And then if we query status in
libvirt
by qemuMigrationUpdateJobStatus, QEMU won't tell us the right status.
MigrationInfo *qmp_query_migrate(Error **errp)
{
...
switch (s->state) {
case MIG_STATE_NONE:
/* no migration has happened ever */
break;
...
}
Another reason why I don't use qemuMigrationUpdateJobStatus is considering job failed
in
libvirt(in my patch 2/2). If job is failed in libvirt, it's not accurate to query job
status from QEMU.
Yes, you're right, that basically what I wanted to know, but was too
busy searching for. ACK to this patch then. Would you be OK with me
rewording the commit message as follows?
The migration job status is traced in qemuMigrationUpdateJobStatus
which is called in qemuMigrationRun. But if migration is cancelled
before the trace such as in qemuMigrationDriveMirror, the jobinfo
type won't be updated to CANCELLED. After this patch, we can get
jobinfo type CANCELLED if migration is cancelled during drive
mirror. Moreover, we can't use qemuMigrationUpdateJobStatus
because from qemu's point of view it's just the drive mirror being
cancelled and the migration hasn't even started yet.
Martin
1:02 a.m.
New subject: [libvirt] [PATCH 1/2] qemu: set jobinfo type to CANCELLED if migration is cancelled in all conditions
On 2014/12/1 4:29, Martin Kletzander wrote:
Yes, you're right, that basically what I wanted to know, but was
too
busy searching for. ACK to this patch then. Would you be OK with me
rewording the commit message as follows?
The migration job status is traced in qemuMigrationUpdateJobStatus
which is called in qemuMigrationRun. But if migration is cancelled
before the trace such as in qemuMigrationDriveMirror, the jobinfo
type won't be updated to CANCELLED. After this patch, we can get
jobinfo type CANCELLED if migration is cancelled during drive
mirror. Moreover, we can't use qemuMigrationUpdateJobStatus
because from qemu's point of view it's just the drive mirror being
cancelled and the migration hasn't even started yet.
Martin
Yes, I'll send V2 right now.
1:16 a.m.
New subject: [libvirt] [PATCH 1/2] qemu: set jobinfo type to CANCELLED if migration is cancelled in all conditions
On Mon, Dec 01, 2014 at 03:02:53PM +0800, Wang Rui wrote:
On 2014/12/1 4:29, Martin Kletzander wrote:
> Yes, you're right, that basically what I wanted to know, but was too
> busy searching for. ACK to this patch then. Would you be OK with me
> rewording the commit message as follows?
>
> The migration job status is traced in qemuMigrationUpdateJobStatus
> which is called in qemuMigrationRun. But if migration is cancelled
> before the trace such as in qemuMigrationDriveMirror, the jobinfo
> type won't be updated to CANCELLED. After this patch, we can get
> jobinfo type CANCELLED if migration is cancelled during drive
> mirror. Moreover, we can't use qemuMigrationUpdateJobStatus
> because from qemu's point of view it's just the drive mirror being
> cancelled and the migration hasn't even started yet.
>
> Martin
Yes, I'll send V2 right now.
No need to, I have it prepared for pushing with this amended message
:-)
So thanks, I'm pushing it in a while.
Martin
2:26 a.m.
New subject: [libvirt] [PATCH 2/2] qemu: set jobinfo type to FAILED if job is failed in qemuMigrationRun
If job is failed in qemuMigrationRun, we expect the jobinfo type as
FAILED. But jobinfo type won't be updated until entering
qemuMigrationWaitForCompletion. We should make it updated in all
conditions.
Signed-off-by: Wang Rui <moon.wangrui(a)huawei.com>
---
src/qemu/qemu_migration.c | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
index c678ba7..75c77ae 100644
--- a/src/qemu/qemu_migration.c
+++ b/src/qemu/qemu_migration.c
@@ -3723,8 +3723,10 @@ qemuMigrationRun(virQEMUDriverPtr driver,
* confirm3 step, but need to release the lock state
*/
if (virDomainObjGetState(vm, NULL) == VIR_DOMAIN_RUNNING) {
- if (qemuMigrationSetOffline(driver, vm) < 0)
+ if (qemuMigrationSetOffline(driver, vm) < 0) {
+ priv->job.current->type = VIR_DOMAIN_JOB_FAILED;
goto cleanup;
+ }
}
ret = 0;
@@ -3748,6 +3750,9 @@ qemuMigrationRun(virQEMUDriverPtr driver,
qemuDomainJobInfoUpdateDowntime(priv->job.completed);
}
+ if (priv->job.current->type == VIR_DOMAIN_JOB_UNBOUNDED)
+ priv->job.current->type = VIR_DOMAIN_JOB_FAILED;
+
cookieFlags |= QEMU_MIGRATION_COOKIE_NETWORK |
QEMU_MIGRATION_COOKIE_STATS;
if (flags & VIR_MIGRATE_PERSIST_DEST)
--
1.7.12.4
3679
days inactive
3683
days old
7 comments
2 participants
participants (2)
-
Martin Kletzander -
Wang Rui