diff options
| author | Kunkun Jiang <jiangkunkun@huawei.com> | 2021-03-16 20:57:15 +0800 |
|---|---|---|
| committer | Dr. David Alan Gilbert <dgilbert@redhat.com> | 2021-05-13 18:21:13 +0100 |
| commit | 23feba906e42747463aa233fb54c58d7f02430c9 (patch) | |
| tree | ee1fb3534404923f15cccb29061fe78792d5758a /migration | |
| parent | 1a37352277763220739290c689867540ec193d06 (diff) | |
| download | qemu-23feba906e42747463aa233fb54c58d7f02430c9.zip qemu-23feba906e42747463aa233fb54c58d7f02430c9.tar.gz qemu-23feba906e42747463aa233fb54c58d7f02430c9.tar.bz2 | |
migration/ram: Reduce unnecessary rate limiting
When the host page is a huge page and something is sent in the
current iteration, migration_rate_limit() should be executed.
If not, it can be omitted.
Signed-off-by: Keqian Zhu <zhukeqian1@huawei.com>
Signed-off-by: Kunkun Jiang <jiangkunkun@huawei.com>
Reviewed-by: David Edmondson <david.edmondson@oracle.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Message-Id: <20210316125716.1243-2-jiangkunkun@huawei.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Diffstat (limited to 'migration')
| -rw-r--r-- | migration/ram.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/migration/ram.c b/migration/ram.c index bee2756..00b579b 100644 --- a/migration/ram.c +++ b/migration/ram.c @@ -2035,8 +2035,13 @@ static int ram_save_host_page(RAMState *rs, PageSearchStatus *pss, pages += tmppages; pss->page++; - /* Allow rate limiting to happen in the middle of huge pages */ - migration_rate_limit(); + /* + * Allow rate limiting to happen in the middle of huge pages if + * something is sent in the current iteration. + */ + if (pagesize_bits > 1 && tmppages > 0) { + migration_rate_limit(); + } } while ((pss->page & (pagesize_bits - 1)) && offset_in_ramblock(pss->block, ((ram_addr_t)pss->page) << TARGET_PAGE_BITS)); |