[lkml]   [2010]   [Jun]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 00/13] jump label v9

    Jason, I'm really at wits end about this patch set. To say
    that trying to test our your patches is frustrating for me
    so far would be an understatement.

    Nothing you ever post builds for me, not one patch set has
    built properly.

    I can also tell that you're just blindly making changes to the
    sparc bits and not trying to build test them at all:

    1) Even though you created the jump_label_t, and made it properly
    a u32 on sparc, you left the assembler using ".xword" to
    record the entries.

    2) The sparc "struct jump_label" still calls it's third member "name",
    it needs to be "key" or else the build breaks.

    3) Eventhough the sparc JUMP_LABEL macro was fixed to have two args,
    the first arg was left as "tag" instead of being renamed to "key"
    and that name change propaged into the asm in the macro expansion.

    I took care of that locally to try and test this, but then I hit the
    current major problem which is that you're using things like
    text_poke_early() unconditionally, but that is an X86-only facility
    implemented by x86's alternative mechanism.

    Also, kernel/jump_label.c only gets the ERR_PTR() definitions
    indirectly on the x86 platform, it needs to include linux/err.h
    directly to make sure those things are available on every platform.

    You gave me the impression a few iterations ago that you were doing
    build testing on sparc64 using cross-compilers, or that you would
    start to do so. You're obviously not, could you please start doing so
    and let me know when you've at least build tested your jump-label
    patch series on sparc64 and at least one architecture that lacks
    jump-label support?


     \ /
      Last update: 2010-06-15 05:51    [W:4.437 / U:0.008 seconds]
    ©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site