From: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
To: Geetha sowjanya <gakula@caviumnetworks.com>,
lv.zheng@intel.com, robin.murphy@arm.com, rjw@rjwysocki.net
Cc: will.deacon@arm.com, hanjun.guo@linaro.org, sudeep.holla@arm.com,
iommu@lists.linux-foundation.org, robert.moore@intel.com,
jcm@redhat.com, linux-kernel@vger.kernel.org,
robert.richter@cavium.com, catalin.marinas@arm.com,
sgoutham@cavium.com, linux-arm-kernel@lists.infradead.org,
linux-acpi@vger.kernel.org, geethasowjanya.akula@gmail.com,
devel@acpica.org, linu.cherian@cavium.com,
Charles.Garcia-Tobin@arm.com, robh@kernel.org
Subject: Re: [PATCH v7 0/3] Cavium ThunderX2 SMMUv3 errata workarounds
Date: Thu, 8 Jun 2017 17:32:13 +0100 [thread overview]
Message-ID: <20170608163213.GA2216@red-moon> (raw)
In-Reply-To: <1496145821-3411-1-git-send-email-gakula@caviumnetworks.com>
On Tue, May 30, 2017 at 05:33:38PM +0530, Geetha sowjanya wrote:
> Cavium ThunderX2 SMMUv3 implementation has two Silicon Erratas.
> 1. Errata ID #74
> SMMU register alias Page 1 is not implemented
> 2. Errata ID #126
> SMMU doesnt support unique IRQ lines and also MSI for gerror,
> eventq and cmdq-sync
>
> The following patchset does software workaround for these two erratas.
>
> This series is based on patchset.
> https://www.spinics.net/lists/arm-kernel/msg578443.html
Yes so it is not standalone. How are we going to merge these
ACPI IORT/ACPICA/SMMU patches - inclusive of:
[1] https://www.spinics.net/lists/arm-kernel/msg586458.html
Rafael, do ACPICA patches go upstream via the ACPI tree pull request ?
To remove dependency on ACPICA changes this series needs updating
anyway and for [1] above I think the only solution is for all the
patches to go via the ACPI tree (if ACPICA updates go upstream with it).
Thanks,
Lorenzo
> Changes since v6:
> - Changed device tree compatible string to vendor specific.
> - Rebased on Robin's latest "Update SMMU models for IORT rev. C" v2 patch.
> https://www.spinics.net/lists/arm-kernel/msg582809.html
>
> Changes since v5:
> - Rebased on Robin's "Update SMMU models for IORT rev. C" patch.
> https://www.spinics.net/lists/arm-kernel/msg580728.html
> - Replaced ACPI_IORT_SMMU_V3_CAVIUM_CN99XX macro with ACPI_IORT_SMMU_CAVIUM_CN99XX
>
> Changes since v4:
> - Replaced all page1 offset macros ARM_SMMU_EVTQ/PRIQ_PROD/CONS with
> arm_smmu_page1_fixup(ARM_SMMU_EVTQ/PRIQ_PROD/CONS, smmu)
>
> Changes since v3:
> - Merged patches 1, 2 and 4 of Version 3.
> - Modified the page1_offset_adjust() and get_irq_flags() implementation as
> suggested by Robin.
>
> Changes since v2:
> - Updated "Documentation/devicetree/bindings/iommu/arm,smmu-v3.txt" document with
> new SMMU option used to enable errata workaround.
>
> Changes since v1:
> - Since the use of MIDR register is rejected and SMMU_IIDR is broken on this
> silicon, as suggested by Will Deacon modified the patches to use ThunderX2
> SMMUv3 IORT model number to enable errata workaround.
>
> Geetha Sowjanya (1):
> iommu/arm-smmu-v3: Add workaround for Cavium ThunderX2 erratum #126
>
> Linu Cherian (2):
> ACPI/IORT: Fixup SMMUv3 resource size for Cavium ThunderX2 SMMUv3
> model
> iommu/arm-smmu-v3: Add workaround for Cavium ThunderX2 erratum
> #74
>
> Documentation/arm64/silicon-errata.txt | 2 +
> .../devicetree/bindings/iommu/arm,smmu-v3.txt | 6 ++
> drivers/acpi/arm64/iort.c | 10 ++-
> drivers/iommu/arm-smmu-v3.c | 93 ++++++++++++++++----
> 4 files changed, 91 insertions(+), 20 deletions(-)
>
next prev parent reply other threads:[~2017-06-08 16:31 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-30 12:03 [PATCH v7 0/3] Cavium ThunderX2 SMMUv3 errata workarounds Geetha sowjanya
2017-05-30 12:03 ` [PATCH v7 1/3] ACPI/IORT: Fixup SMMUv3 resource size for Cavium ThunderX2 SMMUv3 model Geetha sowjanya
2017-06-08 8:58 ` Lorenzo Pieralisi
2017-06-09 6:00 ` Geetha Akula
2017-06-20 8:19 ` Robert Richter
2017-06-20 8:51 ` Robert Richter
2017-06-20 10:31 ` Lorenzo Pieralisi
2017-05-30 12:03 ` [PATCH v7 2/3] iommu/arm-smmu-v3: Add workaround for Cavium ThunderX2 erratum #74 Geetha sowjanya
2017-06-09 10:28 ` Robin Murphy
[not found] ` <CA+7sy7C_44dTy0-nAE=b5BCXmc8ACQx2O6A1jCOCsemZDD5j4w@mail.gmail.com>
2017-06-09 11:38 ` Fwd: " Jayachandran C
2017-06-09 15:43 ` Robin Murphy
2017-06-12 8:12 ` Jayachandran C
2017-05-30 12:03 ` [PATCH v7 3/3] iommu/arm-smmu-v3: Add workaround for Cavium ThunderX2 erratum #126 Geetha sowjanya
2017-06-06 11:03 ` John Garry
2017-06-09 10:00 ` Will Deacon
2017-05-30 14:11 ` [PATCH v7 0/3] Cavium ThunderX2 SMMUv3 errata workarounds Robert Richter
2017-06-08 16:32 ` Lorenzo Pieralisi [this message]
2017-06-08 17:13 ` Rafael J. Wysocki
2017-06-08 17:22 ` Robin Murphy
2017-06-13 11:51 ` Lorenzo Pieralisi
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=20170608163213.GA2216@red-moon \
--to=lorenzo.pieralisi@arm.com \
--cc=Charles.Garcia-Tobin@arm.com \
--cc=catalin.marinas@arm.com \
--cc=devel@acpica.org \
--cc=gakula@caviumnetworks.com \
--cc=geethasowjanya.akula@gmail.com \
--cc=hanjun.guo@linaro.org \
--cc=iommu@lists.linux-foundation.org \
--cc=jcm@redhat.com \
--cc=linu.cherian@cavium.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lv.zheng@intel.com \
--cc=rjw@rjwysocki.net \
--cc=robert.moore@intel.com \
--cc=robert.richter@cavium.com \
--cc=robh@kernel.org \
--cc=robin.murphy@arm.com \
--cc=sgoutham@cavium.com \
--cc=sudeep.holla@arm.com \
--cc=will.deacon@arm.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;
as well as URLs for NNTP newsgroup(s).