On Fri, Nov 26 2021 at 11:11, taoyi ty wrote:

On 11/25/21 11:15 PM, Thomas Gleixner wrote:

quoted

quoted

+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