Received-SPF: pass (google.com: domain of linux-kernel+bounces-191564-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::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 20/35] powerpc: add support for PREEMPT_AUTO
Date: Mon, 27 May 2024 17:35:06 -0700
Message-Id: <20240528003521.979836-21-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?tDn/Ra+yDCPOeTn4CbwB73pyamj3QMo1kMAzLSxCPGZEG8f114DwjX3WvYla?=
 =?us-ascii?Q?2HoNbm65yvwXul2deKVB8J171RkLrmF4E8KGRQ9WKuXQGDG/CXF47HTd4PoP?=
 =?us-ascii?Q?meUxEISCrWcjgtbexhwB9TML41aeoLL7cBDPOrXYxU7bTQsDdRZvyR8Tvxc3?=
 =?us-ascii?Q?+lsTrwBOea3s9Uetr4T9+rsUrfDND4rb6i87M5FIzSU0oHWeJCbHDpk9lxTq?=
 =?us-ascii?Q?ML/weuvcBgL29TAyA02BEuNB0teK32kei8IjvH2Mu5QRHaNqxxqo8DOdeejm?=
 =?us-ascii?Q?ft1V8+WcAw83f+TWaqP9luJWPMM/kegudJVqNKs7J/D+8lzcdgnVDlQH8zKB?=
 =?us-ascii?Q?gkTHUucGLuUK+KUbvbx+ZhSsc3Q8gPxNNfIr4d5BGQtz5lqleOUIB2gzNLS5?=
 =?us-ascii?Q?GiRecHMQCePg/0v3A1Ww1D7kBaNfpFjWfVZDiM1pXRW6RroV4aHTkCQpqedt?=
 =?us-ascii?Q?0/KXVOL2XMTfCm2nJ3BzNKp1Iy3J/riMXniPY4QTAuIgGnggwUFW+Gt+PVNT?=
 =?us-ascii?Q?Rt4Do0csS9zFE6JIExQYabe1Blj4ViS0Mk1zNGnY+ZyDM1E5gUH776ijttIy?=
 =?us-ascii?Q?8O7fXI6z7/zZOWHIdjzBUFzzKPm0AjFTcYZGwYhC/xkR+3F+lXw9uxHDrmv4?=
 =?us-ascii?Q?BH3hxQ+pwzHMkxMGcth5Cl2D/DXsUKXKPkeLTuCKVdq0SVdV3RV4ZZx2EMvX?=
 =?us-ascii?Q?I42cP2OH2TUsKhTe/39rORF12Dr0+8gG+dVs+c0bEQtQGftN18z4X4epVHoN?=
 =?us-ascii?Q?E7Y0rJ8VIKkiwRzytl474UpRtl/3s5WKywZ1Ie17W9X9E4VNf+FljDUbvvK3?=
 =?us-ascii?Q?LKLIZHdcbJNVnNzlzuqGdtr8GoNHdbqv/2QCemwxClIsQRuFjy7mDITPyCm0?=
 =?us-ascii?Q?gWYrUd920fcwHUEWhEyX6QqTyWSBMEPr6TYcV2X+E3uCpXAcBIGUFljQb9CU?=
 =?us-ascii?Q?m1edpCE92P4hY0HVJhUQhDrtYJWDJw76nWEufEfZfwGsxTntZWVZFANdtiDt?=
 =?us-ascii?Q?iSaeA1yPV+UqwZ21y+LnqujtvCJdaNl6hmGgPx8c5SaQHXgvJlU6M/+JLNDg?=
 =?us-ascii?Q?wQ1ZE4N0/f4RDIAUG+GW81pGLNfYI6plJBi3pWQd6Uu8GV/gNaDj577IuPba?=
 =?us-ascii?Q?48K9n5lwuqT6bVo8GAZXVxb6+r4KkJVQa/NopB9xNemEsaBOvbTtJ/3JxPND?=
 =?us-ascii?Q?FeGv3Il2xzmMlj4bHWaDDd4aUrA5U+0x+pZQUKPNa8M57gSBbK5r3VOq2kF+?=
 =?us-ascii?Q?xjp0e9iFXeiAC4KpzCgC0uJaqfeRDEQH3jPs7GrIOOQWF/SPbK3yPbiiUm7J?=
 =?us-ascii?Q?bVor8vmlzPHiDxIm1zFIPI5vPAFOIcULbhhdlpRJrtz9h+0iREL4PWik95RO?=
 =?us-ascii?Q?7FuPbrGPYnDXniMPg1POw2Synd2sAD5rMmPImzNNkPYhahLs7lmwj5H6TeRW?=
 =?us-ascii?Q?11WLyY8alQh1FXZKqX/eE7Mh0wFMNjDmf1SsCilKNSRZOwEvmX41IqJh9Y5o?=
 =?us-ascii?Q?0/QkCrFt12yCiFgGxiSByCZ/UbQ0FBIDVC+ga7PXg8L9+bhOk1c84yx61IBS?=
 =?us-ascii?Q?nNGbDzdFO9nhStpUo7hGYkf/BgMY8MhJX3Qe2djSnjfbTvhNfp7wisP8XXl9?=
 =?us-ascii?Q?qA=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: 
	UcthZTYbaHGU9E3tFu4DIe5SHRO8lNVc+cJHa/e2Is8NQkxtwsEf21l/bW6FKCQho54s878QPxSWy7od49Lzevyen0uIXstmqhl5XLBEAj4UGPcQaodafvabeGqAUJhX/qhw4aUBU+vDCvAOSU0S7diQNIZfoLlTiYQr34TuqIM6lpBMFq/bJms0eGp7iGyeYhrCCXb6qe7Df0AR7cArJUe1rIOxBnb1QGuySl6iQn5pRCU8INDNu3VY44/4zaC1okfiV5RGeGRIi9RFuQVtUKlGAWE6HKKQTmu/t0wWQY3URTc9O0zEqEbdGwgb774pOsFe5NO0PxUDhCwErPRvqy4XPj+WV+q3GokBEDak5Z/fQT72sbF/8qntcS2U2XdjjIrycDKZUai9OkjPAVv2/wNWdVXIbHKFnevsr3F9FQsrktob9LKWkuPDlDnSh7CVH8w18k3khTaNDyCmgGMlzpeMTk+QA9KuOz5WrLG2mMYHgPIokLtd0DLHw6gpXPKlyq9T0HBSTi5S9VNFSrr2iEYbIUMPDUzQqw2ZWTKMt6s6UhqjJX0cJ2CvzUcQsMXz5TLiYBm+N39vsbTzzQ2GyhiBzjk27jCbOiCC4EfKGso=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: fab33e33-70ab-499f-ca80-08dc7eae357d
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:36:26.7406
 (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: +3RPEY1XP/Viy7AKOJdLVTEDezCmqN4gWiooHxhzSkMQapd557z0dL6zgsVPmVFLqUeWgGFWAXVd1WpV+evfLNhwa+X0e3rS89gFLey8TVg=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR10MB7177
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=791 bulkscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2405010000
 definitions=main-2405280003
X-Proofpoint-GUID: VC4V3R1mLwPgrxbdvPxijdEHMQ4OaAEx
X-Proofpoint-ORIG-GUID: VC4V3R1mLwPgrxbdvPxijdEHMQ4OaAEx

From: Shrikanth Hegde <sshegde@linux.ibm.com>

Add PowerPC arch support for PREEMPT_AUTO by defining LAZY bits.

Since PowerPC doesn't use generic exit to functions, Add
NR_LAZY check in exit to user and exit to kernel from interrupt
routines.

Signed-off-by: Shrikanth Hegde <sshegde@linux.ibm.com>
[ Changed TIF_NEED_RESCHED_LAZY to now be defined unconditionally. ]
Signed-off-by: Ankur Arora <ankur.a.arora@oracle.com>
---
 arch/powerpc/Kconfig                   | 1 +
 arch/powerpc/include/asm/thread_info.h | 5 ++++-
 arch/powerpc/kernel/interrupt.c        | 5 +++--
 3 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index 1c4be3373686..11e7008f5dd3 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -268,6 +268,7 @@ config PPC
 	select HAVE_PERF_EVENTS_NMI		if PPC64
 	select HAVE_PERF_REGS
 	select HAVE_PERF_USER_STACK_DUMP
+	select HAVE_PREEMPT_AUTO
 	select HAVE_REGS_AND_STACK_ACCESS_API
 	select HAVE_RELIABLE_STACKTRACE
 	select HAVE_RSEQ
diff --git a/arch/powerpc/include/asm/thread_info.h b/arch/powerpc/include/asm/thread_info.h
index 15c5691dd218..0d170e2be2b6 100644
--- a/arch/powerpc/include/asm/thread_info.h
+++ b/arch/powerpc/include/asm/thread_info.h
@@ -117,11 +117,14 @@ void arch_setup_new_exec(void);
 #endif
 #define TIF_POLLING_NRFLAG	19	/* true if poll_idle() is polling TIF_NEED_RESCHED */
 #define TIF_32BIT		20	/* 32 bit binary */
+#define TIF_NEED_RESCHED_LAZY	21	/* Lazy rescheduling */
 
 /* as above, but as bit values */
 #define _TIF_SYSCALL_TRACE	(1<<TIF_SYSCALL_TRACE)
 #define _TIF_SIGPENDING		(1<<TIF_SIGPENDING)
 #define _TIF_NEED_RESCHED	(1<<TIF_NEED_RESCHED)
+#define _TIF_NEED_RESCHED_LAZY	(1 << TIF_NEED_RESCHED_LAZY)
+
 #define _TIF_NOTIFY_SIGNAL	(1<<TIF_NOTIFY_SIGNAL)
 #define _TIF_POLLING_NRFLAG	(1<<TIF_POLLING_NRFLAG)
 #define _TIF_32BIT		(1<<TIF_32BIT)
@@ -144,7 +147,7 @@ void arch_setup_new_exec(void);
 #define _TIF_USER_WORK_MASK	(_TIF_SIGPENDING | _TIF_NEED_RESCHED | \
 				 _TIF_NOTIFY_RESUME | _TIF_UPROBE | \
 				 _TIF_RESTORE_TM | _TIF_PATCH_PENDING | \
-				 _TIF_NOTIFY_SIGNAL)
+				 _TIF_NOTIFY_SIGNAL | _TIF_NEED_RESCHED_LAZY)
 #define _TIF_PERSYSCALL_MASK	(_TIF_RESTOREALL|_TIF_NOERROR)
 
 /* Bits in local_flags */
diff --git a/arch/powerpc/kernel/interrupt.c b/arch/powerpc/kernel/interrupt.c
index eca293794a1e..0b97cdd4b94e 100644
--- a/arch/powerpc/kernel/interrupt.c
+++ b/arch/powerpc/kernel/interrupt.c
@@ -185,7 +185,7 @@ interrupt_exit_user_prepare_main(unsigned long ret, struct pt_regs *regs)
 	ti_flags = read_thread_flags();
 	while (unlikely(ti_flags & (_TIF_USER_WORK_MASK & ~_TIF_RESTORE_TM))) {
 		local_irq_enable();
-		if (ti_flags & _TIF_NEED_RESCHED) {
+		if (ti_flags & (_TIF_NEED_RESCHED | _TIF_NEED_RESCHED_LAZY)) {
 			schedule();
 		} else {
 			/*
@@ -396,7 +396,8 @@ notrace unsigned long interrupt_exit_kernel_prepare(struct pt_regs *regs)
 		/* Returning to a kernel context with local irqs enabled. */
 		WARN_ON_ONCE(!(regs->msr & MSR_EE));
 again:
-		if (IS_ENABLED(CONFIG_PREEMPT)) {
+
+		if (IS_ENABLED(CONFIG_PREEMPTION)) {
 			/* Return to preemptible kernel context */
 			if (unlikely(read_thread_flags() & _TIF_NEED_RESCHED)) {
 				if (preempt_count() == 0)
-- 
2.31.1