diff options
author | Michael S. Tsirkin <mst@redhat.com> | 2010-10-31 19:06:47 +0200 |
---|---|---|
committer | Anthony Liguori <aliguori@us.ibm.com> | 2010-11-16 08:40:07 -0600 |
commit | 27a6375de3edece2e5d115847d54c01e52331f7e (patch) | |
tree | 2e0668b8e8081e382a998a8d365adf959a0b0c27 /net | |
parent | f6584ee20338a74ef3b05b38b8f9dc5e7a7276a6 (diff) | |
download | qemu-27a6375de3edece2e5d115847d54c01e52331f7e.zip qemu-27a6375de3edece2e5d115847d54c01e52331f7e.tar.gz qemu-27a6375de3edece2e5d115847d54c01e52331f7e.tar.bz2 |
tap: make set_offload a nop after netdev cleanup
virtio-net expects set_offload to succeed after
peer cleanup.
Since we don't have an open fd anymore, make it so.
Fixes warning about the failure of offload setting.
Reported-by: Jason Wang <jasowang@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/tap.c | 6 |
1 files changed, 5 insertions, 1 deletions
@@ -269,8 +269,11 @@ void tap_set_offload(VLANClientState *nc, int csum, int tso4, int tso6, int ecn, int ufo) { TAPState *s = DO_UPCAST(TAPState, nc, nc); + if (s->fd < 0) { + return; + } - return tap_fd_set_offload(s->fd, csum, tso4, tso6, ecn, ufo); + tap_fd_set_offload(s->fd, csum, tso4, tso6, ecn, ufo); } static void tap_cleanup(VLANClientState *nc) @@ -290,6 +293,7 @@ static void tap_cleanup(VLANClientState *nc) tap_read_poll(s, 0); tap_write_poll(s, 0); close(s->fd); + s->fd = -1; } static void tap_poll(VLANClientState *nc, bool enable) |