shithub: riscv

Download patch

ref: 8a4a2dea70aa77475a50968f4c5bd6cdab5d2fb1
parent: 87299152ec6af407392c9069107004c60953b696
author: cinap_lenrek <[email protected]>
date: Sun Apr 7 12:09:46 EDT 2024

kernel: remove some debugging cruft from taslock.c

Only keep the data structures for LOCKCYCLES when
LOCKCYCLES is actually defined (saves 2K in data segment).

Remove unused dumplockmem() function.

--- a/sys/src/9/port/taslock.c
+++ b/sys/src/9/port/taslock.c
@@ -6,6 +6,7 @@
 #include "../port/error.h"
 #include "edf.h"
 
+#ifdef LOCKCYCLES
 long maxlockcycles;
 long maxilockcycles;
 long cumlockcycles;
@@ -12,6 +13,8 @@
 long cumilockcycles;
 uintptr maxlockpc;
 uintptr maxilockpc;
+uintptr ilockpcs[0x100] = { [0xff] = 1 };
+#endif
 
 struct
 {
@@ -20,19 +23,6 @@
 	ulong	inglare;
 } lockstats;
 
-static void
-dumplockmem(char *tag, Lock *l)
-{
-	uchar *cp;
-	int i;
-
-	iprint("%s: ", tag);
-	cp = (uchar*)l;
-	for(i = 0; i < 64; i++)
-		iprint("%2.2ux ", cp[i]);
-	iprint("\n");
-}
-
 void
 lockloop(Lock *l, uintptr pc)
 {
@@ -213,9 +203,6 @@
 		sched();
 }
 
-uintptr ilockpcs[0x100] = { [0xff] = 1 };
-static int n;
-
 void
 iunlock(Lock *l)
 {
@@ -222,6 +209,7 @@
 	ulong sr;
 
 #ifdef LOCKCYCLES
+	static uint n;
 	l->lockcycles += lcycles();
 	cumilockcycles += l->lockcycles;
 	if(l->lockcycles > maxilockcycles){
@@ -229,7 +217,7 @@
 		maxilockpc = l->pc;
 	}
 	if(l->lockcycles > 2400)
-		ilockpcs[n++ & 0xff]  = l->pc;
+		ilockpcs[n++ % nelem(ilockpcs)]  = l->pc;
 #endif
 	if(l->key == 0)
 		print("iunlock(%#p): not locked: pc %#p\n", l, getcallerpc(&l));