Skip to content

Commit

Permalink
[POWERPC] Make cpus_in_xmon static and remove extern mess from hvc_co…
Browse files Browse the repository at this point in the history
…nsole.c

This is a little messier than I'd like because xmon.h only exists
on powerpc and we can't have a static inline and an extern declaration
visible at the same time.

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Paul Mackerras <paulus@samba.org>
  • Loading branch information
Michael Ellerman authored and Paul Mackerras committed May 14, 2008
1 parent 9f1067c commit 1c8950f
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 8 deletions.
7 changes: 6 additions & 1 deletion arch/powerpc/xmon/xmon.c
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
#define skipbl xmon_skipbl

#ifdef CONFIG_SMP
cpumask_t cpus_in_xmon = CPU_MASK_NONE;
static cpumask_t cpus_in_xmon = CPU_MASK_NONE;
static unsigned long xmon_taken = 1;
static int xmon_owner;
static int xmon_gate;
Expand Down Expand Up @@ -327,6 +327,11 @@ static void release_output_lock(void)
{
xmon_speaker = 0;
}

int cpus_are_in_xmon(void)
{
return !cpus_empty(cpus_in_xmon);
}
#endif

static int xmon_core(struct pt_regs *regs, int fromipi)
Expand Down
8 changes: 1 addition & 7 deletions drivers/char/hvc_console.c
Original file line number Diff line number Diff line change
Expand Up @@ -675,12 +675,6 @@ static int hvc_poll(struct hvc_struct *hp)
return poll_mask;
}

#if defined(CONFIG_XMON) && defined(CONFIG_SMP)
extern cpumask_t cpus_in_xmon;
#else
static const cpumask_t cpus_in_xmon = CPU_MASK_NONE;
#endif

/*
* This kthread is either polling or interrupt driven. This is determined by
* calling hvc_poll() who determines whether a console adapter support
Expand All @@ -698,7 +692,7 @@ static int khvcd(void *unused)
hvc_kicked = 0;
try_to_freeze();
wmb();
if (cpus_empty(cpus_in_xmon)) {
if (!cpus_are_in_xmon()) {
spin_lock(&hvc_structs_lock);
list_for_each_entry(hp, &hvc_structs, next) {
poll_mask |= hvc_poll(hp);
Expand Down
10 changes: 10 additions & 0 deletions drivers/char/hvc_console.h
Original file line number Diff line number Diff line change
Expand Up @@ -60,4 +60,14 @@ extern struct hvc_struct * __devinit hvc_alloc(uint32_t vtermno, int irq,
/* remove a vterm from hvc tty operation (modele_exit or hotplug remove) */
extern int __devexit hvc_remove(struct hvc_struct *hp);


#if defined(CONFIG_XMON) && defined(CONFIG_SMP)
#include <asm/xmon.h>
#else
static inline int cpus_are_in_xmon(void)
{
return 0;
}
#endif

#endif // HVC_CONSOLE_H
4 changes: 4 additions & 0 deletions include/asm-powerpc/xmon.h
Original file line number Diff line number Diff line change
Expand Up @@ -25,5 +25,9 @@ static inline void xmon_setup(void) { };
static inline void xmon_register_spus(struct list_head *list) { };
#endif

#if defined(CONFIG_XMON) && defined(CONFIG_SMP)
extern int cpus_are_in_xmon(void);
#endif

#endif /* __KERNEL __ */
#endif /* __ASM_POWERPC_XMON_H */

0 comments on commit 1c8950f

Please sign in to comment.