qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] MTTCG sync-up call today?


From: Alex Bennée
Subject: Re: [Qemu-devel] MTTCG sync-up call today?
Date: Tue, 12 Jan 2016 16:32:56 +0000
User-agent: mu4e 0.9.15; emacs 25.0.50.1

Paolo Bonzini <address@hidden> writes:

> On 12/01/2016 17:13, Alex Bennée wrote:
>> #4  0x00005555556e5b06 in tb_invalidate_phys_range (address@hidden, 
>> address@hidden) at /home/alex/lsrc/qemu/qemu.git/translate-all.c:1303
>> #5  0x00005555556dbe42 in invalidate_and_set_dirty (address@hidden, addr=0, 
>> address@hidden) at /home/alex/lsrc/qemu/qemu.git/exec.c:2420
>> #6  0x00005555556e1890 in address_space_unmap (address@hidden 
>> <address_space_memory>, buffer=<optimised out>, len=<optimised out>,
>>     address@hidden, address@hidden) at 
>> /home/alex/lsrc/qemu/qemu.git/exec.c:2933
>> #7  0x00005555556e19bf in cpu_physical_memory_unmap (buffer=<optimised out>, 
>> len=<optimised out>, address@hidden, address@hidden)
>>     at /home/alex/lsrc/qemu/qemu.git/exec.c:2962
>> #8  0x000055555578219c in virtqueue_unmap_sg (address@hidden, 
>> address@hidden, vq=0x555556e6f020)
>>     at /home/alex/lsrc/qemu/qemu.git/hw/virtio/virtio.c:257
>> #9  0x0000555555782ac0 in virtqueue_fill (address@hidden, address@hidden, 
>> len=4097, address@hidden)
>>     at /home/alex/lsrc/qemu/qemu.git/hw/virtio/virtio.c:282
>> #10 0x0000555555782ccf in virtqueue_push (vq=0x555556e6f020, address@hidden, 
>> len=<optimised out>)
>>     at /home/alex/lsrc/qemu/qemu.git/hw/virtio/virtio.c:308
>> #11 0x000055555573451a in virtio_blk_complete_request (req=0x7ffe782c7ce0, 
>> status=<optimised out>) at 
>> /home/alex/lsrc/qemu/qemu.git/hw/block/virtio-blk.c:58
>> #12 0x0000555555734a13 in virtio_blk_req_complete (status=0 '\000', 
>> req=0x7ffe782c7ce0) at /home/alex/lsrc/qemu/qemu.git/hw/block/virtio-blk.c:64
>> #13 virtio_blk_rw_complete (opaque=<optimised out>, ret=0) at 
>> /home/alex/lsrc/qemu/qemu.git/hw/block/virtio-blk.c:122
>> ---Type <return> to continue, or q <return> to quit---
>> #14 0x0000555555a2d822 in bdrv_co_complete (acb=0x7ffe780189c0) at 
>> block/io.c:2122
>> #15 0x0000555555a87a7a in coroutine_trampoline (i0=<optimised out>, 
>> i1=<optimised out>) at util/coroutine-ucontext.c:80
>> #16 0x00007ffff0afc8b0 in ?? () from /lib/x86_64-linux-gnu/libc.so.6
>> #17 0x00007fff8f5aa6e0 in ?? ()
>> #18 0x0000000000000000 in ?? ()
>>
>> I guess the tb_lock could just be grabbed but there is stuff in that
>> path that assumes current_cpu is valid so I thought the thing to do was
>> defer the operation until a "real" vCPU can deal with it.
>
> I need to look at the branch...  The latest version I have here does
> not require tb_lock taken in tb_invalidate_phys_range.

The tb_locks asserts where added in Fred's branch which makes sense as
we are going to mess with the translation block cache. Looking more
closely at tb_invalidate_phys_page_range I see it jumps through some
hoops when cpu == current_cpu == NULL.


>
> /*
>  * Invalidate all TBs which intersect with the target physical address range
>  * [start;end[. NOTE: start and end may refer to *different* physical pages.
>  * 'is_cpu_write_access' should be true if called from a real cpu write
>  * access: the virtual CPU will exit the current TB if code is modified inside
>  * this TB.
>  *
>  * Called with mmap_lock held for user-mode emulation
>  */
> void tb_invalidate_phys_range(tb_page_addr_t start, tb_page_addr_t end)
> {
>     while (start < end) {
>         tb_invalidate_phys_page_range(start, end, 0);
>         start &= TARGET_PAGE_MASK;
>         start += TARGET_PAGE_SIZE;
>     }
> }
>
> /*
>  * Invalidate all TBs which intersect with the target physical address range
>  * [start;end[. NOTE: start and end must refer to the *same* physical page.
>  * 'is_cpu_write_access' should be true if called from a real cpu write
>  * access: the virtual CPU will exit the current TB if code is modified inside
>  * this TB.
>  *
>  * Called with mmap_lock held for user-mode emulation
>  * If called from generated code, iothread mutex must not be held.
>  */
> void tb_invalidate_phys_page_range(tb_page_addr_t start, tb_page_addr_t end,
>                                    int is_cpu_write_access)
>
>
> Paolo


--
Alex Bennée



reply via email to

[Prev in Thread] Current Thread [Next in Thread]