Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 8 May 2023 13:55:47 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Linus Torvalds <torvalds@linux-foundation.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>, Raj Ashok <ashok.raj@intel.com>,
        "Tian, Kevin" <kevin.tian@intel.com>, Yi Liu <yi.l.liu@intel.com>,
        "Yu, Fenghua" <fenghua.yu@intel.com>,
        "Hansen, Dave" <dave.hansen@intel.com>
Subject: Re: [PATCH] iommu: Add Kconfig help text for IOMMU_SVA
Message-ID: <ZFkpk8y8mUZuZjkV@nvidia.com>
References: <20230506133134.1492395-1-jacob.jun.pan@linux.intel.com>
 <CAHk-=wgUiAtiszwseM1p2fCJ+sC4XWQ+YN4TanFhUgvUqjr9Xw@mail.gmail.com>
 <20230506150741.2e3d2dcc@jacob-builder>
 <CAHk-=wjmOAQnqJF-pW=fzMXb_Rk_J_Oi4ESBLmVPhxwBK4xfGg@mail.gmail.com>
 <20230508094014.53913cf3@jacob-builder>
 <CAHk-=wgobPe0U=Sc-PH08vF-ZbG00KrzftEpQMQ=n0LLNn7y6A@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <CAHk-=wgobPe0U=Sc-PH08vF-ZbG00KrzftEpQMQ=n0LLNn7y6A@mail.gmail.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TnFJcHREZ3dEc1UxYTZwci9JUUhHUVBMZG4xUmNpYllXTnl3SzBidS9yY3NL?=
 =?utf-8?B?YmVKenJUY0UrTjQ4ekNXemxFQU5sdDRzRkc0QmtQVkhSY0VlWm80VldXVXhG?=
 =?utf-8?B?SFc3OGs0MUZtTXplYStXb1RGcytpWjB0Nm9IaGpYdFJNSU5RK1V3d0NqWFU4?=
 =?utf-8?B?NXFwU0J0TUpaWEswQlRtcXhHK0xQY3ljcUJJcnFEOEVaOVVJZktXMmxuUity?=
 =?utf-8?B?ZHBtRnJGRFZxMTBscHZHUkI5OUVua3lxem9aZzdmYmJMQVBSanNibE5OSlZZ?=
 =?utf-8?B?VWxUcXpXL2pGdVZqQWV6Rk5KZk5JQnVwUlN0WVpiWG9FK3RWUy9xMDhkTm5m?=
 =?utf-8?B?L1luc2Z6N1Y3MEFpTEIyV1N3S25vWmY1Z1pUakcyclo5WE1ITTZCRGFCZ0hD?=
 =?utf-8?B?cjZoWHRFQXRvc3I0YWNCeDROYnRrbHEvc2IvVVlrWnZISUN2Zi9jckhxem1q?=
 =?utf-8?B?VFI1dURqU3RjUUVpbGtmOW01VVRCcEdXZDVpT0ZLNGZPcWo1bEVTMHZqZDZG?=
 =?utf-8?B?VWlSZUxhMHpqMUdpcW9Nby9Ba25NRHhKT3laanZRY1FOeVEvZ3JHSE5KVXBE?=
 =?utf-8?B?VldyRnd3VmxRQzFkSlQxQ1hPNHVjbjlyL1FpVDd2TU91cytQQWgrT3hTdTFU?=
 =?utf-8?B?SWVTNkZ5L2Y5WUdRRU1jSCt0SWhDRTBQS2pmaHJRTFcxQ3dSSDc4VlQzM05G?=
 =?utf-8?B?WFZtOVBTN0VRS3IzWGRNRy8xN21JNFpmWkNVSGsxWHBjblphWTRRYXR0Vmpn?=
 =?utf-8?B?K3Y2TDBOWHJtenBaOTVEL201Q3pER1BPZTZjSm5tRmRMejRrWjZrNllJTGYr?=
 =?utf-8?B?SUFIZHN5azlpQmhET01xcUtUM2NseW5WQ2NjR3NjUDk2WTE5clR0Y1NCUWFJ?=
 =?utf-8?B?NWFmczdER1VhTEpvbXZPWXRlT05VWVpkSGs5eTdBdU5udGhhWGJLZDBsVnNW?=
 =?utf-8?B?Tk5KZGIyUWJaWDBhMmk1ZHJsVGQ0Rm1TWnJVM29ZTUFwRXZYU29GNERFTG0w?=
 =?utf-8?B?cTZaM0pJeXRyd3B5bE5RSTBmVmpTMXp1UFp5QlpreURhQ044U2c2REtUUkNq?=
 =?utf-8?B?N0ZGVGlXdE5QQWlrR05qU0hVV0orZHNMT1dabmZnOCt2RFloZy82Rk9ST1pv?=
 =?utf-8?B?ZkNsU0ppNkw2bFZxWDR4ZTcwaDdDclU1bGsxejJrRURKRml6WTVDRHVnWmpv?=
 =?utf-8?B?d3k5TGs2OU1PU0NZanN5TTMrRWk1NVRjcFVFd2JsTkUza1dmUlVRMlVlM1hr?=
 =?utf-8?B?NDBNT0lVVEloaFJkUVluYjUwNVlDZE40ZVZRb3VtbXcvRnVBcy9pVUp0RVVz?=
 =?utf-8?B?TWhNR3BqVVAxNTVmSFhYekNsdk9FRmlsZ2xzb1grb2REcDlrdmxQbkdaOVpW?=
 =?utf-8?B?cnJPZldVd0FIWHdKQjBKMHdvYnNRa0h6em9wREhyanpzL1Jad0NpY0czRlFZ?=
 =?utf-8?B?eEJVZkM3akNJbVV6VTdrOEZ5NGw0OWk0bVhNUnlEMkN0T0pVakZZZE5DdXhL?=
 =?utf-8?B?cHV6b1drbFRjODExeVBvN0RFdXdrbEFiMnZTZERuUXBubHFHRlMyU2ptNUpu?=
 =?utf-8?B?NzVSYXhiVzMwY3B4Yk9ra3JRTkZ6S3JUeGJDSFY0b2NWcXEwemlXTTZKRnI2?=
 =?utf-8?B?UjlwY2djVURvdXh3MUFyUm5nbXlHNmkzNjRmVFNXTm8zRXhnclFuU3VQbWhv?=
 =?utf-8?B?Yk9xY2VvbFdCT2VpOGhKQmpiUjhPSHFQNEhZRHZpbCtMRVFwUkRndXdrMlRD?=
 =?utf-8?B?RTZ0SnFoYUxzM1pjeFdscHRmK2FrMVFLb1pBYUFIZUxGTDhsNjFxczQ0VEVO?=
 =?utf-8?B?TWYwdDRKZE5YOC93VnBMU1VBd3VqSzJMME4rU29LS1cyM2hmamdibWdHWkgr?=
 =?utf-8?B?cU03aHl5UkhrTWtXZlBOQmJRSW9MRkVJRlVXdFFuNng2QXlTeVpLcnJRNmdh?=
 =?utf-8?B?YXlvaldSaW9zVWUyRW1mQjZTSkZiS0wxL0NRU1ArWjdCV0JJOGFsc0YvTXVm?=
 =?utf-8?B?ZUF2TWFwNEtURWdSc1dZcnAvcjNIa1FqTjBYUHRNSHlnUEh2U2JrTndhcGZS?=
 =?utf-8?B?YW1qbEtCaGJtRXdqMzVqK01rQnE2Q01WalIvMURFWExqaUhBSTJ5TWlJTHZt?=
 =?utf-8?Q?Fujk=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4a291cd4-2c62-49c6-1b18-08db4fe51369
X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 May 2023 16:55:49.6549
 (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: ekBiZBUV2NtraLca6MtXMc7eMGuEEvgVFnrgbTvqJHJXSk1Gu4Ke0MA9Sa1mchJi
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR12MB5952
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, May 08, 2023 at 09:42:18AM -0700, Linus Torvalds wrote:
> On Mon, May 8, 2023 at 9:35 AM Jacob Pan <jacob.jun.pan@linux.intel.com> wrote:
> >
> > Conversely, we could also have some part of PASID that is not about SVA.
> 
> If that's the case, then we should *definitely* have that IOMMU_PASID
> kind of config option.
> 
> Then IOMMU_SVA - that needs PASID - can "select" it, but any other use
> - that doesn't bother about SVA but might want PASID for its own
> nefarious purposes - can also select it.

Yes, this has been unwinding slowly, but this mm part is about SVA,
and specifically ENQCMD, and not about PASID, other than SVA and
ENCQCMD are both using PASID.

CONFIG_IOMMU_MM_PASID perhaps? Says what it does without a clue about
why it does it. x86 arch code could select it ideally?

Jason