02 Mar, 2007

1 commit

  • Allow space(s) between "__attribute__" and "((blah))" so that
    kernel-doc does not complain like:

    Warning(/tester/linsrc/linux-2.6.20-git15//kernel/timer.c:939): No description found for parameter 'read_persistent_clock(void'

    Signed-off-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     

27 Feb, 2007

1 commit

  • * master.kernel.org:/pub/scm/linux/kernel/git/kyle/parisc-2.6: (78 commits)
    [PARISC] Use symbolic last syscall in __NR_Linux_syscalls
    [PARISC] Add missing statfs64 and fstatfs64 syscalls
    Revert "[PARISC] Optimize TLB flush on SMP systems"
    [PARISC] Compat signal fixes for 64-bit parisc
    [PARISC] Reorder syscalls to match unistd.h
    Revert "[PATCH] make kernel/signal.c:kill_proc_info() static"
    [PARISC] fix sys_rt_sigqueueinfo
    [PARISC] fix section mismatch warnings in harmony sound driver
    [PARISC] do not export get_register/set_register
    [PARISC] add ENTRY()/ENDPROC() and simplify assembly of HP/UX emulation code
    [PARISC] convert to use CONFIG_64BIT instead of __LP64__
    [PARISC] use CONFIG_64BIT instead of __LP64__
    [PARISC] add ASM_EXCEPTIONTABLE_ENTRY() macro
    [PARISC] more ENTRY(), ENDPROC(), END() conversions
    [PARISC] fix ENTRY() and ENDPROC() for 64bit-parisc
    [PARISC] Fixes /proc/cpuinfo cache output on B160L
    [PARISC] implement standard ENTRY(), END() and ENDPROC()
    [PARISC] kill ENTRY_SYS_CPUS
    [PARISC] clean up debugging printks in smp.c
    [PARISC] factor syscall_restart code out of do_signal
    ...

    Fix conflict in include/linux/sched.h due to kill_proc_info() being made
    publicly available to PARISC again.

    Linus Torvalds
     

21 Feb, 2007

2 commits

  • This patch stops "modpost" from issuing erroneous modpost warnings on ARM
    builds, which it's been doing since since maybe last summer. A canonical
    example would be driver method table entries:

    WARNING: - Section mismatch: reference to .exit.text:_remove
    from .data after '$d' (at offset 0x4)

    That "$d" symbol is generated by tools conformant with ARM ABI specs; in
    this case it's a symbol **in the middle of** a "_driver" struct.

    The erroneous warnings appear to be issued because "modpost" whitelists
    references from "_driver" data into init and exit sections ... but
    doesn't know should also include those "$d" mapping symbols, which are not
    otherwise associated with "_driver" symbols.

    This patch prevents the modpost symbol lookup code from ever returning
    those mapping symbols, so it will return a whitelisted symbol instead.
    Then things work as expected.

    Now to revert various code-bloating "fixes" that got merged because of this
    modpost bug....

    Signed-off-by: David Brownell
    Acked-by: Rusty Russell
    Cc: Roman Zippel
    Acked-by: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    David Brownell
     
  • Output of a function or struct in html mode needs to include the short
    description from the function/struct name line in the output title line.

    Signed-off-by: Randy Dunlap
    Acked-by: Johannes Berg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     

17 Feb, 2007

3 commits


15 Feb, 2007

8 commits

  • * 'for-linus' of git://one.firstfloor.org/home/andi/git/linux-2.6: (94 commits)
    [PATCH] x86-64: Remove mk_pte_phys()
    [PATCH] i386: Fix broken CONFIG_COMPAT_VDSO on i386
    [PATCH] i386: fix 32-bit ioctls on x64_32
    [PATCH] x86: Unify pcspeaker platform device code between i386/x86-64
    [PATCH] i386: Remove extern declaration from mm/discontig.c, put in header.
    [PATCH] i386: Rename cpu_gdt_descr and remove extern declaration from smpboot.c
    [PATCH] i386: Move mce_disabled to asm/mce.h
    [PATCH] i386: paravirt unhandled fallthrough
    [PATCH] x86_64: Wire up compat epoll_pwait
    [PATCH] x86: Don't require the vDSO for handling a.out signals
    [PATCH] i386: Fix Cyrix MediaGX detection
    [PATCH] i386: Fix warning in cpu initialization
    [PATCH] i386: Fix warning in microcode.c
    [PATCH] x86: Enable NMI watchdog for AMD Family 0x10 CPUs
    [PATCH] x86: Add new CPUID bits for AMD Family 10 CPUs in /proc/cpuinfo
    [PATCH] i386: Remove fastcall in paravirt.[ch]
    [PATCH] x86-64: Fix wrong gcc check in bitops.h
    [PATCH] x86-64: survive having no irq mapping for a vector
    [PATCH] i386: geode configuration fixes
    [PATCH] i386: add option to show more code in oops reports
    ...

    Linus Torvalds
     
  • This patch fixes showing empty config list items if "Option/Show All
    Options" is turned on. For example empty items appears on list of 'Block
    Layer' menu.

    Signed-off-by: Cyrill V. Gorcunov
    Cc: Roman Zippel
    Cc: Sam Ravnborg
    Cc: Oleg Verych
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Cyrill V. Gorcunov
     
  • Do "Back" button behaviour normalization so it is enabled starting from
    second-level menu only.

    Signed-off-by: Cyrill V. Gorcunov
    Cc: Roman Zippel
    Cc: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Cyrill V. Gorcunov
     
  • qconf does not clear help text in search window if previous search has been
    failed.

    Signed-off-by: Cyrill V. Gorcunov
    Cc: Roman Zippel
    Cc: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Cyrill V. Gorcunov
     
  • Relocate the qconf search command to the "Edit"->"Find" menu option.

    This is per the discussion on my qconf search dialog patch.

    Cc: Sam Ravnborg
    Cc: Roman Zippel
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Shlomi Fish
     
  • In xconfig's display integer and string values are also shown as part of
    the config item's descriptive text.

    This patch updates the descriptive text, when the corresponding value has
    been changed. Fix for http://bugzilla.kernel.org/show_bug.cgi?id=7744

    Take2 uses updateList() so config values dependending on the changed value
    see the change.

    Signed-off-by: Karsten Wiese
    Cc: Roman Zippel
    Cc: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Karsten Wiese
     
  • Fix a minor bug in mkcompile_h. As one can see, the current locale is used
    while getting the version of gcc. This produces problems when a locale
    other than C or en_US is used. As an example, my /proc/version contains
    Turkish characters in iso-8859-9 encoding.

    This patch fixes this issue by making sure that the C locale is used to get
    gcc's version.

    Cc: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    s situert
     
  • Check to see if `${CROSS_COMPILE}mkimage` exists and if not, fall back to
    the standard `mkimage`

    The Blackfin toolchain includes mkimage, but we dont want to namespace
    collide with any of the user's system setup, so we prefix it with our
    toolchain name.

    Signed-off-by: Mike Frysinger
    Cc: Sam Ravnborg
    Cc: Oleg Verych
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Mike Frysinger
     

13 Feb, 2007

1 commit

  • o Entry startup_32 was in .text section but it was accessing some init
    data too and it prompts MODPOST to generate compilation warnings.

    WARNING: vmlinux - Section mismatch: reference to .init.data:boot_params from
    .text between '_text' (at offset 0xc0100029) and 'startup_32_smp'
    WARNING: vmlinux - Section mismatch: reference to .init.data:boot_params from
    .text between '_text' (at offset 0xc0100037) and 'startup_32_smp'
    WARNING: vmlinux - Section mismatch: reference to
    .init.data:init_pg_tables_end from .text between '_text' (at offset
    0xc0100099) and 'startup_32_smp'

    o Can't move startup_32 to .init.text as this entry point has to be at the
    start of bzImage. Hence moved startup_32 to a new section .text.head and
    instructed MODPOST to not to generate warnings if init data is being
    accessed from .text.head section. This code has been audited.

    o SMP boot up code (startup_32_smp) can go into .init.text if CPU hotplug
    is not supported. Otherwise it generates more warnings

    WARNING: vmlinux - Section mismatch: reference to .init.data:new_cpu_data from
    .text between 'checkCPUtype' (at offset 0xc0100126) and 'is486'
    WARNING: vmlinux - Section mismatch: reference to .init.data:new_cpu_data from
    .text between 'checkCPUtype' (at offset 0xc0100130) and 'is486'

    Signed-off-by: Vivek Goyal
    Signed-off-by: Andrew Morton
    Signed-off-by: Andi Kleen

    Vivek Goyal
     

12 Feb, 2007

5 commits

  • Teach kernel-doc to handle functions that look like the new
    pcim_iomap_table(). Fixes this fatal error in scripts/kernel-doc:

    DOCPROC Documentation/DocBook/kernel-api.xml
    Error(/tester/linsrc/linux-2.6.20-git6//drivers/pci/pci.c:1351): cannot understand prototype: 'void __iomem * const * pcim_iomap_table(struct pci_dev *pdev) '
    make[1]: *** [Documentation/DocBook/kernel-api.xml] Error 1
    make: *** [htmldocs] Error 2

    Signed-off-by: Randy Dunlap
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     
  • - in man and text mode output, if the function return type is empty (like it
    is for macros), don't print the return type and a following space; this
    fixes an output malalignment;

    - in the function short description, strip leading, trailing, and multiple
    embedded spaces (to one space); this makes function name/description output
    spacing consistent;

    - fix a comment typo;

    Signed-off-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     
  • Allow whitespace in pointer-to-function
    [accept "(* done)", not just "(*done)"].

    Allow tabs (spaces are already allowed) between "#define" and a macro name.

    Signed-off-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     
  • In kernel-doc syntax, be a little flexible: allow whitespace between
    a function parameter name and the colon that must follow it, such as:
    @pdev : PCI device to unplug

    (This allows lots of megaraid kernel-doc to work without tons of
    editing.)

    Signed-off-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     
  • The malformed HTML was generated after switch to XSLTPROC
    from SGML tools. The reference title

    struct x

    is converted into two recursive tags

    struct x

    There is more possible solutions for this problem.
    One can be found at

    http://darkk.livejournal.com/

    The proposed solution is based on suggestion provided by Jiri Kosek.

    Signed-off-by: Pavel Pisa
    Acked-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Pavel Pisa
     

10 Feb, 2007

1 commit

  • This adds the remaining changes which should have been part of the
    review process.

    - the define command is inappropriate (it's primarily for rule
    definitions)
    - execute commands in the current dir as all other commands
    - .*.tmp (but not .*.null) files are also removed up by "make clean"
    - printf has other side effects, just use "echo -e"
    - proper quoting
    - proper indentation

    Signed-off-by: Roman Zippel
    Signed-off-by: Linus Torvalds

    Roman Zippel
     

09 Feb, 2007

2 commits

  • parisc and parisc64 seem to name sections a little differently from other
    targets. parisc64 gives spurious warnings like:

    WARNING: drivers/net/dummy.o - Section mismatch: reference to .init.text:dummy_setup from .data.rel.ro between '.LC1' (at offset 0x0) and '.LC6'

    and parisc gives spurious warnings like:

    WARNING: drivers/net/dummy.o - Section mismatch: reference to .init.text:dummy_setup from .rodata.cst4 between '.LC1' (at offset 0x0) and '.LC6'

    Given the other comments in modpost.c, it seems that the best solution is
    to move rodata down to the 'match at start of name' section and add
    .data.rel.ro to the 'match entire name' section.

    Cc: Sam Ravnborg
    Cc: Kyle McMartin
    Signed-off-by: Andrew Morton

    Matthew Wilcox
     
  • Michal Ostrowski points out what the real problem was: the spaces at the
    start of the definition of the 'checker-shell' make function.

    Cc: Michal Ostrowski
    Acked-by: David Miller
    Acked-by: Geert Uytterhoeven
    Acked-by: Oleg Verych
    Signed-off-by: Linus Torvalds

    Linus Torvalds
     

08 Feb, 2007

2 commits


07 Feb, 2007

2 commits


31 Jan, 2007

1 commit

  • The current filename->define translation does not scrub dashes so when
    creating stub defines for like asm-x86_64/ptrace-abi.h, we get: #define
    __ASM_STUB_PTRACE-ABI_H

    gcc just hates that sort of thing :)

    trivial attached patch adds - to the tr list to scrub it to _

    Signed-off-by: Mike Frysinger
    Cc: David Woodhouse
    Cc: Sam Ravnborg
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Mike Frysinger
     

12 Jan, 2007

1 commit


11 Jan, 2007

2 commits

  • o MODPOST generates warning on i386 if kernel is compiled with
    CONFIG_RELOCATABLE=y.

    WARNING: vmlinux - Section mismatch: reference to .init.text:__init_begin from .text between 'free_initmem' (at offset 0xc0114fd3) and 'do_test_wp_bit'
    WARNING: vmlinux - Section mismatch: reference to .init.text:_sinittext from .text between 'core_kernel_text' (at offset 0xc012aeae) and 'kernel_text_address'
    WARNING: vmlinux - Section mismatch: reference to .init.text:_einittext from .text between 'core_kernel_text' (at offset 0xc012aeb7) and 'kernel_text_address'
    WARNING: vmlinux - Section mismatch: reference to .init.text:_sinittext from .text between 'get_symbol_pos' (at offset 0xc0135776) and 'reset_iter'
    WARNING: vmlinux - Section mismatch: reference to .init.text:_einittext from .text between 'get_symbol_pos' (at offset 0xc013577d) and 'reset_iter'

    o These symbols (__init_begin, _sinittext, _einittext) belong to init
    section and generally represent a section boundary. These are special
    symbols in the sense that their size is zero and no memory is allocated
    for them in init section. Their addr and value are same. So even if
    we free the init section, it is ok to reference them.

    o Whitelist access to such select symbols in MODPOST.

    Signed-off-by: Vivek Goyal
    Signed-off-by: Andi Kleen
    Cc: "Eric W. Biederman"
    Cc: Andi Kleen
    Signed-off-by: Andrew Morton

    Vivek Goyal
     
  • o MODPOST generates warning for i386 if compiled with CONFIG_RELOCATABLE=y
    and serial console support is enabled.

    o Serial console setup function, serial8250_console_setup(), is a non __init
    function and it calls functions which are of type __init().
    (uart_parse_options() and uart_set_options()). Assuming, setup will
    be called during init time, changing serial8250_console_setup() to __init.

    o Adding one more pattern to modpost whitelist. Console drivers might
    have *_console structures containing references to setup functions which
    can be of __init type. Don't generate warnings for those.

    WARNING: vmlinux - Section mismatch: reference to .init.text: from .data between 'serial8250_console' (at offset 0xc05a33d8) and 'serial8250_reg'

    Signed-off-by: Vivek Goyal
    Signed-off-by: Andi Kleen
    Cc: "Eric W. Biederman"
    Cc: Andi Kleen
    Signed-off-by: Andrew Morton

    Vivek Goyal
     

06 Jan, 2007

1 commit


23 Dec, 2006

1 commit

  • Make kernel-doc support unnamed (anonymous) structs and unions. There is
    one (union) in include/linux/skbuff.h (inside struct sk_buff) that is
    currently generating a kernel-doc warning, so this fixes that warning.

    Signed-off-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     

14 Dec, 2006

5 commits

  • Clean up a little.

    Signed-off-by: Karsten Wiese
    Cc: Sam Ravnborg
    Cc: Roman Zippel
    Acked-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Karsten Wiese
     
  • Added function sets "void (*conf_changed_callback)(void)". Call it, if
    .config's changed state changes. Use above in qconf.cc to set gui's
    save-widget's sensitvity.

    Signed-off-by: Karsten Wiese
    Cc: Sam Ravnborg
    Cc: Roman Zippel
    Acked-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Karsten Wiese
     
  • Those two functions are
    void sym_set_change_count(int count)
    and
    void sym_add_change_count(int count)

    All write accesses to sym_change_count are replaced by calls to above
    functions.

    Variable and changer-functions are moved to confdata.c. IMO thats ok, as
    sym_change_count is an attribute of the .config's change state.

    Signed-off-by: Karsten Wiese
    Cc: Sam Ravnborg
    Cc: Roman Zippel
    Acked-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Karsten Wiese
     
  • Run "make xconfig" on a freshly untarred kernel-tree. Look at the floppy disk
    icon of the qt application, that has just started: Its in a normal, active
    state.

    Mouse click on it: .config is being saved.

    This patch series changes things so taht
    after the mouse click on the floppy disk icon, the icon is greyed out.
    If you mouse click on it now, nothing happens.

    If you change some CONFIG_*, the floppy disk icon returns to "active state",
    that is, if you mouse click it now, .config is written.

    This patch:

    Returns sym_change_count to reflect the .config's change state.
    All read only accesses of
    sym_change_count
    are replaced by calls to
    conf_get_changed()
    .
    mconfig.c is manipulated to ask for saving only when
    conf_get_changed() returned true.

    Signed-off-by: Karsten Wiese
    Cc: Sam Ravnborg
    Cc: Roman Zippel
    Acked-by: Randy Dunlap
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Karsten Wiese
     
  • Section .parainstructions should not warn about section mismatches.

    WARNING: drivers/net/hamradio/scc.o - Section mismatch: reference to .exit.text: from .parainstructions after '' (at offset 0x0)
    WARNING: drivers/net/hamradio/scc.o - Section mismatch: reference to .exit.text: from .parainstructions after '' (at offset 0x8)

    Signed-off-by: Randy Dunlap
    Cc: Andi Kleen
    Acked-by: Rusty Russell
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Randy Dunlap
     

11 Dec, 2006

1 commit

  • The as-instr/ld-option need to create temporary files, but create them in the
    output directory, when compiling external modules. Reformat them a bit and
    use $(CC) instead of $(AS) as the former is used by kbuild to assemble files.

    Signed-off-by: Roman Zippel
    Cc: Andi Kleen
    Cc: Jan Beulich
    Cc: Sam Ravnborg
    Cc:
    Cc: Horst Schirmeier
    Cc: Daniel Drake
    Signed-off-by: Andrew Morton
    Signed-off-by: Linus Torvalds

    Roman Zippel