[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/4] slirp: Fix for requeuing crash, cleanups
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [PATCH 0/4] slirp: Fix for requeuing crash, cleanups |
Date: |
Wed, 29 Feb 2012 22:33:48 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2012-02-29 22:00, Stefan Weil wrote:
> Am 29.02.2012 20:15, schrieb Jan Kiszka:
>> This is an alternative, more complete approach to fix the requeuing-
>> related crashes reported recently. See patch 2 for details. The rest are
>> simple cleanups.
>>
>> Please check carefully if I messed something up.
>>
>
> Hi Jan,
>
> here is the result of MIPS Malta with your patch series applied:
>
> Program received signal SIGSEGV, Segmentation fault.
> 0x000055555577db5b in slirp_remque (a=0x555556cff360) at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/slirp/misc.c:39
> 39 ((struct quehead *)(element->qh_rlink))->qh_link =
> element->qh_link;
> (gdb) i s
> #0 0x000055555577db5b in slirp_remque (a=0x555556cff360) at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/slirp/misc.c:39
> #1 0x000055555577b7a2 in if_start (slirp=0x5555564bfb80) at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/slirp/if.c:208
> #2 0x000055555577b607 in if_output (so=0x555556ea0b70,
> ifm=0x555556cff9e0) at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/slirp/if.c:139
> #3 0x000055555577d040 in ip_output (so=0x555556ea0b70,
> m0=0x555556cff9e0) at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/slirp/ip_output.c:84
> #4 0x00005555557865d6 in tcp_output (tp=0x555556ea0c20) at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/slirp/tcp_output.c:456
> #5 0x000055555577ff5a in slirp_select_poll (readfds=0x7fffffffda10,
> writefds=0x7fffffffda90, xfds=0x7fffffffdb10, select_error=0)
> at /home/stefan/src/qemu/repo.or.cz/qemu/ar7/slirp/slirp.c:480
> #6 0x000055555572d8c0 in main_loop_wait (nonblocking=0) at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/main-loop.c:469
> #7 0x0000555555721a61 in main_loop () at
> /home/stefan/src/qemu/repo.or.cz/qemu/ar7/vl.c:1558
> #8 0x00005555557284a2 in main (argc=25, argv=0x7fffffffdfe8,
> envp=0x7fffffffe0b8) at /home/stefan/src/qemu/repo.or.cz/qemu/ar7/vl.c:3667
> (gdb) p element
> $1 = (struct quehead *) 0x555556cff360
> (gdb) p *element
> $2 = {qh_link = 0x555556cff360, qh_rlink = 0x0}
> (gdb) p (struct quehead *)(element->qh_rlink)
> $3 = (struct quehead *) 0x0
Hmm. Two options:
- you try to debug what happens to that mbuf, why its queue anchors
get corrupted (maybe while in if_encap?)
- you tell me how to reproduce it (image file, host characteristics)
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 0/4] slirp: Fix for requeuing crash, cleanups, Jan Kiszka, 2012/02/29
- [Qemu-devel] [PATCH 4/4] slirp: Cleanup resources on instance removal, Jan Kiszka, 2012/02/29
- [Qemu-devel] [PATCH 3/4] slirp: Remove unneeded if_queued, Jan Kiszka, 2012/02/29
- [Qemu-devel] [PATCH 1/4] slirp: Keep next_m always valid, Jan Kiszka, 2012/02/29
- [Qemu-devel] [PATCH 2/4] slirp: Fix queue walking in if_start, Jan Kiszka, 2012/02/29
- Re: [Qemu-devel] [PATCH 0/4] slirp: Fix for requeuing crash, cleanups, Jan Kiszka, 2012/02/29
- Re: [Qemu-devel] [PATCH 0/4] slirp: Fix for requeuing crash, cleanups, Stefan Weil, 2012/02/29
- Re: [Qemu-devel] [PATCH 0/4] slirp: Fix for requeuing crash, cleanups,
Jan Kiszka <=