diff options
author | Dr. David Alan Gilbert <dgilbert@redhat.com> | 2020-07-01 10:35:57 +0100 |
---|---|---|
committer | Dr. David Alan Gilbert <dgilbert@redhat.com> | 2020-07-03 16:23:05 +0100 |
commit | 617a32f5295ee4efcc17abadcecc3cf482c98e80 (patch) | |
tree | a24f41cf8f2c56bdf55c6dfd06ee42f789f170dc /migration | |
parent | 3005c099ef3c6b43213e1454296c1c6556345805 (diff) | |
download | qemu-617a32f5295ee4efcc17abadcecc3cf482c98e80.zip qemu-617a32f5295ee4efcc17abadcecc3cf482c98e80.tar.gz qemu-617a32f5295ee4efcc17abadcecc3cf482c98e80.tar.bz2 |
migration: postcopy take proper error return
This function returns a boolean success and we're returning -1;
lets just use the 'out' error path.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Fixes: 58b7c17e226 ("Disable mlock around incoming postcopy")
Buglink: https://bugs.launchpad.net/qemu/+bug/1885720
Message-Id: <20200701093557.130096-1-dgilbert@redhat.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Diffstat (limited to 'migration')
-rw-r--r-- | migration/postcopy-ram.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/migration/postcopy-ram.c b/migration/postcopy-ram.c index a364027..bef2a3a 100644 --- a/migration/postcopy-ram.c +++ b/migration/postcopy-ram.c @@ -389,7 +389,7 @@ bool postcopy_ram_supported_by_host(MigrationIncomingState *mis) */ if (munlockall()) { error_report("%s: munlockall: %s", __func__, strerror(errno)); - return -1; + goto out; } /* |