On Fri, Sep 18, 2015 at 1:40 PM, John W. Linville <[email protected]> wrote: > On Fri, Sep 18, 2015 at 01:30:36PM -0700, Jesse Gross wrote: >> On Thu, Sep 17, 2015 at 1:34 PM, John W. Linville >> <[email protected]> wrote: >> > This seems to have been a "thinko". IP_ECN_decapsulate needs info >> > from both internal and external headers. >> > >> > Signed-off-by: John W. Linville <[email protected]> >> >> This looks good to me although I realized that the transmit path is >> also conditional based on !collect_md metadata. I suppose that means >> that this difference is intentional and I guess I could see arguments >> either way but I still think it is better to be consistent across >> different code paths in this respect. > > I'm happy with the v2 patch if you are. :-)
I'm happy with this part of the code path after this patch. However, my concern was that this patch, while correct, will make receive inconsistent with transmit. Presumably we should update the transmit side as well. -- 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
