From nobody Tue May 21 03:12:19 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=1713276909; cv=none; d=zohomail.com; s=zohoarc; b=CceFMJxHcoi5Fze8Ila+wUCw9/Z4kw8wpUhNolP5N8Hu91fHlO78pkJ8+hhSmL5yDr+OkXMLiH9zJZSLxZZHFurqzgKOb5d5TkPFmLbOTN5TyG2I3oo7xzT2CXQOLhyrsqIPFt06kXBHUxoa+VVwubqMqZ6DCLRBNxVmkfBQ0Ko= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1713276909; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=7YpE1eEq60PPyPPGLOLzheBER822cSy7TAfjKqgTugw=; b=f3UDQS+iekHn8yWp/7PSyDrAGUc4NV0ORnEoDAFmAbTYSpCx5ZJhHF6bbhxdzgJefgp82RFrEHr7VpgCU8DSRzzAKcqjD4dhuZd7q7mD7A32puk3BHtzFbng8z22wwLXM9gBK1W7jAyfunKnuNxymNSgRSoKo/iNRA2UL1b+1Vc= 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 1713276909748346.58606266448135; Tue, 16 Apr 2024 07:15:09 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rwjaA-0004my-K2; Tue, 16 Apr 2024 10:14:47 -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 1rwja5-0004mW-9Q for qemu-devel@nongnu.org; Tue, 16 Apr 2024 10:14:41 -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 1rwja2-0005c7-Se for qemu-devel@nongnu.org; Tue, 16 Apr 2024 10:14:40 -0400 Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-660-dBMF9b63OTCv3rC4_5XKSA-1; Tue, 16 Apr 2024 10:14:34 -0400 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.rdu2.redhat.com [10.11.54.8]) (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 mimecast-mx02.redhat.com (Postfix) with ESMTPS id 587341802A04 for ; Tue, 16 Apr 2024 14:14:34 +0000 (UTC) Received: from t14s.redhat.com (unknown [10.39.194.131]) by smtp.corp.redhat.com (Postfix) with ESMTP id 91EE0C13FA2; Tue, 16 Apr 2024 14:14:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1713276876; 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; bh=7YpE1eEq60PPyPPGLOLzheBER822cSy7TAfjKqgTugw=; b=bM0c/ZXLOnz0HZBEI9Nve1fyG88IfHTdfHbTwKeudmdI4ySmJAh4Ug38tBgbWcQmZjum8J 2rZgrslqEIJm7Mc4uHOJagkUIwY1p/xe5PRZz1EsKNTsphaCrIzJFAkaoxTGISjjll5yUx O3BHTi9/DdC+4d2zwQaCN2D2oCv1HXg= X-MC-Unique: dBMF9b63OTCv3rC4_5XKSA-1 From: David Hildenbrand To: qemu-devel@nongnu.org Cc: David Hildenbrand , Liang Cong , Mario Casquero , "Michael S. Tsirkin" Subject: [PATCH v1] virtio-mem: improve error message when unplug of device fails due to plugged memory Date: Tue, 16 Apr 2024 16:14:26 +0200 Message-ID: <20240416141426.588544-1-david@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.8 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: -38 X-Spam_score: -3.9 X-Spam_bar: --- X-Spam_report: (-3.9 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-1.844, 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: 1713276911949100007 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. Cc: Liang Cong Cc: Mario Casquero Cc: "Michael S. Tsirkin" Signed-off-by: David Hildenbrand Tested-by: Mario Casquero --- 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.44.0