From nobody Sun Feb 8 21:28:48 2026 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1595862740; cv=none; d=zohomail.com; s=zohoarc; b=Wu8djbXLMea/fuTveCdSQWjQ93H2vk29qjPAChb8bbPzcSYscB323OtZMAqjop9gyPNi0t+zuO06+UEnxFPYVW2s0IMYkfV1mQeb5TDBHA2ZulJKsC2+45/qmTs2++FlqQT/QK4GDyxBWoXhdKJcdD7+YQidQ1xNFKO3yHv6uLk= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1595862740; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=WCkLAXBjLGBbuz5WvnpxcNCWRuwJufaoH+ZkmO8gEUo=; b=KxaBU3YtWwI3NHuY5j+jX+WpFriLfzJI1ksxWMqa5eWb0m526qhwPinQxniW/W1Iu/YP7Q90WkSejaFcxkeZHLZxNy7t4AqPOlFDIk8jm1f8kS/LO7qGrKaetcyjBorLjiB61Na3FyrkHYE5kLcbKOaYWP4Yr1f17S2S2a7+uD0= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1595862740507933.5532650177971; Mon, 27 Jul 2020 08:12:20 -0700 (PDT) Received: from localhost ([::1]:47976 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1k04nb-0002ni-9R for importer@patchew.org; Mon, 27 Jul 2020 11:12:19 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:37838) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1k04ki-0006na-S7 for qemu-devel@nongnu.org; Mon, 27 Jul 2020 11:09:20 -0400 Received: from us-smtp-delivery-1.mimecast.com ([205.139.110.120]:21415 helo=us-smtp-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1k04kg-0007gw-5Y for qemu-devel@nongnu.org; Mon, 27 Jul 2020 11:09:20 -0400 Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-442-u1eDLRbGOzecB4PXvwGilQ-1; Mon, 27 Jul 2020 11:09:15 -0400 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 708881932483; Mon, 27 Jul 2020 15:09:14 +0000 (UTC) Received: from localhost.localdomain.com (unknown [10.36.110.5]) by smtp.corp.redhat.com (Postfix) with ESMTP id 9048E60BF4; Mon, 27 Jul 2020 15:09:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1595862557; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=WCkLAXBjLGBbuz5WvnpxcNCWRuwJufaoH+ZkmO8gEUo=; b=IA/T3Nbp2YLuRfzKBKy8DHumr5RdvOVlGv/+aFUDbN6jnfRCXqXxYXMs+kvR5BojRSegS2 Fz9lVOOQ1NWQwrVuT9GA08sqKmdUEhbw8u0yASrH1Eg3jHVqHxGSWp8ZWIfsQyENoNigvZ L9ippMb2sFIcWSn3t54b8be+Gl1kRZU= X-MC-Unique: u1eDLRbGOzecB4PXvwGilQ-1 From: =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= To: qemu-devel@nongnu.org Subject: [PATCH v2 5/6] block: allow specifying name of block device for vmstate storage Date: Mon, 27 Jul 2020 16:08:42 +0100 Message-Id: <20200727150843.3419256-6-berrange@redhat.com> In-Reply-To: <20200727150843.3419256-1-berrange@redhat.com> References: <20200727150843.3419256-1-berrange@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=205.139.110.120; envelope-from=berrange@redhat.com; helo=us-smtp-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/07/27 01:44:14 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -40 X-Spam_score: -4.1 X-Spam_bar: ---- X-Spam_report: (-4.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-1, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-1, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Wolf , Peter Krempa , =?UTF-8?q?Daniel=20P=2E=20Berrang=C3=A9?= , "Denis V. Lunev" , qemu-block@nongnu.org, Juan Quintela , Markus Armbruster , "Dr. David Alan Gilbert" , Max Reitz , John Snow , Pavel Dovgalyuk , Paolo Bonzini Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Currently the vmstate will be stored in the first block device that supports snapshots. Historically this would have usually been the root device, but with UEFI it might be the variable store. There needs to be a way to override the choice of block device to store the state in. Signed-off-by: Daniel P. Berrang=C3=A9 --- block/monitor/block-hmp-cmds.c | 2 +- block/snapshot.c | 64 ++++++++++++++++++++++++++-------- include/block/snapshot.h | 4 ++- migration/savevm.c | 4 +-- 4 files changed, 56 insertions(+), 18 deletions(-) diff --git a/block/monitor/block-hmp-cmds.c b/block/monitor/block-hmp-cmds.c index db76c43cc2..81d1b52262 100644 --- a/block/monitor/block-hmp-cmds.c +++ b/block/monitor/block-hmp-cmds.c @@ -900,7 +900,7 @@ void hmp_info_snapshots(Monitor *mon, const QDict *qdic= t) SnapshotEntry *snapshot_entry; Error *err =3D NULL; =20 - bs =3D bdrv_all_find_vmstate_bs(NULL, &err); + bs =3D bdrv_all_find_vmstate_bs(NULL, NULL, &err); if (!bs) { error_report_err(err); return; diff --git a/block/snapshot.c b/block/snapshot.c index f2600a8c7f..b1ad70e278 100644 --- a/block/snapshot.c +++ b/block/snapshot.c @@ -551,27 +551,63 @@ int bdrv_all_create_snapshot(QEMUSnapshotInfo *sn, return 0; } =20 -BlockDriverState *bdrv_all_find_vmstate_bs(strList *devices, Error **errp) +BlockDriverState *bdrv_all_find_vmstate_bs(const char *vmstate_bs, + strList *devices, + Error **errp) { BlockDriverState *bs; BdrvNextIterator it; =20 - for (bs =3D bdrv_first(&it); bs; bs =3D bdrv_next(&it)) { - AioContext *ctx =3D bdrv_get_aio_context(bs); - bool found; + if (vmstate_bs) { + bool usable =3D false; + for (bs =3D bdrv_first(&it); bs; bs =3D bdrv_next(&it)) { + AioContext *ctx =3D bdrv_get_aio_context(bs); + bool match; =20 - aio_context_acquire(ctx); - found =3D bdrv_all_snapshots_includes_bs(bs, devices) && - bdrv_can_snapshot(bs); - aio_context_release(ctx); + aio_context_acquire(ctx); + if (g_str_equal(vmstate_bs, bdrv_get_node_name(bs))) { + match =3D true; + usable =3D bdrv_can_snapshot(bs); + } + aio_context_release(ctx); + if (match) { + bdrv_next_cleanup(&it); + break; + } + } + if (!bs) { + error_setg(errp, + "block device '%s' does not exist", + vmstate_bs); + return NULL; + } =20 - if (found) { - bdrv_next_cleanup(&it); - break; + if (!usable) { + error_setg(errp, + "block device '%s' does not support snapshots", + vmstate_bs); + return NULL; + } + } else { + for (bs =3D bdrv_first(&it); bs; bs =3D bdrv_next(&it)) { + AioContext *ctx =3D bdrv_get_aio_context(bs); + bool found; + + aio_context_acquire(ctx); + found =3D bdrv_all_snapshots_includes_bs(bs, devices) && + bdrv_can_snapshot(bs); + aio_context_release(ctx); + + if (found) { + bdrv_next_cleanup(&it); + break; + } + } + + if (!bs) { + error_setg(errp, "No block device supports snapshots"); + return NULL; } - } - if (!bs) { - error_setg(errp, "No block device supports snapshots"); } return bs; } diff --git a/include/block/snapshot.h b/include/block/snapshot.h index 1c5b0705a9..05550e5da1 100644 --- a/include/block/snapshot.h +++ b/include/block/snapshot.h @@ -86,6 +86,8 @@ int bdrv_all_create_snapshot(QEMUSnapshotInfo *sn, strList *devices, Error **errp); =20 -BlockDriverState *bdrv_all_find_vmstate_bs(strList *devices, Error **errp); +BlockDriverState *bdrv_all_find_vmstate_bs(const char *vmstate_bs, + strList *devices, + Error **errp); =20 #endif diff --git a/migration/savevm.c b/migration/savevm.c index cdc1f2f2d8..1707fa30db 100644 --- a/migration/savevm.c +++ b/migration/savevm.c @@ -2664,7 +2664,7 @@ int save_snapshot(const char *name, Error **errp) } } =20 - bs =3D bdrv_all_find_vmstate_bs(NULL, errp); + bs =3D bdrv_all_find_vmstate_bs(NULL, NULL, errp); if (bs =3D=3D NULL) { return ret; } @@ -2854,7 +2854,7 @@ int load_snapshot(const char *name, Error **errp) return -1; } =20 - bs_vm_state =3D bdrv_all_find_vmstate_bs(NULL, errp); + bs_vm_state =3D bdrv_all_find_vmstate_bs(NULL, NULL, errp); if (!bs_vm_state) { return -1; } --=20 2.26.2