From nobody Mon Feb 9 23:16:20 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 216.205.24.124 as permitted sender) client-ip=216.205.24.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 216.205.24.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=1614971742; cv=none; d=zohomail.com; s=zohoarc; b=NRdXhQAhL9en27qD9hSz15TIgZErHX4B68dMsRf9LZu4mFMs0tmC+CL5WcUyfP5HxxLEgVfRHAWB1MQemdVTyF0amD5lxqlmREMygEZdqmc3r6qi3DGjPwUILh9ltruDSATUlwhUYEtctGLk9rnP6ARSx1qzHBoNSRfvPV0b/Fg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1614971742; 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=PJeOSXW+I4TdpPdK2LVne/eS2DhNjvwyzbbRfAfReZA=; b=NXrqPcacsifDD5uKZLkCHB9B4fVPbpYH9KM8LskV+VGXt249uzWEn4xm6fsDcz6TQCj34bZDuBfMHDRfOZDMCNy9PyXQfgsOM4I9uhpXrRtCc8yJehX3TVEZHBW+U/IUbvwcZJ8XH+qI16USgooOWGbSwoOcG3fHiIxfl6mKkpY= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 216.205.24.124 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mx.zohomail.com with SMTPS id 16149717424391011.0236169358565; Fri, 5 Mar 2021 11:15:42 -0800 (PST) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-165-cU5C3yeBOce3iUIl4BzPBQ-1; Fri, 05 Mar 2021 14:15:39 -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 62265184215C; Fri, 5 Mar 2021 19:15:31 +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 38BA5100EADF; Fri, 5 Mar 2021 19:15:31 +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 ECF9057DCE; Fri, 5 Mar 2021 19:15:30 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 125JEPWP001858 for ; Fri, 5 Mar 2021 14:14:25 -0500 Received: by smtp.corp.redhat.com (Postfix) id D2E9E5D6DC; Fri, 5 Mar 2021 19:14:25 +0000 (UTC) Received: from harajuku.usersys.redhat.com (unknown [10.40.194.220]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 334895D6B1 for ; Fri, 5 Mar 2021 19:14:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1614971741; 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=PJeOSXW+I4TdpPdK2LVne/eS2DhNjvwyzbbRfAfReZA=; b=R5zt5TBRth2TWPEsUlnmnR0zagdOfwrXKYBdu0QYDOgK8yQpjpAJ8sb5XA2jBS6jxlRhdZ 2civVyPV6kv3TRZeX63shbyb4JmH8HdG/HGlXLVLVY9qf0T3w4nxMNvTr9jZKmiy+z1qEg ihVc3f/C1Bk5jGb5ghFoixG2uwr51sU= X-MC-Unique: cU5C3yeBOce3iUIl4BzPBQ-1 From: Andrea Bolognani To: libvir-list@redhat.com Subject: [libvirt PATCH 13/17] qemu: Don't ignore virProcessGetMaxMemLock() errors Date: Fri, 5 Mar 2021 20:14:00 +0100 Message-Id: <20210305191404.529903-14-abologna@redhat.com> In-Reply-To: <20210305191404.529903-1-abologna@redhat.com> References: <20210305191404.529903-1-abologna@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 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-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) Content-Type: text/plain; charset="utf-8" Now that we've implemented a fallback for the function that obtains the information from /proc, there is no reason we would get a failure unless there's something seriously wrong with the environment we're running in, in which case we're better off reporting the issue to the user rather than pretending everything is fine. Signed-off-by: Andrea Bolognani --- src/qemu/qemu_domain.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c index 73e2473dce..1e9178764b 100644 --- a/src/qemu/qemu_domain.c +++ b/src/qemu/qemu_domain.c @@ -9246,11 +9246,10 @@ qemuDomainAdjustMaxMemLock(virDomainObjPtr vm, if (bytes) { /* If this is the first time adjusting the limit, save the current * value so that we can restore it once memory locking is no longer - * required. Failing to obtain the current limit is not a critical - * failure, it just means we'll be unable to lower it later */ + * required */ if (!vm->originalMemlock) { if (virProcessGetMaxMemLock(vm->pid, &(vm->originalMemlock)) <= 0) - vm->originalMemlock =3D 0; + return -1; } } else { /* Once memory locking is no longer required, we can restore the --=20 2.26.2