diff options
| author | Santiago Leon <santil@us.ibm.com> | 2008-08-20 21:09:19 +0200 |
|---|---|---|
| committer | Jeff Garzik <jgarzik@redhat.com> | 2008-08-27 11:36:57 +0200 |
| commit | 45e15bb734e4e559ab32f26196e44d5cf1417f10 (patch) | |
| tree | e71322bc295822ad97af55b0298668eab093cfec /COPYING | |
| parent | [netdrvr] hso: dev_kfree_skb crash fix (diff) | |
| download | linux-45e15bb734e4e559ab32f26196e44d5cf1417f10.tar.xz linux-45e15bb734e4e559ab32f26196e44d5cf1417f10.zip | |
ibmveth: fix bad UDP checksums
This patch fixes a ibmveth bug where bad UDP checksums are being transmitted
when checksum offloading is enabled.
The hypervisor does checksum offloading only on TCP packets, so ibmveth calls
skb_checksum_help() for any other protocol. The bug happens because
the packet is being modified after the DMA map, so we would need a memory
barrier before making the hypervisor call. Reordering the code so that the
DMA map happens after skb_checksum_help() has the additional advantage of
fixing a DMA map leak if skb_checksum_help() where to fail.
Signed-off-by: Santiago Leon <santil@us.ibm.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions