Don't bother poisoning lists; it costs code size.
authorMichael Brown <mcb30@etherboot.org>
Mon, 24 Apr 2006 15:28:56 +0000 (15:28 +0000)
committerMichael Brown <mcb30@etherboot.org>
Mon, 24 Apr 2006 15:28:56 +0000 (15:28 +0000)
src/include/gpxe/list.h

index 7a951a0..7c6042b 100644 (file)
@@ -1,5 +1,5 @@
-#ifndef _LIST_H
-#define _LIST_H
+#ifndef _GPXE_LIST_H
+#define _GPXE_LIST_H
 
 /** @file
  *
 
 #include <stddef.h>
 
-/*
- * These are non-NULL pointers that will result in page faults
- * under normal circumstances, used to verify that nobody uses
- * non-initialized list entries.
- */
-#define LIST_POISON1 ( ( void * ) 0x00100100 )
-#define LIST_POISON2 ( ( void * ) 0x00200200 )
-
 /*
  * Simple doubly linked list implementation.
  *
@@ -108,8 +100,6 @@ static inline void __list_del ( struct list_head * prev,
  */
 static inline void list_del ( struct list_head *entry ) {
        __list_del ( entry->prev, entry->next );
-       entry->next = LIST_POISON1;
-       entry->prev = LIST_POISON2;
 }
 
 /**
@@ -152,4 +142,4 @@ static inline int list_empty ( const struct list_head *head ) {
              &pos->member != (head);                                         \
              pos = list_entry ( pos->member.next, typeof ( *pos ), member ) )
 
-#endif /* _LIST_H */
+#endif /* _GPXE_LIST_H */