lkml.org 
[lkml]   [2017]   [Feb]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 67/89] sched/headers: Move the task_lock()/unlock() APIs to <linux/sched/task.h>
    Date
    The task_lock()/task_unlock() APIs are not realated to core scheduling,
    they are task lifetime APIs, i.e. they belong into <linux/sched/task.h>.

    Move them and update the code that uses these facilities.

    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Mike Galbraith <efault@gmx.de>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: linux-kernel@vger.kernel.org
    Signed-off-by: Ingo Molnar <mingo@kernel.org>
    ---
    arch/mips/math-emu/dsemul.c | 1 +
    block/blk-ioc.c | 1 +
    block/ioprio.c | 1 +
    drivers/staging/android/ion/ion.c | 1 +
    drivers/staging/lustre/lustre/ptlrpc/sec.c | 1 +
    fs/proc/internal.h | 1 +
    fs/proc/proc_net.c | 1 +
    fs/proc_namespace.c | 2 ++
    include/linux/cpuset.h | 1 +
    include/linux/sched.h | 20 --------------------
    include/linux/sched/task.h | 22 +++++++++++++++++++++-
    ipc/namespace.c | 1 +
    kernel/cpuset.c | 1 +
    kernel/sched/debug.c | 1 +
    kernel/utsname.c | 1 +
    mm/mempolicy.c | 1 +
    mm/mmu_context.c | 1 +
    net/core/net_namespace.c | 2 ++
    net/core/netclassid_cgroup.c | 2 ++
    net/core/netprio_cgroup.c | 2 ++
    20 files changed, 43 insertions(+), 21 deletions(-)

    diff --git a/arch/mips/math-emu/dsemul.c b/arch/mips/math-emu/dsemul.c
    index 6664908514b3..b6bfd3625369 100644
    --- a/arch/mips/math-emu/dsemul.c
    +++ b/arch/mips/math-emu/dsemul.c
    @@ -1,6 +1,7 @@
    #include <linux/err.h>
    #include <linux/slab.h>
    #include <linux/mm_types.h>
    +#include <linux/sched/task.h>

    #include <asm/branch.h>
    #include <asm/cacheflush.h>
    diff --git a/block/blk-ioc.c b/block/blk-ioc.c
    index 381cb50a673c..f646f8b119e2 100644
    --- a/block/blk-ioc.c
    +++ b/block/blk-ioc.c
    @@ -7,6 +7,7 @@
    #include <linux/bio.h>
    #include <linux/blkdev.h>
    #include <linux/slab.h>
    +#include <linux/sched/task.h>

    #include "blk.h"

    diff --git a/block/ioprio.c b/block/ioprio.c
    index 88e916d30180..0354b309ca0e 100644
    --- a/block/ioprio.c
    +++ b/block/ioprio.c
    @@ -27,6 +27,7 @@
    #include <linux/blkdev.h>
    #include <linux/capability.h>
    #include <linux/sched/user.h>
    +#include <linux/sched/task.h>
    #include <linux/syscalls.h>
    #include <linux/security.h>
    #include <linux/pid_namespace.h>
    diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c
    index 937c2d5d7ec3..f913b87c1a01 100644
    --- a/drivers/staging/android/ion/ion.c
    +++ b/drivers/staging/android/ion/ion.c
    @@ -36,6 +36,7 @@
    #include <linux/debugfs.h>
    #include <linux/dma-buf.h>
    #include <linux/idr.h>
    +#include <linux/sched/task.h>

    #include "ion.h"
    #include "ion_priv.h"
    diff --git a/drivers/staging/lustre/lustre/ptlrpc/sec.c b/drivers/staging/lustre/lustre/ptlrpc/sec.c
    index 49f34fd655c3..366f2ce20f5e 100644
    --- a/drivers/staging/lustre/lustre/ptlrpc/sec.c
    +++ b/drivers/staging/lustre/lustre/ptlrpc/sec.c
    @@ -40,6 +40,7 @@
    #include <linux/crypto.h>
    #include <linux/cred.h>
    #include <linux/key.h>
    +#include <linux/sched/task.h>

    #include "../include/obd.h"
    #include "../include/obd_class.h"
    diff --git a/fs/proc/internal.h b/fs/proc/internal.h
    index 6980c9a7e367..378410dee8cb 100644
    --- a/fs/proc/internal.h
    +++ b/fs/proc/internal.h
    @@ -15,6 +15,7 @@
    #include <linux/atomic.h>
    #include <linux/binfmts.h>
    #include <linux/sched/coredump.h>
    +#include <linux/sched/task.h>

    struct ctl_table_header;
    struct mempolicy;
    diff --git a/fs/proc/proc_net.c b/fs/proc/proc_net.c
    index ffd72a6c6e04..5cbc65d7a1e1 100644
    --- a/fs/proc/proc_net.c
    +++ b/fs/proc/proc_net.c
    @@ -17,6 +17,7 @@
    #include <linux/slab.h>
    #include <linux/init.h>
    #include <linux/sched.h>
    +#include <linux/sched/task.h>
    #include <linux/module.h>
    #include <linux/bitops.h>
    #include <linux/mount.h>
    diff --git a/fs/proc_namespace.c b/fs/proc_namespace.c
    index 3f1190d18991..b5713fefb4c1 100644
    --- a/fs/proc_namespace.c
    +++ b/fs/proc_namespace.c
    @@ -10,6 +10,8 @@
    #include <linux/nsproxy.h>
    #include <linux/security.h>
    #include <linux/fs_struct.h>
    +#include <linux/sched/task.h>
    +
    #include "proc/internal.h" /* only for get_proc_task() in ->open() */

    #include "pnode.h"
    diff --git a/include/linux/cpuset.h b/include/linux/cpuset.h
    index c608c39cb161..611fce58d670 100644
    --- a/include/linux/cpuset.h
    +++ b/include/linux/cpuset.h
    @@ -10,6 +10,7 @@

    #include <linux/sched.h>
    #include <linux/sched/topology.h>
    +#include <linux/sched/task.h>
    #include <linux/cpumask.h>
    #include <linux/nodemask.h>
    #include <linux/mm.h>
    diff --git a/include/linux/sched.h b/include/linux/sched.h
    index 6895db6ecf23..bccdc3fecded 100644
    --- a/include/linux/sched.h
    +++ b/include/linux/sched.h
    @@ -1514,26 +1514,6 @@ static inline unsigned long wait_task_inactive(struct task_struct *p,
    }
    #endif

    -/*
    - * Protects ->fs, ->files, ->mm, ->group_info, ->comm, keyring
    - * subscriptions and synchronises with wait4(). Also used in procfs. Also
    - * pins the final release of task.io_context. Also protects ->cpuset and
    - * ->cgroup.subsys[]. And ->vfork_done.
    - *
    - * Nests both inside and outside of read_lock(&tasklist_lock).
    - * It must not be nested with write_lock_irq(&tasklist_lock),
    - * neither inside nor outside.
    - */
    -static inline void task_lock(struct task_struct *p)
    -{
    - spin_lock(&p->alloc_lock);
    -}
    -
    -static inline void task_unlock(struct task_struct *p)
    -{
    - spin_unlock(&p->alloc_lock);
    -}
    -
    /* set thread flags in other task's structures
    * - see asm/thread_info.h for TIF_xxxx flags available
    */
    diff --git a/include/linux/sched/task.h b/include/linux/sched/task.h
    index 5bc61717086a..26bc507eb07e 100644
    --- a/include/linux/sched/task.h
    +++ b/include/linux/sched/task.h
    @@ -1,7 +1,7 @@
    #ifndef _LINUX_SCHED_TASK_H
    #define _LINUX_SCHED_TASK_H

    -#include <linux/spinlock.h>
    +#include <linux/sched.h>

    /*
    * Interface between the scheduler and various task lifetime (fork()/exit())
    @@ -91,4 +91,24 @@ static inline struct vm_struct *task_stack_vm_area(const struct task_struct *t)
    }
    #endif

    +/*
    + * Protects ->fs, ->files, ->mm, ->group_info, ->comm, keyring
    + * subscriptions and synchronises with wait4(). Also used in procfs. Also
    + * pins the final release of task.io_context. Also protects ->cpuset and
    + * ->cgroup.subsys[]. And ->vfork_done.
    + *
    + * Nests both inside and outside of read_lock(&tasklist_lock).
    + * It must not be nested with write_lock_irq(&tasklist_lock),
    + * neither inside nor outside.
    + */
    +static inline void task_lock(struct task_struct *p)
    +{
    + spin_lock(&p->alloc_lock);
    +}
    +
    +static inline void task_unlock(struct task_struct *p)
    +{
    + spin_unlock(&p->alloc_lock);
    +}
    +
    #endif /* _LINUX_SCHED_TASK_H */
    diff --git a/ipc/namespace.c b/ipc/namespace.c
    index 1f1d713ac19c..b4d80f9f7246 100644
    --- a/ipc/namespace.c
    +++ b/ipc/namespace.c
    @@ -14,6 +14,7 @@
    #include <linux/mount.h>
    #include <linux/user_namespace.h>
    #include <linux/proc_ns.h>
    +#include <linux/sched/task.h>

    #include "util.h"

    diff --git a/kernel/cpuset.c b/kernel/cpuset.c
    index a5c46db2855c..0f41292be0fb 100644
    --- a/kernel/cpuset.c
    +++ b/kernel/cpuset.c
    @@ -45,6 +45,7 @@
    #include <linux/rcupdate.h>
    #include <linux/sched.h>
    #include <linux/sched/mm.h>
    +#include <linux/sched/task.h>
    #include <linux/seq_file.h>
    #include <linux/security.h>
    #include <linux/slab.h>
    diff --git a/kernel/sched/debug.c b/kernel/sched/debug.c
    index e865d8bfb881..38f019324f1a 100644
    --- a/kernel/sched/debug.c
    +++ b/kernel/sched/debug.c
    @@ -12,6 +12,7 @@

    #include <linux/proc_fs.h>
    #include <linux/sched/mm.h>
    +#include <linux/sched/task.h>
    #include <linux/seq_file.h>
    #include <linux/kallsyms.h>
    #include <linux/utsname.h>
    diff --git a/kernel/utsname.c b/kernel/utsname.c
    index 06585ad296ff..913fe4336d2b 100644
    --- a/kernel/utsname.c
    +++ b/kernel/utsname.c
    @@ -17,6 +17,7 @@
    #include <linux/cred.h>
    #include <linux/user_namespace.h>
    #include <linux/proc_ns.h>
    +#include <linux/sched/task.h>

    static struct ucounts *inc_uts_namespaces(struct user_namespace *ns)
    {
    diff --git a/mm/mempolicy.c b/mm/mempolicy.c
    index 18e0105810df..75b2745bac41 100644
    --- a/mm/mempolicy.c
    +++ b/mm/mempolicy.c
    @@ -75,6 +75,7 @@
    #include <linux/sched.h>
    #include <linux/sched/mm.h>
    #include <linux/sched/numa_balancing.h>
    +#include <linux/sched/task.h>
    #include <linux/nodemask.h>
    #include <linux/cpuset.h>
    #include <linux/slab.h>
    diff --git a/mm/mmu_context.c b/mm/mmu_context.c
    index 6d93bc207e90..03a9c6bbbb85 100644
    --- a/mm/mmu_context.c
    +++ b/mm/mmu_context.c
    @@ -6,6 +6,7 @@
    #include <linux/mm.h>
    #include <linux/sched.h>
    #include <linux/sched/mm.h>
    +#include <linux/sched/task.h>
    #include <linux/mmu_context.h>
    #include <linux/export.h>

    diff --git a/net/core/net_namespace.c b/net/core/net_namespace.c
    index 3c4bbec39713..652468ff65b7 100644
    --- a/net/core/net_namespace.c
    +++ b/net/core/net_namespace.c
    @@ -16,6 +16,8 @@
    #include <linux/export.h>
    #include <linux/user_namespace.h>
    #include <linux/net_namespace.h>
    +#include <linux/sched/task.h>
    +
    #include <net/sock.h>
    #include <net/netlink.h>
    #include <net/net_namespace.h>
    diff --git a/net/core/netclassid_cgroup.c b/net/core/netclassid_cgroup.c
    index 11fce17274f6..6ae56037bb13 100644
    --- a/net/core/netclassid_cgroup.c
    +++ b/net/core/netclassid_cgroup.c
    @@ -12,6 +12,8 @@
    #include <linux/slab.h>
    #include <linux/cgroup.h>
    #include <linux/fdtable.h>
    +#include <linux/sched/task.h>
    +
    #include <net/cls_cgroup.h>
    #include <net/sock.h>

    diff --git a/net/core/netprio_cgroup.c b/net/core/netprio_cgroup.c
    index 2ec86fc552df..0bf58c5d8eb5 100644
    --- a/net/core/netprio_cgroup.c
    +++ b/net/core/netprio_cgroup.c
    @@ -19,6 +19,8 @@
    #include <linux/cgroup.h>
    #include <linux/rcupdate.h>
    #include <linux/atomic.h>
    +#include <linux/sched/task.h>
    +
    #include <net/rtnetlink.h>
    #include <net/pkt_cls.h>
    #include <net/sock.h>
    --
    2.7.4
    \
     
     \ /
      Last update: 2017-02-06 14:37    [W:4.136 / U:0.172 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site