| From | Paul Burton <> | Subject | Re: [PATCH v3 11/34] mips: mm: Add p?d_large() definitions | Date | Thu, 28 Feb 2019 02:15:28 +0000 |
| |
Hi Steven,
On Wed, Feb 27, 2019 at 05:05:45PM +0000, Steven Price wrote: > For mips, we don't support large pages on 32 bit so add stubs returning 0.
So far so good :)
> For 64 bit look for _PAGE_HUGE flag being set. This means exposing the > flag when !CONFIG_MIPS_HUGE_TLB_SUPPORT.
Here I have to ask why? We could just return 0 like the mips32 case when CONFIG_MIPS_HUGE_TLB_SUPPORT=n, let the compiler optimize the whole thing out and avoid redundant work at runtime.
This could be unified too in asm/pgtable.h - checking for CONFIG_MIPS_HUGE_TLB_SUPPORT should be sufficient to cover the mips32 case along with the subset of mips64 configurations without huge pages.
Thanks, Paul
|