Thread (29 messages) flat view 29 messages, 2 authors, 2021-10-22

Re: [PATCH v3 13/18] powerpc/64e: remove mmu_linear_psize

From: Nicholas Piggin <npiggin@gmail.com>
Date: 2021-10-22 09:22:10

Excerpts from Christophe Leroy's message of October 22, 2021 4:49 pm:

Le 22/10/2021 à 00:30, Nicholas Piggin a écrit :
quoted
mmu_linear_psize is only set at boot once on 64e, is not necessarily
the correct size of the linear map pages, and is never used anywhere.
Remove it.
mmu_linear_psize is defined as a macro in:

     arch/powerpc/include/asm/book3s/32/mmu-hash.h, line 152 (as a macro)
     arch/powerpc/include/asm/nohash/32/mmu-40x.h, line 66 (as a macro)
     arch/powerpc/include/asm/nohash/32/mmu-44x.h, line 150 (as a macro)
     arch/powerpc/include/asm/nohash/32/mmu-8xx.h, line 173 (as a macro)

Is that needed at all or should is be cleaned, if nothing else than 64s 
uses it ?

Otherwise, why not do the same with 64e and define it as a macro ? Maybe 
that would help minimising the amount of ifdefs.
I prefer to remove it entirely, to avoid situations like 64e where it 
was being used without understanding what the value really was.

32e can come in a later cleanup to avoid making this series any bigger.

Thanks,
Nick
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help