From nobody Fri Apr 26 09:36:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1637760586; cv=none; d=zohomail.com; s=zohoarc; b=nN558ZCkMHnFwNwR/xrx5u3SoTgiTsDjudCfoWjiAo+/T5qXG7TORO78GXlDilzJWd+evw6ffCnF8c7tWcnOuuAZE8Hl5l5zWhuK7plerLwLNVKzQNFnWSQWvKqs8m4wPZTRaEUmLOcXTGpddjaAGY4aW+QPzShDxExTu0f5EIM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637760586; h=Content-Type:Content-Transfer-Encoding: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=/EYULd4Ro3nW8ssAPeaTD5o1IFSjmBpkJahyMuYUgBI=; b=Hwm+B2usgjp0lws02ccjGJVx+RF8vC2bRSlFsXC0V9KNhBcNPMfEajIcAsnsCYiy4qdkn+qo9AaHDJ7X3qQwnJTPrGgAYPpptSUIFV6/DWKhiKkvi7cK4f5NFO8PPfzUpxPaRT2bN2LV71R0b5Kp4X6IBVZ7/By8wQDtvz1MsxY= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1637760586205507.41471871519514; Wed, 24 Nov 2021 05:29:46 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-46-BXq5zBUgPXSp5yZ5Zj3Fxg-1; Wed, 24 Nov 2021 08:29:42 -0500 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 0B8E51006AA3; Wed, 24 Nov 2021 13:29:38 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id E2C0760CC9; Wed, 24 Nov 2021 13:29:37 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id B75434A707; Wed, 24 Nov 2021 13:29:37 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 1AODTLh4012581 for ; Wed, 24 Nov 2021 08:29:21 -0500 Received: by smtp.corp.redhat.com (Postfix) id 5EF5418171; Wed, 24 Nov 2021 13:29:21 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.72]) by smtp.corp.redhat.com (Postfix) with ESMTP id DBF545D9CA for ; Wed, 24 Nov 2021 13:29:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1637760584; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=/EYULd4Ro3nW8ssAPeaTD5o1IFSjmBpkJahyMuYUgBI=; b=NCpM/OzMRTUNFNyyNc3tEtTAuKmLtHY1DkSHXihGv8xTCmhLyQz6NazZu276NyfhdWtlqL QMv1H8mviDcNoIU+6VpYuCUk1A0jFSyA5cWSvnztyacdAkClvqjjHTFeE4EF0lUwRA843Q 3JC0D7WalkDXCUPLWV4mSYvPYUyFgt8= X-MC-Unique: BXq5zBUgPXSp5yZ5Zj3Fxg-1 From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Subject: [libvirt PATCH 1/7] qemu: qemuDomainObjExitMonitor: do not warn on unused result Date: Wed, 24 Nov 2021 14:28:49 +0100 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1637760586992100002 This wrapper for qemuDomainObjExitMonitorInternal was extended by my commit dc2fd51fd727bbb6de172e0ca4b7dd307bb99180 to check whether the domain is still alive, because we were observing crashes if the QEMU process died while some of our APIs were in the monitor and the thread processing the EOF event freed the domain definition. This bug was fixed by: commit 81f50cb92d16643bcd749e3ab5b404b8b7cec643 qemu: Avoid calling qemuProcessStop without a job but we kept checking for the return value since. Remove the G_GNUC_WARN_UNUSED_RESULT attribute since all of the calls that could set def->id to -1 are protected by qemuProcessBeginStopJob and cannot happen while we have a job in the monitor. Signed-off-by: J=C3=A1n Tomko Reviewed-by: Michal Privoznik --- src/qemu/qemu_domain.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/qemu/qemu_domain.h b/src/qemu/qemu_domain.h index 6728ab047e..596add616d 100644 --- a/src/qemu/qemu_domain.h +++ b/src/qemu/qemu_domain.h @@ -489,8 +489,7 @@ void qemuDomainObjEnterMonitor(virQEMUDriver *driver, ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); int qemuDomainObjExitMonitor(virQEMUDriver *driver, virDomainObj *obj) - ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2) - G_GNUC_WARN_UNUSED_RESULT; + ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); int qemuDomainObjEnterMonitorAsync(virQEMUDriver *driver, virDomainObj *obj, qemuDomainAsyncJob asyncJob) --=20 2.31.1 From nobody Fri Apr 26 09:36:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1637760573; cv=none; d=zohomail.com; s=zohoarc; b=n/xCNubjDVqYw6Xjjpi46EE44kkvbxxPfh5tqhDXq8EArLp134DJgT6ev3LG2vmYrVJPEhBAH7Z5/u1qew4FXOrKjuZrLkZUmkltRP44vh/xLZcCHTiadSd0sPcSK4oFTm5Oml636nzs3qC06VUtYqWzCuNzf1NhPWPf7L1UGjw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637760573; h=Content-Type:Content-Transfer-Encoding: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=6jrZPoYtin2qjyOpgSebWzlIeZnm9dlSJ1kL+WE+Zhg=; b=J5Ss1XKSrbzNDNOyZ+rD3HT3PeRZ3CaQpNzddoUvVF1DycRwfFvl4z5d0ZRb1WZouF5qiT/cDjfBLTs/fc9CqHt6qeeVaqNmMal6BQB1uFSBQVVwJlkyibFo463UqN36SnFaytmWJX1mZlhSJg+R0LkpjbBd06wELt/KaQu/ZIE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1637760573673954.4202183646024; Wed, 24 Nov 2021 05:29:33 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-573-ER_HtgCzMLCTkYxvJ7XEdw-1; Wed, 24 Nov 2021 08:29:30 -0500 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id B3A0E1023F4E; Wed, 24 Nov 2021 13:29:25 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id CD4115DF21; Wed, 24 Nov 2021 13:29:24 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 8CA364BB7C; Wed, 24 Nov 2021 13:29:24 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 1AODTM4w012586 for ; Wed, 24 Nov 2021 08:29:22 -0500 Received: by smtp.corp.redhat.com (Postfix) id 2FC9EADDC; Wed, 24 Nov 2021 13:29:22 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.72]) by smtp.corp.redhat.com (Postfix) with ESMTP id ACA665DF21 for ; Wed, 24 Nov 2021 13:29:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1637760572; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=6jrZPoYtin2qjyOpgSebWzlIeZnm9dlSJ1kL+WE+Zhg=; b=eHhjuSqjdtMwkjQ0/D7QyWWSgc/xjd/2qO39eMwBpBaEs6IVEsM437tTRWupWPLEjtP6aU 4BGIg2jE9P8hTTs7dEFK1XyNof+xj+sqC1ySRDd1ZBQuuH3CUxQ4QDSNbM2H6Zm0nTUAWK GOQV6YOg5zVbWv0CYU+VlijOIUiMPmo= X-MC-Unique: ER_HtgCzMLCTkYxvJ7XEdw-1 From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Subject: [libvirt PATCH 2/7] qemu: do not propagate return value of qemuDomainObjExitMonitor Date: Wed, 24 Nov 2021 14:28:50 +0100 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1637760575437100005 Signed-off-by: J=C3=A1n Tomko Reviewed-by: Michal Privoznik --- src/qemu/qemu_hotplug.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index 3373ec2cdf..7ae0088c6a 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -1816,7 +1816,8 @@ qemuDomainAddTLSObjects(virQEMUDriver *driver, qemuMonitorAddObject(priv->mon, tlsProps, NULL) < 0) goto error; =20 - return qemuDomainObjExitMonitor(driver, vm); + qemuDomainObjExitMonitor(driver, vm); + return 0; =20 error: virErrorPreserveLast(&orig_err); --=20 2.31.1 From nobody Fri Apr 26 09:36:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) client-ip=170.10.129.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1637760586; cv=none; d=zohomail.com; s=zohoarc; b=M+cmtyWgyWyB4P6jruasKpyUcLnAPoGV7yS83QNP5tZ1+ZFshhDA1cx5Tdjmc0kIL5dO/fkBuCTj2GD1TABNaf98kot5GYz1jjufKHy1shKe+4srewpAVr2KfBwtd0MyQgEaE5uTKukIDvSK+MLItSXz4rXZJ+l3o+ySSE5ZVjw= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637760586; h=Content-Type:Content-Transfer-Encoding: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=dnV6Yfglbqgrv1UwDbS2yESEdLCiaVOJGQzstC3gZRw=; b=E5HimTREBPLh/gXkGvhhwOEWy2G+17bC7SAhQ+0yOLJCTVLqeqArg5ryruOIZceUJjG7cP1Ck2k0Q4mnpMjcxvEHlv3CwmMEMemaIzPIt1egTQEExxhvV3TQV/FXWGm8eyYfNqbfzWDxZ7NNCC0spfIp6e343flcLx4l2nSWzyQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mx.zohomail.com with SMTPS id 1637760586593628.000722187618; Wed, 24 Nov 2021 05:29:46 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-521-VTY182LTNUGEs2VT5eqfQA-1; Wed, 24 Nov 2021 08:29:43 -0500 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id E2424193F561; Wed, 24 Nov 2021 13:29:37 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id C4F4D10246E8; Wed, 24 Nov 2021 13:29:37 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 99C8F1809CB9; Wed, 24 Nov 2021 13:29:37 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 1AODTNY6012592 for ; Wed, 24 Nov 2021 08:29:23 -0500 Received: by smtp.corp.redhat.com (Postfix) id 4F864ADDC; Wed, 24 Nov 2021 13:29:23 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.72]) by smtp.corp.redhat.com (Postfix) with ESMTP id 7EE9218171 for ; Wed, 24 Nov 2021 13:29:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1637760585; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=dnV6Yfglbqgrv1UwDbS2yESEdLCiaVOJGQzstC3gZRw=; b=hagHNPsaK4aXTsZp1qyR2nRHZY1yv2IZGc5vxRAT5uWxCS30qrZsh9g5/LaM6s9U2ruvVX sj6MVqKQaMyjxRXMgNyFbizSYQODg53NGPJr4XuJlQRoMSGHW8SOE/OrIO34vuDBznyjlD PkqYkdKu4jL1/RNNG1TPED+LTkgow1o= X-MC-Unique: VTY182LTNUGEs2VT5eqfQA-1 From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Subject: [libvirt PATCH 3/7] qemu: do not check return value of qemuDomainObjExitMonitor Date: Wed, 24 Nov 2021 14:28:51 +0100 Message-Id: In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1637760587215100004 Remove the unreachable code. Signed-off-by: J=C3=A1n Tomko Reviewed-by: Michal Privoznik --- src/qemu/qemu_backup.c | 9 +- src/qemu/qemu_block.c | 3 +- src/qemu/qemu_blockjob.c | 28 ++--- src/qemu/qemu_checkpoint.c | 3 +- src/qemu/qemu_domain.c | 14 +-- src/qemu/qemu_driver.c | 110 +++++++------------- src/qemu/qemu_hotplug.c | 170 +++++++++---------------------- src/qemu/qemu_migration.c | 27 ++--- src/qemu/qemu_migration_params.c | 6 +- src/qemu/qemu_process.c | 54 ++++------ src/qemu/qemu_snapshot.c | 6 +- 11 files changed, 134 insertions(+), 296 deletions(-) diff --git a/src/qemu/qemu_backup.c b/src/qemu/qemu_backup.c index 9fa8d2f02e..193d4ccd29 100644 --- a/src/qemu/qemu_backup.c +++ b/src/qemu/qemu_backup.c @@ -677,8 +677,7 @@ qemuBackupJobCancelBlockjobs(virDomainObj *vm, =20 rc =3D qemuMonitorBlockJobCancel(priv->mon, job->name, true); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - return; + qemuDomainObjExitMonitor(priv->driver, vm); =20 if (rc =3D=3D 0) { backupdisk->state =3D VIR_DOMAIN_BACKUP_DISK_STATE_CANCELLING; @@ -910,8 +909,7 @@ qemuBackupBegin(virDomainObj *vm, /* note that if the export fails we've already created the checkpo= int * and we will not delete it */ rc =3D qemuBackupBeginPullExportDisks(vm, dd, ndd); - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(priv->driver, vm); =20 if (rc < 0) { qemuBackupJobCancelBlockjobs(vm, priv->backup, false, QEMU_ASY= NC_JOB_BACKUP); @@ -1003,8 +1001,7 @@ qemuBackupNotifyBlockjobEnd(virDomainObj *vm, ignore_value(qemuMonitorDelObject(priv->mon, backup->tlsAlias,= false)); if (backup->tlsSecretAlias) ignore_value(qemuMonitorDelObject(priv->mon, backup->tlsSecret= Alias, false)); - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - return; + qemuDomainObjExitMonitor(priv->driver, vm); =20 /* update the final statistics with the current job's data */ backup->pull_tmp_used +=3D cur; diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c index 53a0b919f9..44579a83ea 100644 --- a/src/qemu/qemu_block.c +++ b/src/qemu/qemu_block.c @@ -2136,8 +2136,7 @@ qemuBlockStorageSourceDetachOneBlockdev(virQEMUDriver= *driver, if (ret =3D=3D 0) ret =3D qemuMonitorBlockdevDel(qemuDomainGetMonitor(vm), src->node= storage); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } diff --git a/src/qemu/qemu_blockjob.c b/src/qemu/qemu_blockjob.c index faf9a9fb7d..e7c3fea946 100644 --- a/src/qemu/qemu_blockjob.c +++ b/src/qemu/qemu_blockjob.c @@ -530,8 +530,7 @@ qemuBlockJobRefreshJobs(virQEMUDriver *driver, if (rc =3D=3D -1 && jobinfo[i]->status =3D=3D QEMU_MONITOR_JOB= _STATUS_CONCLUDED) VIR_WARN("can't cancel job '%s' with invalid data", job->n= ame); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) qemuBlockJobUnregister(job, vm); @@ -847,8 +846,8 @@ qemuBlockJobEventProcessConcludedRemoveChain(virQEMUDri= ver *driver, return; =20 qemuBlockStorageSourceChainDetach(qemuDomainGetMonitor(vm), data); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return; + + qemuDomainObjExitMonitor(driver, vm); =20 qemuDomainStorageSourceChainAccessRevoke(driver, vm, chain); } @@ -964,8 +963,7 @@ qemuBlockJobProcessEventCompletedPullBitmaps(virDomainO= bj *vm, =20 qemuMonitorTransaction(priv->mon, &actions); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(priv->driver, vm); =20 return 0; } @@ -1138,8 +1136,7 @@ qemuBlockJobProcessEventCompletedCommitBitmaps(virDom= ainObj *vm, =20 qemuMonitorTransaction(priv->mon, &actions); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(priv->driver, vm); =20 if (!active) { if (qemuBlockReopenReadOnly(vm, job->data.commit.base, asyncJob) <= 0) @@ -1359,8 +1356,7 @@ qemuBlockJobProcessEventCompletedCopyBitmaps(virDomai= nObj *vm, =20 qemuMonitorTransaction(priv->mon, &actions); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(priv->driver, vm); =20 return 0; } @@ -1442,8 +1438,7 @@ qemuBlockJobProcessEventFailedActiveCommit(virQEMUDri= ver *driver, disk->mirror->nodeformat, "libvirt-tmp-activewrite"); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - return; + qemuDomainObjExitMonitor(priv->driver, vm); =20 /* Ideally, we would make the backing chain read only again (yes, SELi= nux * can do that using different labels). But that is not implemented ye= t and @@ -1491,8 +1486,7 @@ qemuBlockJobProcessEventConcludedCreate(virQEMUDriver= *driver, =20 qemuBlockStorageSourceAttachRollback(qemuDomainGetMonitor(vm), backend= ); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return; + qemuDomainObjExitMonitor(driver, vm); =20 qemuDomainStorageSourceAccessRevoke(driver, vm, job->data.create.src); } @@ -1527,8 +1521,7 @@ qemuBlockJobProcessEventConcludedBackup(virQEMUDriver= *driver, job->disk->src->nodeformat, job->data.backup.bitmap); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return; + qemuDomainObjExitMonitor(driver, vm); =20 if (job->data.backup.store) qemuDomainStorageSourceAccessRevoke(driver, vm, job->data.backup.s= tore); @@ -1641,8 +1634,7 @@ qemuBlockJobEventProcessConcluded(qemuBlockJobData *j= ob, /* dismiss job in qemu */ ignore_value(qemuMonitorJobDismiss(qemuDomainGetMonitor(vm), job->name= )); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 if ((job->newstate =3D=3D QEMU_BLOCKJOB_STATE_COMPLETED || job->newstate =3D=3D QEMU_BLOCKJOB_STATE_FAILED) && diff --git a/src/qemu/qemu_checkpoint.c b/src/qemu/qemu_checkpoint.c index 4902585e5d..38ea57dea7 100644 --- a/src/qemu/qemu_checkpoint.c +++ b/src/qemu/qemu_checkpoint.c @@ -237,8 +237,7 @@ qemuCheckpointDiscardBitmaps(virDomainObj *vm, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorTransaction(priv->mon, &actions); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 relabel: for (next =3D relabelimages; next; next =3D next->next) { diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index 1f45b30975..d0d9a31f37 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -8181,8 +8181,7 @@ qemuDomainUpdateDeviceList(virQEMUDriver *driver, if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0) return -1; rc =3D qemuMonitorGetDeviceAliases(priv->mon, &aliases); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); if (rc < 0) return -1; =20 @@ -8210,10 +8209,7 @@ qemuDomainUpdateMemoryDeviceInfo(virQEMUDriver *driv= er, =20 rc =3D qemuMonitorGetMemoryDeviceInfo(priv->mon, &meminfo); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - virHashFree(meminfo); - return -1; - } + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) return -1; @@ -9480,8 +9476,7 @@ qemuDomainRefreshVcpuInfo(virQEMUDriver *driver, rc =3D qemuMonitorGetCPUInfo(qemuDomainGetMonitor(vm), &info, maxvcpus, hotplug, fast); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) goto cleanup; @@ -10054,8 +10049,7 @@ qemuDomainCheckMonitor(virQEMUDriver *driver, =20 ret =3D qemuMonitorCheck(priv->mon); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index aa48480043..33686f4f23 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -1939,8 +1939,7 @@ qemuDomainShutdownFlagsMonitor(virQEMUDriver *driver, qemuDomainSetFakeReboot(driver, vm, isReboot); qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemPowerdown(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -2065,8 +2064,7 @@ qemuDomainRebootMonitor(virQEMUDriver *driver, qemuDomainSetFakeReboot(driver, vm, isReboot); qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemPowerdown(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -2155,8 +2153,7 @@ qemuDomainReset(virDomainPtr dom, unsigned int flags) priv =3D vm->privateData; qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemReset(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 priv->fakeReboot =3D false; =20 @@ -2471,8 +2468,7 @@ static int qemuDomainSetMemoryStatsPeriod(virDomainPt= r dom, int period, =20 qemuDomainObjEnterMonitor(driver, vm); r =3D qemuMonitorSetMemoryStatsPeriod(priv->mon, def->memballoon, = period); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); if (r < 0) { virReportError(VIR_ERR_OPERATION_INVALID, "%s", _("unable to set balloon driver collection peri= od")); @@ -2529,8 +2525,7 @@ static int qemuDomainInjectNMI(virDomainPtr domain, u= nsigned int flags) =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorInjectNMI(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -2589,8 +2584,7 @@ static int qemuDomainSendKey(virDomainPtr domain, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSendKey(priv->mon, holdtime, keycodes, nkeycodes); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -3309,8 +3303,7 @@ qemuDomainCoreDumpWithFormat(virDomainPtr dom, if ((ret =3D=3D 0) && (flags & VIR_DUMP_RESET)) { qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemReset(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); } =20 if (resume && virDomainObjIsActive(vm)) { @@ -3427,8 +3420,7 @@ qemuDomainScreenshot(virDomainPtr dom, ignore_value(qemuDomainObjExitMonitor(driver, vm)); goto endjob; } - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); =20 if (VIR_CLOSE(tmp_fd) < 0) { virReportSystemError(errno, _("unable to close %s"), tmp); @@ -3928,8 +3920,7 @@ processNicRxFilterChangedEvent(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorQueryRxFilter(priv->mon, devAlias, &guestFilter); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); if (ret < 0) goto endjob; =20 @@ -4975,8 +4966,7 @@ qemuDomainGetIOThreadsMon(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetIOThreads(priv->mon, iothreads, niothreads); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -5263,8 +5253,7 @@ qemuDomainHotplugAddIOThread(virQEMUDriver *driver, if (qemuMonitorGetIOThreads(priv->mon, &new_iothreads, &new_niothreads= ) < 0) goto exit_monitor; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 if (new_niothreads !=3D exp_niothreads) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -5349,8 +5338,7 @@ qemuDomainHotplugModIOThread(virQEMUDriver *driver, =20 rc =3D qemuMonitorSetIOThread(priv->mon, &iothread); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) return -1; @@ -5386,8 +5374,7 @@ qemuDomainHotplugDelIOThread(virQEMUDriver *driver, if (qemuMonitorGetIOThreads(priv->mon, &new_iothreads, &new_niothreads= ) < 0) goto exit_monitor; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 if (new_niothreads !=3D exp_niothreads) { virReportError(VIR_ERR_INTERNAL_ERROR, @@ -9921,8 +9908,7 @@ qemuDomainBlockResize(virDomainPtr dom, ignore_value(qemuDomainObjExitMonitor(driver, vm)); goto endjob; } - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); =20 ret =3D 0; =20 @@ -10638,8 +10624,7 @@ qemuDomainMemoryStatsInternal(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetMemoryStats(qemuDomainGetMonitor(vm), vm->def->memballoon, stats, nr_sta= ts); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0 || ret >=3D nr_stats) return ret; @@ -10816,8 +10801,7 @@ qemuDomainMemoryPeek(virDomainPtr dom, goto endjob; } } - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); =20 /* Read the memory file into buffer. */ if (saferead(fd, buffer, size) =3D=3D (ssize_t)-1) { @@ -12784,8 +12768,7 @@ qemuDomainAbortJobMigration(virDomainObj *vm) qemuDomainObjAbortAsyncJob(vm); qemuDomainObjEnterMonitor(priv->driver, vm); ret =3D qemuMonitorMigrateCancel(priv->mon); - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(priv->driver, vm); =20 return ret; } @@ -13342,8 +13325,7 @@ qemuDomainMigrateStartPostCopy(virDomainPtr dom, VIR_DEBUG("Starting post-copy"); qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorMigrateStartPostCopy(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -14057,8 +14039,7 @@ static int qemuDomainQemuMonitorCommand(virDomainPt= r domain, const char *cmd, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorArbitraryCommand(priv->mon, cmd, result, hmp); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -14333,8 +14314,7 @@ qemuDomainBlockPivot(virQEMUDriver *driver, } else { ret =3D qemuMonitorDrivePivot(priv->mon, job->name); } - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 /* The pivot failed. The block job in QEMU remains in the synchronised= state */ if (ret < 0) @@ -14465,8 +14445,7 @@ qemuDomainBlockPullCommon(virDomainObj *vm, (!baseSource || basePath)) ret =3D qemuMonitorBlockStream(priv->mon, device, jobname, persist= job, basePath, nodebase, backingPath, speed); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0) goto endjob; @@ -14540,10 +14519,7 @@ qemuDomainBlockJobAbort(virDomainPtr dom, } else { qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorBlockJobCancel(priv->mon, job->name, false); - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - ret =3D -1; - goto endjob; - } + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0) goto endjob; @@ -14758,8 +14734,7 @@ qemuDomainBlockJobSetSpeed(virDomainPtr dom, ret =3D qemuMonitorBlockJobSetSpeed(qemuDomainGetMonitor(vm), job->name, speed); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -15142,8 +15117,7 @@ qemuDomainBlockCopyCommon(virDomainObj *vm, if (data) { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuBlockStorageSourceChainAttach(priv->mon, data); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) goto endjob; @@ -15177,8 +15151,7 @@ qemuDomainBlockCopyCommon(virDomainObj *vm, } =20 virDomainAuditDisk(vm, NULL, mirror, "mirror", ret >=3D 0); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); if (ret < 0) { qemuDomainStorageSourceChainAccessRevoke(driver, vm, mirror); goto endjob; @@ -15707,8 +15680,7 @@ qemuDomainOpenGraphics(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorOpenGraphics(priv->mon, protocol, fd, "graphicsfd", (flags & VIR_DOMAIN_OPEN_GRAPHICS_SKIPAU= TH) !=3D 0); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -15784,8 +15756,7 @@ qemuDomainOpenGraphicsFD(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorOpenGraphics(priv->mon, protocol, pair[1], "graphic= sfd", (flags & VIR_DOMAIN_OPEN_GRAPHICS_SKIPAU= TH)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); qemuDomainObjEndJob(driver, vm); if (ret < 0) goto cleanup; @@ -16221,8 +16192,7 @@ qemuDomainSetBlockIoTune(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSetBlockIoThrottle(priv->mon, drivealias, q= devid, &info); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); if (ret < 0) goto endjob; ret =3D -1; @@ -16352,8 +16322,7 @@ qemuDomainGetBlockIoTune(virDomainPtr dom, } qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetBlockIoThrottle(priv->mon, drivealias, qdevi= d, &reply); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); if (ret < 0) goto endjob; } @@ -16472,8 +16441,7 @@ qemuDomainGetDiskErrors(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); table =3D qemuMonitorGetBlockInfo(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); if (!table) goto endjob; =20 @@ -16804,8 +16772,7 @@ qemuDomainPMWakeup(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorSystemWakeup(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); @@ -17365,8 +17332,7 @@ qemuDomainSetTime(virDomainPtr dom, if (virQEMUCapsGet(priv->qemuCaps, QEMU_CAPS_RTC_RESET_REINJECTION)) { qemuDomainObjEnterMonitor(driver, vm); rv =3D qemuMonitorRTCResetReinjection(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); =20 if (rv < 0) goto endjob; @@ -18498,8 +18464,7 @@ qemuDomainGetStatsBlock(virQEMUDriver *driver, if (fetchnodedata) nodedata =3D qemuMonitorQueryNamedBlockNodes(priv->mon); =20 - if (qemuDomainObjExitMonitor(driver, dom) < 0) - return -1; + qemuDomainObjExitMonitor(driver, dom); =20 /* failure to retrieve stats is fine at this point */ if (rc < 0 || (fetchnodedata && !nodedata)) @@ -18637,8 +18602,7 @@ qemuDomainGetStatsDirtyRateMon(virQEMUDriver *drive= r, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorQueryDirtyRate(priv->mon, info); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -20008,8 +19972,7 @@ qemuDomainGetSEVMeasurement(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); tmp =3D qemuMonitorGetSEVMeasurement(QEMU_DOMAIN_PRIVATE(vm)->mon); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); =20 if (!tmp) goto endjob; @@ -20678,8 +20641,7 @@ qemuDomainStartDirtyRateCalc(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorStartDirtyRateCalc(priv->mon, seconds); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 endjob: qemuDomainObjEndJob(driver, vm); diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index 7ae0088c6a..59b3e57fd3 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -109,10 +109,7 @@ qemuDomainDeleteDevice(virDomainObj *vm, =20 rc =3D qemuMonitorDelDevice(priv->mon, alias); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - /* Domain is no longer running. No cleanup needed. */ - return -1; - } + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) { /* Deleting device failed. Let's check if DEVICE_DELETED @@ -272,8 +269,7 @@ qemuDomainChangeMediaLegacy(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorEjectMedia(priv->mon, driveAlias, force); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 /* If the tray is present wait for it to open. */ if (!force && diskPriv->tray) { @@ -305,8 +301,7 @@ qemuDomainChangeMediaLegacy(virQEMUDriver *driver, driveAlias, sourcestr, format); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); } =20 if (rc < 0) @@ -354,8 +349,7 @@ qemuHotplugAttachDBusVMState(virQEMUDriver *driver, if (ret =3D=3D 0) priv->dbusVMState =3D true; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -391,8 +385,7 @@ qemuHotplugRemoveDBusVMState(virQEMUDriver *driver, if (ret =3D=3D 0) priv->dbusVMState =3D false; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -479,8 +472,7 @@ qemuHotplugRemoveManagedPR(virQEMUDriver *driver, goto cleanup; ignore_value(qemuMonitorDelObject(priv->mon, qemuDomainGetManagedPRAli= as(), false)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 qemuProcessKillManagedPRDaemon(vm); =20 @@ -757,8 +749,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriver *driver, =20 rc =3D qemuBlockStorageSourceChainAttach(priv->mon, data); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -2; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) goto rollback; @@ -808,8 +799,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriver *driver, VIR_WARN("failed to set blkdeviotune for '%s' of '%s'", disk->= dst, vm->def->name); } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -2; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) goto rollback; @@ -825,8 +815,7 @@ qemuDomainAttachDiskGeneric(virQEMUDriver *driver, =20 qemuBlockStorageSourceChainDetach(priv->mon, data); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -2; + qemuDomainObjExitMonitor(driver, vm); =20 return -1; } @@ -891,11 +880,7 @@ int qemuDomainAttachControllerDevice(virQEMUDriver *dr= iver, ignore_value(qemuDomainDetachExtensionDevice(priv->mon, &controlle= r->info)); =20 exit_monitor: - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - releaseaddr =3D false; - ret =3D -1; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 if (ret =3D=3D 0) virDomainControllerInsertPreAlloced(vm->def, controller); @@ -1486,8 +1471,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, } netdevPlugged =3D true; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 for (i =3D 0; i < tapfdSize; i++) VIR_FORCE_CLOSE(tapfd[i]); @@ -1511,8 +1495,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 /* set link state */ if (net->linkstate =3D=3D VIR_DOMAIN_NET_INTERFACE_LINK_STATE_DOWN) { @@ -1528,8 +1511,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, goto try_remove; } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); } /* link set to down */ } @@ -1728,8 +1710,7 @@ qemuDomainAttachHostPCIDevice(virQEMUDriver *driver, ignore_value(qemuDomainDetachExtensionDevice(priv->mon, hostdev->i= nfo)); =20 exit_monitor: - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto error; + qemuDomainObjExitMonitor(driver, vm); =20 virDomainAuditHostdev(vm, hostdev, "attach", ret =3D=3D 0); if (ret < 0) @@ -1940,8 +1921,7 @@ qemuDomainDelChardevTLSObjects(virQEMUDriver *driver, if (secAlias) ignore_value(qemuMonitorDelObject(priv->mon, secAlias, false)); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return 0; } @@ -1989,8 +1969,7 @@ int qemuDomainAttachRedirdevDevice(virQEMUDriver *dri= ver, if (qemuMonitorAddDeviceProps(priv->mon, &devprops) < 0) goto exit_monitor; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto audit; + qemuDomainObjExitMonitor(driver, vm); =20 def->redirdevs[def->nredirdevs++] =3D redirdev; ret =3D 0; @@ -2255,8 +2234,7 @@ int qemuDomainAttachChrDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto audit; + qemuDomainObjExitMonitor(driver, vm); =20 qemuDomainChrInsertPreAlloced(vmdef, chr); ret =3D 0; @@ -2363,10 +2341,7 @@ qemuDomainAttachRNGDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - releaseaddr =3D false; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 VIR_APPEND_ELEMENT_INPLACE(vm->def->rngs, vm->def->nrngs, rng); =20 @@ -2393,8 +2368,7 @@ qemuDomainAttachRNGDevice(virQEMUDriver *driver, ignore_value(qemuMonitorDelObject(priv->mon, objAlias, false)); if (rng->backend =3D=3D VIR_DOMAIN_RNG_BACKEND_EGD && chardevAdded) ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - releaseaddr =3D false; + qemuDomainObjExitMonitor(driver, vm); virErrorRestore(&orig_err); =20 qemuDomainDelTLSObjects(driver, vm, QEMU_ASYNC_JOB_NONE, @@ -2487,11 +2461,7 @@ qemuDomainAttachMemory(virQEMUDriver *driver, if (qemuMonitorAddDeviceProps(priv->mon, &devprops) < 0) goto exit_monitor; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - /* we shouldn't touch mem now, as the def might be freed */ - mem =3D NULL; - goto audit; - } + qemuDomainObjExitMonitor(driver, vm); =20 event =3D virDomainEventDeviceAddedNewFromObj(vm, objalias); virObjectEventStateQueue(driver->domainEventState, event); @@ -2531,8 +2501,7 @@ qemuDomainAttachMemory(virQEMUDriver *driver, virErrorPreserveLast(&orig_err); if (objAdded) ignore_value(qemuMonitorDelObject(priv->mon, objalias, false)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - mem =3D NULL; + qemuDomainObjExitMonitor(driver, vm); =20 if (objAdded && mem) ignore_value(qemuProcessDestroyMemoryBackingPath(driver, vm, mem)); @@ -2597,10 +2566,7 @@ qemuDomainAttachHostUSBDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorAddDeviceProps(priv->mon, &devprops); - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - ret =3D -1; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); virDomainAuditHostdev(vm, hostdev, "attach", ret =3D=3D 0); if (ret < 0) goto cleanup; @@ -2694,8 +2660,7 @@ qemuDomainAttachHostSCSIDevice(virQEMUDriver *driver, if (qemuMonitorAddDeviceProps(priv->mon, &devprops) < 0) goto exit_monitor; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 virDomainAuditHostdev(vm, hostdev, "attach", true); =20 @@ -2914,10 +2879,7 @@ qemuDomainAttachMediatedDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorAddDeviceProps(priv->mon, &devprops); - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - ret =3D -1; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 virDomainAuditHostdev(vm, hostdev, "attach", ret =3D=3D 0); if (ret < 0) @@ -3072,10 +3034,7 @@ qemuDomainAttachShmemDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - release_address =3D false; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 /* Doing a copy here just so the pointer doesn't get nullified * because we need it in the audit function */ @@ -3104,8 +3063,7 @@ qemuDomainAttachShmemDevice(virQEMUDriver *driver, ignore_value(qemuMonitorDelObject(priv->mon, memAlias, false)); } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - release_address =3D false; + qemuDomainObjExitMonitor(driver, vm); =20 virErrorRestore(&orig_err); =20 @@ -3205,10 +3163,7 @@ qemuDomainAttachWatchdog(virQEMUDriver *driver, if (rv >=3D 0) rv =3D qemuMonitorAddDeviceProps(priv->mon, &props); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - releaseAddress =3D false; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 if (rv < 0) goto cleanup; @@ -3296,10 +3251,7 @@ qemuDomainAttachInputDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - releaseaddr =3D false; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 VIR_APPEND_ELEMENT_COPY_INPLACE(vm->def->inputs, vm->def->ninputs, inp= ut); =20 @@ -3325,10 +3277,7 @@ qemuDomainAttachInputDevice(virQEMUDriver *driver, return ret; =20 exit_monitor: - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - releaseaddr =3D false; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); goto audit; } =20 @@ -3386,10 +3335,7 @@ qemuDomainAttachVsockDevice(virQEMUDriver *driver, if ((ret =3D qemuMonitorAddDeviceProps(priv->mon, &devprops)) < 0) goto exit_monitor; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - releaseaddr =3D false; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 vm->def->vsock =3D g_steal_pointer(&vsock); =20 @@ -3410,8 +3356,7 @@ qemuDomainAttachVsockDevice(virQEMUDriver *driver, ignore_value(qemuMonitorCloseFileHandle(priv->mon, fdname)); if (removeextension) ignore_value(qemuDomainDetachExtensionDevice(priv->mon, &vsock->in= fo)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - releaseaddr =3D false; + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } =20 @@ -3480,10 +3425,7 @@ qemuDomainAttachFSDevice(virQEMUDriver *driver, goto exit_monitor; } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) { - releaseaddr =3D false; - goto cleanup; - } + qemuDomainObjExitMonitor(driver, vm); =20 VIR_APPEND_ELEMENT_COPY(vm->def->fss, vm->def->nfss, fs); =20 @@ -3507,8 +3449,7 @@ qemuDomainAttachFSDevice(virQEMUDriver *driver, virErrorPreserveLast(&origErr); if (chardevAdded) ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - releaseaddr =3D false; + qemuDomainObjExitMonitor(driver, vm); virErrorRestore(&origErr); goto audit; } @@ -3677,8 +3618,7 @@ int qemuDomainChangeNetLinkState(virQEMUDriver *drive= r, dev->linkstate =3D linkstate; =20 cleanup: - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -4272,8 +4212,7 @@ qemuDomainChangeGraphicsPasswords(virQEMUDriver *driv= er, ret =3D qemuMonitorExpirePassword(priv->mon, type, expire); =20 end_job: - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -4552,8 +4491,7 @@ qemuDomainRemoveDiskDevice(virQEMUDriver *driver, if (diskBackend) qemuBlockStorageSourceChainDetach(priv->mon, diskBackend); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 virDomainAuditDisk(vm, disk->src, NULL, "detach", true); =20 @@ -4629,8 +4567,7 @@ qemuDomainRemoveMemoryDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorDelObject(priv->mon, backendAlias, true); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - rc =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 virDomainAuditMemory(vm, oldmem, newmem, "update", rc =3D=3D 0); if (rc < 0) @@ -4729,8 +4666,7 @@ qemuDomainRemoveHostDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); qemuBlockStorageSourceAttachRollback(priv->mon, detachscsi); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); } =20 if (hostdev->parentnet) { @@ -4847,8 +4783,7 @@ qemuDomainRemoveNetDevice(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); if (qemuMonitorRemoveNetdev(priv->mon, hostnet_name) < 0) { - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); virDomainAuditNet(vm, net, NULL, "detach", false); return -1; } @@ -4894,8 +4829,7 @@ qemuDomainRemoveNetDevice(virQEMUDriver *driver, } =20 =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (QEMU_DOMAIN_NETWORK_PRIVATE(net)->slirp) qemuSlirpStop(QEMU_DOMAIN_NETWORK_PRIVATE(net)->slirp, vm, driver,= net); @@ -4969,8 +4903,7 @@ qemuDomainRemoveChrDevice(virQEMUDriver *driver, if (monitor) { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorDetachCharDev(priv->mon, charAlias); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); } =20 if (rc =3D=3D 0 && @@ -5036,8 +4969,7 @@ qemuDomainRemoveRNGDevice(virQEMUDriver *driver, qemuMonitorDetachCharDev(priv->mon, charAlias) < 0) rc =3D -1; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (rng->backend =3D=3D VIR_DOMAIN_RNG_BACKEND_EGD && rc =3D=3D 0 && @@ -5091,8 +5023,7 @@ qemuDomainRemoveShmemDevice(virQEMUDriver *driver, else rc =3D qemuMonitorDelObject(priv->mon, memAlias, true); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 virDomainAuditShmem(vm, shmem, "detach", rc =3D=3D 0); =20 @@ -5186,8 +5117,7 @@ qemuDomainRemoveRedirdevDevice(virQEMUDriver *driver, */ ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (qemuDomainDelChardevTLSObjects(driver, vm, dev->source, charAlias)= < 0) return -1; @@ -5224,8 +5154,7 @@ qemuDomainRemoveFSDevice(virQEMUDriver *driver, if (qemuMonitorDetachCharDev(priv->mon, charAlias) < 0) rc =3D -1; =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); } =20 virDomainAuditFS(vm, fs, NULL, "detach", rc =3D=3D 0); @@ -5966,8 +5895,7 @@ qemuDomainDetachDeviceChr(virQEMUDriver *driver, int rc; qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorRemoveNetdev(priv->mon, tmpChr->info.alias); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - rc =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) goto cleanup; @@ -6503,8 +6431,7 @@ qemuDomainHotplugAddVcpu(virQEMUDriver *driver, =20 rc =3D qemuMonitorAddDeviceProps(qemuDomainGetMonitor(vm), &vcpuprops); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 virDomainAuditVcpu(vm, oldvcpus, oldvcpus + nvcpus, "update", rc =3D= =3D 0); =20 @@ -7000,8 +6927,7 @@ qemuDomainChangeMemoryRequestedSize(virQEMUDriver *dr= iver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorChangeMemoryRequestedSize(priv->mon, alias, requeste= dsize); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 return rc; } diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c index 8001792f5a..3fd0db15bb 100644 --- a/src/qemu/qemu_migration.c +++ b/src/qemu/qemu_migration.c @@ -515,8 +515,7 @@ qemuMigrationDstStartNBDServer(virQEMUDriver *driver, =20 if (qemuBlockExportAddNBD(vm, diskAlias, disk->src, diskAlias, tru= e, NULL) < 0) goto exit_monitor; - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); } =20 if (server.transport =3D=3D VIR_STORAGE_NET_HOST_TRANS_TCP) @@ -551,8 +550,7 @@ qemuMigrationDstStopNBDServer(virQEMUDriver *driver, =20 if (qemuMonitorNBDServerStop(priv->mon) < 0) VIR_WARN("Unable to stop NBD server"); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 virPortAllocatorRelease(priv->nbdPort); priv->nbdPort =3D 0; @@ -873,8 +871,7 @@ qemuMigrationSrcCancelRemoveTempBitmaps(virDomainObj *v= m, if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0) return -1; qemuMonitorBitmapRemove(priv->mon, t->nodename, t->bitmapname); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); } =20 return 0; @@ -2039,8 +2036,7 @@ qemuMigrationSrcGraphicsRelocate(virQEMUDriver *drive= r, ret =3D qemuMonitorGraphicsRelocate(priv->mon, type, listenAddress, port, tlsPort, tlsSubject); jobPriv->spiceMigration =3D !ret; - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); } =20 cleanup: @@ -3811,8 +3807,7 @@ qemuMigrationSrcContinue(virQEMUDriver *driver, =20 ret =3D qemuMonitorMigrateContinue(priv->mon, status); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -3835,8 +3830,7 @@ qemuMigrationSetDBusVMState(virQEMUDriver *driver, =20 rv =3D qemuMonitorSetDBusVMStateIdList(priv->mon, priv->dbusVMStat= eIds); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - rv =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 return rv; } else { @@ -5932,8 +5926,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, qemuMonitorSetMigrationSpeed(priv->mon, QEMU_DOMAIN_MIG_BANDWIDTH_MAX); priv->migMaxBandwidth =3D QEMU_DOMAIN_MIG_BANDWIDTH_MAX; - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); } } =20 @@ -5984,8 +5977,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, VIR_CLOSE(pipeFD[1]) < 0) VIR_WARN("failed to close intermediate pipe"); } - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); if (rc < 0) goto cleanup; =20 @@ -6057,8 +6049,7 @@ qemuMigrationSrcCancel(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ignore_value(qemuMonitorMigrateCancel(priv->mon)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 for (i =3D 0; i < vm->def->ndisks; i++) { virDomainDiskDef *disk =3D vm->def->disks[i]; diff --git a/src/qemu/qemu_migration_params.c b/src/qemu/qemu_migration_par= ams.c index 8df7aac93a..7bfae29de4 100644 --- a/src/qemu/qemu_migration_params.c +++ b/src/qemu/qemu_migration_params.c @@ -870,8 +870,7 @@ qemuMigrationParamsApply(virQEMUDriver *driver, ret =3D 0; =20 cleanup: - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (xbzrleCacheSize_old) migParams->params[xbzrle].set =3D true; @@ -1412,8 +1411,7 @@ qemuMigrationCapsCheck(virQEMUDriver *driver, =20 rc =3D qemuMonitorSetMigrationCapabilities(priv->mon, &json); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) { virResetLastError(); diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index c355a39e15..1148e6072f 100644 --- a/src/qemu/qemu_process.c +++ b/src/qemu/qemu_process.c @@ -473,8 +473,7 @@ qemuProcessFakeReboot(void *opaque) qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSystemReset(priv->mon); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto endjob; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) goto endjob; @@ -1956,8 +1955,7 @@ qemuProcessInitMonitor(virQEMUDriver *driver, =20 ret =3D qemuMonitorSetCapabilities(QEMU_DOMAIN_PRIVATE(vm)->mon); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 return ret; } @@ -2252,8 +2250,7 @@ qemuRefreshVirtioChannelState(virQEMUDriver *driver, goto cleanup; =20 ret =3D qemuMonitorGetChardevInfo(priv->mon, &info); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0) goto cleanup; @@ -2306,8 +2303,7 @@ qemuRefreshPRManagerState(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetPRManagerInfo(priv->mon, &info); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0) goto cleanup; @@ -2337,8 +2333,7 @@ qemuRefreshRTC(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); now =3D time(NULL); rv =3D qemuMonitorGetRTCTime(priv->mon, &thenbits); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - rv =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (rv < 0) return; @@ -2427,8 +2422,7 @@ qemuProcessWaitForMonitor(virQEMUDriver *driver, goto cleanup; ret =3D qemuMonitorGetChardevInfo(priv->mon, &info); VIR_DEBUG("qemuMonitorGetChardevInfo returned %i", ret); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret =3D=3D 0) { if ((ret =3D qemuProcessFindCharDevicePTYsMonitor(vm, info)) < 0) @@ -2470,8 +2464,7 @@ qemuProcessDetectIOThreadPIDs(virQEMUDriver *driver, if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) < 0) goto cleanup; ret =3D qemuMonitorGetIOThreads(priv->mon, &iothreads, &niothreads); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); if (ret < 0) goto cleanup; =20 @@ -2646,8 +2639,7 @@ qemuProcessSetLinkStates(virQEMUDriver *driver, ret =3D 0; =20 cleanup: - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); return ret; } =20 @@ -3194,8 +3186,7 @@ qemuProcessUpdateVideoRamSize(virQEMUDriver *driver, =20 } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 cfg =3D virQEMUDriverGetConfig(driver); ret =3D virDomainObjSave(vm, driver->xmlopt, cfg->stateDir); @@ -3326,8 +3317,7 @@ qemuProcessStartCPUs(virQEMUDriver *driver, virDomain= Obj *vm, goto release; =20 ret =3D qemuMonitorStartCPUs(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0) goto release; @@ -3364,8 +3354,7 @@ int qemuProcessStopCPUs(virQEMUDriver *driver, goto cleanup; =20 ret =3D qemuMonitorStopCPUs(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0) goto cleanup; @@ -3476,8 +3465,7 @@ qemuProcessUpdateState(virQEMUDriver *driver, virDoma= inObj *vm) =20 qemuDomainObjEnterMonitor(driver, vm); ret =3D qemuMonitorGetStatus(priv->mon, &running, &reason); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (ret < 0) return -1; @@ -3713,8 +3701,7 @@ qemuProcessRecoverJob(virQEMUDriver *driver, case QEMU_ASYNC_JOB_SNAPSHOT: qemuDomainObjEnterMonitor(driver, vm); ignore_value(qemuMonitorMigrateCancel(priv->mon)); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); /* resume the domain but only if it was paused as a result of * running a migration-to-file operation. Although we are * recovering an async job, this function is run at startup @@ -4355,8 +4342,7 @@ qemuProcessFetchGuestCPU(virQEMUDriver *driver, rc =3D qemuMonitorGetGuestCPUx86(priv->mon, &dataEnabled, &dataDis= abled); } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - return -1; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc =3D=3D -1) return -1; @@ -5206,8 +5192,7 @@ qemuProcessSetupBalloon(virQEMUDriver *driver, ret =3D 0; =20 cleanup: - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); return ret; } =20 @@ -6115,8 +6100,7 @@ qemuProcessSetupHotpluggableVcpus(virQEMUDriver *driv= er, =20 rc =3D qemuMonitorAddDeviceProps(qemuDomainGetMonitor(vm), &vcpupr= ops); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); =20 if (rc < 0) goto cleanup; @@ -7023,8 +7007,7 @@ qemuProcessSetupDiskThrottlingBlockdev(virQEMUDriver = *driver, ret =3D 0; =20 cleanup: - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); return ret; } =20 @@ -8337,8 +8320,7 @@ qemuProcessRefreshDisks(virQEMUDriver *driver, =20 if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) =3D=3D 0) { table =3D qemuMonitorGetBlockInfo(priv->mon); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - goto cleanup; + qemuDomainObjExitMonitor(driver, vm); } =20 if (!table) diff --git a/src/qemu/qemu_snapshot.c b/src/qemu/qemu_snapshot.c index 661aeeb8aa..ff827d5313 100644 --- a/src/qemu/qemu_snapshot.c +++ b/src/qemu/qemu_snapshot.c @@ -330,8 +330,7 @@ qemuSnapshotCreateActiveInternal(virQEMUDriver *driver, } =20 ret =3D qemuMonitorCreateSnapshot(priv->mon, snap->def->name); - if (qemuDomainObjExitMonitor(driver, vm) < 0) - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); if (ret < 0) goto cleanup; =20 @@ -1289,8 +1288,7 @@ qemuSnapshotDiskCreate(qemuSnapshotDiskContext *snapc= txt) =20 rc =3D qemuMonitorTransaction(priv->mon, &snapctxt->actions); =20 - if (qemuDomainObjExitMonitor(driver, snapctxt->vm) < 0) - rc =3D -1; + qemuDomainObjExitMonitor(driver, snapctxt->vm); =20 for (i =3D 0; i < snapctxt->ndd; i++) { qemuSnapshotDiskData *dd =3D snapctxt->dd + i; --=20 2.31.1 From nobody Fri Apr 26 09:36:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1637760623; cv=none; d=zohomail.com; s=zohoarc; b=VIG4Qo0ffqkYuFRARwljg3f3OCSHpmnHe4Dzb2hnzpzHoH90h5ss8GvLoqtTERnGrVM2sUoNddQIRaqllraomVBUaU66mD46cAzsg7+3LYjSvE54dbeZBLH5hlzzO+E74csqPO3w/Kp6ii1m54awnWQPJDiiC6YHLPAnj4ynBlc= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637760623; h=Content-Type:Content-Transfer-Encoding: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=3UaXLJ3RwgYqtEwtydo26EV+BohpSiY/FMrCWX8SOvY=; b=MEbav2SaCxBWnA4BsLGL+hhtasMX6ZnKb+YB0CZx9ls3G51+lAwauO86qbWHAy7YyK5U4Lve8hHWBgjw813a2C1WTuxkyf66RvjdikwutNT+9goqCQJ//AANeo+6qNP2uxY9yWTWuX5rU6hnZEgIe8KyHKbfJ2cnpPxAZyZ+Rn4= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1637760623834678.89838726533; Wed, 24 Nov 2021 05:30:23 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-476-b_webplpNyecM5kk6hwyDQ-1; Wed, 24 Nov 2021 08:30:20 -0500 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 4CE26193F566; Wed, 24 Nov 2021 13:30:13 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 2D47B60C17; Wed, 24 Nov 2021 13:30:13 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id EDD061809CB9; Wed, 24 Nov 2021 13:30:12 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 1AODTQ1T012616 for ; Wed, 24 Nov 2021 08:29:26 -0500 Received: by smtp.corp.redhat.com (Postfix) id B832DADDC; Wed, 24 Nov 2021 13:29:26 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.72]) by smtp.corp.redhat.com (Postfix) with ESMTP id 3F2485DF21 for ; Wed, 24 Nov 2021 13:29:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1637760622; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=3UaXLJ3RwgYqtEwtydo26EV+BohpSiY/FMrCWX8SOvY=; b=aZRjcPNklALGM8fOGsYfZt3Wpw6RA2ggBg636AEN+3tC/b80dJfTAnD4pbe97et2cHPFaB z/UAw6RmL8hoib40bl0UPAjO/hBvaSYm25g9kyqN/1PjN5e381B3fb3aQH518lL6p4LKDT qLe3FLGalZWnuWnzkTCWSiiukxiln20= X-MC-Unique: b_webplpNyecM5kk6hwyDQ-1 From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Subject: [libvirt PATCH 4/7] qemu: do not check return value of qemuDomainObjExitMonitor Date: Wed, 24 Nov 2021 14:28:52 +0100 Message-Id: <569bc9bfcf820ef0028355c88f0361becf3d4543.1637760513.git.jtomko@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1637760625713100001 Remove the check from conditions where it's coupled with some other checks. Signed-off-by: J=C3=A1n Tomko Reviewed-by: Michal Privoznik --- src/qemu/qemu_backup.c | 9 ++++--- src/qemu/qemu_block.c | 23 +++++++++++----- src/qemu/qemu_blockjob.c | 3 ++- src/qemu/qemu_checkpoint.c | 9 ++++--- src/qemu/qemu_domain.c | 4 ++- src/qemu/qemu_driver.c | 45 ++++++++++++++++++++------------ src/qemu/qemu_hotplug.c | 15 +++++++---- src/qemu/qemu_migration.c | 24 +++++++++++------ src/qemu/qemu_migration_cookie.c | 3 ++- src/qemu/qemu_migration_params.c | 6 +++-- src/qemu/qemu_process.c | 19 +++++++++----- src/qemu/qemu_snapshot.c | 3 ++- 12 files changed, 109 insertions(+), 54 deletions(-) diff --git a/src/qemu/qemu_backup.c b/src/qemu/qemu_backup.c index 193d4ccd29..8e191d344e 100644 --- a/src/qemu/qemu_backup.c +++ b/src/qemu/qemu_backup.c @@ -474,7 +474,8 @@ qemuBackupDiskPrepareOneStorage(virDomainObj *vm, =20 rc =3D qemuBlockStorageSourceAttachApply(priv->mon, dd->crdata->sr= cdata[0]); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) return -1; } =20 @@ -888,7 +889,8 @@ qemuBackupBegin(virDomainObj *vm, if (rc =3D=3D 0) rc =3D qemuMonitorTransaction(priv->mon, &actions); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) goto endjob; =20 job_started =3D true; @@ -1158,7 +1160,8 @@ qemuBackupGetJobInfoStats(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetJobInfo(priv->mon, &blockjobs, &nblockjobs); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto cleanup; =20 /* count in completed jobs */ diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c index 44579a83ea..04f5e383bc 100644 --- a/src/qemu/qemu_block.c +++ b/src/qemu/qemu_block.c @@ -326,7 +326,9 @@ qemuBlockNodeNamesDetect(virQEMUDriver *driver, data =3D qemuMonitorQueryNamedBlockNodes(qemuDomainGetMonitor(vm)); blockstats =3D qemuMonitorQueryBlockstats(qemuDomainGetMonitor(vm)); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || !data || !blockstats) + qemuDomainObjExitMonitor(driver, vm); + + if (!data || !blockstats) return -1; =20 if (!(disktable =3D qemuBlockNodeNameGetBackingChain(data, blockstats)= )) @@ -2711,7 +2713,8 @@ qemuBlockStorageSourceCreateGeneric(virDomainObj *vm, rc =3D qemuMonitorBlockdevCreate(priv->mon, job->name, props); props =3D NULL; =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) goto cleanup; =20 qemuBlockJobStarted(job, vm); @@ -2857,7 +2860,8 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, =20 rc =3D qemuBlockStorageSourceAttachApplyStorageDeps(priv->mon, data); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) goto cleanup; =20 if (qemuBlockStorageSourceCreateStorage(vm, src, chain, asyncJob) < 0) @@ -2871,7 +2875,8 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, if (rc =3D=3D 0) rc =3D qemuBlockStorageSourceAttachApplyFormatDeps(priv->mon, data= ); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) goto cleanup; =20 if (qemuBlockStorageSourceCreateFormat(vm, src, backingStore, chain, @@ -2889,7 +2894,8 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, =20 rc =3D qemuBlockStorageSourceAttachApplyFormat(priv->mon, data); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) goto cleanup; =20 ret =3D 0; @@ -3021,7 +3027,9 @@ qemuBlockGetNamedNodeData(virDomainObj *vm, =20 blockNamedNodeData =3D qemuMonitorBlockGetNamedNodeData(priv->mon, sup= ports_flat); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || !blockNamedNodeData) + qemuDomainObjExitMonitor(driver, vm); + + if (!blockNamedNodeData) return NULL; =20 return g_steal_pointer(&blockNamedNodeData); @@ -3377,7 +3385,8 @@ qemuBlockReopenFormat(virDomainObj *vm, =20 rc =3D qemuBlockReopenFormatMon(priv->mon, src); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 return 0; diff --git a/src/qemu/qemu_blockjob.c b/src/qemu/qemu_blockjob.c index e7c3fea946..e57902df8c 100644 --- a/src/qemu/qemu_blockjob.c +++ b/src/qemu/qemu_blockjob.c @@ -507,7 +507,8 @@ qemuBlockJobRefreshJobs(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetJobInfo(priv->mon, &jobinfo, &njobinfo); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto cleanup; =20 for (i =3D 0; i < njobinfo; i++) { diff --git a/src/qemu/qemu_checkpoint.c b/src/qemu/qemu_checkpoint.c index 38ea57dea7..3ac5912def 100644 --- a/src/qemu/qemu_checkpoint.c +++ b/src/qemu/qemu_checkpoint.c @@ -524,7 +524,8 @@ qemuCheckpointCreate(virQEMUDriver *driver, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorTransaction(qemuDomainGetMonitor(vm), &actions); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) { + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) { qemuCheckpointRollbackMetadata(vm, chk); return NULL; } @@ -734,7 +735,8 @@ qemuCheckpointGetXMLDescUpdateSize(virDomainObj *vm, if (rc =3D=3D 0) rc =3D qemuMonitorTransaction(priv->mon, &mergeactions); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto endjob; =20 /* now do a final refresh */ @@ -746,7 +748,8 @@ qemuCheckpointGetXMLDescUpdateSize(virDomainObj *vm, =20 rc =3D qemuMonitorTransaction(priv->mon, &cleanupactions); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto endjob; =20 /* update disks */ diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index d0d9a31f37..b0b2dd0258 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -9628,7 +9628,9 @@ qemuDomainRefreshVcpuHalted(virQEMUDriver *driver, QEMU_CAPS_QUERY_CPUS_FAST); haltedmap =3D qemuMonitorGetCpuHalted(qemuDomainGetMonitor(vm), maxvcp= us, fast); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || !haltedmap) + qemuDomainObjExitMonitor(driver, vm); + + if (!haltedmap) return -1; =20 for (i =3D 0; i < maxvcpus; i++) { diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 33686f4f23..4a48354df1 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -2388,7 +2388,8 @@ static int qemuDomainSetMemoryFlags(virDomainPtr dom,= unsigned long newmem, priv =3D vm->privateData; qemuDomainObjEnterMonitor(driver, vm); r =3D qemuMonitorSetBalloon(priv->mon, newmem); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || r < 0) + qemuDomainObjExitMonitor(driver, vm); + if (r < 0) goto endjob; =20 /* Lack of balloon support is a fatal error */ @@ -3126,7 +3127,8 @@ qemuDumpToFd(virQEMUDriver *driver, =20 ret =3D qemuMonitorDumpToFd(priv->mon, fd, dumpformat, detach); =20 - if ((qemuDomainObjExitMonitor(driver, vm) < 0) || ret < 0) + qemuDomainObjExitMonitor(driver, vm); + if (ret < 0) return -1; =20 if (detach) @@ -10001,7 +10003,9 @@ qemuDomainBlocksStatsGather(virQEMUDriver *driver, rc =3D qemuMonitorBlockStatsUpdateCapacity(priv->mon, blocksta= ts); } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || nstats < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + + if (nstats < 0 || rc < 0) goto cleanup; =20 *retstats =3D g_new0(qemuBlockStats, 1); @@ -12573,7 +12577,8 @@ qemuDomainGetJobInfoDumpStats(virQEMUDriver *driver, =20 rc =3D qemuMonitorQueryDump(priv->mon, &stats); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 jobInfo->stats.dump =3D stats; @@ -12909,7 +12914,8 @@ qemuDomainMigrateSetMaxDowntime(virDomainPtr dom, } else { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetMigrationDowntime(priv->mon, downtime); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto endjob; } =20 @@ -13024,7 +13030,8 @@ qemuDomainMigrateGetCompressionCache(virDomainPtr d= om, } else { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorGetMigrationCacheSize(priv->mon, cacheSize); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto endjob; } =20 @@ -13089,7 +13096,8 @@ qemuDomainMigrateSetCompressionCache(virDomainPtr d= om, } else { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetMigrationCacheSize(priv->mon, cacheSize); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto endjob; } =20 @@ -13177,7 +13185,8 @@ qemuDomainMigrateSetMaxSpeed(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetMigrationSpeed(priv->mon, bandwidth); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto endjob; } =20 @@ -14663,7 +14672,9 @@ qemuDomainGetBlockJobInfo(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); blockjobstats =3D qemuMonitorGetAllBlockJobInfo(qemuDomainGetMonitor(v= m), true); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || !blockjobstats) + qemuDomainObjExitMonitor(driver, vm); + + if (!blockjobstats) goto endjob; =20 rawInfo =3D g_hash_table_lookup(blockjobstats, job->name); @@ -15587,10 +15598,10 @@ qemuDomainBlockCommit(virDomainPtr dom, topPath, nodetop, basePath, nodebase, backingPath, speed); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || ret < 0) { - ret =3D -1; + qemuDomainObjExitMonitor(driver, vm); + + if (ret < 0) goto endjob; - } =20 if (mirror) { disk->mirror =3D g_steal_pointer(&mirror); @@ -16607,8 +16618,8 @@ qemuDomainProbeQMPCurrentMachine(virQEMUDriver *dri= ver, =20 qemuDomainObjEnterMonitor(driver, vm); rv =3D qemuMonitorGetCurrentMachineInfo(priv->mon, &info); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || - rv < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rv < 0) return -1; =20 *wakeupSupported =3D info.wakeupSuspendSupport; @@ -19681,7 +19692,8 @@ qemuDomainSetBlockThreshold(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorSetBlockThreshold(priv->mon, nodename, threshold); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto endjob; =20 /* we need to remember whether the threshold was registered with an ex= plicit @@ -19796,7 +19808,8 @@ qemuDomainModifyLifecycleActionLive(virDomainObj *v= m, QEMU_MONITOR_ACTION_WATCHDOG_KEEP, QEMU_MONITOR_ACTION_PANIC_KEEP); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 return 0; diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index 59b3e57fd3..e886fbc35d 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -280,7 +280,8 @@ qemuDomainChangeMediaLegacy(virQEMUDriver *driver, /* re-issue ejection command to pop out the media */ qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorEjectMedia(priv->mon, driveAlias, false); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 } else { @@ -432,7 +433,8 @@ qemuHotplugAttachManagedPR(virQEMUDriver *driver, =20 rc =3D qemuMonitorAddObject(priv->mon, &props, NULL); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto cleanup; =20 ret =3D 0; @@ -529,7 +531,8 @@ qemuDomainChangeMediaBlockdev(virQEMUDriver *driver, if (diskPriv->tray && disk->tray_status !=3D VIR_DOMAIN_DISK_TRAY_OPEN= ) { qemuDomainObjEnterMonitor(driver, vm); rc =3D qemuMonitorBlockdevTrayOpen(priv->mon, diskPriv->qomName, f= orce); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 if (!force && qemuHotplugWaitForTrayEject(vm, disk) < 0) @@ -567,7 +570,8 @@ qemuDomainChangeMediaBlockdev(virQEMUDriver *driver, if (rc < 0 && newbackend) qemuBlockStorageSourceChainDetach(priv->mon, newbackend); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 return 0; @@ -2786,7 +2790,8 @@ qemuDomainAttachSCSIVHostDevice(virQEMUDriver *driver, ignore_value(qemuMonitorCloseFileHandle(priv->mon, vhostfdName)); if (removeextension) ignore_value(qemuDomainDetachExtensionDevice(priv->mon, hostdev->i= nfo)); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || ret < 0) + qemuDomainObjExitMonitor(driver, vm); + if (ret < 0) goto audit; =20 vm->def->hostdevs[vm->def->nhostdevs++] =3D hostdev; diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c index 3fd0db15bb..a544e9e2fd 100644 --- a/src/qemu/qemu_migration.c +++ b/src/qemu/qemu_migration.c @@ -750,7 +750,8 @@ qemuMigrationSrcNBDCopyCancelOne(virQEMUDriver *driver, * consistency on the destination so that we can force cancel the mirr= or */ rv =3D qemuMonitorBlockJobCancel(priv->mon, job->name, abortMigration); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rv < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rv < 0) return -1; =20 return 0; @@ -963,7 +964,8 @@ qemuMigrationSrcNBDStorageCopyBlockdev(virQEMUDriver *d= river, if (mon_ret !=3D 0) qemuBlockStorageSourceAttachRollback(qemuDomainGetMonitor(vm), dat= a); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || mon_ret < 0) + qemuDomainObjExitMonitor(driver, vm); + if (mon_ret < 0) return -1; =20 diskPriv->migrSource =3D g_steal_pointer(©src); @@ -1004,7 +1006,8 @@ qemuMigrationSrcNBDStorageCopyDriveMirror(virQEMUDriv= er *driver, diskAlias, nbd_dest, "raw", mirror_speed, 0, 0, mirror_shallow, t= rue); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || mon_ret < 0) + qemuDomainObjExitMonitor(driver, vm); + if (mon_ret < 0) return -1; =20 return 0; @@ -1673,7 +1676,8 @@ qemuMigrationAnyFetchStats(virQEMUDriver *driver, =20 rv =3D qemuMonitorGetMigrationStats(priv->mon, &stats, error); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rv < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rv < 0) return -1; =20 jobInfo->stats.mig =3D stats; @@ -2140,7 +2144,8 @@ qemuMigrationDstRun(virQEMUDriver *driver, rv =3D qemuMonitorMigrateIncoming(priv->mon, uri); =20 exit_monitor: - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rv < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rv < 0) return -1; =20 if (asyncJob =3D=3D QEMU_ASYNC_JOB_MIGRATION_IN) { @@ -3930,7 +3935,8 @@ qemuMigrationSrcRunPrepareBlockDirtyBitmapsMerge(virD= omainObj *vm, =20 rc =3D qemuMonitorTransaction(priv->mon, &actions); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 jobPriv->migTempBitmaps =3D g_steal_pointer(&tmpbitmaps); @@ -4240,7 +4246,8 @@ qemuMigrationSrcRun(virQEMUDriver *driver, break; } =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) goto error; =20 /* From this point onwards we *must* call cancel to abort the @@ -6247,7 +6254,8 @@ qemuMigrationSrcFetchMirrorStats(virQEMUDriver *drive= r, =20 blockinfo =3D qemuMonitorGetAllBlockJobInfo(priv->mon, false); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || !blockinfo) + qemuDomainObjExitMonitor(driver, vm); + if (!blockinfo) return -1; =20 memset(stats, 0, sizeof(*stats)); diff --git a/src/qemu/qemu_migration_cookie.c b/src/qemu/qemu_migration_coo= kie.c index eaf360932d..bffab7c13d 100644 --- a/src/qemu/qemu_migration_cookie.c +++ b/src/qemu/qemu_migration_cookie.c @@ -496,7 +496,8 @@ qemuMigrationCookieAddNBD(qemuMigrationCookie *mig, rc =3D qemuMonitorBlockStatsUpdateCapacityBlockdev(priv->mon, stat= s); else rc =3D qemuMonitorBlockStatsUpdateCapacity(priv->mon, stats); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 for (i =3D 0; i < vm->def->ndisks; i++) { diff --git a/src/qemu/qemu_migration_params.c b/src/qemu/qemu_migration_par= ams.c index 7bfae29de4..837ee6d635 100644 --- a/src/qemu/qemu_migration_params.c +++ b/src/qemu/qemu_migration_params.c @@ -1079,7 +1079,8 @@ qemuMigrationParamsFetch(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetMigrationParams(priv->mon, &jsonParams); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 if (!(*migParams =3D qemuMigrationParamsFromJSON(jsonParams))) @@ -1379,7 +1380,8 @@ qemuMigrationCapsCheck(virQEMUDriver *driver, =20 rc =3D qemuMonitorGetMigrationCapabilities(priv->mon, &caps); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 if (!caps) diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index 1148e6072f..dcd547b279 100644 --- a/src/qemu/qemu_process.c +++ b/src/qemu/qemu_process.c @@ -2372,7 +2372,8 @@ qemuProcessRefreshBalloonState(virQEMUDriver *driver, return -1; =20 rc =3D qemuMonitorGetBalloonInfo(qemuDomainGetMonitor(vm), &balloon); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 /* We want the balloon size stored in domain definition to @@ -4457,7 +4458,8 @@ qemuProcessFetchCPUDefinitions(virQEMUDriver *driver, =20 rc =3D virQEMUCapsFetchCPUModels(priv->mon, vm->def->os.arch, &models); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 *cpuModels =3D g_steal_pointer(&models); @@ -7127,7 +7129,8 @@ qemuProcessSetupDisksTransientHotplug(virDomainObj *v= m, =20 rc =3D qemuMonitorSystemReset(priv->mon); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) return -1; } =20 @@ -7178,7 +7181,8 @@ qemuProcessSetupLifecycleActions(virDomainObj *vm, QEMU_MONITOR_ACTION_WATCHDOG_KEEP, QEMU_MONITOR_ACTION_PANIC_KEEP); =20 - if (qemuDomainObjExitMonitor(priv->driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(priv->driver, vm); + if (rc < 0) return -1; =20 return 0; @@ -8390,7 +8394,8 @@ qemuProcessRefreshCPUMigratability(virQEMUDriver *dri= ver, =20 rc =3D qemuMonitorGetCPUMigratable(priv->mon, &migratable); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; =20 if (rc =3D=3D 1) @@ -8547,7 +8552,9 @@ qemuProcessRefreshLegacyBlockjobs(virQEMUDriver *driv= er, =20 qemuDomainObjEnterMonitor(driver, vm); blockJobs =3D qemuMonitorGetAllBlockJobInfo(qemuDomainGetMonitor(vm), = true); - if (qemuDomainObjExitMonitor(driver, vm) < 0 || !blockJobs) + qemuDomainObjExitMonitor(driver, vm); + + if (!blockJobs) goto cleanup; =20 if (virHashForEach(blockJobs, qemuProcessRefreshLegacyBlockjob, vm) < = 0) diff --git a/src/qemu/qemu_snapshot.c b/src/qemu/qemu_snapshot.c index ff827d5313..d0b2d40e94 100644 --- a/src/qemu/qemu_snapshot.c +++ b/src/qemu/qemu_snapshot.c @@ -1032,7 +1032,8 @@ qemuSnapshotDiskPrepareOneBlockdev(virQEMUDriver *dri= ver, rc =3D qemuBlockStorageSourceAttachApply(qemuDomainGetMonitor(vm), dd->crdata->srcdata[0]); =20 - if (qemuDomainObjExitMonitor(driver, vm) < 0 || rc < 0) + qemuDomainObjExitMonitor(driver, vm); + if (rc < 0) return -1; } else { if (qemuBlockStorageSourceCreateDetectSize(blockNamedNodeData, --=20 2.31.1 From nobody Fri Apr 26 09:36:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1637760635; cv=none; d=zohomail.com; s=zohoarc; b=HVkKy9MJuA/iZRW0xz9dQiNQEr55DoZAKDLPSZlsXNsWFpE/WrlbE7m+GfMFs4hxMW7J/jqCWZsnRpxyuJF/67srrBegeZ3iBr0VHph7HmDrFWCfxR1DHGDBD76HCXngefbLfQo4KLVjO5rK8AdQeS7YFVvRpIbIpJW19JurEZQ= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637760635; h=Content-Type:Content-Transfer-Encoding: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=rnyxoVnZtnjQVlHp7jeNhnqk06PGfmybctnEFzTsr6A=; b=gVIZzvy6vz2qTdHwo4TC9Riz63a3Y++QIG3QDXFmKOZHKSrRjdARfw3PLTDflHTo2evvGDsT3YDBmhT7IhLWeqxlWF8aeFQHj0UvSz5bHcPeBTObgdSEVO/KmBVnQyywazADuAEwPHal9YBYwyOvPq0Sfc92ipIntX0F2XnH7jU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1637760635722413.4228598293795; Wed, 24 Nov 2021 05:30:35 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-428-R8trclF9MBW2gY3-aYO7EA-1; Wed, 24 Nov 2021 08:30:30 -0500 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 6A69481EE71; Wed, 24 Nov 2021 13:30:25 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 1A0992B0B2; Wed, 24 Nov 2021 13:30:25 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id DF5F61802E23; Wed, 24 Nov 2021 13:30:24 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 1AODTWJx012639 for ; Wed, 24 Nov 2021 08:29:32 -0500 Received: by smtp.corp.redhat.com (Postfix) id 1A528ADDC; Wed, 24 Nov 2021 13:29:32 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.72]) by smtp.corp.redhat.com (Postfix) with ESMTP id 96AF15DF37 for ; Wed, 24 Nov 2021 13:29:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1637760634; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=rnyxoVnZtnjQVlHp7jeNhnqk06PGfmybctnEFzTsr6A=; b=TL/IexPyWuFeoRLRGkenGv+iOZrLhRoSSuWnFXBBTOVUdtxQ0cTtC8eeToaSL3dQWGXN38 PO8b/w6D7nJSt4SzWWqbPTJXkQcelHfC1RBwMFSrMJ8a/DS4DwDL3i7UUie+u/rLL6877l MnR5mMutptS/cIbi8N97LaE1O6ZlJqo= X-MC-Unique: R8trclF9MBW2gY3-aYO7EA-1 From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Subject: [libvirt PATCH 5/7] qemu: remove ignore_value for qemuDomainObjExitMonitor Date: Wed, 24 Nov 2021 14:28:53 +0100 Message-Id: <825b22adb7aa1720c815ca5274a4b7a376be3842.1637760513.git.jtomko@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1637760636019100001 Signed-off-by: J=C3=A1n Tomko Reviewed-by: Michal Privoznik --- src/qemu/qemu_backup.c | 4 ++-- src/qemu/qemu_block.c | 2 +- src/qemu/qemu_domain.c | 2 +- src/qemu/qemu_driver.c | 18 +++++++++--------- src/qemu/qemu_hotplug.c | 26 +++++++++++++------------- src/qemu/qemu_migration.c | 14 +++++++------- src/qemu/qemu_process.c | 2 +- src/qemu/qemu_snapshot.c | 2 +- 8 files changed, 35 insertions(+), 35 deletions(-) diff --git a/src/qemu/qemu_backup.c b/src/qemu/qemu_backup.c index 8e191d344e..304a0d5a4f 100644 --- a/src/qemu/qemu_backup.c +++ b/src/qemu/qemu_backup.c @@ -132,7 +132,7 @@ qemuBackupDiskDataCleanupOne(virDomainObj *vm, if (dd->added) { qemuDomainObjEnterMonitor(priv->driver, vm); qemuBlockStorageSourceAttachRollback(priv->mon, dd->crdata->sr= cdata[0]); - ignore_value(qemuDomainObjExitMonitor(priv->driver, vm)); + qemuDomainObjExitMonitor(priv->driver, vm); } =20 if (dd->created) { @@ -935,7 +935,7 @@ qemuBackupBegin(virDomainObj *vm, ignore_value(qemuMonitorDelObject(priv->mon, tlsAlias, false)); if (tlsSecretAlias) ignore_value(qemuMonitorDelObject(priv->mon, tlsSecretAlias, f= alse)); - ignore_value(qemuDomainObjExitMonitor(priv->driver, vm)); + qemuDomainObjExitMonitor(priv->driver, vm); } =20 if (ret < 0 && !job_started && priv->backup) diff --git a/src/qemu/qemu_block.c b/src/qemu/qemu_block.c index 04f5e383bc..8a10d2aa0c 100644 --- a/src/qemu/qemu_block.c +++ b/src/qemu/qemu_block.c @@ -2906,7 +2906,7 @@ qemuBlockStorageSourceCreate(virDomainObj *vm, qemuDomainObjEnterMonitorAsync(priv->driver, vm, asyncJob) =3D=3D = 0) { =20 qemuBlockStorageSourceAttachRollback(priv->mon, data); - ignore_value(qemuDomainObjExitMonitor(priv->driver, vm)); + qemuDomainObjExitMonitor(priv->driver, vm); } =20 return ret; diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index b0b2dd0258..94b4081da6 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -6968,7 +6968,7 @@ qemuDomainSnapshotDiscard(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); /* we continue on even in the face of error */ qemuMonitorDeleteSnapshot(priv->mon, snap->def->name); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); } } =20 diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 4a48354df1..28a9ae09e5 100644 --- a/src/qemu/qemu_driver.c +++ b/src/qemu/qemu_driver.c @@ -3120,7 +3120,7 @@ qemuDumpToFd(virQEMUDriver *driver, _("unsupported dumpformat '%s' " "for this QEMU binary"), dumpformat); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); return -1; } } @@ -3419,7 +3419,7 @@ qemuDomainScreenshot(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); if (qemuMonitorScreendump(priv->mon, videoAlias, screen, tmp) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto endjob; } qemuDomainObjExitMonitor(driver, vm); @@ -5302,7 +5302,7 @@ qemuDomainHotplugAddIOThread(virQEMUDriver *driver, if (qemuMonitorDelObject(priv->mon, alias, true) < 0) VIR_WARN("deletion of iothread object %d of domain %s fail= ed when cleanup", iothread_id, vm->def->name); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); } } =20 @@ -5317,7 +5317,7 @@ qemuDomainHotplugAddIOThread(virQEMUDriver *driver, return ret; =20 exit_monitor: - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } =20 @@ -5405,7 +5405,7 @@ qemuDomainHotplugDelIOThread(virQEMUDriver *driver, return ret; =20 exit_monitor: - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } =20 @@ -9907,7 +9907,7 @@ qemuDomainBlockResize(virDomainPtr dom, =20 qemuDomainObjEnterMonitor(driver, vm); if (qemuMonitorBlockResize(priv->mon, device, nodename, size) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto endjob; } qemuDomainObjExitMonitor(driver, vm); @@ -10796,12 +10796,12 @@ qemuDomainMemoryPeek(virDomainPtr dom, qemuDomainObjEnterMonitor(driver, vm); if (flags =3D=3D VIR_MEMORY_VIRTUAL) { if (qemuMonitorSaveVirtualMemory(priv->mon, offset, size, tmp) < 0= ) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto endjob; } } else { if (qemuMonitorSavePhysicalMemory(priv->mon, offset, size, tmp) < = 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto endjob; } } @@ -15184,7 +15184,7 @@ qemuDomainBlockCopyCommon(virDomainObj *vm, qemuBlockStorageSourceChainDetach(priv->mon, data); if (crdata) qemuBlockStorageSourceAttachRollback(priv->mon, crdata->sr= cdata[0]); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); } if (need_revoke) qemuDomainStorageSourceChainAccessRevoke(driver, vm, mirror); diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index e886fbc35d..43d86dbb65 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -1442,7 +1442,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, if (qemuMonitorAddFileHandleToSet(priv->mon, vdpafd, -1, net->data.vdpa.devicepath, &fdinfo) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } vdpafdName =3D g_strdup_printf("/dev/fdset/%d", fdinfo.fdset); @@ -1452,13 +1452,13 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, tapfdName, tapfdSize, vhostfdName, vhostfdSize, slirpfdName, vdpafdName))) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } =20 if (actualType =3D=3D VIR_DOMAIN_NET_TYPE_VHOSTUSER) { if (qemuHotplugChardevAttach(priv->mon, charDevAlias, net->data.vh= ostuser) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto cleanup; } @@ -1469,7 +1469,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, tapfd, tapfdName, tapfdSize, vhostfd, vhostfdName, vhostfdSize, slirpfd, slirpfdName) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } @@ -1488,14 +1488,14 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); =20 if (qemuDomainAttachExtensionDevice(priv->mon, &net->info) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } =20 if (qemuMonitorAddDeviceProps(priv->mon, &nicprops) < 0) { ignore_value(qemuDomainDetachExtensionDevice(priv->mon, &net->info= )); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } @@ -1510,7 +1510,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, qemuDomainObjEnterMonitor(driver, vm); =20 if (qemuMonitorSetLink(priv->mon, net->info.alias, VIR_DOMAIN_= NET_INTERFACE_LINK_STATE_DOWN) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virDomainAuditNet(vm, NULL, net, "attach", false); goto try_remove; } @@ -1609,7 +1609,7 @@ qemuDomainAttachNetDevice(virQEMUDriver *driver, qemuMonitorRemoveNetdev(priv->mon, netdev_name) < 0) VIR_WARN("Failed to remove network backend for netdev %s", netdev_name); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virErrorRestore(&originalError); goto cleanup; } @@ -1769,7 +1769,7 @@ qemuDomainDelTLSObjects(virQEMUDriver *driver, if (secAlias) ignore_value(qemuMonitorDelObject(priv->mon, secAlias, false)); =20 - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); =20 cleanup: virErrorRestore(&orig_err); @@ -1806,7 +1806,7 @@ qemuDomainAddTLSObjects(virQEMUDriver *driver, =20 error: virErrorPreserveLast(&orig_err); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virErrorRestore(&orig_err); qemuDomainDelTLSObjects(driver, vm, asyncJob, secAlias, NULL); =20 @@ -1989,7 +1989,7 @@ int qemuDomainAttachRedirdevDevice(virQEMUDriver *dri= ver, /* detach associated chardev on error */ if (chardevAdded) ignore_value(qemuMonitorDetachCharDev(priv->mon, charAlias)); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virErrorRestore(&orig_err); qemuDomainDelTLSObjects(driver, vm, QEMU_ASYNC_JOB_NONE, secAlias, tlsAlias); @@ -2264,7 +2264,7 @@ int qemuDomainAttachChrDevice(virQEMUDriver *driver, /* detach associated chardev on error */ if (chardevAttached) qemuMonitorDetachCharDev(priv->mon, charAlias); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virErrorRestore(&orig_err); =20 qemuDomainDelTLSObjects(driver, vm, QEMU_ASYNC_JOB_NONE, @@ -2690,7 +2690,7 @@ qemuDomainAttachHostSCSIDevice(virQEMUDriver *driver, exit_monitor: virErrorPreserveLast(&orig_err); qemuBlockStorageSourceAttachRollback(priv->mon, data); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); virErrorRestore(&orig_err); =20 virDomainAuditHostdev(vm, hostdev, "attach", false); diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c index a544e9e2fd..3cb39264e3 100644 --- a/src/qemu/qemu_migration.c +++ b/src/qemu/qemu_migration.c @@ -529,7 +529,7 @@ qemuMigrationDstStartNBDServer(virQEMUDriver *driver, return ret; =20 exit_monitor: - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } =20 @@ -4368,7 +4368,7 @@ qemuMigrationSrcRun(virQEMUDriver *driver, qemuDomainObjEnterMonitorAsync(driver, vm, QEMU_ASYNC_JOB_MIGRATION_OUT) = =3D=3D 0) { qemuMonitorMigrateCancel(priv->mon); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); } =20 /* cancel any outstanding NBD jobs */ @@ -4389,7 +4389,7 @@ qemuMigrationSrcRun(virQEMUDriver *driver, goto cleanup; =20 exit_monitor: - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto error; } =20 @@ -5970,11 +5970,11 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDo= mainObj *vm, if (virSetCloseExec(pipeFD[1]) < 0) { virReportSystemError(errno, "%s", _("Unable to set cloexec flag")); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } if (virCommandRunAsync(compressor, NULL) < 0) { - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); goto cleanup; } rc =3D qemuMonitorMigrateToFd(priv->mon, @@ -5997,7 +5997,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, if (virDomainObjIsActive(vm) && qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) =3D= =3D 0) { qemuMonitorMigrateCancel(priv->mon); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); } } goto cleanup; @@ -6024,7 +6024,7 @@ qemuMigrationSrcToFile(virQEMUDriver *driver, virDoma= inObj *vm, } else { if (qemuDomainObjEnterMonitorAsync(driver, vm, asyncJob) =3D= =3D 0) { qemuMonitorSetMigrationSpeed(priv->mon, saveMigBandwidth); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); } } priv->migMaxBandwidth =3D saveMigBandwidth; diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c index dcd547b279..34302b404a 100644 --- a/src/qemu/qemu_process.c +++ b/src/qemu/qemu_process.c @@ -3195,7 +3195,7 @@ qemuProcessUpdateVideoRamSize(virQEMUDriver *driver, return ret; =20 error: - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); return -1; } =20 diff --git a/src/qemu/qemu_snapshot.c b/src/qemu/qemu_snapshot.c index d0b2d40e94..394956a14d 100644 --- a/src/qemu/qemu_snapshot.c +++ b/src/qemu/qemu_snapshot.c @@ -883,7 +883,7 @@ qemuSnapshotDiskCleanup(qemuSnapshotDiskData *data, =20 qemuBlockStorageSourceAttachRollback(qemuDomainGetMoni= tor(vm), data[i].crdata->s= rcdata[0]); - ignore_value(qemuDomainObjExitMonitor(driver, vm)); + qemuDomainObjExitMonitor(driver, vm); } } =20 --=20 2.31.1 From nobody Fri Apr 26 09:36:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1637760664; cv=none; d=zohomail.com; s=zohoarc; b=WZraMjk6c1Ure3mQDRL/A+AXapsrLfujK6OcS/y9/Ae4DZqOcPT4eOqZjQOlz7uEKRQF96xfdXXclWGJ+poY9eXIbZXs7M5WHK/MC5FsiEqyscbZUZaEgY+hKwEfcSEsQv/usuItQDN7fqtvCEmWmmpnlb1WlQ1U8AKMf6HiplA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637760664; h=Content-Type:Content-Transfer-Encoding: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=b+BTmJZG5JnlS/q7VTScLuRRi8iIx3uuJTc0OZ2hVsY=; b=PMPPiZ3IPMJIcU/dfDlGLzOLjkKFgXKhWP+1OAyNC4wbCuV0wUniFu3lmhO6bIB5Jwi2bAn4te9EBKngEDOe9TuCVK7+sVND7jhPUW5AjL1/ix+F/jwIkgeg5vyUsOXZ46M4CmQyKG71O56PjxR0ncMNrqFfGUDnUhfaWC4SEdw= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1637760664641445.5544809704285; Wed, 24 Nov 2021 05:31:04 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-191-fc1nW4W-MxeIoVu9QJIlmg-1; Wed, 24 Nov 2021 08:30:30 -0500 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 6F23F81EE72; Wed, 24 Nov 2021 13:30:25 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 0F4EC19733; Wed, 24 Nov 2021 13:30:25 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id D5E064A705; Wed, 24 Nov 2021 13:30:24 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 1AODTWr0012644 for ; Wed, 24 Nov 2021 08:29:32 -0500 Received: by smtp.corp.redhat.com (Postfix) id DF5B7ADDC; Wed, 24 Nov 2021 13:29:32 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.72]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6808A5DF21 for ; Wed, 24 Nov 2021 13:29:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1637760663; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=b+BTmJZG5JnlS/q7VTScLuRRi8iIx3uuJTc0OZ2hVsY=; b=gucKvp9CfZVizw7D/oJy2uz8c/ur1SOW3BQYFTabDHTPXdVR1kvtUZNlFlkQjRrqz9hEfj BgFpTnvmJePHbeTkw4p2z0T10oov2GeLxm01Od7JsNZ0uzdOli6CNqVyo064M10ZDn4lm+ j+sMfaj7gEi/fnjRlrqBrPdRV4llDhs= X-MC-Unique: fc1nW4W-MxeIoVu9QJIlmg-1 From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Subject: [libvirt PATCH 6/7] qemu: turn qemuDomainObjExitMonitor into void Date: Wed, 24 Nov 2021 14:28:54 +0100 Message-Id: <84a7b74fb1c136574a57a547866a022ad26e9baa.1637760513.git.jtomko@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1637760666240100001 This reverts my commit dc2fd51fd727bbb6de172e0ca4b7dd307bb99180 Check for domain liveness in qemuDomainObjExitMonitor which fixed the symptoms of the bug later fixed by commit 81f50cb92d16643bcd749e3ab5b404b8b7cec643 qemu: Avoid calling qemuProcessStop without a job Signed-off-by: J=C3=A1n Tomko Reviewed-by: Michal Privoznik --- src/qemu/THREADS.txt | 5 ----- src/qemu/qemu_domain.c | 15 +-------------- src/qemu/qemu_domain.h | 4 ++-- 3 files changed, 3 insertions(+), 21 deletions(-) diff --git a/src/qemu/THREADS.txt b/src/qemu/THREADS.txt index ff10bef9a0..98aa3165e3 100644 --- a/src/qemu/THREADS.txt +++ b/src/qemu/THREADS.txt @@ -170,11 +170,6 @@ To acquire the QEMU monitor lock - Acquires the virDomainObj *lock =20 These functions must not be used by an asynchronous job. - Note that the virDomainObj is unlocked during the time in - monitor and it can be changed, e.g. if QEMU dies, qemuProcessStop - may free the live domain definition and put the persistent - definition back in vm->def. The callers should check the return - value of ExitMonitor to see if the domain is still alive. =20 =20 To acquire the QEMU monitor lock as part of an asynchronous job diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index 94b4081da6..dea7dae4b3 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -5893,24 +5893,11 @@ void qemuDomainObjEnterMonitor(virQEMUDriver *drive= r, /* obj must NOT be locked before calling * * Should be paired with an earlier qemuDomainObjEnterMonitor() call - * - * Returns -1 if the domain is no longer alive after exiting the monitor. - * In that case, the caller should be careful when using obj's data, - * e.g. the live definition in vm->def has been freed by qemuProcessStop - * and replaced by the persistent definition, so pointers stolen - * from the live definition could no longer be valid. */ -int qemuDomainObjExitMonitor(virQEMUDriver *driver, +void qemuDomainObjExitMonitor(virQEMUDriver *driver, virDomainObj *obj) { qemuDomainObjExitMonitorInternal(driver, obj); - if (!virDomainObjIsActive(obj)) { - if (virGetLastErrorCode() =3D=3D VIR_ERR_OK) - virReportError(VIR_ERR_OPERATION_FAILED, "%s", - _("domain is no longer running")); - return -1; - } - return 0; } =20 /* diff --git a/src/qemu/qemu_domain.h b/src/qemu/qemu_domain.h index 596add616d..8173c39cdb 100644 --- a/src/qemu/qemu_domain.h +++ b/src/qemu/qemu_domain.h @@ -487,8 +487,8 @@ qemuMonitor *qemuDomainGetMonitor(virDomainObj *vm) void qemuDomainObjEnterMonitor(virQEMUDriver *driver, virDomainObj *obj) ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); -int qemuDomainObjExitMonitor(virQEMUDriver *driver, - virDomainObj *obj) +void qemuDomainObjExitMonitor(virQEMUDriver *driver, + virDomainObj *obj) ATTRIBUTE_NONNULL(1) ATTRIBUTE_NONNULL(2); int qemuDomainObjEnterMonitorAsync(virQEMUDriver *driver, virDomainObj *obj, --=20 2.31.1 From nobody Fri Apr 26 09:36:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) client-ip=170.10.133.124; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-124.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1637760632; cv=none; d=zohomail.com; s=zohoarc; b=aI8QttaggyN57LxX6cZ6TkHCVLLPjeXfXZ14cvFCkBfFINAsa5m6rih4kClNz0smf6XdxqNGEJC6gb/UW0ohwCt54Vy0JEFbZy4T0BU/4z/M2ZnLu3YU/94bQct7HrTGWv+30hf3SWsSZS4gMuPPJwQej4sqhN+KTTxFDMldE3c= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1637760632; h=Content-Type:Content-Transfer-Encoding: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=ZcZTBBHN0QFjr5QQ70+4muyYw0LS1A7xI97pmOgrtLM=; b=SECVD31vJ2GBGXjhssFhDa6YRP1bVT4cuJuK9nwVbM6zOBnNnxU+/IhDqn+DnUMUU2SNKKYIZXCcTexyrNc3dWrYq6MAHbZjqmFYiDxrGM1tg5zwbuIiTPHW3Bk/jpz8dHZ9n4WyfTJSlZ/5TG2LdZFYcYvl8J5L1i4L/amu+SU= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 170.10.133.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mx.zohomail.com with SMTPS id 1637760632916223.79453513644262; Wed, 24 Nov 2021 05:30:32 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-500-XHaTArE8OceelvG0kxox8g-1; Wed, 24 Nov 2021 08:30:29 -0500 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 EE2ED83DD25; Wed, 24 Nov 2021 13:30:24 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.20]) by smtp.corp.redhat.com (Postfix) with ESMTPS id D001160C05; Wed, 24 Nov 2021 13:30:24 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id A72D31809CB7; Wed, 24 Nov 2021 13:30:24 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 1AODTX3S012649 for ; Wed, 24 Nov 2021 08:29:33 -0500 Received: by smtp.corp.redhat.com (Postfix) id B048FADDC; Wed, 24 Nov 2021 13:29:33 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.43.2.72]) by smtp.corp.redhat.com (Postfix) with ESMTP id 38E215DF21 for ; Wed, 24 Nov 2021 13:29:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1637760631; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=ZcZTBBHN0QFjr5QQ70+4muyYw0LS1A7xI97pmOgrtLM=; b=EuuemaxSw7CLdzY/xfW3pYavDBbDD/qjO57inFTGJCpIkCcbwsGqeGRLGrIzXXm1eOoZpo 9nXTrrC4CBNl88ZNvalu9/ekrIvREhXNfAaQ0fFMFfZc+4hSrsz4vz9l978pIfOHZbeE+7 uWmRqMVZvqUSAQXSHrI6Qa9w/eVgDdw= X-MC-Unique: XHaTArE8OceelvG0kxox8g-1 From: =?UTF-8?q?J=C3=A1n=20Tomko?= To: libvir-list@redhat.com Subject: [libvirt PATCH 7/7] qemu: absorb qemuDomainObjExitMonitorInternal Date: Wed, 24 Nov 2021 14:28:55 +0100 Message-Id: <84ad5de85acaf2b716d867835b5e335ed86fdc46.1637760513.git.jtomko@redhat.com> In-Reply-To: References: MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=libvir-list-bounces@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1637760633772100002 qemuDomainObjExitMonitor is just an alias for it at this point. This also removes the incomplete ATTRIBUTE_NONNULL(1) annotation. Signed-off-by: J=C3=A1n Tomko Reviewed-by: Michal Privoznik --- src/qemu/qemu_domain.c | 21 ++++++++------------- 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index dea7dae4b3..080510e683 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -5857,9 +5857,14 @@ qemuDomainObjEnterMonitorInternal(virQEMUDriver *dri= ver, return 0; } =20 -static void ATTRIBUTE_NONNULL(1) -qemuDomainObjExitMonitorInternal(virQEMUDriver *driver, - virDomainObj *obj) +/* obj must NOT be locked before calling + * + * Should be paired with an earlier qemuDomainObjEnterMonitor() call + * + */ +void +qemuDomainObjExitMonitor(virQEMUDriver *driver, + virDomainObj *obj) { qemuDomainObjPrivate *priv =3D obj->privateData; bool hasRefs; @@ -5890,16 +5895,6 @@ void qemuDomainObjEnterMonitor(virQEMUDriver *driver, QEMU_ASYNC_JOB_NONE)); } =20 -/* obj must NOT be locked before calling - * - * Should be paired with an earlier qemuDomainObjEnterMonitor() call - */ -void qemuDomainObjExitMonitor(virQEMUDriver *driver, - virDomainObj *obj) -{ - qemuDomainObjExitMonitorInternal(driver, obj); -} - /* * obj must be locked before calling * --=20 2.31.1