[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 29/30] migration: Drop rs->f
From: |
Juan Quintela |
Subject: |
[PULL 29/30] migration: Drop rs->f |
Date: |
Tue, 15 Nov 2022 16:35:13 +0100 |
From: Peter Xu <peterx@redhat.com>
Now with rs->pss we can already cache channels in pss->pss_channels. That
pss_channel contains more infromation than rs->f because it's per-channel.
So rs->f could be replaced by rss->pss[RAM_CHANNEL_PRECOPY].pss_channel,
while rs->f itself is a bit vague now.
Note that vanilla postcopy still send pages via pss[RAM_CHANNEL_PRECOPY],
that's slightly confusing but it reflects the reality.
Then, after the replacement we can safely drop rs->f.
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
---
migration/ram.c | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 88e61b0aeb..29e413b97b 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -351,8 +351,6 @@ struct RAMSrcPageRequest {
/* State of RAM for migration */
struct RAMState {
- /* QEMUFile used for this migration */
- QEMUFile *f;
/*
* PageSearchStatus structures for the channels when send pages.
* Protected by the bitmap_mutex.
@@ -2560,8 +2558,6 @@ static int ram_find_and_save_block(RAMState *rs)
}
if (found) {
- /* Cache rs->f in pss_channel (TODO: remove rs->f) */
- pss->pss_channel = rs->f;
pages = ram_save_host_page(rs, pss);
}
} while (!pages && again);
@@ -3117,7 +3113,7 @@ static void ram_state_resume_prepare(RAMState *rs,
QEMUFile *out)
ram_state_reset(rs);
/* Update RAMState cache of output QEMUFile */
- rs->f = out;
+ rs->pss[RAM_CHANNEL_PRECOPY].pss_channel = out;
trace_ram_state_resume_prepare(pages);
}
@@ -3208,7 +3204,7 @@ static int ram_save_setup(QEMUFile *f, void *opaque)
return -1;
}
}
- (*rsp)->f = f;
+ (*rsp)->pss[RAM_CHANNEL_PRECOPY].pss_channel = f;
WITH_RCU_READ_LOCK_GUARD() {
qemu_put_be64(f, ram_bytes_total_common(true) |
RAM_SAVE_FLAG_MEM_SIZE);
@@ -3343,7 +3339,7 @@ static int ram_save_iterate(QEMUFile *f, void *opaque)
out:
if (ret >= 0
&& migration_is_setup_or_active(migrate_get_current()->state)) {
- ret = multifd_send_sync_main(rs->f);
+ ret = multifd_send_sync_main(rs->pss[RAM_CHANNEL_PRECOPY].pss_channel);
if (ret < 0) {
return ret;
}
@@ -3413,7 +3409,7 @@ static int ram_save_complete(QEMUFile *f, void *opaque)
return ret;
}
- ret = multifd_send_sync_main(rs->f);
+ ret = multifd_send_sync_main(rs->pss[RAM_CHANNEL_PRECOPY].pss_channel);
if (ret < 0) {
return ret;
}
--
2.38.1
- [PULL 21/30] migration: Use atomic ops properly for page accountings, (continued)
- [PULL 21/30] migration: Use atomic ops properly for page accountings, Juan Quintela, 2022/11/15
- [PULL 23/30] migration: Introduce pss_channel, Juan Quintela, 2022/11/15
- [PULL 19/30] migration: Remove RAMState.f references in compression code, Juan Quintela, 2022/11/15
- [PULL 17/30] migration: Cleanup xbzrle zero page cache update logic, Juan Quintela, 2022/11/15
- [PULL 18/30] migration: Trivial cleanup save_page_header() on same block check, Juan Quintela, 2022/11/15
- [PULL 24/30] migration: Add pss_init(), Juan Quintela, 2022/11/15
- [PULL 25/30] migration: Make PageSearchStatus part of RAMState, Juan Quintela, 2022/11/15
- [PULL 22/30] migration: Teach PSS about host page, Juan Quintela, 2022/11/15
- [PULL 27/30] migration: Send requested page directly in rp-return thread, Juan Quintela, 2022/11/15
- [PULL 30/30] migration: Block migration comment or code is wrong, Juan Quintela, 2022/11/15
- [PULL 29/30] migration: Drop rs->f,
Juan Quintela <=
- [PULL 28/30] migration: Remove old preempt code around state maintainance, Juan Quintela, 2022/11/15
- [PULL 26/30] migration: Move last_sent_block into PageSearchStatus, Juan Quintela, 2022/11/15
- Re: [PULL 00/30] Next patches, Daniel P . Berrangé, 2022/11/15
- Re: [PULL 00/30] Next patches, Stefan Hajnoczi, 2022/11/15
- Re: [PULL 00/30] Next patches, Stefan Hajnoczi, 2022/11/15