On 2021-03-17 19:43 +0100, Vincent Blut wrote: > Le 2021-03-17 15:49, Wookey a écrit : > > On 2021-03-17 14:52 +0100, Vincent Blut wrote: > > > Le 2021-01-27 12:57, Wookey a écrit : > > > > Version: Please enable ARM CMN-600 power management on arm64 > > > > > > > > This requires CONFIG_ARM_CMN=y > > > > > > Does it really have to be built-in instead of being provided as a module? > > > Last I > > > checked, Fedora and Ubuntu provide it as a module. > > > > No it should really be a module. Perf is driven from userspace so you > > never need to use it before modules can be loaded. > > Agreed.
> > I see that > > CONFIG_THUNDERX2_PMU=y > > CONFIG_ARM_SMMU_V3_PMU=y > > are also set as builtins. That's probably wrong too. > > It seems your arm64 kernel config deviates from the one we provide in Debian. > CONFIG_THUNDERX2_PMU is compiled as a module while CONFIG_ARM_SMMU_V3_PMU is > not set, at least in linux 5.10.19-1. Hmm. I was looking at the (built, with CONFIG_ARM_CMN=y) sources for 5.10.9-1 and the (unbuilt) sources for 5.10.19-1. So yes, slightly different and the built version is not up to date any more. If we already have CONFIG_THUNDERX2_PMU=m already then that's great (Ah yes - that's the upstream default). Adding CONFIG_ARM_SMMU_V3_PMU=m would be good too. Adding it as a module should be pretty harmless then at least it's available? I'll set off a build now to check it works. > > […] > > > I also checked the state of the other perf configs with the arm kernel team > > and got feedback that we have all the ones that should sensibly be set set > > once > > CONFIG_ARM_CMN=m > > and > > CONFIG_THUNDERX2_PMU=m > > is added > > This means updating the arm64 kernel config to only include ARM_CMN as a > module. > To me it is acceptable for Bullseye as this seems uncontroversial, but note > that > I can't speak for the kernel team. Will you ask them, or should I? It seems like prodding someone would be good as this was filed back on 27th jan and there have been uploads since, so I guess no-one has noticed till now. > > Upstream enables > > CONFIG_FSL_IMX8_DDR_PMU=m > > by default too. IMX8 hardware is available so we should probably turn this > > on too > > Contrary to Ubuntu, we do not provide support for the i.MX8M SoC family, > so enabling this option in the arm64 kernel config is not an option, right? Ah OK. I didn't realise IMX8 was not enabled in the debian kernel (A subject for a different bug). In that case, no this is not appropriate. Wookey -- Principal hats: Linaro, Debian, Wookware, ARM http://wookware.org/
signature.asc
Description: PGP signature
