On 06/02/17 06:20, Andrew Pinski wrote:
> Hi,
> When I implemented the -mcpu=thunderx2t99 I did not have the Cavium
> partno for ThunderX CN99xx, only the original part no. This patch
> adds the new part no for the future versions of the chip.
>
> OK? Bootstrapped and tested on aarch64-linux-gnu with no regressions.
>
> Thanks,
> Andrew
>
> ChangeLog:
> * config/aarch64/aarch64-cores.def (thunderx2t99): Move to under 'C"
> cores and change the partno/implementer to be correct.
> (thunderx2t99p1): New core which replaces thunderx2t99 and still has
> the 'B" as the implementer.
>
OK.
R.
>
> midrthunderx2t99.diff.txt
>
>
> Index: config/aarch64/aarch64-cores.def
> ===
> --- config/aarch64/aarch64-cores.def (revision 245203)
> +++ config/aarch64/aarch64-cores.def (working copy)
> @@ -67,6 +67,7 @@ AARCH64_CORE("thunderxt88p1", thunderxt8
> AARCH64_CORE("thunderxt88", thunderxt88, thunderx, 8A,
> AARCH64_FL_FOR_ARCH8 | AARCH64_FL_CRC | AARCH64_FL_CRYPTO | AARCH64_FL_LSE,
> thunderx, 0x43, 0x0a1, -1)
> AARCH64_CORE("thunderxt81", thunderxt81, thunderx, 8_1A,
> AARCH64_FL_FOR_ARCH8 | AARCH64_FL_CRC | AARCH64_FL_CRYPTO | AARCH64_FL_LSE,
> thunderx, 0x43, 0x0a2, -1)
> AARCH64_CORE("thunderxt83", thunderxt83, thunderx, 8_1A,
> AARCH64_FL_FOR_ARCH8 | AARCH64_FL_CRC | AARCH64_FL_CRYPTO | AARCH64_FL_LSE,
> thunderx, 0x43, 0x0a3, -1)
> +AARCH64_CORE("thunderx2t99", thunderx2t99, thunderx2t99, 8_1A,
> AARCH64_FL_FOR_ARCH8_1 | AARCH64_FL_CRYPTO, thunderx2t99, 0x43, 0x0af, -1)
>
> /* APM ('P') cores. */
> AARCH64_CORE("xgene1", xgene1,xgene1,8A, AARCH64_FL_FOR_ARCH8,
> xgene1, 0x50, 0x000, -1)
> @@ -74,7 +75,7 @@ AARCH64_CORE("xgene1", xgene1,x
> /* V8.1 Architecture Processors. */
>
> /* Broadcom ('B') cores. */
> -AARCH64_CORE("thunderx2t99", thunderx2t99, thunderx2t99, 8_1A,
> AARCH64_FL_FOR_ARCH8_1 | AARCH64_FL_CRYPTO, thunderx2t99, 0x42, 0x516, -1)
> +AARCH64_CORE("thunderx2t99p1", thunderx2t99p1, thunderx2t99, 8_1A,
> AARCH64_FL_FOR_ARCH8_1 | AARCH64_FL_CRYPTO, thunderx2t99, 0x42, 0x516, -1)
> AARCH64_CORE("vulcan", vulcan, thunderx2t99, 8_1A, AARCH64_FL_FOR_ARCH8_1
> | AARCH64_FL_CRYPTO, thunderx2t99, 0x42, 0x516, -1)
>
> /* V8 big.LITTLE implementations. */
> Index: config/aarch64/aarch64-tune.md
> ===
> --- config/aarch64/aarch64-tune.md(revision 245203)
> +++ config/aarch64/aarch64-tune.md(working copy)
> @@ -1,5 +1,5 @@
> ;; -*- buffer-read-only: t -*-
> ;; Generated automatically by gentune.sh from aarch64-cores.def
> (define_attr "tune"
> -
> "cortexa35,cortexa53,cortexa57,cortexa72,cortexa73,exynosm1,falkor,qdf24xx,thunderx,thunderxt88p1,thunderxt88,thunderxt81,thunderxt83,xgene1,thunderx2t99,vulcan,cortexa57cortexa53,cortexa72cortexa53,cortexa73cortexa35,cortexa73cortexa53"
> +
> "cortexa35,cortexa53,cortexa57,cortexa72,cortexa73,exynosm1,falkor,qdf24xx,thunderx,thunderxt88p1,thunderxt88,thunderxt81,thunderxt83,thunderx2t99,xgene1,thunderx2t99p1,vulcan,cortexa57cortexa53,cortexa72cortexa53,cortexa73cortexa35,cortexa73cortexa53"
> (const (symbol_ref "((enum attr_tune) aarch64_tune)")))
>