From nobody Sun Nov 24 17:33:37 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; 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=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1720634525; cv=none; d=zohomail.com; s=zohoarc; b=aM9OlBLzh11HsWDXFHLi63TLyiy+z3K60MhyQVFy0lYV3Z/SS41b4s85cTXDBGLnO7IaFDuigE4801Br3Nt/XwIgUZagUX+mqNat1aAQryswUe/XGiBnMkmcRzSC7XbzIjmZqypGsigDI19M4qXPK4UMPgEsB5qAUU5xLtX9GK0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1720634525; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=J+znbF9pwZIVd04bzjM+SUKkhSFYNYBhE43jpaEEmEw=; b=Uz3f38vjaxgvYgaly3DbPYLSYcN+SS89+H8zOImk/j0DiPckz2kJyP6dJ4qACiF3s+1TLTMhGbsC8Spu7wo82AaR5xYFnhioPkLCckteidk0PVN8PwtWapJb9b7nydb98LVQBQUuPy5nDuZZ/i7UPvVK2CIcPn6PU4S+RINujOQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; 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=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1720634525693330.45850393560397; Wed, 10 Jul 2024 11:02:05 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1sRbce-0008CG-RP; Wed, 10 Jul 2024 14:00:56 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1sRbcb-00082L-Ty for qemu-devel@nongnu.org; Wed, 10 Jul 2024 14:00:53 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1sRbcZ-0003JP-QV for qemu-devel@nongnu.org; Wed, 10 Jul 2024 14:00:53 -0400 Received: from mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-288-beyLO8Q4Phyes2mEywwywA-1; Wed, 10 Jul 2024 14:00:47 -0400 Received: from mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.17]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 3F5B919560AD for ; Wed, 10 Jul 2024 18:00:46 +0000 (UTC) Received: from t14s.redhat.com (unknown [10.39.192.75]) by mx-prod-int-05.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTP id 544B81955F40; Wed, 10 Jul 2024 18:00:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1720634450; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=J+znbF9pwZIVd04bzjM+SUKkhSFYNYBhE43jpaEEmEw=; b=OylzPiIW+/IdblRxRuhPMsEvNFpiOOuyG9XEAnbxPejnK1N7C+IdzpFRjWCNpKS/ZRPrxc 4ZFdBWjD+qE7EMhrPW+TGkhhULYEPiMeLu372OhjzmQJ6soFMkp+fbSCINHcJox21FasZP gunRyFdgR0Gck20yScAti8InUhVAePc= X-MC-Unique: beyLO8Q4Phyes2mEywwywA-1 From: David Hildenbrand To: qemu-devel@nongnu.org Cc: Mario Casquero , Liang Cong , "Michael S . Tsirkin" , David Hildenbrand Subject: [PULL 1/1] virtio-mem: improve error message when unplug of device fails due to plugged memory Date: Wed, 10 Jul 2024 20:00:36 +0200 Message-ID: <20240710180036.101149-2-david@redhat.com> In-Reply-To: <20240710180036.101149-1-david@redhat.com> References: <20240710180036.101149-1-david@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.0 on 10.30.177.17 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=170.10.133.124; envelope-from=david@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -21 X-Spam_score: -2.2 X-Spam_bar: -- X-Spam_report: (-2.2 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.144, 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_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1720634526221116600 Content-Type: text/plain; charset="utf-8" The error message is actually expressive, considering QEMU only. But when called from Libvirt, talking about "size" can be confusing, because in Libvirt "size" translates to the memory backend size in QEMU (maximum size) and "current" translates to the QEMU "size" property. Let's simply avoid talking about the "size" property and spell out that some device memory is still plugged. Message-ID: <20240416141426.588544-1-david@redhat.com> Tested-by: Mario Casquero Cc: Liang Cong Cc: Mario Casquero Cc: "Michael S. Tsirkin" Signed-off-by: David Hildenbrand --- hw/virtio/virtio-mem.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/virtio/virtio-mem.c b/hw/virtio/virtio-mem.c index ffd119ebac..ef64bf1b4a 100644 --- a/hw/virtio/virtio-mem.c +++ b/hw/virtio/virtio-mem.c @@ -1832,8 +1832,8 @@ static void virtio_mem_unplug_request_check(VirtIOMEM= *vmem, Error **errp) } =20 if (vmem->size) { - error_setg(errp, "virtio-mem device cannot get unplugged while" - " '" VIRTIO_MEM_SIZE_PROP "' !=3D '0'"); + error_setg(errp, "virtio-mem device cannot get unplugged while som= e" + " of its memory is still plugged"); return; } if (vmem->requested_size) { --=20 2.45.2