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

Reply via email to