esp6: Support RX checksum with crypto offload
Keep the device's reported ip_summed indication in case crypto was offloaded by the device. Subtract the csum values of the stripped parts (esp header+iv, esp trailer+auth_data) to keep value correct. Note: CHECKSUM_COMPLETE should be indicated only if skb->csum has the post-decryption offload csum value. Signed-off-by: Ariel Levkovich <lariel@mellanox.com> Signed-off-by: Ilan Tayari <ilant@mellanox.com> Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
This commit is contained in:
Родитель
ec9567a9e0
Коммит
e51a647270
|
@ -470,7 +470,8 @@ int esp6_input_done2(struct sk_buff *skb, int err)
|
||||||
int hlen = sizeof(struct ip_esp_hdr) + crypto_aead_ivsize(aead);
|
int hlen = sizeof(struct ip_esp_hdr) + crypto_aead_ivsize(aead);
|
||||||
int elen = skb->len - hlen;
|
int elen = skb->len - hlen;
|
||||||
int hdr_len = skb_network_header_len(skb);
|
int hdr_len = skb_network_header_len(skb);
|
||||||
int padlen;
|
int padlen, trimlen;
|
||||||
|
__wsum csumdiff;
|
||||||
u8 nexthdr[2];
|
u8 nexthdr[2];
|
||||||
|
|
||||||
if (!xo || (xo && !(xo->flags & CRYPTO_DONE)))
|
if (!xo || (xo && !(xo->flags & CRYPTO_DONE)))
|
||||||
|
@ -492,8 +493,15 @@ int esp6_input_done2(struct sk_buff *skb, int err)
|
||||||
|
|
||||||
/* ... check padding bits here. Silly. :-) */
|
/* ... check padding bits here. Silly. :-) */
|
||||||
|
|
||||||
pskb_trim(skb, skb->len - alen - padlen - 2);
|
trimlen = alen + padlen + 2;
|
||||||
__skb_pull(skb, hlen);
|
if (skb->ip_summed == CHECKSUM_COMPLETE) {
|
||||||
|
csumdiff = skb_checksum(skb, skb->len - trimlen, trimlen, 0);
|
||||||
|
skb->csum = csum_block_sub(skb->csum, csumdiff,
|
||||||
|
skb->len - trimlen);
|
||||||
|
}
|
||||||
|
pskb_trim(skb, skb->len - trimlen);
|
||||||
|
|
||||||
|
skb_pull_rcsum(skb, hlen);
|
||||||
if (x->props.mode == XFRM_MODE_TUNNEL)
|
if (x->props.mode == XFRM_MODE_TUNNEL)
|
||||||
skb_reset_transport_header(skb);
|
skb_reset_transport_header(skb);
|
||||||
else
|
else
|
||||||
|
|
|
@ -209,10 +209,12 @@ out:
|
||||||
static int esp6_input_tail(struct xfrm_state *x, struct sk_buff *skb)
|
static int esp6_input_tail(struct xfrm_state *x, struct sk_buff *skb)
|
||||||
{
|
{
|
||||||
struct crypto_aead *aead = x->data;
|
struct crypto_aead *aead = x->data;
|
||||||
|
struct xfrm_offload *xo = xfrm_offload(skb);
|
||||||
|
|
||||||
if (!pskb_may_pull(skb, sizeof(struct ip_esp_hdr) + crypto_aead_ivsize(aead)))
|
if (!pskb_may_pull(skb, sizeof(struct ip_esp_hdr) + crypto_aead_ivsize(aead)))
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
|
if (!(xo->flags & CRYPTO_DONE))
|
||||||
skb->ip_summed = CHECKSUM_NONE;
|
skb->ip_summed = CHECKSUM_NONE;
|
||||||
|
|
||||||
return esp6_input_done2(skb, 0);
|
return esp6_input_done2(skb, 0);
|
||||||
|
|
Загрузка…
Ссылка в новой задаче