diff options
| author | Richard Guy Briggs <rgb@redhat.com> | 2014-03-16 14:00:19 -0400 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2015-11-17 15:54:45 +0000 |
| commit | 4421196453ad90ff54c97b5d600d65464a3965b2 (patch) | |
| tree | dcc59ca3ccee72fbc7c69aaae66ad2890898408d /include/linux | |
| parent | cc1875ecbc3c9fb2774097e03870280c91c1e0e1 (diff) | |
| download | lwn-4421196453ad90ff54c97b5d600d65464a3965b2.tar.gz lwn-4421196453ad90ff54c97b5d600d65464a3965b2.zip | |
sched: declare pid_alive as inline
commit 80e0b6e8a001361316a2d62b748fe677ec46b860 upstream.
We accidentally declared pid_alive without any extern/inline connotation.
Some platforms were fine with this, some like ia64 and mips were very angry.
If the function is inline, the prototype should be inline!
on ia64:
include/linux/sched.h:1718: warning: 'pid_alive' declared inline after
being called
Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
Signed-off-by: Eric Paris <eparis@redhat.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Cc: Neal Gompa <ngompa13@gmail.com>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/sched.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/sched.h b/include/linux/sched.h index 44e5f4782861..9b9ac291f66e 100644 --- a/include/linux/sched.h +++ b/include/linux/sched.h @@ -1690,7 +1690,7 @@ static inline pid_t task_tgid_vnr(struct task_struct *tsk) } -static int pid_alive(const struct task_struct *p); +static inline int pid_alive(const struct task_struct *p); static inline pid_t task_ppid_nr_ns(const struct task_struct *tsk, struct pid_namespace *ns) { pid_t pid = 0; |