Date:   Tue, 14 Feb 2023 09:06:18 -0400
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Jean-Philippe Brucker <jean-philippe@linaro.org>
Cc:     Jacob Pan <jacob.jun.pan@linux.intel.com>,
        LKML <linux-kernel@vger.kernel.org>, iommu@lists.linux.dev,
        Lu Baolu <baolu.lu@linux.intel.com>,
        Joerg Roedel <joro@8bytes.org>,
        Jean-Philippe Brucker <jean-philippe@linaro.com>,
        Robin Murphy <robin.murphy@arm.com>,
        Will Deacon <will@kernel.org>,
        David Woodhouse <dwmw2@infradead.org>,
        Raj Ashok <ashok.raj@intel.com>,
        "Tian, Kevin" <kevin.tian@intel.com>, Yi Liu <yi.l.liu@intel.com>
Subject: Re: [PATCH v2 3/3] iommu: Remove ioasid infrastructure
Message-ID: <Y+uHSry/tpgGNr/b@nvidia.com>
References: <20230214064414.1038058-1-jacob.jun.pan@linux.intel.com>
 <20230214064414.1038058-4-jacob.jun.pan@linux.intel.com>
 <Y+tVMRzRQbcc7Guz@myrica>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Y+tVMRzRQbcc7Guz@myrica>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?rE/totnrezW9kMYC2qESI0gC8ww/EObihMuJFa1lJIPbWG3dTYfcXqAwyV3c?=
 =?us-ascii?Q?NqnvwGQv7QOUsLHjTqbgnMq75yo4a7sSn5o5dogh7UwntqCc+/Wpt8nDvnaQ?=
 =?us-ascii?Q?23c2mbmCti/4O9568UY0zjAYgbMEF+ol+82GIfp9kI+6vJGCnxXgYBw2DDDD?=
 =?us-ascii?Q?NhFY4bODwtDNc5BWWGnx8Hfvu3N99dnjuJzfCW9r8bJActcrkZYj7L1N6OSh?=
 =?us-ascii?Q?JC7IW1mKA+LJSdJvGB/Cp3MuhBL3Cxwg0kECuCwyY8BbObn0P9DnJt8WMXXo?=
 =?us-ascii?Q?Jo/miAo/vbsxgmHZ/pL9oBRvvpv5i1MiQeE10HpLDjwHZ5kUD29hN83DJKGw?=
 =?us-ascii?Q?7aw9u5Nokig0p0izGoQpNHsS/0BUuCg23wD1OcVwtOyjnM+tV2Z/CXqJ6/RY?=
 =?us-ascii?Q?2X37mw5pgcq4v4AH6J86F2qU/2DfijzAdd0j0sjA6ItRfsLnjA+yTZCa+cCA?=
 =?us-ascii?Q?X3ZQZS9TBE8wAjgaD1o6gCZPfVhWzG+0FcLA0yJqVgHAMMNpWgqepWBg0l8g?=
 =?us-ascii?Q?/JXMliNci5WozQJh7Eo6ngHlrSIdouxey+n3sIKGwQWZk3UOxq22zqNBsfu0?=
 =?us-ascii?Q?yfaDwVbJoGIfj7V2+BzTrRjBqHr3CdsJeZTlk04yGQCYpomLbJea396CORHq?=
 =?us-ascii?Q?fUvETnulkG3waO2aZyJsEvJV48NlmM6esXhT/9Dg1n6tJhphtJpDrSM1EStj?=
 =?us-ascii?Q?7oaZeBAky+sZuT8tNzgQ+vMdWEPB7IMYWYj9GN4Y4/tGMKqgDgEE7Ce4aSWZ?=
 =?us-ascii?Q?DHLO8Zot8uMFYYlgz++NJvo0SM6smALzqS6hhcQ1RX7ZrUfsbVVTK40ZkK+Y?=
 =?us-ascii?Q?A/wBgzBVWFEC734mh8TwpzxOYFf+UKhYrdnZJME9DYDSp1sLh7WZzW+V0CiA?=
 =?us-ascii?Q?NvlRq9GQLQ2NBaCQCC3PKUKeEo6wa5dI01hQgjNuZlyh3GfBdNc+cLQCvEro?=
 =?us-ascii?Q?pKwJOc+haLY0vnXit/aY8fvSSg146f9gZ2QTyBBVaFmMrjN30i1/exQLjbGo?=
 =?us-ascii?Q?wwBJ7Mwzorwko3in5Abh5jvxkKnfz6n7u3LR7KAg5pJd8VfXocn6RGl5a2D7?=
 =?us-ascii?Q?L2+BVSNkpC8bPO3gULBOT6q3+NlNACNRlOt/iV4/ZVy/FgEu8ZBbBDtO8fBn?=
 =?us-ascii?Q?L0TvvnE/NMVMFEw84YUDpoUMg7q00VC5N5OQ/MnHPDYjPZStrDy7Lv6JbVaj?=
 =?us-ascii?Q?U1qNIOuv0GDnhaRPNWIY2KtsqoBIIyAhTizXz/7eGLNOwXQi9RagLUFAE3Cp?=
 =?us-ascii?Q?BoYP0xDGlaGNnNOc/pEsTxyLpq62wWVWijtfUSDf5V6bArG+ozFPhDQwpx9Q?=
 =?us-ascii?Q?xqtogu9JKBMi5OvMFNbpavznQbu3sJkB8ijPlgvrSfeIzRPlf1nutnu2GJzF?=
 =?us-ascii?Q?XKqFIIVXQvOy1057gaFsgn7vJTi6InwMkkyBuUcWHJihpADFiiIxvph/7iPe?=
 =?us-ascii?Q?NBFYqOxssSlwU/Eu+ujcbo7yooPp0uAWFt3aH41pDmiH9gHuoT/GOfEgWdqS?=
 =?us-ascii?Q?7MnE9OIglAlDTo1FgB00gpNyk/3Jew54wHqL3vEabhi+LBaSF0hwF16Nt0ej?=
 =?us-ascii?Q?xEtj+3Pwjawlpt7dRuf2U2cvzM01pEnT/qHW8cyF?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2a941801-6f5d-4baf-92a7-08db0e8c438e
X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Feb 2023 13:06:19.5621
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: m6XWxfUXXIVSBWZeefoxAq4IEYDdkJixHAoQ3rTk1YyEzvIf66Zzb1y42Y2Q1zQR
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA1PR12MB8721
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, Feb 14, 2023 at 09:32:33AM +0000, Jean-Philippe Brucker wrote:
> > diff --git a/include/uapi/linux/iommu.h b/include/uapi/linux/iommu.h
> > index 65d8b0234f69..a95f07f0c98c 100644
> > --- a/include/uapi/linux/iommu.h
> > +++ b/include/uapi/linux/iommu.h
> > @@ -157,5 +157,6 @@ struct iommu_page_response {
> >  	__u32	grpid;
> >  	__u32	code;
> >  };
> > +#define IOMMU_PASID_INVALID	(-1U)
> 
> Why change UAPI?

There is no uapi, this whole file needs to be deleted too

JAson