[PATCH 06/38] powerpc: Move 64-bit feature fixup earlier

Aneesh Kumar K.V aneesh.kumar at linux.vnet.ibm.com
Tue Jun 28 21:49:26 AEST 2016


Benjamin Herrenschmidt <benh at kernel.crashing.org> writes:

> On Tue, 2016-06-28 at 16:35 +0530, Aneesh Kumar K.V wrote:
>> Why not move it immediately after we finish all the device tress
>> scan.
>
> Do you need it in early_init_mmu ? The important thing is to have it
> done before we *turn on* the MMU but yeah, there's no big deal
> moving it even further up I think..

If we can do feature fixup early. Then this patch series become simpler.

https://lkml.kernel.org/r/1465887288-12952-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com

ie, we can now start using cpu_has_feature/mmu_has_feature early instead
of using __cpu/mmu_has_feature.

>
>> ie,
>>         early_init_devtree(__va(dt_ptr));
>> 
>>         epapr_paravirt_early_init();
>> 
>>   +      apply_feature_fixups();
>>         /* Now we know the logical id of our boot cpu, setup the
>> paca. */
>>         setup_paca(&paca[boot_cpuid]);
>>         fixup_boot_paca();
>> 
>> Also with a comment explaining why we can't move it further up ?
>
> Yup, adding some comments about the why things are done in that order
> would be good, I'll improve that
>
> Cheers,
> Ben.

-aneesh



More information about the Linuxppc-dev mailing list