[PATCH] Fix comment in include/linux/nodemask.h

From: Matthew Dobson
Date: Thu Aug 26 2004 - 17:55:25 EST


Andrew, here's a small comment fix for include/linux/nodemask.h. It's a
comment that is only relevant to cpumasks and accidentally got copied
over during my (failed) attempt to correctly post the nodemask code.
The patch just updates the comment to make sense for nodemasks.

Thanks!

-Matt

--- linux-2.6.9-rc1-mm1/include/linux/nodemask.h 2004-08-26 15:38:42.000000000 -0700
+++ linux-2.6.9-rc1-mm1/include/linux/nodemask.h.fixed 2004-08-26 15:39:04.000000000 -0700
@@ -69,12 +69,9 @@
*
* Subtlety:
* 1) The 'type-checked' form of node_isset() causes gcc (3.3.2, anyway)
- * to generate slightly worse code. Note for example the additional
- * 40 lines of assembly code compiling the "for each possible node"
- * loops buried in the disk_stat_read() macros calls when compiling
- * drivers/block/genhd.c (arch i386, CONFIG_SMP=y). So use a simple
- * one-line #define for node_isset(), instead of wrapping an inline
- * inside a macro, the way we do the other calls.
+ * to generate slightly worse code. So use a simple one-line #define
+ * for node_isset(), instead of wrapping an inline inside a macro, the
+ * way we do the other calls.
*/

#include <linux/kernel.h>


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/