[PATCH] inclusion of <asm/system.h> into spinlock.h breaks sparc

From: Andreas Beckmann (andreas_at_abeckmann.de)
Date: 12/06/03

  • Next message: Dan Creswell: "Irq balancing problem?"
    Date:	Sat, 06 Dec 2003 23:56:04 +0100
    To: linux-kernel@vger.kernel.org
    
    
    

    Hi,

    the recent inclusion of <asm/system.h> into spinlock.h causes a cyclic
    inclusion under the sparc architecture. This breaks the build (at the
    beginning of make dep) in oplib.h because spinlock_t is not yet defined.

    linux/spinlock.h
       asm/system.h
         asm/oplib.h
           linux/spinlock.h
         ERROR! (asm/oplib.h)

    I was working with 2.4.23-bk3 and -bk4.

    The change was introduced in the following changeset:
    --------------------
    ChangeSet 1.1192.4.2 2003/12/02 11:25:25 ralf@linux-mips.org
       [PATCH] Include <asm/system.h> into spinlock.h

       <linux/spinlock.h> uses local_irq_save() etc. from <asm/spinlock.h> but
       relies on this header file having been dragged in on some other way.
       So if things are just right the build may blow up ...
    include/linux/spinlock.h 1.9 2003/11/28 10:38:30 ralf@linux-mips.org
       Include <asm/system.h> into spinlock.h
    --------------------

    Not including asm/oplib.h from asm/system.h seems to fix this.
    The patch drops this include.

    Andreas

    Please CC: me in your replies.

    
    

    --- linux-2.4.23/include/asm-sparc/system.h.orig 2003-11-28 19:26:21.000000000 +0100
    +++ linux-2.4.23/include/asm-sparc/system.h 2003-12-06 05:49:05.000000000 +0100
    @@ -10,7 +10,6 @@
     
     #ifdef __KERNEL__
     #include <asm/page.h>
    -#include <asm/oplib.h>
     #include <asm/psr.h>
     #include <asm/ptrace.h>
     #include <asm/btfixup.h>

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/


  • Next message: Dan Creswell: "Irq balancing problem?"

    Relevant Pages

    • Re: [PATCH][MMC] Secure Digital (SD) support
      ... I tried to get some feedback on inclusion of this ... >interface area, I am convinced that it is fundamentally incorrect ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [RFC] atomic open(..., O_CREAT | ...)
      ... > We've already got a patch that does this, and that I'm queueing up for ... > inclusion. ... dentry_openneeds the original open flags, ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH] 2.4.x Linux Medley RAID Version 7
      ... >> Patch for inclusion should have this cleaned up. ... > Medley superblock without comparing these magic words with the PCI values. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH][MMC] Secure Digital (SD) support
      ... I tried to get some feedback on inclusion of this ... interface area, I am convinced that it is fundamentally incorrect ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [RFC] __initdata strings
      ... >>This patch is not intended for inclusion, ... any scheme that moves strings in __init functions to .data.init has ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)