All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rahul Singh <Rahul.Singh@arm.com>
To: Julien Grall <julien@xen.org>
Cc: xen-devel <xen-devel@lists.xenproject.org>,
	Bertrand Marquis <Bertrand.Marquis@arm.com>,
	Stefano Stabellini <sstabellini@kernel.org>,
	Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>
Subject: Re: [PATCH 1/5] xen/arm: smmuv1: Handle stream IDs more dynamically
Date: Tue, 16 Mar 2021 17:03:00 +0000	[thread overview]
Message-ID: <33CC3689-401C-49DC-9F62-2A30AEEA4F46@arm.com> (raw)
In-Reply-To: <0ff580cf-0e06-ae17-32c9-bf8dce26aead@xen.org>

Hello Julien

> On 16 Mar 2021, at 3:16 pm, Julien Grall <julien@xen.org> wrote:
> 
> Hi Rahul,
> 
> On 09/03/2021 18:19, Rahul Singh wrote:
>> Backport commit 21174240e4f4439bb8ed6c116cdbdc03eba2126e
>> "iommu/arm-smmu: Handle stream IDs more dynamically" from the Linux
>> ernel.
>> This patch is the preparatory work to fix the stream match conflict
>> when two devices have the same stream-id.
>> Original commit message:
>> Rather than assuming fixed worst-case values for stream IDs and SMR
>> masks, keep track of whatever implemented bits the hardware actually
>> reports. This also obviates the slightly questionable validation of SMR
>> fields in isolation - rather than aborting the whole SMMU probe for a
>> hardware configuration which is still architecturally valid, we can
>> simply refuse masters later if they try to claim an unrepresentable ID
>> or mask (which almost certainly implies a DT error anyway).
> 
> For single backported and verbatim commit, it is common to keep the origin tags (I usually indent them) to show who is the original author of the patch.
> 
> Since 7936671da9fbf645d6bb207608f7b81c27f992de from Wei Chen as an example.

Thanks for reviewing the series. I will add the origin tags in the next version.

Regards,
Rahul
> 
> Cheers,
> 
> -- 
> Julien Grall



  reply	other threads:[~2021-03-16 17:04 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-09 18:19 [PATCH 0/5] xen/arm: smmuv1: Fix stream match conflict issue Rahul Singh
2021-03-09 18:19 ` [PATCH 1/5] xen/arm: smmuv1: Handle stream IDs more dynamically Rahul Singh
2021-03-16 15:16   ` Julien Grall
2021-03-16 17:03     ` Rahul Singh [this message]
2021-03-09 18:19 ` [PATCH 2/5] xen/arm: smmuv1: Consolidate stream map entry state Rahul Singh
2021-03-09 18:19 ` [PATCH 3/5] xen/arm: smmuv1: Keep track of S2CR state Rahul Singh
2021-03-09 18:19 ` [PATCH 4/5] xen/arm: smmuv1: Add a stream map entry iterator Rahul Singh
2021-03-09 18:19 ` [PATCH 5/5] xen/arm: smmuv1: Intelligent SMR allocation Rahul Singh
2021-03-16 15:08   ` Julien Grall
2021-03-16 17:54     ` Rahul Singh
2021-03-20 12:01       ` Julien Grall
2021-03-22  8:45         ` Rahul Singh
2021-03-12  0:38 ` [PATCH 0/5] xen/arm: smmuv1: Fix stream match conflict issue Stefano Stabellini
2021-03-12  7:58 ` Bertrand Marquis
2021-03-16 15:04 ` Julien Grall
2021-03-16 17:06   ` Rahul Singh

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=33CC3689-401C-49DC-9F62-2A30AEEA4F46@arm.com \
    --to=rahul.singh@arm.com \
    --cc=Bertrand.Marquis@arm.com \
    --cc=Volodymyr_Babchuk@epam.com \
    --cc=julien@xen.org \
    --cc=sstabellini@kernel.org \
    --cc=xen-devel@lists.xenproject.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.