linux-kernel - Re: include/linux/irq.h:452: error: implicit declaration of function 'cpu_to

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4990A899.8070302@sgi.com>
Date:	Mon, 09 Feb 2009 14:05:13 -0800
From:	Mike Travis <travis@....com>
To:	Alexey Dobriyan <adobriyan@...il.com>
CC:	linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: include/linux/irq.h:452: error: implicit declaration of	function
 'cpu_to_node'

Alexey Dobriyan wrote:
> On Wed, Jan 21, 2009 at 03:38:25PM -0800, Mike Travis wrote:
>> Alexey Dobriyan wrote:
>>> On mips-malta:
>>>
>>>   CC      arch/mips/mti-malta/malta-int.o
>>> In file included from arch/mips/mti-malta/malta-int.c:25:
>>> include/linux/irq.h: In function 'init_alloc_desc_masks':
>>> include/linux/irq.h:452: error: implicit declaration of function 'cpu_to_node'
>>> include/linux/irq.h:454: error: 'GFP_ATOMIC' undeclared (first use in this function)
>>>
>>> I _think_ init_alloc_desc_masks() should be made out of line to fix this.
>> Hi Alexey,
>>
>> Does adding #include <linux/slab.h> fix the error?  If so, I can add this to
>> linux/irq.h.
>>
>> The primary reason it's inline is that unless CONFIG_CPUMASK_OFFSTACK=y, then the
>> entire routine is a NOP which optimizes out more completely as an inline function.
> 
> linux/slab.h helps.

Hi Alexey,

Since you reported this problem a similar issue was discovered on s390.  However
the fix for that was to include interrupt.h instead of irq.h.

Could you try this as well?

Here's a tail end pointer to the thread and Andrew's commit msg including the
patch is below.

http://marc.info/?l=linux-mm-commits&m=123360349631362&w=4

Thanks!
Mike
--- 

Subject: + sysrq-include-interrupth-instead-of-irqh.patch added to -mm tree
To: mm-commits@...r.kernel.org
Cc: heiko.carstens@...ibm.com, mingo@...e.hu, schwidefsky@...ibm.com,
        travis@....com
From: akpm@...ux-foundation.org
Date: Tue, 27 Jan 2009 15:24:08 -0800

The patch titled
     sysrq: include interrupt.h instead of irq.h
has been added to the -mm tree.  Its filename is
     sysrq-include-interrupth-instead-of-irqh.patch

Before you just go and hit "reply", please:
   a) Consider who else should be cc'ed
   b) Prefer to cc a suitable mailing list as well
   c) Ideally: find the original patch on the mailing list and do a
      reply-to-all to that, adding suitable additional cc's

*** Remember to use Documentation/SubmitChecklist when testing your code ***

See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find
out what to do about this

The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/

------------------------------------------------------
Subject: sysrq: include interrupt.h instead of irq.h
From: Heiko Carstens <heiko.carstens@...ibm.com>

With "cpumask: update irq_desc to use cpumask_var_t" in linux-next
we get this compile bug on s390:

  CC      drivers/char/sysrq.o
In file included from drivers/char/sysrq.c:38:
include/linux/irq.h: In function 'init_alloc_desc_masks':
include/linux/irq.h:442: error: dereferencing pointer to incomplete type

drivers/char/sysrq.c should include interrupt.h instead of irq.h.

Cc: Mike Travis <travis@....com>
Cc: Martin Schwidefsky <schwidefsky@...ibm.com>
Cc: Ingo Molnar <mingo@...e.hu>
Signed-off-by: Heiko Carstens <heiko.carstens@...ibm.com>
Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
---

 drivers/char/sysrq.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -puN drivers/char/sysrq.c~sysrq-include-interrupth-instead-of-irqh drivers/char/sysrq.c
--- a/drivers/char/sysrq.c~sysrq-include-interrupth-instead-of-irqh
+++ a/drivers/char/sysrq.c
@@ -35,7 +35,7 @@
 #include <linux/vt_kern.h>
 #include <linux/workqueue.h>
 #include <linux/kexec.h>
-#include <linux/irq.h>
+#include <linux/interrupt.h>
 #include <linux/hrtimer.h>
 #include <linux/oom.h>
 
_

Patches currently in -mm which might be from heiko.carstens@...ibm.com are

origin.patch
alpha-use-syscall-wrappers.patch
linux-next.patch
sysrq-include-interrupth-instead-of-irqh.patch
kmalloc-return-null-instead-of-link-failure.patch
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/