Re: [PASEMI] Nemo board doesn't boot anymore after the commit "powerpc/book3s64/pkeys: Simplify pkey disable branch"

["Aneesh Kumar K.V" <aneesh.kumar@linux.ibm.com>]

On 8/9/20 7:42 PM, Christian Zigotzky wrote:
> Hello,
> 
> The Nemo board (A-EON AmigaOne X1000) [1] doesn't start with the latest 
> Git kernel anymore after the commit "powerpc/book3s64/pkeys: Simplify 
> pkey disable branch" [2].
> 
> I bisected today [3].
> 
> Result: powerpc/book3s64/pkeys: Simplify pkey disable branch 
> (a4678d4b477c3d2901f101986ca01406f3b7eaea) [2] is the first bad commit.
> 
> Unfortunately I wasn't able to revert the first bad commit. The first 
> bad commit depends on many other commits, which unfortunately I don't 
> know. I tried to remove the modifications of the files from the first 
> bad commit but without any success. There are just too many dependencies.
> 
> Additionally I reverted the commit "selftests/powerpc: Fix pkey syscall 
> redefinitions" [4] and compiled a new kernel but without any success.
> 
> Could you please check the first bad commit?
> 
> Thanks,
> Christian
> 


Can you share a successful boot log of the system so that i can double 
check the cpu_feature and mmu_feature reported ? I am looking for 
details similar to below.

[    0.000000] cpu_features      = 0x0001c07f8f5f91a7
[    0.000000]   possible        = 0x0001fbffcf5fb1a7
[    0.000000]   always          = 0x00000003800081a1
[    0.000000] cpu_user_features = 0xdc0065c2 0xefe00000
[    0.000000] mmu_features      = 0x7c006001
[    0.000000] firmware_features = 0x0000001fc45bfc57
[    0.000000] vmalloc start     = 0xc008000000000000
[    0.000000] IO start          = 0xc00a000000000000
[    0.000000] vmemmap start     = 0xc00c000000000000


IIUC this is P5+? (ISA 2.04). On that pkey should be marked disabled via

static int scan_pkey_feature(void)
{
	int ret;
	int pkeys_total = 0;

	....

	/*
	 * Only P7 and above supports SPRN_AMR update with MSR[PR] = 1
	 */
	if (!early_cpu_has_feature(CPU_FTR_ARCH_206))
		return 0;


}

Can you boot with CONFIG_PPC_MEM_KEYS=n ?

-aneesh