Re: [PATCH 4/6] sched: Add get_current_state()

[PATCH 0/6] sched: Cleanup task_struct::state · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
[PATCH 5/6] sched,timer: Use __set_current_state() · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 5/6] sched,timer: Use __set_current_state() · Will Deacon <will@kernel.org> · 2021-06-02
Re: [PATCH 5/6] sched,timer: Use __set_current_state() · Davidlohr Bueso <dave@stgolabs.net> · 2021-06-02
Re: [PATCH 5/6] sched,timer: Use __set_current_state() · Peter Zijlstra <peterz@infradead.org> · 2021-06-03
Re: [PATCH 5/6] sched,timer: Use __set_current_state() · Thomas Gleixner <hidden> · 2021-06-18
[PATCH 1/6] sched: Unbreak wakeups · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 1/6] sched: Unbreak wakeups · Greg Kroah-Hartman <gregkh@linuxfoundation.org> · 2021-06-02
Re: [PATCH 1/6] sched: Unbreak wakeups · Will Deacon <will@kernel.org> · 2021-06-02
Re: [PATCH 1/6] sched: Unbreak wakeups · Davidlohr Bueso <dave@stgolabs.net> · 2021-06-02
[PATCH 2/6] sched: Introduce task_is_running() · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 2/6] sched: Introduce task_is_running() · Will Deacon <will@kernel.org> · 2021-06-02
Re: [PATCH 2/6] sched: Introduce task_is_running() · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 2/6] sched: Introduce task_is_running() · Davidlohr Bueso <dave@stgolabs.net> · 2021-06-02
[PATCH 3/6] sched,perf,kvm: Fix preemption condition · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 3/6] sched,perf,kvm: Fix preemption condition · Mathieu Desnoyers <mathieu.desnoyers@efficios.com> · 2021-06-02
Re: [PATCH 3/6] sched,perf,kvm: Fix preemption condition · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 3/6] sched,perf,kvm: Fix preemption condition · Mark Rutland <mark.rutland@arm.com> · 2021-06-02
Re: [PATCH 3/6] sched,perf,kvm: Fix preemption condition · Mathieu Desnoyers <mathieu.desnoyers@efficios.com> · 2021-06-02
Re: [PATCH 3/6] sched,perf,kvm: Fix preemption condition · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
[PATCH 4/6] sched: Add get_current_state() · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 4/6] sched: Add get_current_state() · Mathieu Desnoyers <mathieu.desnoyers@efficios.com> · 2021-06-02
Re: [PATCH 4/6] sched: Add get_current_state() · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 4/6] sched: Add get_current_state() · Will Deacon <will@kernel.org> · 2021-06-02
[PATCH 6/6] sched: Change task_struct::state · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 6/6] sched: Change task_struct::state · Mathieu Desnoyers <mathieu.desnoyers@efficios.com> · 2021-06-02
Re: [PATCH 6/6] sched: Change task_struct::state · Peter Zijlstra <peterz@infradead.org> · 2021-06-02
Re: [PATCH 6/6] sched: Change task_struct::state · Will Deacon <will@kernel.org> · 2021-06-02
Re: [PATCH 6/6] sched: Change task_struct::state · Daniel Thompson <hidden> · 2021-06-07
Re: [PATCH 6/6] sched: Change task_struct::state · Peter Zijlstra <peterz@infradead.org> · 2021-06-07

From: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Date: 2021-06-02 14:01:37
Also in: cgroups, dm-devel, kvm, linux-block, linux-fsdevel, linux-mm, linux-pm, linux-usb, lkml, netdev, rcu

----- On Jun 2, 2021, at 9:12 AM, Peter Zijlstra peterz@infradead.org wrote:

Remove yet another few p->state accesses.

[...]

quoted hunk ↗ jump to hunk

--- a/include/linux/sched.h
+++ b/include/linux/sched.h

@@ -212,6 +212,8 @@ struct task_group;

#endif

+#define get_current_state()	READ_ONCE(current->state)

Why use a macro rather than a static inline here ?

Thanks,

Mathieu

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help