On Wed, 12 Dec 2012, Kirill A. Shutemov wrote: > > What's the benefit from doing this other than generalizing some per-arch > > code? It simply adds on more layer of redirection to try to find the > > implementation that matters for the architecture you're hacking on. > > The idea of asm-generic is consolidation arch code which can be re-used > for different arches. It also makes support of new arches easier. > Yeah, but you're moving is_zero_pfn() unnecessarily into a header file when it is only used mm/memory.c and it adds a __HAVE_* definition that we always try to reduce (both __HAVE and __ARCH definitions are frowned upon). I don't think it's much of a win to obfuscate the code because mips and s390 implement colored zero pages. -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html