[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 0/4] slirp: Fix for requeuing crash, cleanups
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] [PATCH v2 0/4] slirp: Fix for requeuing crash, cleanups |
Date: |
Fri, 2 Mar 2012 19:57:49 +0100 |
Well, this requeuing bug seems to have a long breath. Previous attempts
to fix it (mine included) neglected the fact that we need to walk the
queue of pending packets, not just restart from the beginning after a
requeue. This version should get it Right(TM).
This also comes with a fix for resource cleanups on slirp shutdown. At
least valgrind is happy now.
Changes in v2:
- fixed corner case of session list walk that Stefan Weil reported
CC: Fabien Chouteau <address@hidden>
CC: Michael S. Tsirkin <address@hidden>
CC: Stefan Weil <address@hidden>
CC: Zhi Yong Wu <address@hidden>
Jan Kiszka (4):
slirp: Keep next_m always valid
slirp: Fix queue walking in if_start
slirp: Remove unneeded if_queued
slirp: Cleanup resources on instance removal
slirp/if.c | 64 +++++++++++++++++++++++++++++------------------------
slirp/ip_icmp.c | 7 ++++++
slirp/ip_icmp.h | 1 +
slirp/ip_input.c | 7 ++++++
slirp/mbuf.c | 21 +++++++++++++++++
slirp/mbuf.h | 1 +
slirp/slirp.c | 10 +++-----
slirp/slirp.h | 3 +-
slirp/tcp_subr.c | 7 ++++++
slirp/udp.c | 8 ++++++
slirp/udp.h | 1 +
11 files changed, 94 insertions(+), 36 deletions(-)
--
1.7.3.4
- [Qemu-devel] [PATCH v2 0/4] slirp: Fix for requeuing crash, cleanups,
Jan Kiszka <=
- [Qemu-devel] [PATCH v2 3/4] slirp: Remove unneeded if_queued, Jan Kiszka, 2012/03/02
- [Qemu-devel] [PATCH v2 4/4] slirp: Cleanup resources on instance removal, Jan Kiszka, 2012/03/02
- [Qemu-devel] [PATCH v2 1/4] slirp: Keep next_m always valid, Jan Kiszka, 2012/03/02
- [Qemu-devel] [PATCH v2 2/4] slirp: Fix queue walking in if_start, Jan Kiszka, 2012/03/02
- Re: [Qemu-devel] [PATCH v2 0/4] slirp: Fix for requeuing crash, cleanups, Stefan Weil, 2012/03/02