diff options
| author | Paul Durrant <Paul.Durrant@citrix.com> | 2013-12-03 15:06:25 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-12-03 17:49:30 +0100 |
| commit | 67fa36609fe0a0a4b4c99120e5093599556e4c5b (patch) | |
| tree | 4c607aeaa3de8da2b7b3de8d2457195c643281cb /net/rds | |
| parent | Revert "net: Handle CHECKSUM_COMPLETE more adequately in pskb_trim_rcsum()." (diff) | |
| download | linux-67fa36609fe0a0a4b4c99120e5093599556e4c5b.tar.xz linux-67fa36609fe0a0a4b4c99120e5093599556e4c5b.zip | |
xen-netback: clear vif->task on disconnect
xenvif_start_xmit() relies on checking vif->task for NULL to determine
whether the vif is ready to accept packets. The task thread is stopped in
xenvif_disconnect() but task is not set to NULL. Thus, on a re-connect the
check will give a false positive.
Also since commit ea732dff5cfa10789007bf4a5b935388a0bb2a8f (Handle backend
state transitions in a more robust way) it should not be possible for
xenvif_connect() to be called if the vif is already connected so change the
check of vif->tx_irq to a BUG_ON() and also add a BUG_ON(vif->task).
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: David Vrabel <david.vrabel@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/rds')
0 files changed, 0 insertions, 0 deletions