On Thu, 2007-10-18 at 07:01 -0500, Josh Boyer wrote: > On Thu, 18 Oct 2007 21:45:14 +1000 > Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote: > > > > > On Thu, 2007-10-18 at 05:44 -0500, Josh Boyer wrote: > > > On Thu, 2007-10-18 at 11:08 +0400, Yuri Tikhonov wrote: > > > > Hello, > > > > > > > > The following patch adds support for 256KB PAGE_SIZE on ppc44x-based > > > > boards. > > > > The applications to be run on the kernel with 256KB PAGE_SIZE have to > > > > be > > > > built using the modified version of binutils, where the MAXPAGESIZE > > > > definition is set to 0x40000 (as opposite to standard 0x10000). > > > > > > Sorry, this is against arch/ppc which is bug fix only. New features > > > should be done against arch/powerpc. Also, I'd rather see something > > > along the lines of hugetlbfs support instead. > > > > I slightly disagree on that one. It does make sense in embedded > > applications to use larger page sizes like that to compensate for small > > TLBs, and hugetlbfs has serious constraints that may well make it > > impractical. > > Out of curiosity, what constraints are those?
You have to open & map a specific file in specific areas, so you don't just get existing code build & run with larger page size without change, there are various sneaky limitations here or there too, such as allocations more likely to fail etc... and in general, switching the base page size provides overall more performance improvement than just hacking an app to use hugetlbfs. > > Based on that, I'd be tempted to let that in provided it doesn't > > requires ugly hacks, which seems to be the case. It still needs to be > > adapted to arch/powerpc however, and get closer scrutiny that I didn't > > have time to do yet. > > > > You are the maintainer, so you decide, but my opinion here is that > > wanting that is fair enough. > > I always reserve the right to change my mind. If something makes sense > and the code is decent enough then it might very well be acceptable. > Requiring a modified binutils makes me a bit nervous though. >From a kernel point of view, I totally don't care about the modified binutils to build userspace as long as it's not required to build the kernel and that option is not enabled by default (and explicitely documented as having that requirement). If it is necessary for building the kernel, then I'm a bit cooler about the whole thing indeed, the max page size needs to be added at least as a command line or linker script param so a different build of binutils isn't needed. My main base for judging is really how invasive/ugly the patch is. If it's neat and fits in without much damage, and serves a purpose, then there is no reason not to get it in. Ben. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
