All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Will Deacon <will@kernel.org>,
	Robin Murphy <robin.murphy@arm.com>,
	Joerg Roedel <joro@8bytes.org>,
	Jordan Crouse <jcrouse@codeaurora.org>,
	Rob Clark <robdclark@chromium.org>,
	Sibi Sankar <sibis@codeaurora.org>,
	linux-arm-kernel@lists.infradead.org,
	iommu@lists.linux-foundation.org, linux-kernel@vger.kernel.org,
	linux-arm-msm@vger.kernel.org
Subject: Re: [PATCH v3 5/8] iommu/arm-smmu-qcom: Consistently initialize stream mappings
Date: Fri, 11 Sep 2020 08:26:29 +0000	[thread overview]
Message-ID: <010101747c45e395-e7b8299e-f3bf-4dbb-828f-c3608b2121d3-000000@us-west-2.amazonses.com> (raw)
In-Reply-To: <20200904155513.282067-6-bjorn.andersson@linaro.org>

On 2020-09-04 21:25, Bjorn Andersson wrote:
> Firmware that traps writes to S2CR to translate BYPASS into FAULT also
> ignores writes of type FAULT. As such booting with "disable_bypass" set
> will result in all S2CR registers left as configured by the bootloader.
> 
> This has been seen to result in indeterministic results, as these
> mappings might linger and reference context banks that Linux is
> reconfiguring.
> 
> Use the fact that BYPASS writes result in FAULT type to force all 
> stream
> mappings to FAULT.
> 
> Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> ---

Reviewed-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
Tested-by: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>


-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a 
member
of Code Aurora Forum, hosted by The Linux Foundation

  parent reply	other threads:[~2020-09-11  8:26 UTC|newest]

Thread overview: 99+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-04 15:55 [PATCH v3 0/8] iommu/arm-smmu: Support maintaining bootloader mappings Bjorn Andersson
2020-09-04 15:55 ` Bjorn Andersson
2020-09-04 15:55 ` Bjorn Andersson
2020-09-04 15:55 ` [PATCH v3 1/8] iommu/arm-smmu: Refactor context bank allocation Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-08 18:42   ` Jordan Crouse
2020-09-08 18:42   ` Jordan Crouse
2020-09-08 18:42   ` Jordan Crouse
2020-09-08 18:46     ` Jordan Crouse
2020-09-08 18:46     ` Jordan Crouse
2020-09-08 18:46     ` Jordan Crouse
2020-09-11  8:18   ` Sai Prakash Ranjan
2020-09-11  8:18   ` Sai Prakash Ranjan
2020-09-11  8:18   ` Sai Prakash Ranjan
2020-09-04 15:55 ` [PATCH v3 2/8] iommu/arm-smmu: Delay modifying domain during init Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-11  8:20   ` Sai Prakash Ranjan
2020-09-11  8:20   ` Sai Prakash Ranjan
2020-09-11  8:20   ` Sai Prakash Ranjan
2020-09-04 15:55 ` [PATCH v3 3/8] iommu/arm-smmu: Consult context bank allocator for identify domains Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-11  8:21   ` Sai Prakash Ranjan
2020-09-11  8:21   ` Sai Prakash Ranjan
2020-09-11  8:21   ` Sai Prakash Ranjan
2020-09-11  8:24   ` Sai Prakash Ranjan
2020-09-11  8:24   ` Sai Prakash Ranjan
2020-09-11  8:24   ` Sai Prakash Ranjan
2020-09-04 15:55 ` [PATCH v3 4/8] iommu/arm-smmu-qcom: Emulate bypass by using context banks Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-11  8:25   ` Sai Prakash Ranjan
2020-09-11  8:25   ` Sai Prakash Ranjan
2020-09-11  8:25   ` Sai Prakash Ranjan
2020-09-04 15:55 ` [PATCH v3 5/8] iommu/arm-smmu-qcom: Consistently initialize stream mappings Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-11  8:26   ` Sai Prakash Ranjan
2020-09-11  8:26   ` Sai Prakash Ranjan
2020-09-11  8:26   ` Sai Prakash Ranjan [this message]
2020-09-04 15:55 ` [PATCH v3 6/8] iommu/arm-smmu: Add impl hook for inherit boot mappings Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-11  8:27   ` Sai Prakash Ranjan
2020-09-11  8:27   ` Sai Prakash Ranjan
2020-09-11  8:27   ` Sai Prakash Ranjan
2020-09-11 17:13   ` Robin Murphy
2020-09-11 17:13     ` Robin Murphy
2020-09-11 17:13     ` Robin Murphy
2020-09-13  3:25     ` Bjorn Andersson
2020-09-13  3:25       ` Bjorn Andersson
2020-09-13  3:25       ` Bjorn Andersson
2020-09-21 21:08       ` Will Deacon
2020-09-21 21:08         ` Will Deacon
2020-09-21 21:08         ` Will Deacon
2020-09-24 15:55         ` Bjorn Andersson
2020-09-24 15:55           ` Bjorn Andersson
2020-09-24 15:55           ` Bjorn Andersson
2020-10-12  7:31         ` Bjorn Andersson
2020-10-12  7:31           ` Bjorn Andersson
2020-10-12  7:31           ` Bjorn Andersson
2020-10-13 16:47           ` Robin Murphy
2020-10-13 16:47             ` Robin Murphy
2020-10-13 16:47             ` Robin Murphy
2020-09-04 15:55 ` [PATCH v3 7/8] iommu/arm-smmu: Provide helper for allocating identity domain Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-11  8:28   ` Sai Prakash Ranjan
2020-09-11  8:28   ` Sai Prakash Ranjan
2020-09-11  8:28   ` Sai Prakash Ranjan
2020-09-04 15:55 ` [PATCH v3 8/8] iommu/arm-smmu-qcom: Setup identity domain for boot mappings Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-04 15:55   ` Bjorn Andersson
2020-09-11  8:29   ` Sai Prakash Ranjan
2020-09-11  8:29   ` Sai Prakash Ranjan
2020-09-11  8:29   ` Sai Prakash Ranjan
2020-09-11 17:29   ` Robin Murphy
2020-09-11 17:29     ` Robin Murphy
2020-09-11 17:29     ` Robin Murphy
2020-09-05 22:27 ` [PATCH v3 0/8] iommu/arm-smmu: Support maintaining bootloader mappings Rob Clark
2020-09-05 22:27   ` Rob Clark
2020-09-05 22:27   ` Rob Clark
2020-09-09 14:46 ` Laurentiu Tudor
2020-09-09 14:46   ` Laurentiu Tudor
2020-09-09 14:46   ` Laurentiu Tudor
2020-09-10 22:56 ` John Stultz
2020-09-10 22:56   ` John Stultz
2020-09-10 22:56   ` John Stultz
2020-09-11  8:16 ` Sai Prakash Ranjan
2020-09-11  8:16 ` Sai Prakash Ranjan
2020-09-11  8:16 ` Sai Prakash Ranjan
2020-09-11 16:10 ` Amit Pundir
2020-09-11 16:10   ` Amit Pundir
2020-09-11 16:10   ` Amit Pundir
2020-09-16 10:09 ` Laurentiu Tudor
2020-09-16 10:09   ` Laurentiu Tudor
2020-09-16 10:09   ` Laurentiu Tudor

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=010101747c45e395-e7b8299e-f3bf-4dbb-828f-c3608b2121d3-000000@us-west-2.amazonses.com \
    --to=saiprakash.ranjan@codeaurora.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=iommu@lists.linux-foundation.org \
    --cc=jcrouse@codeaurora.org \
    --cc=joro@8bytes.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robdclark@chromium.org \
    --cc=robin.murphy@arm.com \
    --cc=sibis@codeaurora.org \
    --cc=will@kernel.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.