diff options
| author | Dr. David Alan Gilbert | 2017-02-24 19:28:44 +0100 |
|---|---|---|
| committer | Dr. David Alan Gilbert | 2017-02-28 12:30:24 +0100 |
| commit | 665414ad06aa1bc92e615db9641e58fb13d07de1 (patch) | |
| tree | 3eb2c360c674a94203e08f9b110c9a1eba6137ba /migration | |
| parent | postcopy: Add doc about hugepages and postcopy (diff) | |
| download | qemu-665414ad06aa1bc92e615db9641e58fb13d07de1.tar.gz qemu-665414ad06aa1bc92e615db9641e58fb13d07de1.tar.xz qemu-665414ad06aa1bc92e615db9641e58fb13d07de1.zip | |
postcopy: Add extra check for COPY function
As an extra sanity check, make sure the region we're registering
can perform UFFDIO_COPY; the COPY will fail later but this
gives a cleaner failure.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Laurent Vivier <lvivier@redhat.com>
Message-Id: <20170224182844.32452-17-dgilbert@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Diffstat (limited to 'migration')
| -rw-r--r-- | migration/postcopy-ram.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/migration/postcopy-ram.c b/migration/postcopy-ram.c index 102fb61485..effbeb64fb 100644 --- a/migration/postcopy-ram.c +++ b/migration/postcopy-ram.c @@ -378,6 +378,10 @@ static int ram_block_enable_notify(const char *block_name, void *host_addr, error_report("%s userfault register: %s", __func__, strerror(errno)); return -1; } + if (!(reg_struct.ioctls & ((__u64)1 << _UFFDIO_COPY))) { + error_report("%s userfault: Region doesn't support COPY", __func__); + return -1; + } return 0; } |