David Miller <[EMAIL PROTECTED]> writes: > From: Stephen Hemminger <[EMAIL PROTECTED]> > Date: Fri, 14 Sep 2007 22:07:14 +0200 > >> Could we just make it so dev->init is not allowed to fail? Then it >> can be a void function and the nasty unwind code can go? > > Someone (not me :-) need to do an audit to find all current > users of this function and determine if they all can live > without returning errors. > > If so, sure let's make the change and simplify things.
I did that audit when I replied to Stephen the first time and I just redid it to verify myself. We are calling functions that can fail from the init function (kmalloc in the most common). So the init function can fail. So short of adding a bunch of BUG_ON's to the kernel to trap those failure cases we can't remove the backwards list walk. Especially since I can initiate this code path as root by calling "clone(CLONE_NEWNET...)". Eric - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
