[lkml]   [2006]   [Jul]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: 2.6.18-rc1-mm2
    On 7/14/06, Dave Hansen <> wrote:
    > > +#define PFN_UP(x) (((x) + PAGE_SIZE - 1) >> PAGE_SHIFT)
    > > +#define PFN_DOWN(x) ((x) >> PAGE_SHIFT)
    > > +#define PFN_PHYS(x) ((x) << PAGE_SHIFT)
    > Please use include/linux/pfn.h instead of defining these

    Ah, of course.

    > > Since there's only a single board available, and that board has no use for
    > > discontigmem or sparsemem anyway, I figured it's better to just turn it off
    > > until a need for it arises.
    > How about we help you get sparsemem working properly, and you can kill
    > all of the discontigmem support from your arch? You can be the first
    > non-legacy-impeded architecture. ;)

    That would be great. I think maybe I should start by ripping out the
    discontigmem stuff altogether, since there are no boards available
    that requires it. But I can perhaps test sparsemem on a prototype

    > Feel free to mail Andy or myself with your compile errors, and I'm sure
    > we can iron it out. I'd try myself, but I don't have a cross-compiler
    > for your arch yet. Do you have one handy?

    Thanks. There are patches for binutils and gcc available from

    For binutils you should probably grab the pre-patched source tarball
    unless you want to mess around with autoconf. I tried to make a patch
    after regenerating all the makefiles, but the patch ended up being 5x
    the size of the original one...

    > It would also be nice to see a _couple_ of patches that perhaps abstract
    > a thing or two into generic code. I know that new architectures usually
    > begin with a 'cp -r', but it would be nice to see a wee bit of code
    > refactoring as a small price of admission. Some of the ioremap and
    > other pagetable code looked pretty generic to me.

    The pagetable code in ioremap.c is indeed copied verbatim from another
    architecture (I think it was MIPS.) I'll look at all the
    implementations of remap_area_pages() and see if it's possible to
    consolidate them when I get back to work in a bit less than two weeks.

    I won't be submitting any patches until then, but your mail will wait
    for me in my inbox so I won't forget about it. Thanks for taking the
    time to review this.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2006-07-18 23:51    [W:3.175 / U:0.364 seconds]
    ©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site