Re: [PATCH net-next] net: skb_needs_check() removes CHECKSUM_NONE check for tx.

2017-08-10 Thread Tonghao Zhang
Hi Willem, because we change the CHECKSUM_NONE to CHECKSUM_UNNECESSARY in skb_needs_check(), I can't revert 6e7bc478c9a0 ("net: skb_needs_check() accepts CHECKSUM_NONE for tx"). I change it directly and resubmit it. On Fri, Aug 11, 2017 at 1:03 AM, Willem de Bruijn wrote: > On Wed, Aug 9, 2017 a

Re: [PATCH net-next] net: skb_needs_check() removes CHECKSUM_NONE check for tx.

2017-08-10 Thread Willem de Bruijn
On Wed, Aug 9, 2017 at 6:02 PM, Tonghao Zhang wrote: > Thanks for your work. You, too. > On Thu, Aug 10, 2017 at 2:30 AM, Willem de Bruijn > wrote: >> On Wed, Aug 9, 2017 at 5:04 AM, Tonghao Zhang >> wrote: >>> This patch reverts the commit 6e7bc478c9a0 >>> ("net: skb_needs_check() accepts CH

Re: [PATCH net-next] net: skb_needs_check() removes CHECKSUM_NONE check for tx.

2017-08-09 Thread Tonghao Zhang
Thanks for your work. On Thu, Aug 10, 2017 at 2:30 AM, Willem de Bruijn wrote: > On Wed, Aug 9, 2017 at 5:04 AM, Tonghao Zhang > wrote: >> This patch reverts the commit 6e7bc478c9a0 >> ("net: skb_needs_check() accepts CHECKSUM_NONE for tx"), >> because we removed the UFO support. >> >> Cc: Eric

Re: [PATCH net-next] net: skb_needs_check() removes CHECKSUM_NONE check for tx.

2017-08-09 Thread Willem de Bruijn
On Wed, Aug 9, 2017 at 5:04 AM, Tonghao Zhang wrote: > This patch reverts the commit 6e7bc478c9a0 > ("net: skb_needs_check() accepts CHECKSUM_NONE for tx"), > because we removed the UFO support. > > Cc: Eric Dumazet > Cc: Willem de Bruijn > Signed-off-by: Tonghao Zhang I would wait until net