From: "K. Y. Srinivasan" <[email protected]> Date: Fri, 25 Apr 2014 20:55:23 -0700
> @@ -467,6 +467,9 @@ static int netvsc_start_xmit(struct sk_buff *skb, struct > net_device *net) > if (skb_is_gso(skb)) > goto do_lso; > > + if (skb->ip_summed != CHECKSUM_PARTIAL) > + goto do_send; > + There are many possible values of ip_summed, why would you go to do_send if for example it was set to CHECKSUM_COMPLETE? I think you are just rushing this change, take your time and implement the fix properly. For example, if you only expect two possible values here (CHECKSUM_PARTIAL and something else), design your test so that it only allows those two values and therefore you'll be documenting this invariant. -- 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/
