On Wed, 13 Mar 2013 04:00:12 +0900 Paul Mundt <lethal@xxxxxxxxxxxx> wrote: > On Tue, Mar 12, 2013 at 07:48:25PM +0100, Gerald Schaefer wrote: > > This patch introduces those huge_pte_xxx functions and their > > implementation on all architectures supporting hugetlbfs. This change > > will be a no-op for all architectures other than s390. > > > .. > > > arch/ia64/include/asm/hugetlb.h | 36 ++++++++++++++++++++++++ > > arch/mips/include/asm/hugetlb.h | 36 ++++++++++++++++++++++++ > > arch/powerpc/include/asm/hugetlb.h | 36 ++++++++++++++++++++++++ > > arch/s390/include/asm/hugetlb.h | 56 +++++++++++++++++++++++++++++++++++++- > > arch/s390/include/asm/pgtable.h | 20 -------------- > > arch/s390/mm/hugetlbpage.c | 2 +- > > arch/sh/include/asm/hugetlb.h | 36 ++++++++++++++++++++++++ > > arch/sparc/include/asm/hugetlb.h | 36 ++++++++++++++++++++++++ > > arch/tile/include/asm/hugetlb.h | 36 ++++++++++++++++++++++++ > > arch/x86/include/asm/hugetlb.h | 36 ++++++++++++++++++++++++ > > mm/hugetlb.c | 23 ++++++++-------- > > 11 files changed, 320 insertions(+), 33 deletions(-) > > > None of these wrappers are doing anything profound for most platforms, so > this would be a good candidate for an asm-generic/hugetlb.h (after which > s390 can continue to be special and no one else has to care). Yes, that was also my first idea, but I vaguely remembered some discussion with Andrew when I sent the original s390 hugetlb support patch (which also went for the asm-generic approach). So I tried to dig out that thread, and it turned out that the ugliness of ARCH_HAS_xxx actually resulted in my original patch to be changed into removing lots of those and therefore creating the individual arch header files, for the sake of readability and maintainability. So I guess it would be straightforward to extend those header files now, instead of re-introducing some of the ugliness. See also here http://marc.info/?l=linux-kernel&m=120536577402075&w=2 and here http://marc.info/?l=linux-kernel&m=120732788201196&w=2. Thanks, Gerald -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
