[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] REGRESSION: pcspk: convert "pit" property type from ptr
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] REGRESSION: pcspk: convert "pit" property type from ptr to link |
Date: |
Thu, 30 Jun 2016 15:16:42 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 |
On 30/06/2016 14:28, Peter Lieven wrote:
> The above patch:
>
> commit 873b4d3f0571a1e415cf089a67a230ea8a12d059
> Author: Efimov Vasily <address@hidden>
> Date: Wed Jun 22 15:24:46 2016 +0300
>
> pcspk: convert "pit" property type from ptr to link
>
> adds a regression (segfault). Was this patch actually tested?
>
> Cmdline:
> x86_64-softmmu/qemu-system-x86_64 -enable-kvm
>
> Backtrace:
> #0 object_get_class (address@hidden) at qom/object.c:744
> No locals.
> #1 0x00005555559aa061 in pit_get_channel_info (dev=<optimized out>,
> channel=<optimized out>, info=0x7ffff0d1d840) at
> hw/timer/i8254_common.c:147
> pit = 0x0
> __func__ = "pit_get_channel_info"
> s = 0x1f0
> c = <optimized out>
> #2 0x00005555558d7b2c in pcspk_io_read (opaque=0x555557db4e30,
> addr=<optimized out>, size=<optimized out>) at hw/audio/pcspk.c:136
> ch = {gate = 255, mode = 0, initial_count = 1528560384, out =
> -1873283211}
> #3 0x000055555579202c in memory_region_read_accessor
> (mr=0x555557db4eb0, addr=0, value=0x7ffff0d1da30, size=1, shift=0,
> mask=255, attrs=...) at /home/lieven/git/qemu/memory.c:434
> tmp = <optimized out>
> #4 0x000055555578fa4b in access_with_adjusted_size (address@hidden,
> address@hidden, address@hidden,
> access_size_min=<optimized out>, access_size_max=<optimized out>,
> address@hidden <memory_region_read_accessor>,
> address@hidden, address@hidden) at
> /home/lieven/git/qemu/memory.c:591
> access_mask = 255
> access_size = 1
> i = <optimized out>
> r = 0
> #5 0x00005555557940de in memory_region_dispatch_read1 (attrs=...,
> size=1, pval=0x7ffff0d1da30, addr=0, mr=0x555557db4eb0) at
> /home/lieven/git/qemu/memory.c:1181
> No locals.
> #6 memory_region_dispatch_read (address@hidden,
> address@hidden, address@hidden, address@hidden,
> address@hidden) at /home/lieven/git/qemu/memory.c:1212
> No locals.
> #7 0x000055555574cf9a in address_space_read_continue
> (address@hidden <address_space_io>, address@hidden,
> attrs=..., address@hidden, address@hidden " \030",
> address@hidden,
> addr1=0, l=1, mr=0x555557db4eb0) at /home/lieven/git/qemu/exec.c:2680
> ptr = <optimized out>
> val = 0
> result = 0
> release_lock = true
> #8 0x000055555574d047 in address_space_read_full (as=0x5555560c1080
> <address_space_io>, addr=97, address@hidden, attrs=...,
> buf=0x7ffff7fef000 " \030", len=1, address@hidden)
> at /home/lieven/git/qemu/exec.c:2725
> l = 1
> addr1 = 0
> mr = <optimized out>
> result = 0
> #9 0x000055555574d1be in address_space_read (len=0, buf=<optimized
> out>, attrs=..., addr=93824994760285, as=<optimized out>) at
> /home/lieven/git/qemu/include/exec/memory.h:1454
> result = 0
> #10 address_space_rw (as=<optimized out>, address@hidden, attrs=...,
> address@hidden, buf=<optimized out>, address@hidden,
> address@hidden) at /home/lieven/git/qemu/exec.c:2739
> No locals.
> #11 0x000055555578e675 in kvm_handle_io (count=1, size=1,
> direction=<optimized out>, data=<optimized out>, attrs=..., port=97) at
> /home/lieven/git/qemu/kvm-all.c:1772
> i = 0
> ptr = <optimized out>
> #12 kvm_cpu_exec (address@hidden) at
> /home/lieven/git/qemu/kvm-all.c:1936
> attrs = {unspecified = 0, secure = 0, user = 0, requester_id = 0}
> run = 0x7ffff7fee000
> ret = <optimized out>
> run_ret = <optimized out>
>
>
> Peter
Yup, patch on the way...
Paolo