[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mut
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread |
Date: |
Fri, 22 Jun 2012 12:37:27 +0200 |
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-06-22 12:24, liu ping fan wrote:
> On Thu, Jun 21, 2012 at 11:23 PM, Jan Kiszka <address@hidden> wrote:
>> On 2012-06-21 16:49, Liu Ping Fan wrote:
>>> Nowadays, we use qemu_mutex_lock_iothread()/qemu_mutex_unlock_iothread() to
>>> protect the race to access the emulated dev launched by vcpu threads &
>>> iothread.
>>>
>>> But this lock is too big. We can break it down.
>>> These patches separate the CPUArchState's protection from the other
>>> devices, so we
>>> can have a per-cpu lock for each CPUArchState, not the big lock any longer.
>>
>> Anything that reduces lock dependencies is generally welcome. But can
>> you specify in more details what you gain, and under which conditions?
>>
> In fact, there are several steps to break down the Qemu big lock. This
> step just aims to shrink the code area protected by
> qemu_mutex_lock_iothread()/qemu_mutex_unlock_iothread(). And I am
> working on the following steps, which focus on breaking down the big
> lock when calling handle_{io,mmio}
Then let us discuss the strategy. This is important as it is unrealistic
to break up the lock for all code paths. We really need to focus on
goals that provide benefits for relevant use cases.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] [PATCH 1/2] CPUArchState: introduce per-cpu lock, (continued)
- [Qemu-devel] [PATCH 1/2] CPUArchState: introduce per-cpu lock, Liu Ping Fan, 2012/06/21
- [Qemu-devel] [PATCH 2/2] kvm: use per-cpu lock to free vcpu thread out of the big lock, Liu Ping Fan, 2012/06/21
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, Jan Kiszka, 2012/06/21
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, liu ping fan, 2012/06/22
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread,
Jan Kiszka <=
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, Anthony Liguori, 2012/06/22
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, Jan Kiszka, 2012/06/22
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, Anthony Liguori, 2012/06/22
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, Jan Kiszka, 2012/06/22
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, Anthony Liguori, 2012/06/22
- Re: [Qemu-devel] [RFC] use little granularity lock to substitue qemu_mutex_lock_iothread, Jan Kiszka, 2012/06/23