[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <AM4PR0501MB27690D8721451EEE5082E593C5D70@AM4PR0501MB2769.eurprd05.prod.outlook.com>
Date: Mon, 26 Nov 2018 13:55:37 +0000
From: Ran Rozenstein <ranro@...lanox.com>
To: "paulmck@...ux.ibm.com" <paulmck@...ux.ibm.com>,
Joel Fernandes <joel@...lfernandes.org>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"mingo@...nel.org" <mingo@...nel.org>,
"jiangshanlai@...il.com" <jiangshanlai@...il.com>,
"dipankar@...ibm.com" <dipankar@...ibm.com>,
"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
"mathieu.desnoyers@...icios.com" <mathieu.desnoyers@...icios.com>,
"josh@...htriplett.org" <josh@...htriplett.org>,
"tglx@...utronix.de" <tglx@...utronix.de>,
"peterz@...radead.org" <peterz@...radead.org>,
"rostedt@...dmis.org" <rostedt@...dmis.org>,
"dhowells@...hat.com" <dhowells@...hat.com>,
"edumazet@...gle.com" <edumazet@...gle.com>,
"fweisbec@...il.com" <fweisbec@...il.com>,
"oleg@...hat.com" <oleg@...hat.com>,
Maor Gottlieb <maorg@...lanox.com>,
Tariq Toukan <tariqt@...lanox.com>,
Eran Ben Elisha <eranbe@...lanox.com>,
Leon Romanovsky <leonro@...lanox.com>
Subject: RE: [PATCH tip/core/rcu 02/19] rcu: Defer reporting RCU-preempt
quiescent states when disabled
>
> Hearing no objections, here is the updated patch.
>
> Thanx, Paul
>
> ------------------------------------------------------------------------
>
> commit 970cab5d3d206029ed27274a98ea1c3d7e780e53
> Author: Paul E. McKenney <paulmck@...ux.ibm.com>
> Date: Mon Oct 29 07:36:50 2018 -0700
>
> rcu: Avoid signed integer overflow in rcu_preempt_deferred_qs()
>
> Subtracting INT_MIN can be interpreted as unconditional signed integer
> overflow, which according to the C standard is undefined behavior.
> Therefore, kernel build arguments notwithstanding, it would be good to
> future-proof the code. This commit therefore substitutes INT_MAX for
> INT_MIN in order to avoid undefined behavior.
>
> While in the neighborhood, this commit also creates some meaningful
> names
> for INT_MAX and friends in order to improve readability, as suggested
> by Joel Fernandes.
>
> Reported-by: Ran Rozenstein <ranro@...lanox.com>
> Signed-off-by: Paul E. McKenney <paulmck@...ux.ibm.com>
>
> squash! rcu: Avoid signed integer overflow in rcu_preempt_deferred_qs()
>
> While in the neighborhood, use macros to give meaningful names.
>
> Signed-off-by: Paul E. McKenney <paulmck@...ux.ibm.com>
>
Hi,
What is the acceptance status of this patch?
Thanks,
Ran
Powered by blists - more mailing lists