Hi Michael, Here is the updated version, could you please help to review it again?
As you suggested, this version didn't copy the code, but splitted the logic into a helper function, so both kernel_map_linear_page() and __hash_page_huge() can use. Also patch #1 moves some unnecessary code out of the repeating loop, so the splitting is easier. Patch #3 removes the HPTE_V_BOLTED flag in kernel_map_linear_page(), it seems not needed based on my understanding. Changes are split into smaller ones, so each one did only one thing. Thanks, Zhong Li Zhong (4): powerpc: Move the setting of rflags out of loop in __hash_page_huge powerpc: Split the code trying to insert hpte repeatedly as an helper function powerpc: Don't bolt the hpte in kernel_map_linear_page() powerpc: Try to insert the hptes repeatedly in kernel_map_linear_page() arch/powerpc/mm/hash_utils_64.c | 45 +++++++++++++++++++++++++++++++--- arch/powerpc/mm/hugetlbpage-hash64.c | 31 +++++------------------ 2 files changed, 47 insertions(+), 29 deletions(-) -- 1.7.9.5 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev