lkml.org 
[lkml]   [2008]   [Jan]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] x86: fix unconditional arch/x86/kernel/pcspeaker.c?compiling

    * Matt Mackall <mpm@selenic.com> wrote:

    > > btw., are there any pending arch/x86 bits in -tiny? (stupid
    > > question: were can i get the most uptodate version of -tiny from?)
    >
    > It's not a stupid question. I dropped updating the tree regulary some
    > time ago to focus on merging bits and then got a bit side-tracked by
    > this little thing called "version control".
    >
    > Michael is attempting to get the tree started again and has put a
    > quilt up here:
    >
    > http://elinux.org/images/3/3c/Tiny-quilt-2.6.23-0.tar.bz2

    cool! A few comments about the patches that affect arch/x86:

    dmi_blacklist.patch:

    +#ifdef CONFIG_DMI_SCAN
    dmi_scan_machine();
    +#endif

    put that #ifdef into the include file and make it an inlined NOP in the
    !DMI_SCAN case. That de-#ifdef-ifies the .c files.

    cpu-support.patch: ditto.

    mtrr.patch: ditto.

    inflate-error.patch: please use:

    quilt refresh --diffstat --sort --no-timestamps -p 1

    when generating patches, to make them reviewable, uniform and
    noise-free.

    no-doublefault.patch: looks ok. Please add proper metadata and submit to
    lkml.

    sbf.patch: looks ok.

    sysenter.patch:

    +#ifdef CONFIG_SYSENTER
    sysenter_setup();
    enable_sep_cpu();
    +#endif

    hide these #ifdefs in the include file. (as it already does) Looks ok
    otherwise.

    threadinfo-ool.patch: doesnt this break the scheduler? Had something
    like this in -rt, i turned 'current' into a const actually, but had
    trouble with keeping from gcc to do the right thing in sched.c. I guess
    we could live with current_non_const() that gets it from assembly. (and
    hence gcc wont be able to optimize it away in the middle of the
    context-switch) But when done properly, this is both a nice, measurable
    speedup on UP, and a nice space-saver.

    tiny-cflags.patch: obsolete? Isnt CFLAGS already extendable? Question to
    Sam i guess.

    tsc.patch: look ok. needs x86.git porting.

    Ingo


    \
     
     \ /
      Last update: 2008-01-22 15:45    [W:2.294 / U:0.924 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site