On Fri, Nov 26 2021 at 11:11, taoyi ty wrote: > On 11/25/21 11:15 PM, Thomas Gleixner wrote: >>> +void setup_pks(void) >>> +{ >>> + if (!cpu_feature_enabled(X86_FEATURE_PKS)) >>> + return; >>> + >>> + write_pkrs(pkrs_init_value); >> >> Is the init value set up _before_ this function is invoked for the first >> time? >> > Setting up for cpu0 is before create_initial_pkrs_value. therefore pkrs > value of cpu0 won't be set correctly. > > [root@AliYun ~]# rdmsr -a 0x000006E1 > 0 > 55555554 > 55555554 > 55555554 > 55555554 > 55555554 > 55555554 > 55555554 > 55555554 > 55555554 > > Here are my test results after applying the patches
Thanks for confirming what I assumed from looking at the patches! tglx