From nobody Sat Nov 23 19:24:28 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1730786517; cv=none; d=zohomail.com; s=zohoarc; b=bJMfY/uF2110vx+x9fn/40vqDtP6zAwQDIghTPkU6bknEC2tjohEF8jfhgtetMdhAjEd7qITDdkYsnAhKs7SQNxKGSXBciDsezSyIrXlQPLqzZNh7vsW1KX1qMH0555uXwsYwQmmDR/M7NZdLW7vAh5OEu0FlCDyOEtTV4R2+0g= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1730786517; h=Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Sender:Subject:Subject:To:To:Message-Id:Reply-To; bh=FlsApH3WJa68TNfE88herv+JBSulfa3TOMSPh2EF6Tk=; b=FSPqnx4y7WaucieVVJKfqR9akRHWqJI4LM9SsGB1xizBp/VmwtFtYKtTCAYZ6Ob2oSPlOP+N8bm5PqVE/ekqRcwQpZgrt6SMIQkaDt0EwkthIGAEkK9Af/BeOIol4IHRS0TSX9hHuyO3BdW8TSzrwSVg8Hpwk8/IT5SCJJzaNUI= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1730786517285393.5900160198478; Mon, 4 Nov 2024 22:01:57 -0800 (PST) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1t8Cct-0006Ac-83; Tue, 05 Nov 2024 01:01:15 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1t8Ccq-0006AE-4v for qemu-devel@nongnu.org; Tue, 05 Nov 2024 01:01:12 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1t8Ccm-0006y1-Dv for qemu-devel@nongnu.org; Tue, 05 Nov 2024 01:01:11 -0500 Received: from mx-prod-mc-02.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-411-XWMcmzE2PGiCpIWwQwhI9w-1; Tue, 05 Nov 2024 01:01:03 -0500 Received: from mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.40]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-02.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 1AF251955EA7; Tue, 5 Nov 2024 06:01:02 +0000 (UTC) Received: from kaapi.redhat.com (unknown [10.74.18.102]) by mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 1CEB919560AA; Tue, 5 Nov 2024 06:00:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1730786466; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=FlsApH3WJa68TNfE88herv+JBSulfa3TOMSPh2EF6Tk=; b=DHJtOhj6O8D8RgPyCORtwPwooXg1G9eQfUuqqiQWQ6EiLKjIPom6xiF6bXVo83LOdrf9HJ hy/0KXcXgePQUtLoj2DPIExmJTnyFFc34YQDNvUKch20BdUHgBS4umnAlwgnJrP7BbBopr fueBBrdQqajLhDiwm0W4V41U1v6+Ti8= X-MC-Unique: XWMcmzE2PGiCpIWwQwhI9w-1 From: Prasad Pandit To: qemu-devel@nongnu.org Cc: "Michael S . Tsirkin" , Stefano Garzarella , Prasad Pandit Subject: [PATCH] vhost: fail device start if iotlb update fails Date: Tue, 5 Nov 2024 11:30:53 +0530 Message-ID: <20241105060053.61973-1-ppandit@redhat.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Scanned-By: MIMEDefang 3.0 on 10.30.177.40 Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=170.10.129.124; envelope-from=ppandit@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -23 X-Spam_score: -2.4 X-Spam_bar: -- X-Spam_report: (-2.4 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.34, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_VALIDITY_CERTIFIED_BLOCKED=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZohoMail-DKIM: pass (identity @redhat.com) X-ZM-MESSAGEID: 1730786518842116600 Content-Type: text/plain; charset="utf-8" From: Prasad Pandit While starting a vhost device, updating iotlb entries via 'vhost_device_iotlb_miss' may return an error. qemu-kvm: vhost_device_iotlb_miss: 700871,700871: Fail to update device iotlb Fail device start when such an error occurs. Signed-off-by: Prasad Pandit --- hw/virtio/vhost.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) Note: - Earlier this patch was submitted as part of a series to fix vhost device related issue. It remained unreviewed, because the other patch in that series did not fix the issue entirely. - The error fixed in this patch is fairly independent of the other issue. It can be reviewed as is, without waiting for the other patch in that series. Not sure when the other patch will be revised again. So sending this one alone, instead of holding it back. * https://lore.kernel.org/qemu-devel/20240808095147.291626-2-ppandit@redhat= .com/ * https://lore.kernel.org/qemu-devel/20240711131424.181615-3-ppandit@redhat= .com/ diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c index 06fc71746e..a70b7422b5 100644 --- a/hw/virtio/vhost.c +++ b/hw/virtio/vhost.c @@ -2151,7 +2151,11 @@ int vhost_dev_start(struct vhost_dev *hdev, VirtIODe= vice *vdev, bool vrings) * vhost-kernel code requires for this.*/ for (i =3D 0; i < hdev->nvqs; ++i) { struct vhost_virtqueue *vq =3D hdev->vqs + i; - vhost_device_iotlb_miss(hdev, vq->used_phys, true); + r =3D vhost_device_iotlb_miss(hdev, vq->used_phys, true); + if (r) { + VHOST_OPS_DEBUG(r, "vhost_device_iotlb_miss failed"); + goto fail_start; + } } } vhost_start_config_intr(hdev); -- 2.47.0