Page MenuHomeFreeBSD

Execute PL310_ERRATA_727915 only for related revisions

Authored by on Mar 31 2017, 2:35 PM.



Part of PL310 erratum 727915 in pl310_wbinv_range() was
executed uncoditionally for all possible controllers'
revisions. This patch adds appropriate condition, since
extra operations are required only for revisions between
r2p0 and r3p0.

Submitted by: Marcin Wojtas <>
Obtained from: Semihalf
Sponsored by: Stormshield
Reviewed by:
Differential revision:

Diff Detail

rS FreeBSD src repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

andrew requested changes to this revision.Mar 31 2017, 3:34 PM

This should be runtime detected.

This revision now requires changes to proceed.Mar 31 2017, 3:34 PM

Armada38x have Marvell specific (or modified) PL310 ? If not then all errata fixes (but two at line 274 and 295) are checked online. Why do you want to disable them? Do you have measured impact caused by these tests?

Thanks. I'll confirm this and either abandon this patch or get back with something cleaner.

Ok, checked - Armada38x comprises r3p3 cache controller revision. I'm uploading second version of the patch, which adds missing condition checks for PL310_ERRATA_727915. edited edge metadata. retitled this revision from Do not activate errata in PL310 for Armada38x to Execute PL310_ERRATA_727915 only for related revisions. edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Apr 16 2017, 1:21 PM
This revision was automatically updated to reflect the committed changes.