Re: [Qemu-devel] [PULL 00/21] pci,net,misc enhancements

[Amos Kong]

On Sun, Jun 23, 2013 at 02:38:04PM +0300, Michael S. Tsirkin wrote:
> On Fri, Jun 21, 2013 at 08:10:15AM -0500, Anthony Liguori wrote:
> > "Michael S. Tsirkin" <address@hidden> writes:
> > 
> > > From: Michael S. Tsirkin <address@hidden>
> > >
> > > The following changes since commit 
> > > 90a2541b763b31d2b551b07e24aae3de5266d31b:
> > >
> > >   target-i386: fix over 80 chars warnings (2013-06-15 17:50:38 +0000)
> > >
> > > are available in the git repository at:
> > >
> > >   git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git tags/for_anthony
> > >
> > > for you to fetch changes up to f96c30047009f8a9c3cecf68104d8d99f989f54d:
> > >
> > >   pci: Fold host_buses list into PCIHostState functionality
> > >   (2013-06-19 18:35:05 +0300)
> > 
> > SEGV's during pretty much any test I try to run during startup:
> > 
> > Core was generated by 
> > `/home/aliguori/build/qemu/x86_64-softmmu/qemu-system-x86_64 -kernel 
> > /usr/local/'.
> > Program terminated with signal 11, Segmentation fault.
> > #0  qemu_get_queue (nic=0x0) at /home/aliguori/git/qemu/net/net.c:263
> 
> 
> It's caused by the rx filter patch, sorry about not catching it.

If we don't assign 'id' for virtio-net nic device, crash[1] will occur.
Your fix is right for that.

But anthony's calltrace seems different from that.

> I've fixed it up but will send a pull request without it,
> so it can get a bit more testing.

BTW, we can use 'nc->name' to avoid the rx-filter crash[1].
I have sent a patch for mst/pci.

-- 
                        Amos.

crash [1]

#0  0x00007ffff59236b1 in __strlen_sse2_pminub () from /lib64/libc.so.6
#1  0x0000555555873a8d in qstring_from_str (str=0x0) at qobject/qstring.c:72
#2  0x0000555555876dae in parse_escape (ap=0x7ffff1c3c910, ctxt=0x7fffe4015650) 
at qobject/json-parser.c:614
#3  parse_value (address@hidden, address@hidden) at qobject/json-parser.c:691
#4  0x0000555555876fd2 in parse_pair (address@hidden, address@hidden, 
address@hidden) at qobject/json-parser.c:393
#5  0x00005555558765fe in parse_object (ap=0x7ffff1c3c910, ctxt=0x7fffe4015650) 
at qobject/json-parser.c:454
#6  parse_value (address@hidden, address@hidden) at qobject/json-parser.c:686
#7  0x0000555555877193 in json_parser_parse_err (tokens=<optimized out>, 
ap=0x7ffff1c3c910, address@hidden) at qobject/json-parser.c:717
#8  0x000055555587729f in json_parser_parse (tokens=<optimized out>, 
ap=<optimized out>) at qobject/json-parser.c:705
#9  0x0000555555875254 in parse_json (parser=0x7ffff1c3c8a0, tokens=<optimized 
out>) at qobject/qjson.c:34
#10 0x0000555555875d6a in json_message_process_token (lexer=0x7ffff1c3c8a8, 
token=0x7fffe4015440, type=JSON_OPERATOR, x=26, y=0) at 
qobject/json-streamer.c:87
#11 0x000055555588a7af in json_lexer_feed_char (address@hidden, ch=125 '}', 
address@hidden) at qobject/json-lexer.c:303
#12 0x000055555588a8f6 in json_lexer_feed (address@hidden, address@hidden "{ 
'name': %s, 'path': %s }", size=<optimized out>)
    at qobject/json-lexer.c:356
#13 0x0000555555875f81 in json_message_parser_feed (address@hidden, 
address@hidden "{ 'name': %s, 'path': %s }", size=<optimized out>)
    at qobject/json-streamer.c:110
#14 0x0000555555875a2d in qobject_from_jsonv (string=0x5555558d1882 "{ 'name': 
%s, 'path': %s }", address@hidden) at qobject/qjson.c:44
#15 0x0000555555875b39 in qobject_from_jsonf (address@hidden "{ 'name': %s, 
'path': %s }") at qobject/qjson.c:66
#16 0x00005555557b5477 in rxfilter_notify (address@hidden) at 
/home/devel/qemu/hw/net/virtio-net.c:203
#17 0x00005555557b6fbe in virtio_net_handle_rx_mode (iov_cnt=<optimized out>, 
iov=0x7ffff1c44ab0, cmd=0 '\000', n=0x555556756528) at 
/home/devel/qemu/hw/net/virtio-net.c:542
#18 virtio_net_handle_ctrl (vdev=0x555556756528, vq=0x555556767850) at 
/home/devel/qemu/hw/net/virtio-net.c:755
#19 0x00005555557c7cf2 in access_with_adjusted_size (address@hidden, 
address@hidden, size=2, access_size_min=<optimized out>, 
    access_size_max=<optimized out>, address@hidden 
<memory_region_write_accessor>, address@hidden) at /home/devel/qemu/memory.c:399
#20 0x00005555557c93d7 in memory_region_iorange_write (iorange=<optimized out>, 
offset=16, width=2, data=2) at /home/devel/qemu/memory.c:475
#21 0x00005555557c6a16 in kvm_handle_io (count=1, size=2, direction=1, 
data=<optimized out>, port=49232) at /home/devel/qemu/kvm-all.c:1510
#22 kvm_cpu_exec (address@hidden) at /home/devel/qemu/kvm-all.c:1659
#23 0x00005555557706d5 in qemu_kvm_cpu_thread_fn (arg=0x5555566d6660) at 
/home/devel/qemu/cpus.c:759
#24 0x00007ffff6985d15 in start_thread () from /lib64/libpthread.so.0
#25 0x00007ffff58b548d in clone () from /lib64/libc.so.6