From patchwork Wed Mar 6 20:32:39 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Eric Auger X-Patchwork-Id: 13584636 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 3B486C54E41 for ; Wed, 6 Mar 2024 20:35:08 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rhxxp-0004F4-KJ; Wed, 06 Mar 2024 15:34:10 -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 1rhxxi-0004Dr-AZ for qemu-devel@nongnu.org; Wed, 06 Mar 2024 15:34:03 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rhxxg-0001mr-IP for qemu-devel@nongnu.org; Wed, 06 Mar 2024 15:34:02 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1709757239; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=E1H6p/sNzqNQ9c2q9TA9R9BfO5OvideIBffXBx5RiDI=; b=EtXYnP4DIxFWMjLFUXG9z9oJXN1jTsuvlB52UT9H1iPW7Y+DReiJ8+OIjvtfm5On5tCDTW +E2pLIEzLve4MrkGMFQy1f80zJqTR/1W7Arl+UFqh1lNkvEzl+USHGGRPmCOTnIHNlEWWf PaGMsVf9s6EhDg3ik8qwTkPfWFKLVfc= Received: from mimecast-mx02.redhat.com (mx-ext.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-634-xaJ3RYrAOSGO5jcqmJIkNA-1; Wed, 06 Mar 2024 15:33:55 -0500 X-MC-Unique: xaJ3RYrAOSGO5jcqmJIkNA-1 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.rdu2.redhat.com [10.11.54.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 049493800083; Wed, 6 Mar 2024 20:33:55 +0000 (UTC) Received: from laptop.redhat.com (unknown [10.39.193.114]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6054037FC; Wed, 6 Mar 2024 20:33:51 +0000 (UTC) From: Eric Auger To: eric.auger.pro@gmail.com, eric.auger@redhat.com, qemu-devel@nongnu.org, qemu-arm@nongnu.org, mst@redhat.com, jean-philippe@linaro.org, imammedo@redhat.com, peter.maydell@linaro.org, clg@redhat.com, yanghliu@redhat.com, zhenzhong.duan@intel.com Cc: alex.williamson@redhat.com, jasowang@redhat.com, pbonzini@redhat.com, berrange@redhat.com Subject: [PATCH v7 0/9] VIRTIO-IOMMU: Introduce aw-bits and granule options Date: Wed, 6 Mar 2024 21:32:39 +0100 Message-ID: <20240306203348.65776-1-eric.auger@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.1 Received-SPF: pass client-ip=170.10.133.124; envelope-from=eric.auger@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -24 X-Spam_score: -2.5 X-Spam_bar: -- X-Spam_report: (-2.5 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.365, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org This is a respin of [1] [PATCH v5 0/4] VIRTIO-IOMMU: Introduce an aw-bits option (https://lore.kernel.org/all/20240215084315.863897-1-eric.auger@redhat.com/) which now also integrates [PATCH v6 0/3] VIRTIO-IOMMU: Set default granule to host page size (https://lore.kernel.org/all/20240227165730.14099-1-eric.auger@redhat.com/) The introduction of those 2 new options and their new default values fix bugs when assigning VFIO devices protected by a virtio-iommu. patches 1 - 4: intro of the granule property, collected reviews - we used to set the default granule to 4k. This causes failures when hotplugging a VFIO device on a 64kB/64kB host/guest config: "vfio: DMA mapping failed, unable to continue". When the device is hotplugged the granule is already frozen to 4k wheras 64k is needed. This series introduces a new granule option which is set by default to the host page size. patches 5 - 9: intro of the aw-bits property, needs further review - we used to set the input address width to 64b. This causes failures with some assigned devices where the guest driver tries to use the full 64b input range whereas the physical IOMMU supports less bits (39/48 gaw for instance on VTD). New default usually match the host HW capability. For more details please see the cover letter of [1] and [2]. This series can be found at: https://github.com/eauger/qemu/tree/granule_aw_bits_v7 History: v6 -> v7: - Made property static in virt and pc_q35. Fix qtest 32 limit. Eric Auger (9): qdev: Add a granule_mode property virtio-iommu: Add a granule property virtio-iommu: Change the default granule to the host page size qemu-options.hx: Document the virtio-iommu-pci granule option virtio-iommu: Trace domain range limits as unsigned int virtio-iommu: Add an option to define the input range width hw/i386/q35: Set virtio-iommu aw-bits default value to 39 hw/arm/virt: Set virtio-iommu aw-bits default value to 48 qemu-options.hx: Document the virtio-iommu-pci aw-bits option qapi/virtio.json | 18 +++++++++++++++ include/hw/qdev-properties-system.h | 3 +++ include/hw/virtio/virtio-iommu.h | 3 +++ hw/arm/virt.c | 17 ++++++++++++++ hw/core/machine.c | 6 ++++- hw/core/qdev-properties-system.c | 14 ++++++++++++ hw/i386/pc_q35.c | 9 ++++++++ hw/virtio/virtio-iommu.c | 35 +++++++++++++++++++++++++---- tests/qtest/virtio-iommu-test.c | 2 +- hw/virtio/trace-events | 2 +- qemu-options.hx | 11 +++++++++ 11 files changed, 113 insertions(+), 7 deletions(-)