Commit 9a848896 authored by Benoit Boissinot's avatar Benoit Boissinot Committed by Linus Torvalds

[PATCH] cpuset: remove function attribute const

gcc-4 warns with
include/linux/cpuset.h:21: warning: type qualifiers ignored on function
return type

cpuset_cpus_allowed is declared with const
extern const cpumask_t cpuset_cpus_allowed(const struct task_struct *p);

First const should be __attribute__((const)), but the gcc manual
explains that:

"Note that a function that has pointer arguments and examines the data
pointed to must not be declared const. Likewise, a function that calls a
non-const function usually must not be const. It does not make sense for
a const function to return void."

The following patch remove const from the function declaration.
Signed-off-by: default avatarBenoit Boissinot <benoit.boissinot@ens-lyon.org>
Acked-by: default avatarPaul Jackson <pj@sgi.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent b52402c7
...@@ -18,7 +18,7 @@ extern int cpuset_init(void); ...@@ -18,7 +18,7 @@ extern int cpuset_init(void);
extern void cpuset_init_smp(void); extern void cpuset_init_smp(void);
extern void cpuset_fork(struct task_struct *p); extern void cpuset_fork(struct task_struct *p);
extern void cpuset_exit(struct task_struct *p); extern void cpuset_exit(struct task_struct *p);
extern const cpumask_t cpuset_cpus_allowed(const struct task_struct *p); extern cpumask_t cpuset_cpus_allowed(const struct task_struct *p);
void cpuset_init_current_mems_allowed(void); void cpuset_init_current_mems_allowed(void);
void cpuset_update_current_mems_allowed(void); void cpuset_update_current_mems_allowed(void);
void cpuset_restrict_to_mems_allowed(unsigned long *nodes); void cpuset_restrict_to_mems_allowed(unsigned long *nodes);
......
...@@ -1432,7 +1432,7 @@ void cpuset_exit(struct task_struct *tsk) ...@@ -1432,7 +1432,7 @@ void cpuset_exit(struct task_struct *tsk)
* tasks cpuset. * tasks cpuset.
**/ **/
const cpumask_t cpuset_cpus_allowed(const struct task_struct *tsk) cpumask_t cpuset_cpus_allowed(const struct task_struct *tsk)
{ {
cpumask_t mask; cpumask_t mask;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment