lkml.org 
[lkml]   [2011]   [Jan]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: Bug#609371: linux-image-2.6.37-trunk-sparc64: module scsi_mod: Unknown relocation: 36
    From
    From: "Bernhard R. Link" <brl+ccmadness@pcpool00.mathematik.uni-freiburg.de>
    Date: Mon, 17 Jan 2011 15:39:54 +0100

    >> I think we want none of this, and I think we should elide the align
    >> directives entirely, or at least fix them so we don't get unaligned
    >> stuff on 64-bit.
    >
    > One fix might be to move the __attribute__ from include/trace/ftrace.h
    > (and from include/linux/syscalls.h) to include/linux/ftrace_event.h
    > and attach it to the struct there. This way it should only increase it.

    I'm beginning to think that the align directive is there purposely to
    down-align the structure so that the amount of space that tracing
    information consumes is minimized.

    I honestly can't tell, only Steven Rostedt can tell us for sure,
    because there are no commit messages or comments that explain why
    these things need to be there.

    If the align directives exist for that reason, then your suggestion
    would likely completely undo what these directives are trying to
    achieve.

    Someone mentioned that the default struct alignment on x86-64 is
    something rather rediculious like 32 bytes or something like that.
    Yet someone else suggested to use __aligned__(32) to fix this, so
    color me completely confused.



    \
     
     \ /
      Last update: 2011-01-18 07:29    [W:3.125 / U:0.128 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site