> > The last hunk that you removed had a role and was by no means > dead-code, right? so... (1) why it's correct to remove it? (2) if you > want to introduce different way to implement what was done here, why > in this patch? maybe add pre-patch for that In a way you are right. This hunk does not insert a bug and even improves correctness but it acutally belongs to an earlier patch (dbf727de7440f73c4b92be4b958cbc24977e8ca2 IB/core: Use GID table in AH creation and dmac resolution) -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html