[PATCH 2/3] Fix Spinlock compile errors

[PATCH 2/3] Fix Spinlock compile errors

[Ben Guthro]

[-- Attachment #1: Type: text/plain, Size: 518 bytes --]

Remove extra parentheses in netif_tx_lock_bh and etif_tx_unlock_bh 
macros, as it caused problems
when compiling against a SLES9 tree.

While at first glance - the existing code should work, these extraneous 
parentheses were causing build errors.
It seems to me this was either a buggy kernel, or a buggy compiler, for 
this to fail.

However, this change, (though subtle) does fix the compile error, and 
does not cause any other problems in newer guest kernels.

Signed-off-by: Ben Guthro <bguthro@virtualiron.com>

[-- Attachment #2: xen-legacy-spinlock.patch --]
[-- Type: text/x-patch, Size: 778 bytes --]

diff -r e9f35dc18ae9 unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h
--- a/unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h	Wed Aug 15 16:15:06 2007 -0400
+++ b/unmodified_drivers/linux-2.6/compat-include/xen/platform-compat.h	Wed Aug 15 16:15:28 2007 -0400
@@ -108,8 +108,8 @@ extern char *kasprintf(gfp_t gfp, const 
 #endif
 
 #if defined(_LINUX_NETDEVICE_H) && LINUX_VERSION_CODE < KERNEL_VERSION(2,6,18)
-#define netif_tx_lock_bh(dev) (spin_lock_bh(&(dev)->xmit_lock))
-#define netif_tx_unlock_bh(dev) (spin_unlock_bh(&(dev)->xmit_lock))
+#define netif_tx_lock_bh(dev) spin_lock_bh(&(dev)->xmit_lock)
+#define netif_tx_unlock_bh(dev) spin_unlock_bh(&(dev)->xmit_lock)
 #endif
 
 #if defined(__LINUX_SEQLOCK_H) && !defined(DEFINE_SEQLOCK)

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel