Skip to content

Commit

Permalink
Revert broken "statement with no effect" warning fix
Browse files Browse the repository at this point in the history
It may shut up gcc, but it also incorrectly changes the semantics of the
smp_call_function() helpers.

You can fix the warning other ways if you are interested (create another
inline function that takes no arguments and returns zero), but
preferably gcc just shouldn't complain about unused return values from
statement expressions in the first place.
  • Loading branch information
Linus Torvalds committed Jul 28, 2005
1 parent 7b70888 commit 2ac6608
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 23 deletions.
9 changes: 2 additions & 7 deletions include/asm-alpha/smp.h
Expand Up @@ -50,16 +50,11 @@ extern cpumask_t cpu_online_map;
extern int smp_num_cpus;
#define cpu_possible_map cpu_present_mask

int smp_call_function_on_cpu(void (*) (void *), void *, int, int, cpumask_t);
int smp_call_function_on_cpu(void (*func) (void *info), void *info,int retry, int wait, cpumask_t cpu);

#else /* CONFIG_SMP */

static inline int
smp_call_function_on_cpu (void (*func) (void *), void *info, int retry,
int wait, cpumask_t cpu)
{
return 0;
}
#define smp_call_function_on_cpu(func,info,retry,wait,cpu) ({ 0; })

#endif /* CONFIG_SMP */

Expand Down
20 changes: 4 additions & 16 deletions include/linux/smp.h
Expand Up @@ -94,23 +94,11 @@ void smp_prepare_boot_cpu(void);
*/
#define raw_smp_processor_id() 0
#define hard_smp_processor_id() 0
#define num_booting_cpus() 1

static inline int smp_call_function(void (*func) (void *), void *info,
int retry, int wait)
{
return 0;
}

static inline int on_each_cpu(void (*func) (void *), void *info,
int retry, int wait)
{
func(info);
return 0;
}

#define smp_call_function(func,info,retry,wait) ({ 0; })
#define on_each_cpu(func,info,retry,wait) ({ func(info); 0; })
static inline void smp_send_reschedule(int cpu) { }
static inline void smp_prepare_boot_cpu(void) { }
#define num_booting_cpus() 1
#define smp_prepare_boot_cpu() do {} while (0)

#endif /* !SMP */

Expand Down

0 comments on commit 2ac6608

Please sign in to comment.