All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keith Owens <kaos@sgi.com>
To: linux-ia64@vger.kernel.org
Subject: [patch 2.6.19-rc2] Correct definition of handle_IPI
Date: Wed, 18 Oct 2006 05:36:49 +0000	[thread overview]
Message-ID: <8916.1161149809@kao2.melbourne.sgi.com> (raw)

The declaration of handle_IPI in arch/ia64/kernel/smp.c was changed but
not the definition of this function.  Remove struct pt_regs from
handle_IPI().

The declaration of handle_IPI should really be in a header file instead
of a .c file, but which one?  Adding

  extern irqreturn_t handle_IPI(int irq, void *dev_id);

to include/asm-ia64/smp.h is the obvious place but that leads to
include hell.  Using irqreturn_t requires linux/interrupt.h which
leads to a recursive mess.  sched.h includes smp.h quit early, adding
interrupt.h to smp.h uses data from sched.h before most of sched.h is
expanded.

  CC      arch/ia64/kernel/asm-offsets.s
In file included from linux/include/linux/hardirq.h:5,
                 from linux/include/linux/interrupt.h:11,
                 from include/asm-ia64/smp.h:17,
                 from linux/include/linux/smp.h:18,
                 from linux/include/linux/sched.h:65,
                 from linux/arch/ia64/kernel/asm-offsets.c:9:
linux/include/linux/smp_lock.h: In function reacquire_kernel_lock:
linux/include/linux/smp_lock.h:35: error: dereferencing pointer to incomplete type
In file included from linux/include/linux/slab.h:14,
                 from linux/include/linux/percpu.h:5,
                 from linux/include/asm-generic/irq_regs.h:15,
                 from include/asm-ia64/irq_regs.h:1,
                 from linux/include/linux/irq.h:24,
                 from include/asm-ia64/hardirq.h:11,
                 from linux/include/linux/hardirq.h:7,
                 from linux/include/linux/interrupt.h:11,
                 from include/asm-ia64/smp.h:17,
                 from linux/include/linux/smp.h:18,
                 from linux/include/linux/sched.h:65,
                 from linux/arch/ia64/kernel/asm-offsets.c:9:
linux/include/linux/gfp.h: In function alloc_pages_node:
linux/include/linux/gfp.h:131: warning: implicit declaration of function raw_smp_processor_id
In file included from linux/include/linux/irq.h:185,
                 from include/asm-ia64/hardirq.h:11,
                 from linux/include/linux/hardirq.h:7,
                 from linux/include/linux/interrupt.h:11,
                 from include/asm-ia64/smp.h:17,
                 from linux/include/linux/smp.h:18,
                 from linux/include/linux/sched.h:65,
                 from linux/arch/ia64/kernel/asm-offsets.c:9:
include/asm-ia64/hw_irq.h: In function ia64_resend_irq:
include/asm-ia64/hw_irq.h:102: warning: implicit declaration of function smp_processor_id
make[2]: *** [arch/ia64/kernel/asm-offsets.s] Error 1

Signed-off-by: Keith Owens <kaos@sgi.com>

---

Sorry for any duplicates.  Cut and paste of color gcc messages into
exmh generates MIME8 messages which then get rejected by vger's
bogofilter.

 arch/ia64/kernel/smp.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: linux/arch/ia64/kernel/smp.c
=================================--- linux.orig/arch/ia64/kernel/smp.c
+++ linux/arch/ia64/kernel/smp.c
@@ -108,7 +108,7 @@ cpu_die(void)
 }
 
 irqreturn_t
-handle_IPI (int irq, void *dev_id, struct pt_regs *regs)
+handle_IPI (int irq, void *dev_id)
 {
 	int this_cpu = get_cpu();
 	unsigned long *pending_ipis = &__ia64_per_cpu_var(ipi_operation);





--k9I8DriA003673.1161159233/omx2.sgi.com--




------- End of Forwarded Message



             reply	other threads:[~2006-10-18  5:36 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-18  5:36 Keith Owens [this message]
2006-10-18 10:10 ` [patch 2.6.19-rc2] Correct definition of handle_IPI David Howells

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8916.1161149809@kao2.melbourne.sgi.com \
    --to=kaos@sgi.com \
    --cc=linux-ia64@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.