Date:   Mon, 13 Feb 2023 19:19:35 -0400
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Jacob Pan <jacob.jun.pan@linux.intel.com>
Cc:     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>,
        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 1/2] iommu/vt-d: Remove virtual command interface
Message-ID: <Y+rFhwxyMstovCiX@nvidia.com>
References: <20230210230206.3160144-1-jacob.jun.pan@linux.intel.com>
 <20230210230206.3160144-2-jacob.jun.pan@linux.intel.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230210230206.3160144-2-jacob.jun.pan@linux.intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?3+jW2FQH224HbdW4lhZUViCKU1U50QOOntIcl2uEnI47r2RabdrlXGZubpuy?=
 =?us-ascii?Q?2a1i09UwDjTjPCR31t6izz62xvFYew327nqfECS1zuQ0Ej4RuCXtY+6MxTQ5?=
 =?us-ascii?Q?U7FIlfVAXkr7BS/Kb/l2Oi2WK2TLq0UlamCk42izOdrGAWlO5w4rUn6v99vV?=
 =?us-ascii?Q?aiFvLyvLcsOJ8KKLMIHmw5kYWnMmfjwPZj31LEbpsnyzs+mqiy8mL47hSz45?=
 =?us-ascii?Q?FGVZNT5XjTdRJDYINZDpS47bMHqojrJm8aRZWC8V6v8qIInbyCMLw5b8Pk8T?=
 =?us-ascii?Q?sfOacH7Hrq3ML5keLXwc/0MtFqEetI/C8G2nH3I0vq1J5OthlnDhCPDjRLj5?=
 =?us-ascii?Q?VXc/2oe5m4AEz719ke4MTNX72Y7CZnefNrPYNMWLHhN8Ip9BOFhmscivfmBh?=
 =?us-ascii?Q?0QMhApEIp+Yr0UEUGlCCQTgUTZwWlhloE0jA/KLFWsVggA/OezUNxzfn+TWy?=
 =?us-ascii?Q?zi4d7CQayzbh54bORKRBe4AN9Ty1FnttgcbjTUYLTeK8C0wOwwlLyZW2pDF7?=
 =?us-ascii?Q?wq7niC1k9AwhQe1hEUOe1xYOnq642quzgk/mG2HFa5UTq9eq7iLlEMQF6Osb?=
 =?us-ascii?Q?BXL5whtWg7BEpsDDJDdH7eCCmN19bnvyrd+MENSXculBgl8C7Xvq8+pAZWDR?=
 =?us-ascii?Q?73GLnMGJ4KapLbwRD2mYgTLa4Pi8PmFd+D9G2ZWgxus9pQF3DHQZO9zXhd2z?=
 =?us-ascii?Q?GzSIzHEnzQRJKaC0HlxsoLKZjEChG93iQbyn2VBSvfHwb/WEA6y1st42Ad8q?=
 =?us-ascii?Q?/zBKAJQtR+NqJe5ApWfPwiphRGbFVp2SoTup1eCIzXl+IwD1fJ8pxXyDJv8o?=
 =?us-ascii?Q?Lw+jocbr1lVoia0Lf25l7O3dSYcmVowDEZXAqwKdWFHIaJWhyZ+ZRUx0IHdz?=
 =?us-ascii?Q?8ZfJstXr5F1cnA/OEAGbmsUtej1t2CXQFpFbLNO6+ReEukSmVyDE15OyuKLJ?=
 =?us-ascii?Q?p4Qz+qY3Ly8bkTRxYkIV1JNLKJkamWG1nDs2WVXcRWvnNEXEv8kpE9XQNCej?=
 =?us-ascii?Q?Qlxd7TkQbNrI7xroxuJq0eee9x+aFkoCNkfGD7t1eYQ81SMiTSaVDPloB9re?=
 =?us-ascii?Q?80yuAaL1uhGkvEND0b9a6jelpvo2rmJ0eFIMIv7bwZ8HgyduTRGLP1GmX+pV?=
 =?us-ascii?Q?qjRTVJ/YI3qOQEQkJzR6w8VM9rQ9IhI0pfe+LeTTvlzLQEz+Unv5nmVhZqIK?=
 =?us-ascii?Q?eNUnm2TN5wd9IRspKmgnnmbfkj4GR0bpOqJnbwhOzvv5lv3awrngN0xe5q8J?=
 =?us-ascii?Q?12kOYxFRLhtdAH9MMsf3gSPbB2JF6/7o3gwmUax+OR7zI0saYERpT6HevV7G?=
 =?us-ascii?Q?2rgrykgn4LjS6QjQh9WtZWOQHywu7shx8edVq9xDwpt7dhIWKD/hWCZrNh2V?=
 =?us-ascii?Q?kA5dQH/YekBgxxMkbAA0mcH6b3NpQe5EdK2FU8L6RH403zSjSB9wyC4nSXrZ?=
 =?us-ascii?Q?Cb+BsWTkEhaMVXqtyhGhspHKP37ciMFKN8bNM4qWdQJ+N/yzRyMnbQ9RhY27?=
 =?us-ascii?Q?anfQtLszxNL1YIV3Mp2vRS/LbUkYq8E1snV0u8doOU9fC4LSxvTlS496mPOT?=
 =?us-ascii?Q?zzsXpJKjeVJQLpOP6iXxkIdkIQXeTKSXBHACoQA2?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 37fb9abe-d3d6-47bc-53b4-08db0e18c5d8
X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Feb 2023 23:19:36.5912
 (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: yQlJhTLFRn6zU6sbvhYDpy9HtRtza+rX5sKExUWcD9LULe/OOY9vvy8JJ4Q12egW
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6058
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Feb 10, 2023 at 03:02:05PM -0800, Jacob Pan wrote:
> Virtual command interface was introduced to allow using host PASIDs
> inside VMs. It is unused and abandoned due to architectural change.
> 
> With this patch, we can safely remove this feature and the related helpers.
> 
> Signed-off-by: Jacob Pan <jacob.jun.pan@linux.intel.com>
> ---
>  drivers/iommu/intel/cap_audit.c |  2 -
>  drivers/iommu/intel/dmar.c      |  2 -
>  drivers/iommu/intel/iommu.c     | 85 ---------------------------------
>  drivers/iommu/intel/iommu.h     |  8 ----
>  4 files changed, 97 deletions(-)

Reviewed-by: Jason Gunthorpe <jgg@nvidia.com>

Jason