Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1321069 > unrolled thread

Re: [PATCH] net: mv643xx_eth: fix packet corruption with TSO and tiny unaligned packets.

Started byPhilipp Kirchhofer <philipp@familie-kirchhofer.de>
First post2016-01-28 22:30 +0100
Last post2016-01-28 22:30 +0100
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH] net: mv643xx_eth: fix packet corruption with TSO and tiny  unaligned packets. Philipp Kirchhofer <philipp@familie-kirchhofer.de> - 2016-01-28 22:30 +0100

#1321069 — Re: [PATCH] net: mv643xx_eth: fix packet corruption with TSO and tiny unaligned packets.

FromPhilipp Kirchhofer <philipp@familie-kirchhofer.de>
Date2016-01-28 22:30 +0100
SubjectRe: [PATCH] net: mv643xx_eth: fix packet corruption with TSO and tiny unaligned packets.
Message-ID<qW2iC-UG-19@gated-at.bofh.it>
Am 26.01.2016 um 16:12 schrieb Nicolas Schichan:
> The code in txq_put_data() would use txq->tx_curr_desc to index the
> tso_hdrs/tso_hdrs_dma buffers, for less than 8 bytes unaligned
> fragments, which is already moved to the next descriptor at the
> beginning of the function.
>
> If that fragment was the last of the the skb, the next skb would use
> that same space to place the ip headers, overwritting that small
> fragment data.
>
> Fixes: 91986fd3d335 (net: mv643xx_eth: Ensure proper data alignment in TSO TX path)
> Signed-off-by: Nicolas Schichan <nschichan@freebox.fr>

Hello Nicolas,

thanks for catching this bug. Fix is good.

Reviewed-by: Philipp Kirchhofer <philipp@familie-kirchhofer.de>

Best wishes,
   Philipp

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web