Received-SPF: pass (google.com: domain of linux-kernel+bounces-186816-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 22 May 2024 20:28:33 -0300
From: Jason Gunthorpe <jgg@nvidia.com>
To: Nicolin Chen <nicolinc@nvidia.com>
Cc: "Tian, Kevin" <kevin.tian@intel.com>,
	"will@kernel.org" <will@kernel.org>,
	"robin.murphy@arm.com" <robin.murphy@arm.com>,
	"suravee.suthikulpanit@amd.com" <suravee.suthikulpanit@amd.com>,
	"joro@8bytes.org" <joro@8bytes.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"iommu@lists.linux.dev" <iommu@lists.linux.dev>,
	"linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org>,
	"linux-tegra@vger.kernel.org" <linux-tegra@vger.kernel.org>,
	"Liu, Yi L" <yi.l.liu@intel.com>,
	"eric.auger@redhat.com" <eric.auger@redhat.com>,
	"vasant.hegde@amd.com" <vasant.hegde@amd.com>,
	"jon.grimm@amd.com" <jon.grimm@amd.com>,
	"santosh.shukla@amd.com" <santosh.shukla@amd.com>,
	"Dhaval.Giani@amd.com" <Dhaval.Giani@amd.com>,
	"shameerali.kolothum.thodi@huawei.com" <shameerali.kolothum.thodi@huawei.com>
Subject: Re: [PATCH RFCv1 00/14] Add Tegra241 (Grace) CMDQV Support (part 2/2)
Message-ID: <20240522232833.GH20229@nvidia.com>
References: <cover.1712978212.git.nicolinc@nvidia.com>
 <BN9PR11MB527641C15DD88FB0C44323D08CEB2@BN9PR11MB5276.namprd11.prod.outlook.com>
 <20240522164818.GB20229@nvidia.com>
 <Zk5Lx7IqvnE2q7a7@nvidia.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <Zk5Lx7IqvnE2q7a7@nvidia.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?us-ascii?Q?a6vw5+LWVbxnYmTqGVRmUC+KIMv4XKVSXTmMMjYJl4wzZXMZUB0ZNRMyK+ax?=
 =?us-ascii?Q?L9X4ZOIa9v7CdHN/YVEH7sPoNlFl1EOJl1nQoJXPaC+dHUZaW89HujA41PSD?=
 =?us-ascii?Q?J1wAfq5gs8bA5XwqgcSOPE7H2A+J00mwkdIesaGQnB2cA79VdllflsQ0U/5b?=
 =?us-ascii?Q?kfmbkIJqbOXbzmW0ASZ30UScmDFoYNgh6AxFSjq81Yt8MlV8XRmq+XLosk2j?=
 =?us-ascii?Q?rXdiHScbqinUkEbBRz8H+qH4++pP1xomZeAAjC5nUndEmUldYRSSMAH1U3KC?=
 =?us-ascii?Q?u+U3/r0v4uTCuKUEhvIxQ1SWopfXnd9fV5T2MNv8E1tKhTayZ07UdgWYML5W?=
 =?us-ascii?Q?CG94lRXEhXSh9wokA0nZOXJ7tVhCzS+3GUAEnbDB3ijLGtwKEoyEepdoR9Gl?=
 =?us-ascii?Q?gXORE77y7TuZzNgI8ZwX2rPAibz/QS22dbH0t+Ac5eqOBggYXmH2bkuq3Wu5?=
 =?us-ascii?Q?Cc+V8ZO14/RTCtGyrxVEMokR1sTvQYZCm7QdXWq5jHW4yyNNXyC3EnTkpBT+?=
 =?us-ascii?Q?KHJOdry38REwXlinhRuJ31ySog5q9TCJF6qSqKSuosJ4Ugh3AwBtcH0756mN?=
 =?us-ascii?Q?x6N8iy/qx6FiWwougKc4lalPce3reCSUHTN91XxObJGylNp8haVXWtd7VtWK?=
 =?us-ascii?Q?mYwa9weO7vzAGGcjbQ7OFPrzxZUjhU4seBBvN3Uy536ird/G5gSwUZtnZT9s?=
 =?us-ascii?Q?/DVi3PPpxaGB9xrFgYDQPIlDoMLqgh1qLifjRsQqpvA3HxI6HF8mHkieY5A7?=
 =?us-ascii?Q?tkefe9Arr1HgPuxT6dbdajvE23Jz69RBljAwpC8G40Hh9Y4tE8uIEIXzevxK?=
 =?us-ascii?Q?4PosBDe2v6Vi0aswkg7gsO9DNPJML9yGGiYfsmtvJS8ZLQ+0tv6H3hVjbg8s?=
 =?us-ascii?Q?8vBa/JbjEgMYz06EAyST3z7YgbjjuojE6eIA6nMy/nDD+uyQq5fsTpfr9+zp?=
 =?us-ascii?Q?L9gAD5jnOqVzwgLCD0+oSNIrL27MjIzMs8jPhsX516nIl4Y4bq16U9dibTzL?=
 =?us-ascii?Q?Sub1GCMLHhl5hsLDoqHpq0UsbscW2IP2h56Vn7q8abwv4TEQQbc0D1lVnMat?=
 =?us-ascii?Q?/jaaokFKzFbFuWBIltVKZQ9CiYj4GVx9JfrSBNepSZtKCvSy+6NzPdxJk71P?=
 =?us-ascii?Q?bvbu7w1puM48ReIJUldArN4hdmr4phfHkrsetkqXnWbxPn4Fj6xdM+PCsa/h?=
 =?us-ascii?Q?qH5Np1/NUmiPS9wLW7V+DerKpuDvmbeh9veiGdoCx9IMzNhCVixbbIr8o6V5?=
 =?us-ascii?Q?+OR4OKpVxlzNCGVYQX9bPplCSQQPeYe7C0JWhFy5lMb9vR8NRoxRP9OcPgvQ?=
 =?us-ascii?Q?mGoigBB6XTsmOxwDbGB17RWm3sxkJYeWpHbtZR0Et7nEwjwSH/xjBbYeEgHn?=
 =?us-ascii?Q?uNuTwov9ekhQbN5d6yLk+8aKFdV+cLoQYeITspoPnwqp82uTdD+4oUlKD9C0?=
 =?us-ascii?Q?g3pT32VohSiSHxN4ZAIQoE4rhV9WmlUDfWt9H7Gm6IG3Mj37QoTTgF7vdF5z?=
 =?us-ascii?Q?R+Y4ArAM9z19EDiUBOZ3OgbbJ9Mcuo9F2Rj7UA+dHilAMrTHE+8XXL4pQF8R?=
 =?us-ascii?Q?i3+Ztc+7rbQfqbox8XI=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4f9df21d-6f2f-4ea6-1ed4-08dc7ab6e70b
X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB3849.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 May 2024 23:28:36.0633
 (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: 06s5UPDxkPAJ38p79YjtLL64IGdmpgedwzNIiCH1AmMU9+0iwNV/Gp6xBaKnb9eB
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR12MB8102

On Wed, May 22, 2024 at 12:47:19PM -0700, Nicolin Chen wrote:
> On Wed, May 22, 2024 at 01:48:18PM -0300, Jason Gunthorpe wrote:
> > On Wed, May 22, 2024 at 08:40:00AM +0000, Tian, Kevin wrote:
> > > > From: Nicolin Chen <nicolinc@nvidia.com>
> > > > Sent: Saturday, April 13, 2024 11:47 AM
> > > > 
> > > > This is an experimental RFC series for VIOMMU infrastructure, using NVIDIA
> > > > Tegra241 (Grace) CMDQV as a test instance.
> > > > 
> > > > VIOMMU obj is used to represent a virtual interface (iommu) backed by an
> > > > underlying IOMMU's HW-accelerated feature for virtualizaion: for example,
> > > > NVIDIA's VINTF (v-interface for CMDQV) and AMD"s vIOMMU.
> > > > 
> > > > VQUEUE obj is used to represent a virtual command queue (buffer) backed
> > > > by
> > > > an underlying IOMMU command queue to passthrough for VMs to use
> > > > directly:
> > > > for example, NVIDIA's Virtual Command Queue and AMD's Command Buffer.
> > > > 
> > > 
> > > is VCMDQ more accurate? AMD also supports fault queue passthrough
> > > then VQUEUE sounds broader than a cmd queue...
> > 
> > Is there a reason VQUEUE couldn't handle the fault/etc queues too? The
> > only difference is direction, there is still a doorbell/etc.
> 
> Yea, SMMU also has Event Queue and PRI queue. Though I haven't
> got time to sit down to look at Baolu's work closely, the uAPI
> seems to be a unified one for all IOMMUs. And though I have no
> intention to be against that design, yet maybe there could be
> an alternative in a somewhat HW specific language as we do for
> invalidation? Or not worth it?

I was thinking not worth it, I expect a gain here is to do as AMD has
done and make the HW dma the queues directly to guest memory.

IMHO the primary issue with the queues is DOS, as having any shared
queue across VMs is dangerous in that way. Allowing each VIOMMU to
have its own private queue and own flow control helps with that.

Jason