Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
To:     linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org
Cc:     joro@8bytes.org, will@kernel.org,
        Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
Subject: [PATCH v4 00/13] iommu/amd: Add Generic IO Page Table Framework Support
Date:   Tue, 15 Dec 2020 01:36:52 -0600
Message-Id: <20201215073705.123786-1-suravee.suthikulpanit@amd.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from ethanolx5673host.amd.com (165.204.78.2) by SN2PR01CA0060.prod.exchangelabs.com (2603:10b6:800::28) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3654.12 via Frontend Transport; Tue, 15 Dec 2020 07:34:31 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-HT: Tenant
X-MS-Office365-Filtering-Correlation-Id: 60de447e-dda5-4362-e43e-08d8a0cbde9e
X-MS-TrafficTypeDiagnostic: BY5PR12MB3794:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <BY5PR12MB37944257B7EEB290104EE19FF3C60@BY5PR12MB3794.namprd12.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:9508;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: hEcVyGGEtCrZ2ibdEd0aw/XXB7BAUaO5foQ/EbHIzQoXcvlpFG7kTc6ELUFHhTXi5L5zhjRCK8FjlnwTtvJkjQQAuRnjHcUksZ3AQDJ3gxFovy1tsb11ldx62mO5NSOkPZ5Z2CmRMw1VWTfKV0vzhfHRAHhdblXvbFTnJnkz0x2XU3VzuZmHp0rC45L3g14CurSvJH/vkdHCfN6iJwU2W29cn3+KizlfwIrqUajXrYNEDK3m4114j1yAxnrRhR1oBvMHmbDcqCcOfZaINwDHiHcS0MeWSQoCkmxt2IgLwH+5xEd7/EhGDGSGrtIiJanKo+IzYl9RpnTBOlpOB+1gUPThlEO83OwrPXA0KPpry0Pik4/7+ZzKy57Q+2GyXQoSRZOX7DFsBBaF1D+gKksUb57Gbzq+bmegCzMcMSsQxqiXe2UJXCN+58lBQQiIyLLvbHjb8AFLSdFnmmpoPhsKMFZZhNQMngvbWvYREFk7Vxs=
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BYAPR12MB4597.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(136003)(366004)(346002)(376002)(6666004)(2616005)(66556008)(16526019)(86362001)(956004)(66946007)(6486002)(34490700003)(26005)(52116002)(186003)(66476007)(1076003)(7696005)(36756003)(83380400001)(5660300002)(44832011)(2906002)(8676002)(508600001)(8936002)(4326008)(41533002);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?Ba/+uChY99AODYPc/LWUN8cJIrBXatWxZqnJMa6vaKfmeSY5K622LV6lXcCO?=
 =?us-ascii?Q?PX94KasX77Aa9uwjYrNCQNT3+xX0YM14H4uTGduyBP/SP99SdDf3w01qiR/L?=
 =?us-ascii?Q?pNUbFV08RFwkG2H8sWMPD6ocHBEwiHNBZysI1a1TsxtjmrP15lBynqrwRpRi?=
 =?us-ascii?Q?CV/cHT545Ni2mgzXOCMcgwlxgrcCJtUJ0WsFyt5pJryGspCedhxz0TXrKpCn?=
 =?us-ascii?Q?sXcvsSZpPwoebij9cwvUZXpdwU0lSQPsJZjXiYQiGR6qgK4MbrIXgkWz7Mn7?=
 =?us-ascii?Q?QjiVgfBfHeAqbx/8FwLzG/E/8bz4McP2TmXZJl3ZfSVuIu38MgKjj1VtxyLz?=
 =?us-ascii?Q?T+8vFMvIWwtnaQHj3WZhcwp/HKcp87aDpfOTT7E6qfHralv8A1VMKl2pw6R/?=
 =?us-ascii?Q?8u/X/D44avLwrVdaW2kpVNBN/yFGQfAs1z52ERlOxzSOmQX0gWY7VVk6vnfu?=
 =?us-ascii?Q?gd3uTKRS0hZ8AQLfjB+DMlV6O3eOgiqW6ekkbxPqhJp1CbDzO4g0FtDev/a4?=
 =?us-ascii?Q?OF7g/AzfcQTBDDBkGstV8iYhNAw4XQ/1EZxxAo/9+tHtP1mi6YtMmlCiymQQ?=
 =?us-ascii?Q?dkbIcpFK94vtx6p2gIhUPYa7/PZgBe87cB2BaRmnPhw6pF92qXx/1K8GhSsf?=
 =?us-ascii?Q?2Jpbe+NJvi9v48rjcHQi4C/Z7PEWIydsdYlhZn+tKMoJNWRnxBd+SYyuuoxN?=
 =?us-ascii?Q?aIYtcnf7qqm3hIU1q5DEn6GoLzV7i2UQ7B72e1Ln31yYLCl281RXbHkCuxIB?=
 =?us-ascii?Q?w8eFgNegeWssqGH0x0VreMck5Uy5nueJfZ0+XK9W3yMnomTzTZSh+ANW2iat?=
 =?us-ascii?Q?jn8G9aqtzWhVBEp73TWYC+hfE4trl4GoVj0CL1xhZw5h9wXnbu4c/kbDv1Wj?=
 =?us-ascii?Q?CA8+EHRlopoeMZMyZ8O42seyDvRcxBczqDQVI0dCU9UFZUFVoXosi9uGz0DU?=
 =?us-ascii?Q?6bVx3eXkfJTmTO5SxtuUJITUb/c0ECsNwmyJuvvvAv0TLP2JFLzeY2HehHgX?=
 =?us-ascii?Q?RU6f?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-AuthSource: BYAPR12MB4597.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Dec 2020 07:34:33.6681
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-Network-Message-Id: 60de447e-dda5-4362-e43e-08d8a0cbde9e
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 48YPaP2CJJSem4Jf+29+qfBAZOJmlNc+8r7g1jpXrPWpiRc9cod/7M8dl6jPSYrpETqBMXdgTkiSpCUafmhQzQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR12MB3794
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The framework allows callable implementation of IO page table.
This allows AMD IOMMU driver to switch between different types
of AMD IOMMU page tables (e.g. v1 vs. v2).

This series refactors the current implementation of AMD IOMMU v1 page table
to adopt the framework. There should be no functional change.
Subsequent series will introduce support for the AMD IOMMU v2 page table.

Thanks,
Suravee

Change from V3 (https://lore.kernel.org/linux-iommu/20201004014549.16065-1-suravee.suthikulpanit@amd.com/)
  - Rebase to v5.10
  - Patch  2: Add struct iommu_flush_ops (previously in patch 13 of v3)
  - Patch  7: Consolidate logic into v1_free_pgtable() instead of amd_iommu_free_pgtable()
  - Patch 12: Check ops->[map|unmap] before calling.
  - Patch 13: Setup page table when allocating domain (instead of when attaching device).

Change from V2 (https://lore.kernel.org/lkml/835c0d46-ed96-9fbe-856a-777dcffac967@amd.com/T/#t)
  - Patch  2: Introduce helper function io_pgtable_cfg_to_data.
  - Patch 13: Put back the struct iommu_flush_ops since patch v2 would run into
    NULL pointer bug when calling free_io_pgtable_ops if not defined.

Change from V1 (https://lkml.org/lkml/2020/9/23/251)
  - Do not specify struct io_pgtable_cfg.coherent_walk, since it is
    not currently used. (per Robin)
  - Remove unused struct iommu_flush_ops.  (patch 2/13)
  - Move amd_iommu_setup_io_pgtable_ops to iommu.c instead of io_pgtable.c
    patch 13/13)

Suravee Suthikulpanit (13):
  iommu/amd: Re-define amd_iommu_domain_encode_pgtable as inline
  iommu/amd: Prepare for generic IO page table framework
  iommu/amd: Move pt_root to struct amd_io_pgtable
  iommu/amd: Convert to using amd_io_pgtable
  iommu/amd: Declare functions as extern
  iommu/amd: Move IO page table related functions
  iommu/amd: Restructure code for freeing page table
  iommu/amd: Remove amd_iommu_domain_get_pgtable
  iommu/amd: Rename variables to be consistent with struct
    io_pgtable_ops
  iommu/amd: Refactor fetch_pte to use struct amd_io_pgtable
  iommu/amd: Introduce iommu_v1_iova_to_phys
  iommu/amd: Introduce iommu_v1_map_page and iommu_v1_unmap_page
  iommu/amd: Adopt IO page table framework for AMD IOMMU v1 page table

 drivers/iommu/amd/Kconfig           |   1 +
 drivers/iommu/amd/Makefile          |   2 +-
 drivers/iommu/amd/amd_iommu.h       |  22 +
 drivers/iommu/amd/amd_iommu_types.h |  43 +-
 drivers/iommu/amd/init.c            |   2 +
 drivers/iommu/amd/io_pgtable.c      | 564 +++++++++++++++++++++++
 drivers/iommu/amd/iommu.c           | 672 ++++------------------------
 drivers/iommu/io-pgtable.c          |   3 +
 include/linux/io-pgtable.h          |   2 +
 9 files changed, 707 insertions(+), 604 deletions(-)
 create mode 100644 drivers/iommu/amd/io_pgtable.c

-- 
2.17.1