Commit b4759dcd authored by Davide Caratti's avatar Davide Caratti Committed by David S. Miller

sk_buff.h: improve description of CHECKSUM_{COMPLETE, UNNECESSARY}

Add FCoE to the list of protocols that can set CHECKSUM_UNNECESSARY; add a
note to CHECKSUM_COMPLETE section to specify that it does not apply to SCTP
and FCoE protocols.
Suggested-by: default avatarTom Herbert <tom@herbertland.com>
Signed-off-by: default avatarDavide Caratti <dcaratti@redhat.com>
Acked-by: default avatarTom Herbert <tom@herbertland.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 7529390d
...@@ -109,6 +109,7 @@ ...@@ -109,6 +109,7 @@
* may perform further validation in this case. * may perform further validation in this case.
* GRE: only if the checksum is present in the header. * GRE: only if the checksum is present in the header.
* SCTP: indicates the CRC in SCTP header has been validated. * SCTP: indicates the CRC in SCTP header has been validated.
* FCOE: indicates the CRC in FC frame has been validated.
* *
* skb->csum_level indicates the number of consecutive checksums found in * skb->csum_level indicates the number of consecutive checksums found in
* the packet minus one that have been verified as CHECKSUM_UNNECESSARY. * the packet minus one that have been verified as CHECKSUM_UNNECESSARY.
...@@ -126,8 +127,10 @@ ...@@ -126,8 +127,10 @@
* packet as seen by netif_rx() and fills out in skb->csum. Meaning, the * packet as seen by netif_rx() and fills out in skb->csum. Meaning, the
* hardware doesn't need to parse L3/L4 headers to implement this. * hardware doesn't need to parse L3/L4 headers to implement this.
* *
* Note: Even if device supports only some protocols, but is able to produce * Notes:
* skb->csum, it MUST use CHECKSUM_COMPLETE, not CHECKSUM_UNNECESSARY. * - Even if device supports only some protocols, but is able to produce
* skb->csum, it MUST use CHECKSUM_COMPLETE, not CHECKSUM_UNNECESSARY.
* - CHECKSUM_COMPLETE is not applicable to SCTP and FCoE protocols.
* *
* CHECKSUM_PARTIAL: * CHECKSUM_PARTIAL:
* *
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment