[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] migration: not necessary to check ops again
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH] migration: not necessary to check ops again |
Date: |
Tue, 14 May 2019 12:26:48 +0100 |
User-agent: |
Mutt/1.11.4 (2019-03-13) |
* Wei Yang (address@hidden) wrote:
> During each iteration, se->ops is checked before each loop. So it is not
> necessary to check it again and simplify the following check a little.
>
> Signed-off-by: Wei Yang <address@hidden>
Queued
> ---
> migration/savevm.c | 14 ++++++--------
> 1 file changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/migration/savevm.c b/migration/savevm.c
> index 1415001d1c..5f0ca7fac2 100644
> --- a/migration/savevm.c
> +++ b/migration/savevm.c
> @@ -1157,15 +1157,13 @@ int qemu_savevm_state_iterate(QEMUFile *f, bool
> postcopy)
> if (!se->ops || !se->ops->save_live_iterate) {
> continue;
> }
> - if (se->ops && se->ops->is_active) {
> - if (!se->ops->is_active(se->opaque)) {
> - continue;
> - }
> + if (se->ops->is_active &&
> + !se->ops->is_active(se->opaque)) {
> + continue;
> }
> - if (se->ops && se->ops->is_active_iterate) {
> - if (!se->ops->is_active_iterate(se->opaque)) {
> - continue;
> - }
> + if (se->ops->is_active_iterate &&
> + !se->ops->is_active_iterate(se->opaque)) {
> + continue;
> }
> /*
> * In the postcopy phase, any device that doesn't know how to
> --
> 2.19.1
>
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH] migration: not necessary to check ops again,
Dr. David Alan Gilbert <=