From:   "Tian, Kevin" <kevin.tian@intel.com>
To:     Jacob Pan <jacob.jun.pan@linux.intel.com>,
        Lu Baolu <baolu.lu@linux.intel.com>
CC:     "iommu@lists.linux-foundation.org" <iommu@lists.linux-foundation.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Joerg Roedel <joro@8bytes.org>,
        "Jason Gunthorpe" <jgg@nvidia.com>,
        Christoph Hellwig <hch@infradead.org>,
        "Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
        Jean-Philippe Brucker <jean-philippe@linaro.com>,
        "Pan, Jacob jun" <jacob.jun.pan@intel.com>,
        "Raj, Ashok" <ashok.raj@intel.com>,
        "Kumar, Sanjay K" <sanjay.k.kumar@intel.com>,
        "Jiang, Dave" <dave.jiang@intel.com>,
        "Luck, Tony" <tony.luck@intel.com>,
        "Liu, Yi L" <yi.l.liu@intel.com>, Barry Song <21cnbao@gmail.com>,
        "Zanussi, Tom" <tom.zanussi@intel.com>,
        "Williams, Dan J" <dan.j.williams@intel.com>
Subject: RE: [PATCH 2/4] iommu: Add PASID support for DMA mapping API users
Thread-Topic: [PATCH 2/4] iommu: Add PASID support for DMA mapping API users
Thread-Index: AQHX67q0UnEyPcLAH0q1jKWtvfT+gqwn3/wAgAERQ4CAAHVD0A==
Date:   Thu, 9 Dec 2021 01:56:37 +0000
Message-ID: <BN9PR11MB5276676474FA6A35016B6BB88C709@BN9PR11MB5276.namprd11.prod.outlook.com>
References: <1638884834-83028-1-git-send-email-jacob.jun.pan@linux.intel.com>
        <1638884834-83028-3-git-send-email-jacob.jun.pan@linux.intel.com>
        <16408193-c8bc-3046-b32f-9274bf0b415c@linux.intel.com>
 <20211208104939.732fa5b9@jacob-builder>
In-Reply-To: <20211208104939.732fa5b9@jacob-builder>
Accept-Language: en-US
Content-Language: en-US
dlp-version: 11.6.200.16
dlp-product: dlpe-windows
dlp-reaction: no-action
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?YCSb5p0cZ6H9PC4IWFrqbm3kXB7kmPQ8U6ruLrzA404Akm79i9vXvuo/ReRx?=
 =?us-ascii?Q?ct0WSn+YuJ67cILzU8II2rd5sCX2PjqbM+BqTPMIaN77B73tXlCBn3Zg5Nyr?=
 =?us-ascii?Q?30VI2wQmznqWoAIa514ES93c/EUkMu/jP9F01oA5V6NH+zEN/xt++NodIXBO?=
 =?us-ascii?Q?WZufoU5meNP6tt2Q8yWo1eluTQyoX9SNtXFiQatN0Cx8MS+gDzhXwfDjtlPX?=
 =?us-ascii?Q?ubCh9Gk6KsCSheIpozr3sbeXFS4GJbzTFTEac0xa+RXHRFgDuNejYJRlmmbj?=
 =?us-ascii?Q?dSNJ/xGP6NvaveET3B4FGZ90w/EMVNquSbSUqVDX+tmakGcLuUBw+ua72Evl?=
 =?us-ascii?Q?+IyARkf6ULR7B7iSD8fdIBnr+EfW3F9AtXcOopM80Ir2wYBeGWqqD/kdv+SF?=
 =?us-ascii?Q?efTIwmxktkpRJP3pJDBylprM9m501SNHOewgDjg+6LktuQQxslb0fWMBcfR3?=
 =?us-ascii?Q?q6IseUNBPUrpDg7f3HGBZU4LBVG1u1bE8r72RUuFZRG50jhREsrqHfantlMF?=
 =?us-ascii?Q?3n9L4CWfKpchmOv4nLhFg7IRTooyX4phBIT8VMTbpCOCa8fe6JCJccaRD4gk?=
 =?us-ascii?Q?MEaRBWrwS7T/rF/4+7RwXtA+db+9/IffBm3T4RfyaPnf4LNmeAD7oEBSMJg3?=
 =?us-ascii?Q?DlxEICkKHLV0TGa6W1wU+GWW+UhhRD0jygIeyJ5h7zAPEyJ7q5VQSDKNUTYB?=
 =?us-ascii?Q?aWI/VEtSgdMd6zXAsf7vAkKfEE+kc2DUwJwIbO1iMlBC1zbCvwLs6K+/T/kI?=
 =?us-ascii?Q?g+1rpfo2Mcc/dQ12jjIMNlIsYQB7BZLizoaimlep9M/NDzaWLGn7eVxfoKIc?=
 =?us-ascii?Q?kZCt7tDUSSVg8AZHnGC9YZf42R79hWGS/gSpXgKV0Ju5ZXUkP2awHns3EQcD?=
 =?us-ascii?Q?pgAaJqH1ovoTwpZ4Jw6N4+gusz8K2bn0RKPBmLC2hQtwdvIWxYNTRn1ngY0K?=
 =?us-ascii?Q?ahw1wT8xdinXnLH1Jv8xPfkrTfDHgUf78LgubWlEkEksAMat0G4kj2nV6L+I?=
 =?us-ascii?Q?Uf6d1qcdWi+Ki/c4Qs+HfbKBhxSFWWzmNBmt3fxOCFkSNlQXwJvrNkDz47Nd?=
 =?us-ascii?Q?0r3pRO/eOWWlmHFbA8M/lTMxwa0VScZRvBppCPMEMp/aOUF2LJvmBN32pLxT?=
 =?us-ascii?Q?vJI35ZdZRzaeZLPwYHrI6UBRJ4BdKmMCXO0q5tf2WtQHlrA0bo+t30Fw9WVv?=
 =?us-ascii?Q?yhNe2jUXmbba1J3GxXj3L6ZArpDNa7oqn/pHV7BLiRxAE6zxcVpA43GyI3Od?=
 =?us-ascii?Q?9VyqF45A1+gGrAOM5iD/ylZNRANKnbGWSdUha9KeKUjCDzJUDrkEwNghWbTi?=
 =?us-ascii?Q?z0Jpe6EdBvo2sIpOdHpQBoA8AFMP2JH5tQCoUuR9rrVz6KYqblonZ/fAUIfZ?=
 =?us-ascii?Q?C+UhXnsYY5HlD5FqvdUzjd/wl7MAiibKn/+DTgJ7kx3AfgPf1dnm0mR0bf6I?=
 =?us-ascii?Q?nAnADs1A7EyERU+O+WRpAsEQ5YHBbeNEaG43qRT/I+CwJwZYwVBJrLFAyQ07?=
 =?us-ascii?Q?1JJflW2h58qah+Or8PKjyh6cGV+dv32E1woS5VQ2EfF0CQ+fr86VGO2ghal/?=
 =?us-ascii?Q?wh4b7oSsn75Sok/mKRWrZRgYj6lxHdSYzregNsUS/MfSKGYlNyOplWBzGw1j?=
 =?us-ascii?Q?ybTRIrM6iFhpWAIRBKCMw1E=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BN9PR11MB5276.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b41fbb28-0dd6-4fd9-4081-08d9bab7230b
X-MS-Exchange-CrossTenant-originalarrivaltime: 09 Dec 2021 01:56:37.7087
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: UagS4MzAKelHfNqc1QsIrXQ/NeGMC4IxD06xAkL09IcN8YoyIl/9VOKiim44bgDYqcmS9MKx/0tygjx0OsCwzQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR11MB1489
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> From: Jacob Pan <jacob.jun.pan@linux.intel.com>
> Sent: Thursday, December 9, 2021 2:50 AM
>=20
> > Can a device issue DMA requests with PASID even there's no system
> IOMMU
> > or the system IOMMU is disabled?
> >
> Good point.
> If IOMMU is not enabled, device cannot issue DMA requests with PASID. Thi=
s
> API will not be available. Forgot to add dummy functions to the header.
>=20

PASID is a PCI thing, not defined by IOMMU.

I think the key is physically if IOMMU is disabled, how will root complex
handle a PCI memory request including a PASID TLP prefix? Does it block=20
such request due to no IOMMU to consume PASID or simply ignore PASID
and continue routing the request to the memory controller?

If block, then having an iommu interface makes sense.

If ignore, possibly a DMA API call makes more sense instead, implying that
this extension can be used even when iommu is disabled.

I think that is what Baolu wants to point out.

Thanks
Kevin