Received-SPF: pass (google.com: domain of linux-kernel+bounces-191547-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Ankur Arora <ankur.a.arora@oracle.com>
To: linux-kernel@vger.kernel.org
Cc: tglx@linutronix.de, peterz@infradead.org, torvalds@linux-foundation.org,
        paulmck@kernel.org, rostedt@goodmis.org, mark.rutland@arm.com,
        juri.lelli@redhat.com, joel@joelfernandes.org, raghavendra.kt@amd.com,
        sshegde@linux.ibm.com, boris.ostrovsky@oracle.com,
        konrad.wilk@oracle.com, Ankur Arora <ankur.a.arora@oracle.com>
Subject: [PATCH v2 03/35] sched: make test_*_tsk_thread_flag() return bool
Date: Mon, 27 May 2024 17:34:49 -0700
Message-Id: <20240528003521.979836-4-ankur.a.arora@oracle.com>
In-Reply-To: <20240528003521.979836-1-ankur.a.arora@oracle.com>
References: <20240528003521.979836-1-ankur.a.arora@oracle.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: 
	=?us-ascii?Q?KE/KRskRe4fFTuuCS4Uud12VQLJ7jUnGBxtVnhVDQKmba3hA1ySs5EwZOx6y?=
 =?us-ascii?Q?y7FVd2/AaMpaombaoRWEP2cmh1abS9ieeDnEKb+MgC4fE/0PGu+O4t0O4OCu?=
 =?us-ascii?Q?s/M3285mEYBLGwtPd2907WTEudPy7JCJKLi5XhCBuaWNYWSKyt3d+8vUO2kJ?=
 =?us-ascii?Q?UANZkryd/JKHI4VDhLbP8hYzuq+6RpY6M9JAcdaB3GJiepVQFYRDZQFRlynh?=
 =?us-ascii?Q?X0JiqXCRaBqMYH4n1aEeCmm9AkXMhHwS2qJ+HxDJegOD209q1YlwTN69LfOA?=
 =?us-ascii?Q?7iZuytnylsG4U+JWpr7B2GInV72R8cj2bvA+rLGD+yWjYyXbl03p8XLyPTec?=
 =?us-ascii?Q?lTg+o0DMjAbd76mlsBC1hlXJNXUq0IN5zJ7FjXJ6FYNYZ/KTxy3tdn00QL+b?=
 =?us-ascii?Q?4w00OcO3R7bBVQgCAKJPuOX9L8ex8Lo9JKohPoUDa7iW+iACZH2HWO3xZYPn?=
 =?us-ascii?Q?83lfzWe97ylqvfTYhdWor5qQ/qpnYhGA/IAeMSJe2FxOq4s7cCxBJ8H3lq5l?=
 =?us-ascii?Q?ZabAZRcUugvGENbVatfBuWLFMG/ho7vpSqvp9RXN7kCxiwBPLJ43WSFLYNeY?=
 =?us-ascii?Q?9JBCMbCA6atBBL2CZrnKn8Pz+Hr/c0CTZXe4SNJGSjlOUbaGvYEijoVjqbW9?=
 =?us-ascii?Q?VjHwltfp4qPN0ohCFoZS5Gbz5vN7HPHCZ9nC7ubkaNlM84mLIqg+n3tJoZ22?=
 =?us-ascii?Q?t79eFBxPDpKz65zfbRSyXR4MPW1Lvhx1hemFBPBsJb33N40EaWbzfPhyfX8R?=
 =?us-ascii?Q?u6HRxCEXxnqJIwcV7OgqB6nlFrY5mzcVXEaAXykl3a45XtGtxjMa0kIHDI2y?=
 =?us-ascii?Q?5F80wdCsZXwuDAZbCLTHlqzidWBoUsQVchJHI72vLfzdVBVVL6zv/xdlnvPW?=
 =?us-ascii?Q?OGiEzTeV90kPYXtO2l0BX5qoo/P2aUMC3Ht001Ba25kxI+ArOAIeVJjZT9rW?=
 =?us-ascii?Q?YaR+dI8qYXy5r1tfUcSb8K46zaVemPOEjOmFqs25AAJVVS7EW9xusA1DVH55?=
 =?us-ascii?Q?G+et3CRD43HBCJAO0AuGyX3eRPxE71KfGHeF62kPU7fi5pErDz+URYtvZint?=
 =?us-ascii?Q?e/K8DRWN4Wbvgj5mPpVf3RrJrUlC6Jg0Uw6sNOGGc91J5KgsxN5IVAPogqu4?=
 =?us-ascii?Q?DlX8IqRf7+Nd/uWFJOSgIytIjw+3a58CXXRylc5HZ/wIsEg8HFIzcSIPN4AV?=
 =?us-ascii?Q?4pPrZryDpfH/HbXpClIJg+JgI3G7aLtcD7YYcEcwKqKx82v0KHl0nB9toO/L?=
 =?us-ascii?Q?PvA/c/6Tvpcr3ytbW+BVfJVVsI3/L5nHquElrvodbgtxjgNea42oCWKXaVjk?=
 =?us-ascii?Q?47RI13ds1U8s5yHPCOk1J9/7IYmf4wmaa4kjIlDrC8XpqNtLjuGH8gMFvXla?=
 =?us-ascii?Q?i6kasI5TPhDnQmdIfnsBYDmOQtCP8F4ihrU9qj7CqwnLPkjHfJgNG/lmO4Cu?=
 =?us-ascii?Q?8t00yVqjweVunbTRZ5GWFR9C4y4rJ60xk6ZBlgPOrsVgii1adiETRxGD24qq?=
 =?us-ascii?Q?81hcnnOrPsFpYURAUXsxLfROSgECpb3nThYAHakcpuk0vZi2WFjgEDu2CMLR?=
 =?us-ascii?Q?lSFNQwXTZgTdwAoW6NSFD1pRvh3QkDhMiTuCpsUxOVm5ssPUOPb31e+hkGE8?=
 =?us-ascii?Q?/w=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	wPcGx3WphRv5sS95XAGqRxx3/bq0/r3LzsIwwkLD1rJpo/BVimf3IF31xSgaDS2BF9QTBg6RjBmaue1OHogUVbFtFnnNDzlrS/VoEYIZTdXuY7Cab2XTNdvY1tHPE2FO37pCLvpAqYnrTwdZR2YsJymak4HiQzk4ozuU9VPDWPjbHc3K0OxMHkIPREnQBPu0muf5bD8w/0MJ3u5gWb8qnlHHloH9gofqwhk+lW1x+V6Oq+alK/EAE/YUVzAhRbLkBEYOKtcIIXGPJo87xm1v5Sg5CFGma3vac+tk4jiWAdsiCEOa+cK2yNOe72xGfHsZId9D4sWZ6Wq6HbVbJ111Xoyq+oWmh77Gb4IIwxRnyP+ynAPO1IKeF3RpPiCdjk6yeQh01tve0VfGW8VGAlknw+D+VbbJCS+gcmJVmWwDHcK3mWJGYJz5kAPyRkGuE7ua0uUarb6t+nahpMrIQsniQk2ry7PpaZtlQBZdeBeh3jCitVrUPnwzRVVCIBy8isu3Ck5jzcrT9ufu03ABF8u1wt2cNoAjF8dIxjkSHb9WRCQqqtQ8L0f+C7Tkxn6OxOsiZvFpx5wX27T9IF5Lx/+MmbPYpzXf/lHUqsmtjZEiiCg=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3c5f899f-0336-47a0-f7ee-08dc7eae16d8
X-MS-Exchange-CrossTenant-AuthSource: CO6PR10MB5409.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 May 2024 00:35:35.7649
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: mxG9fzBkrFtkfHrSTpnJB1/8C54J8OuUiTC3Su6bVG5h4xjv19fkgx7lV3xr6Jhz14geY+R+26Do5twzmNm8+6Pjmi5Xl35SRIkyXbCj+3A=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR10MB6510
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1039,Hydra:6.0.650,FMLib:17.12.28.16
 definitions=2024-05-27_06,2024-05-27_01,2024-05-17_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 spamscore=0 suspectscore=0
 adultscore=0 phishscore=0 malwarescore=0 mlxlogscore=999 bulkscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2405010000
 definitions=main-2405280003
X-Proofpoint-GUID: 6kkzXL2AOwXXBCOOrklisDGk2QOS5_kG
X-Proofpoint-ORIG-GUID: 6kkzXL2AOwXXBCOOrklisDGk2QOS5_kG

All users of test_*_tsk_thread_flag() treat the result as boolean.
This is also true for the underlying test_and_*_bit() operations.

Change the return type to bool.

Cc: Peter Ziljstra <peterz@infradead.org>
Originally-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/lkml/87jzshhexi.ffs@tglx/
Signed-off-by: Ankur Arora <ankur.a.arora@oracle.com>
Acked-by: Mark Rutland <mark.rutland@arm.com>
---
 include/linux/sched.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/include/linux/sched.h b/include/linux/sched.h
index 73a3402843c6..4808e5dd4f69 100644
--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -1937,17 +1937,17 @@ static inline void update_tsk_thread_flag(struct task_struct *tsk, int flag,
 	update_ti_thread_flag(task_thread_info(tsk), flag, value);
 }
 
-static inline int test_and_set_tsk_thread_flag(struct task_struct *tsk, int flag)
+static inline bool test_and_set_tsk_thread_flag(struct task_struct *tsk, int flag)
 {
 	return test_and_set_ti_thread_flag(task_thread_info(tsk), flag);
 }
 
-static inline int test_and_clear_tsk_thread_flag(struct task_struct *tsk, int flag)
+static inline bool test_and_clear_tsk_thread_flag(struct task_struct *tsk, int flag)
 {
 	return test_and_clear_ti_thread_flag(task_thread_info(tsk), flag);
 }
 
-static inline int test_tsk_thread_flag(struct task_struct *tsk, int flag)
+static inline bool test_tsk_thread_flag(struct task_struct *tsk, int flag)
 {
 	return test_ti_thread_flag(task_thread_info(tsk), flag);
 }
@@ -1962,7 +1962,7 @@ static inline void clear_tsk_need_resched(struct task_struct *tsk)
 	clear_tsk_thread_flag(tsk,TIF_NEED_RESCHED);
 }
 
-static inline int test_tsk_need_resched(struct task_struct *tsk)
+static inline bool test_tsk_need_resched(struct task_struct *tsk)
 {
 	return unlikely(test_tsk_thread_flag(tsk,TIF_NEED_RESCHED));
 }
-- 
2.31.1