lkml.org 
[lkml]   [2009]   [Jan]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: Doubts in usr/include/linux/elf.h
    On Wed, Jan 21, 2009 at 05:46:26AM +0530, Jaswinder Singh Rajput wrote:
    > usr/include/linux/elf.h is giving 4 'make headers_check' warnings:
    > usr/include/linux/elf.h:379: extern's make no sense in userspace
    > usr/include/linux/elf.h:387: extern's make no sense in userspace
    > usr/include/linux/elf.h:401: extern's make no sense in userspace
    > usr/include/linux/elf.h:402: extern's make no sense in userspace
    >
    >
    > usr/include/linux/elf.h:
    > --
    > #if ELF_CLASS == ELFCLASS32
    >
    > extern Elf32_Dyn _DYNAMIC [];
    > #define elfhdr elf32_hdr
    > #define elf_phdr elf32_phdr
    > #define elf_note elf32_note
    > #define elf_addr_t Elf32_Off
    >
    > #else
    >
    > extern Elf64_Dyn _DYNAMIC [];
    > #define elfhdr elf64_hdr
    > #define elf_phdr elf64_phdr
    > #define elf_note elf64_note
    > #define elf_addr_t Elf64_Off
    >
    > #endif

    If my quick grep is correct we fo not define any variable named _DYNAMIC.
    So you can just drop these two lines.

    >
    > /* Optional callbacks to write extra ELF notes. */
    > #ifndef ARCH_HAVE_EXTRA_ELF_NOTES
    > static __inline__ int elf_coredump_extra_notes_size(void) { return 0; }
    > static __inline__ int elf_coredump_extra_notes_write(struct file *file,
    > loff_t *foffset) { return 0; }
    > #else
    > extern int elf_coredump_extra_notes_size(void);
    > extern int elf_coredump_extra_notes_write(struct file *file, loff_t *foffset);
    > #endif

    And the above can all be wrapped in __KERNEL__.
    They got renamed in 2007:
    http://ozlabs.org/pipermail/cbe-oss-dev/2007-September/003276.html

    So they are hardly used by userspace anyway.

    Sam


    \
     
     \ /
      Last update: 2009-01-21 05:19    [W:0.021 / U:123.564 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site