On Fri, 2010-01-01 at 11:08 +0800, Américo Wang wrote: > This patch looks fine for me, except that I don't think it's necessary > to introduce an inline function for that... Actually, I really think we do. The whole reason we got into this mess in the first place is that it wasn't obvious from the code that if we altered the loadability of sections, there were a ton of dependent places we also had to update. Putting all that knowledge into a single inline makes the same mistake impossible to make because there's now only one place to go to adjust the loadability of sections, and it automatically takes care of all the dependencies. > Reviewed-by: WANG Cong <xiyou.wangcong@xxxxxxxxx> Thanks, James -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html