Skip to contentSkip to conversation

to remove unused hcl_spinlock

PATCH New David C. Hansen 1 participant 1 message Sep 7, 2001

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 to remove unused hcl_spinlock

I was looking at spinlocks in the kernel and came across hcl_spinlock. If you look at this paper:
http://lse.sourceforge.net/lockhier/global-spin-lock , you can see a part on hcl_spinlock. It is initialized. but never used. Was this lock meant to guard something? Is that thing still in the code? If it isn't, here is a patch to remove hcl_spinlock.

arch/ia64/sn/io/hcl.c +0 -3
@@ -49,7 +49,6 @@
 /*
  * Some Global definitions.
  */
-spinlock_t hcl_spinlock;
 devfs_handle_t hcl_handle = NULL;
 
 invplace_t invplace_none = {
@@ -152,8 +151,6 @@
 	printk ("%s: hcl_debug: 0x%0x\n", HCL_NAME, hcl_debug);
 	printk ("\n%s: boot_options: 0x%0x\n", HCL_NAME, boot_options);
 #endif
-
-	spin_lock_init(&hcl_spinlock);
 
 	/*
 	 * Create the hwgraph_root on devfs.
include/asm-ia64/sn/hcl.h +0 -1
@@ -10,7 +10,6 @@
 #ifndef _ASM_SN_HCL_H
 #define _ASM_SN_HCL_H
 
-extern spinlock_t hcl_spinlock;
 extern devfs_handle_t hcl_handle; /* HCL driver */
 extern devfs_handle_t hwgraph_root;
 extern devfs_handle_t linux_busnum;

--
David C. Hansen
[email protected]
IBM LTC Base/OS Group
(503)578-4080