[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] migration: cleanup check on ops in savevm.ha
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH v2] migration: cleanup check on ops in savevm.handlers iterations |
Date: |
Thu, 12 Sep 2019 10:55:43 +0100 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
* Wei Yang (address@hidden) wrote:
> During migration, there are several places to iterate on
> savevm.handlers. And on each iteration, we need to check its ops and
> related callbacks before invoke it.
>
> Generally, ops is the first element to check, and it is only necessary
> to check it once.
>
> This patch clean all the related part in savevm.c to check ops only once
> in those iterations.
>
> Signed-off-by: Wei Yang <address@hidden>
Queued
>
> ---
> v2: rebase on latest upstream
> ---
> migration/savevm.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/migration/savevm.c b/migration/savevm.c
> index 4a86128ac4..35426d1db8 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -1100,7 +1100,7 @@ void qemu_savevm_state_setup(QEMUFile *f)
> if (!se->ops || !se->ops->save_setup) {
> continue;
> }
> - if (se->ops && se->ops->is_active) {
> + if (se->ops->is_active) {
> if (!se->ops->is_active(se->opaque)) {
> continue;
> }
> @@ -1131,7 +1131,7 @@ int qemu_savevm_state_resume_prepare(MigrationState *s)
> if (!se->ops || !se->ops->resume_prepare) {
> continue;
> }
> - if (se->ops && se->ops->is_active) {
> + if (se->ops->is_active) {
> if (!se->ops->is_active(se->opaque)) {
> continue;
> }
> @@ -1227,7 +1227,7 @@ void qemu_savevm_state_complete_postcopy(QEMUFile *f)
> if (!se->ops || !se->ops->save_live_complete_postcopy) {
> continue;
> }
> - if (se->ops && se->ops->is_active) {
> + if (se->ops->is_active) {
> if (!se->ops->is_active(se->opaque)) {
> continue;
> }
> @@ -1264,7 +1264,7 @@ int
> qemu_savevm_state_complete_precopy_iterable(QEMUFile *f, bool in_postcopy)
> continue;
> }
>
> - if (se->ops && se->ops->is_active) {
> + if (se->ops->is_active) {
> if (!se->ops->is_active(se->opaque)) {
> continue;
> }
> @@ -1413,7 +1413,7 @@ void qemu_savevm_state_pending(QEMUFile *f, uint64_t
> threshold_size,
> if (!se->ops || !se->ops->save_live_pending) {
> continue;
> }
> - if (se->ops && se->ops->is_active) {
> + if (se->ops->is_active) {
> if (!se->ops->is_active(se->opaque)) {
> continue;
> }
> @@ -2334,7 +2334,7 @@ static int qemu_loadvm_state_setup(QEMUFile *f)
> if (!se->ops || !se->ops->load_setup) {
> continue;
> }
> - if (se->ops && se->ops->is_active) {
> + if (se->ops->is_active) {
> if (!se->ops->is_active(se->opaque)) {
> continue;
> }
> --
> 2.17.1
>
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2] migration: cleanup check on ops in savevm.handlers iterations,
Dr. David Alan Gilbert <=