* [Linux-ia64] [PATCH] to remove unused hcl_spinlock
@ 2001-09-07 22:01 David C. Hansen
0 siblings, 0 replies; only message in thread
From: David C. Hansen @ 2001-09-07 22:01 UTC (permalink / raw)
To: linux-ia64
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.
--- linux-2.4.9/arch/ia64/sn/io/hcl.c Thu Apr 5 12:51:47 2001
+++ linux/arch/ia64/sn/io/hcl.c Fri Sep 7 14:52:40 2001
@@ -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.
--- linux-2.4.9/include/asm-ia64/sn/hcl.h Thu Apr 5 12:51:47 2001
+++ linux/include/asm-ia64/sn/hcl.h Fri Sep 7 14:53:53 2001
@@ -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
haveblue@us.ibm.com
IBM LTC Base/OS Group
(503)578-4080
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2001-09-07 22:01 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-09-07 22:01 [Linux-ia64] [PATCH] to remove unused hcl_spinlock David C. Hansen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox