Hi Geoff, > +#define list_continue_each_entry(_list, _pos, _member) \ > + for (; &_pos->_member != &(_list)->head; \ > + _pos = list_entry(_pos->_member.next, typeof(*_pos), _member)) > + Let's make this consistent and use list_for_each_entry_continue. Cheers, Jeremy