diff options
| author | Peter Xu <peterx@redhat.com> | 2022-01-19 16:09:17 +0800 |
|---|---|---|
| committer | Juan Quintela <quintela@redhat.com> | 2022-01-28 15:38:23 +0100 |
| commit | 2d1c37c603ab3ed9ec1abfc4f4491b8680b23a77 (patch) | |
| tree | 57eaa239d89182f7efb29ef7f44867da86660af9 | |
| parent | 258f5c9825a9fb77aeae8a9d8c0877e714c090e2 (diff) | |
| download | qemu-2d1c37c603ab3ed9ec1abfc4f4491b8680b23a77.zip qemu-2d1c37c603ab3ed9ec1abfc4f4491b8680b23a77.tar.gz qemu-2d1c37c603ab3ed9ec1abfc4f4491b8680b23a77.tar.bz2 | |
migration: Enable UFFD_FEATURE_THREAD_ID even without blocktime feat
This patch allows us to read the tid even without blocktime feature enabled.
It's useful when tracing postcopy fault thread on faulted pages to show thread
id too with the address.
Remove the comments - they're merely not helpful at all.
Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
| -rw-r--r-- | migration/postcopy-ram.c | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/migration/postcopy-ram.c b/migration/postcopy-ram.c index d18b5d0..2176ed6 100644 --- a/migration/postcopy-ram.c +++ b/migration/postcopy-ram.c @@ -283,15 +283,13 @@ static bool ufd_check_and_apply(int ufd, MigrationIncomingState *mis) } #ifdef UFFD_FEATURE_THREAD_ID - if (migrate_postcopy_blocktime() && mis && - UFFD_FEATURE_THREAD_ID & supported_features) { - /* kernel supports that feature */ - /* don't create blocktime_context if it exists */ - if (!mis->blocktime_ctx) { - mis->blocktime_ctx = blocktime_context_new(); - } - + if (UFFD_FEATURE_THREAD_ID & supported_features) { asked_features |= UFFD_FEATURE_THREAD_ID; + if (migrate_postcopy_blocktime()) { + if (!mis->blocktime_ctx) { + mis->blocktime_ctx = blocktime_context_new(); + } + } } #endif |