On Fri, Jul 24, 2020 at 11:34:12AM +0800, Wang Xin wrote:
The shmem 'name' specifies the shared memory path in '/dev/shm/', however, we may need to change it to avoid filename conflict when VM migrate to other host. This patch remove shmem name consistency check.
There is one extra thing you need to consider when allowing this. And that is "is that value part of the migration stream"? Even though this applies only to QEMU (in which case this could be handled there only, but the whole feature is QEMU-only, so...) we need to check that before allowing it. The easiest check is to actually just migrate such machine. I tried it and works so it's fine. Reviewed-by: Martin Kletzander <mkletzan@xxxxxxxxxx> Same as with the first one I'll push it after the release. Thanks for the patches.
Signed-off-by: Wang Xin <wangxinxin.wang@xxxxxxxxxx>
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index bede53d5aa..59cc61ea49 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -23768,13 +23768,6 @@ static bool
virDomainShmemDefCheckABIStability(virDomainShmemDefPtr src,
virDomainShmemDefPtr dst)
{
- if (STRNEQ_NULLABLE(src->name, dst->name)) {
- virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
- _("Target shared memory name '%s' does not match source "
- "'%s'"), dst->name, src->name);
- return false;
- }
-
if (src->role != dst->role) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
_("Target shared memory role '%s' does not match "
--
2.23.0
Attachment:
signature.asc
Description: PGP signature
