From: Tom Herbert <tom@herbertland.com>
To: <netdev@vger.kernel.org>
Subject: [PATCH net-next 0/2] net: Fix "hw csum failure" message flood for ppp tunnel
Date: Mon, 20 Apr 2015 14:10:03 -0700 [thread overview]
Message-ID: <1429564205-1661247-1-git-send-email-tom@herbertland.com> (raw)
This patch set addresses bug "Bug 95171 - "hw csum failure" message
flood for ppp tunnel since upgrade to 3.16". The problem is that pppoe
is being used over UDP with UDP checksusm enabled. On receive
checksum conversion turns checksum-unnecessary in checksum-
complete. The PPP receive functions do no properly pull
the checksum over its headers, so that when an encapsulated
checksums is considered the checksum-complete value is incorrect.
This patch adds skb_checksum_complete_unset which can be called
in the receive path in lieu of pulling checksum complete in
layer. This is useful when the packet is being modified (e.g.
decompressed) and the checksum-complete value is no longer
relevant.
In the ppp_receive_frame we call skb_checksum_complete_unset to toss
out checksum-complete. This should eliminate the reported messages.
Alternatively, we could add skb_postpull_rcsum and probably
special case handling for VJ compression if maintaining the
checksum-complete is needed (not clear to me this is worth the
effort).
I haven't tested this since setting up the failure scenario doesn't
seem trivial to configure.
Tom Herbert (2):
net: add skb_checksum_complete_unset
ppp: call skb_checksum_complete_unset in ppp_receive_frame
drivers/net/ppp/ppp_generic.c | 1 +
include/linux/skbuff.h | 12 ++++++++++++
2 files changed, 13 insertions(+)
--
1.8.1
next reply other threads:[~2015-04-20 21:10 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-20 21:10 Tom Herbert [this message]
2015-04-20 21:10 ` [PATCH net-next 1/2] net: add skb_checksum_complete_unset Tom Herbert
2015-04-20 21:10 ` [PATCH net-next 2/2] ppp: call skb_checksum_complete_unset in ppp_receive_frame Tom Herbert
2015-04-21 2:01 ` [PATCH net-next 0/2] net: Fix "hw csum failure" message flood for ppp tunnel David Miller
[not found] <1429369717-4084627-1-git-send-email-tom@herbertland.com>
2015-04-20 18:15 ` David Miller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1429564205-1661247-1-git-send-email-tom@herbertland.com \
--to=tom@herbertland.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).