From 1eededf17fadaefcec0adcfaabbe00bbbcf327fc Mon Sep 17 00:00:00 2001
From: Augustin Cavalier <waddlesplash@gmail.com>
Date: Sun, 17 Feb 2019 14:34:36 -0500
Subject: [PATCH] kernel: Only sanity-check interrupts in switch_sem_etc under
 KDEBUG.

"are_interrupts_enabled()" is a "relatively" expensive function, so avoiding
it makes sense. This is the same way it's done elsewhere in sem.cpp, as well
as for mutexes and condition variables also.
---
 src/system/kernel/sem.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/system/kernel/sem.cpp b/src/system/kernel/sem.cpp
index eb06cbb410..841df6fe45 100644
--- a/src/system/kernel/sem.cpp
+++ b/src/system/kernel/sem.cpp
@@ -763,11 +763,12 @@ switch_sem_etc(sem_id semToBeReleased, sem_id id, int32 count,
 		return B_OK;
 	if (sSemsActive == false)
 		return B_NO_MORE_SEMS;
-
+#if KDEBUG
 	if (!are_interrupts_enabled()) {
 		panic("switch_sem_etc: called with interrupts disabled for sem "
 			"%" B_PRId32 "\n", id);
 	}
+#endif
 
 	if (id < 0)
 		return B_BAD_SEM_ID;