Hello. Not sure who maintains list.h, hopefully this is -mm material.
It is really annoying that we do not have list_next/prev_entry, they can have a lot of users, including list.h itself. And in fact list_next_entry() is already defined in events/core.c and bnx2x_sp.c, this proves that it is really handy. That is why 1/4 comes as a separate patch because it touches the code outside of list.h. 3/4 is minor, but I think makes sense anyway. 4/4 is just the random example of usage outside of list.h, list_entry(child->d_u.d_child.next, struct dentry, d_u.d_child) is hardly readable. None of this patches changes the compiled code. Oleg. drivers/net/ethernet/broadcom/bnx2x/bnx2x_sp.c | 3 - fs/debugfs/inode.c | 3 +- include/linux/list.h | 79 ++++++++++++++++-------- kernel/events/core.c | 3 - 4 files changed, 54 insertions(+), 34 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
