Received-SPF: pass (google.com: domain of linux-kernel+bounces-126630-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Content-Type: text/plain;
	charset=utf-8
Precedence: bulk
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3774.300.61.1.2\))
Subject: Re: [PATCH 02/10] rcu: Move rcu_preempt_depth_set() to rcupdate.h
From: Alan Huang <mmpgouride@gmail.com>
In-Reply-To: <CAJhGHyDjoQ3TrApksYJ5o1ooU4s8rzys2UnfF6EFzGwkaSVaKA@mail.gmail.com>
Date: Mon, 1 Apr 2024 19:40:24 +0800
Cc: Joel Fernandes <joel@joelfernandes.org>,
 linux-kernel@vger.kernel.org,
 rcu@vger.kernel.org,
 x86@kernel.org,
 Lai Jiangshan <jiangshan.ljs@antgroup.com>,
 "Paul E. McKenney" <paulmck@kernel.org>,
 Peter Zijlstra <peterz@infradead.org>,
 Frederic Weisbecker <frederic@kernel.org>,
 Neeraj Upadhyay <quic_neeraju@quicinc.com>,
 Josh Triplett <josh@joshtriplett.org>,
 Boqun Feng <boqun.feng@gmail.com>,
 Steven Rostedt <rostedt@goodmis.org>,
 Mathieu Desnoyers <mathieu.desnoyers@efficios.com>,
 Zqiang <qiang.zhang1211@gmail.com>
Content-Transfer-Encoding: quoted-printable
Message-Id: <073E72BD-63AF-4D88-95F8-9A002252C554@gmail.com>
References: <20240328075318.83039-3-jiangshanlai@gmail.com>
 <769AEAAB-7DEA-4D94-8979-F01EDFD6EE46@joelfernandes.org>
 <CAJhGHyDjoQ3TrApksYJ5o1ooU4s8rzys2UnfF6EFzGwkaSVaKA@mail.gmail.com>
To: Lai Jiangshan <jiangshanlai@gmail.com>

2024=E5=B9=B44=E6=9C=881=E6=97=A5 00:16=EF=BC=8CLai Jiangshan =
<jiangshanlai@gmail.com> wrote=EF=BC=9A
>=20
> On Sun, Mar 31, 2024 at 7:10=E2=80=AFPM Joel Fernandes =
<joel@joelfernandes.org> wrote:
>>=20
>>=20
>>=20
>>> On Mar 28, 2024, at 1:20=E2=80=AFPM, Lai Jiangshan =
<jiangshanlai@gmail.com> wrote:
>>>=20
>>> =EF=BB=BFFrom: Lai Jiangshan <jiangshan.ljs@antgroup.com>
>>>=20
>>> Prepare for arch-specific-defined rcu_preempt_depth_set().
>>>=20
>>> No functionality change intended, but it has to be defined as a =
macro
>>> as rcupdate.h is a very low level header included from areas that =
don't
>>> even know about the task struct "current".
>>=20
>> Sorry I did not follow changelog. If some rcupdate.h includers do not =
know
>> about task_struct, how does adding a macro that uses current help?
>>=20
>=20
> Hello
>=20
> This is how macro works and it expands blindly based on tokens on the
> usage-sites.

But =E2=80=98current=E2=80=99 still needs to be expanded at last, it =
seems to me that it only affects=20
the including order of the header files=EF=BC=9F

Or what am I missing?

>=20
> And rcu_preempt_depth() & rcu_preempt_depth_set() are not universally
> used wrappers, the user can simply also include linux/sched.h to make
> they work.
>=20
> Thanks
> Lai
>=20