public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgg@ziepe.ca>
To: Tina Zhang <tina.zhang@intel.com>
Cc: Kevin Tian <kevin.tian@intel.com>,
	Lu Baolu <baolu.lu@linux.intel.com>,
	iommu@lists.linux.dev, linux-kernel@vger.kernel.org
Subject: Re: [v6 PATCH 2/5] iommu: Add mm_get_enqcmd_pasid() helper function
Date: Wed, 11 Oct 2023 12:44:35 -0300	[thread overview]
Message-ID: <20231011154435.GE55194@ziepe.ca> (raw)
In-Reply-To: <20231011065132.102676-3-tina.zhang@intel.com>

On Wed, Oct 11, 2023 at 02:51:29PM +0800, Tina Zhang wrote:
> mm_get_enqcmd_pasid() is for getting enqcmd pasid value.
> 
> The motivation is to replace mm->pasid with an iommu private data
> structure that is introduced in a later patch.

When you do v7 how about:

===

mm_get_enqcmd_pasid() should be used by architecture code and closely
related to learn the PASID value that the x86 ENQCMD operation should
use for the mm.

For the moment SMMUv3 uses this without any connection to ENQCMD, it
will be cleaned up similar to how the prior patch made VT-d use the
PASID argument of set_dev_pasid().

The motivation is to replace mm->pasid with an iommu private data
structure that is introduced in a later patch.

===

Jason

  reply	other threads:[~2023-10-11 15:44 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-11  6:51 [v6 PATCH 0/5] Share sva domains with all devices bound to a mm Tina Zhang
2023-10-11  6:51 ` [v6 PATCH 1/5] iommu/vt-d: Remove mm->pasid in intel_sva_bind_mm() Tina Zhang
2023-10-11  6:51 ` [v6 PATCH 2/5] iommu: Add mm_get_enqcmd_pasid() helper function Tina Zhang
2023-10-11 15:44   ` Jason Gunthorpe [this message]
2023-10-11  6:51 ` [v6 PATCH 3/5] mm: Add structure to keep sva information Tina Zhang
2023-10-11  6:51 ` [v6 PATCH 4/5] iommu: Support mm PASID 1:n with sva domains Tina Zhang
2023-10-11 12:39   ` Jason Gunthorpe
2023-10-11 13:26     ` Tina Zhang
2023-10-11 19:33       ` Nicolin Chen
2023-10-11  6:51 ` [v6 PATCH 5/5] mm: Deprecate pasid field Tina Zhang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20231011154435.GE55194@ziepe.ca \
    --to=jgg@ziepe.ca \
    --cc=baolu.lu@linux.intel.com \
    --cc=iommu@lists.linux.dev \
    --cc=kevin.tian@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tina.zhang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox