All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v7 0/3] Cavium ThunderX2 SMMUv3 errata workarounds
@ 2017-05-30 12:03 ` Geetha sowjanya
  0 siblings, 0 replies; 58+ messages in thread
From: Geetha sowjanya @ 2017-05-30 12:03 UTC (permalink / raw)
  To: will.deacon, robin.murphy, lorenzo.pieralisi, hanjun.guo,
	sudeep.holla, iommu
  Cc: robert.moore, lv.zheng, rjw, jcm, linux-kernel, robert.richter,
	catalin.marinas, sgoutham, linux-arm-kernel, linux-acpi,
	geethasowjanya.akula, devel, linu.cherian, Charles.Garcia-Tobin,
	robh, Geetha sowjanya

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

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(-)


^ permalink raw reply	[flat|nested] 58+ messages in thread
* Re: [Devel] [PATCH v7 0/3] Cavium ThunderX2 SMMUv3 errata workarounds
@ 2017-06-13 11:51 ` Lorenzo Pieralisi
  0 siblings, 0 replies; 58+ messages in thread
From: Lorenzo Pieralisi @ 2017-06-13 11:51 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 1864 bytes --]

Hi Rafael, Lv,

On Thu, Jun 08, 2017 at 07:13:24PM +0200, Rafael J. Wysocki wrote:
> On Thu, Jun 8, 2017 at 6:32 PM, Lorenzo Pieralisi
> <lorenzo.pieralisi(a)arm.com> wrote:
> > 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 ?
> 
> Not as a rule.

So I take it as the can they go in as separate pull (standalone ACPICA
updates) ?

> > 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).
> 
> I think we may ask Lv to backport the header changes once they have
> been merged into Linux.
> 
> Lv, would that work?

I was asking to understand how to queue some patches for the upcoming
merge window that have an ACPICA dependency, how are we supposed to
handle that ? I would like to avoid cross tree dependencies, that's why
I asked about the ACPI pull request, so that IORT patches could go via
ACPI tree too this time along with ACPICA changes just trying to make
it simple.

Please let us know, thanks a lot.

Lorenzo

^ permalink raw reply	[flat|nested] 58+ messages in thread
* Re: [Devel] [PATCH v7 1/3] ACPI/IORT: Fixup SMMUv3 resource size for Cavium ThunderX2 SMMUv3 model
  2017-06-20  8:51           ` Robert Richter
  (?)
@ 2017-06-20 10:31 ` Lorenzo Pieralisi
  -1 siblings, 0 replies; 58+ messages in thread
From: Lorenzo Pieralisi @ 2017-06-20 10:31 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 1124 bytes --]

On Tue, Jun 20, 2017 at 10:51:23AM +0200, Robert Richter wrote:
> On 20.06.17 10:19:43, Robert Richter wrote:
> > On 30.05.17 17:33:39, Geetha sowjanya wrote:
> > > From: Linu Cherian <linu.cherian(a)cavium.com>
> 
> > > +	/*
> > > +	 * Override the size, for Cavium ThunderX2 implementation
> > > +	 * which doesn't support the page 1 SMMU register space.
> > > +	 */
> > > +	if (smmu->model == ACPI_IORT_SMMU_CAVIUM_CN99XX)
> > 
> > Geetha,
> > 
> > please resubmit the series since the macro changed to
> > ACPI_IORT_SMMU_V3_CAVIUM_CN99XX:
> > 
> >  https://github.com/acpica/acpica/commit/d00a4eb86e64bb4fa70f57ab5e5ca0a4ca2ad8ef#diff-a572aac2ccc26fe4a901616d7fdba859R1053
> 
> Rafael, Bob,
> 
> btw, I haven't seen
> 
>  https://github.com/acpica/acpica/commit/d00a4eb86e64
> 
> yet in linux-pm:linux-next. We would like to see this in 4.13 also. Is
> it on the way already?

Yes it would be great to understand how we can make sure IORT and
related ACPICA changes can be kept in sync for this merge window, it
is -rc6 already so it is getting a bit late in the cycle.

Thanks,
Lorenzo

^ permalink raw reply	[flat|nested] 58+ messages in thread

end of thread, other threads:[~2017-06-20 10:31 UTC | newest]

Thread overview: 58+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-30 12:03 [PATCH v7 0/3] Cavium ThunderX2 SMMUv3 errata workarounds Geetha sowjanya
2017-05-30 12:03 ` 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-05-30 12:03   ` Geetha sowjanya
2017-06-08  8:58   ` [Devel] " Lorenzo Pieralisi
2017-06-08  8:58     ` Lorenzo Pieralisi
2017-06-08  8:58     ` Lorenzo Pieralisi
2017-06-08  8:58     ` Lorenzo Pieralisi
2017-06-09  6:00     ` Geetha Akula
2017-06-09  6:00       ` Geetha Akula
2017-06-09  6:00       ` Geetha Akula
     [not found]   ` <1496145821-3411-2-git-send-email-gakula-M3mlKVOIwJVv6pq1l3V1OdBPR1lH4CV8@public.gmane.org>
2017-06-20  8:19     ` Robert Richter
2017-06-20  8:19       ` Robert Richter
2017-06-20  8:19       ` Robert Richter
     [not found]       ` <20170620081943.GT658-vWBEXY7mpu582hYKe6nXyg@public.gmane.org>
2017-06-20  8:51         ` Robert Richter
2017-06-20  8:51           ` Robert Richter
2017-06-20  8:51           ` Robert Richter
2017-05-30 12:03 ` [PATCH v7 2/3] iommu/arm-smmu-v3: Add workaround for Cavium ThunderX2 erratum #74 Geetha sowjanya
2017-05-30 12:03   ` Geetha sowjanya
     [not found]   ` <1496145821-3411-3-git-send-email-gakula-M3mlKVOIwJVv6pq1l3V1OdBPR1lH4CV8@public.gmane.org>
2017-06-09 10:28     ` Robin Murphy
2017-06-09 10:28       ` Robin Murphy
2017-06-09 10:28       ` Robin Murphy
     [not found]       ` <CA+7sy7C_44dTy0-nAE=b5BCXmc8ACQx2O6A1jCOCsemZDD5j4w@mail.gmail.com>
     [not found]         ` <CA+7sy7C_44dTy0-nAE=b5BCXmc8ACQx2O6A1jCOCsemZDD5j4w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-06-09 11:38           ` Fwd: " Jayachandran C
2017-06-09 11:38             ` Jayachandran C
2017-06-09 11:38             ` Jayachandran C
2017-06-09 15:43             ` Robin Murphy
2017-06-09 15:43               ` Robin Murphy
     [not found]               ` <ee0fb6c3-1c5c-f6d1-b063-ead8102d0c67-5wv7dgnIgG8@public.gmane.org>
2017-06-12  8:12                 ` Jayachandran C
2017-06-12  8:12                   ` Jayachandran C
2017-06-12  8:12                   ` Jayachandran C
     [not found] ` <1496145821-3411-1-git-send-email-gakula-M3mlKVOIwJVv6pq1l3V1OdBPR1lH4CV8@public.gmane.org>
2017-05-30 12:03   ` [PATCH v7 3/3] iommu/arm-smmu-v3: Add workaround for Cavium ThunderX2 erratum #126 Geetha sowjanya
2017-05-30 12:03     ` Geetha sowjanya
2017-05-30 12:03     ` Geetha sowjanya
     [not found]     ` <1496145821-3411-4-git-send-email-gakula-M3mlKVOIwJVv6pq1l3V1OdBPR1lH4CV8@public.gmane.org>
2017-06-06 11:03       ` John Garry
2017-06-06 11:03         ` John Garry
2017-06-06 11:03         ` John Garry
2017-06-09 10:00       ` Will Deacon
2017-06-09 10:00         ` Will Deacon
2017-06-09 10:00         ` Will Deacon
2017-05-30 14:11 ` [PATCH v7 0/3] Cavium ThunderX2 SMMUv3 errata workarounds Robert Richter
2017-05-30 14:11   ` Robert Richter
2017-06-08 16:32 ` [Devel] " Lorenzo Pieralisi
2017-06-08 16:32   ` Lorenzo Pieralisi
2017-06-08 16:32   ` Lorenzo Pieralisi
2017-06-08 17:13   ` [Devel] " Rafael J. Wysocki
2017-06-08 17:13     ` Rafael J. Wysocki
2017-06-08 17:13     ` Rafael J. Wysocki
2017-06-08 17:13     ` Rafael J. Wysocki
2017-06-08 17:22     ` Robin Murphy
2017-06-08 17:22       ` Robin Murphy
2017-06-08 17:22       ` Robin Murphy
  -- strict thread matches above, loose matches on Subject: below --
2017-06-13 11:51 [Devel] " Lorenzo Pieralisi
2017-06-13 11:51 ` Lorenzo Pieralisi
2017-06-13 11:51 ` Lorenzo Pieralisi
2017-06-13 11:51 ` Lorenzo Pieralisi
2017-06-20 10:31 [Devel] [PATCH v7 1/3] ACPI/IORT: Fixup SMMUv3 resource size for Cavium ThunderX2 SMMUv3 model Lorenzo Pieralisi
2017-06-20 10:31 ` Lorenzo Pieralisi
2017-06-20 10:31 ` Lorenzo Pieralisi

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.