From patchwork Tue Mar 5 18:06:18 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Eric Auger X-Patchwork-Id: 13582823 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 1FB48C54798 for ; Tue, 5 Mar 2024 18:08:41 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rhZCh-0001q4-Uu; Tue, 05 Mar 2024 13:07:51 -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 1rhZCg-0001p4-D5 for qemu-devel@nongnu.org; Tue, 05 Mar 2024 13:07:50 -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 1rhZCe-0008S9-E5 for qemu-devel@nongnu.org; Tue, 05 Mar 2024 13:07:50 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1709662067; 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=ffhGhTFKTfoOEdehWkcuXBNltZdfqeZn8lAiBMxVS6Y=; b=PaaKuSvd5yRD+P00d3NZ7+oM8c4GlMi67irXYhG8JF4Tf0//KbPohBV9hFAA5jHyDrNg8z b+K0fs47fmsHDTu7G4zB2o0zjFirGkpXqz+DKcK1jY2tMjxI+tSVbvgP2WtsV3jIOr8l3F 3tMfF0NcMxtJSeGKutWu7vONpSUWU+U= Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-683-eB6L_hBqNIGezc45DXgtAg-1; Tue, 05 Mar 2024 13:07:42 -0500 X-MC-Unique: eB6L_hBqNIGezc45DXgtAg-1 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.rdu2.redhat.com [10.11.54.8]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id D0AB686C047; Tue, 5 Mar 2024 18:07:40 +0000 (UTC) Received: from laptop.redhat.com (unknown [10.39.193.114]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1C172C041F0; Tue, 5 Mar 2024 18:07:36 +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 v6 0/9] VIRTIO-IOMMU: Introduce aw-bits and granule options Date: Tue, 5 Mar 2024 19:06:18 +0100 Message-ID: <20240305180734.48515-1-eric.auger@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.8 Received-SPF: pass client-ip=170.10.129.124; envelope-from=eric.auger@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -26 X-Spam_score: -2.7 X-Spam_bar: -- X-Spam_report: (-2.7 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.568, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, 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_v4 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/i386/pc.h | 3 +++ 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.c | 6 +++++ hw/i386/pc_q35.c | 2 ++ hw/virtio/virtio-iommu.c | 35 +++++++++++++++++++++++++---- tests/qtest/virtio-iommu-test.c | 2 +- hw/virtio/trace-events | 2 +- qemu-options.hx | 11 +++++++++ 13 files changed, 115 insertions(+), 7 deletions(-)