Re: [Qemu-devel] [PATCH v6 0/8] push mmio dispatch out of big lock

[liu ping fan <qemulist@gmail.com>]

Ping? Any further comments?

Thanks and regards,
Pingfan

On Mon, Nov 5, 2012 at 3:00 PM, Jan Kiszka <jan.kiszka@web.de> wrote:
> On 2012-11-05 06:38, Liu Ping Fan wrote:
>> From: Liu Ping Fan <pingfank@linux.vnet.ibm.com>
>>
>> v1:
>> https://lists.gnu.org/archive/html/qemu-devel/2012-07/msg03312.html
>>
>> v2:
>> http://lists.gnu.org/archive/html/qemu-devel/2012-08/msg01275.html
>>
>> v3:
>> http://lists.nongnu.org/archive/html/qemu-devel/2012-09/msg01474.html
>>
>> v4:
>> http://lists.gnu.org/archive/html/qemu-devel/2012-10/msg03857.html
>>
>> v5:
>> https://lists.gnu.org/archive/html/qemu-devel/2012-10/msg04867.html
>>
>> changes v5->v6:
>>  Apply fine-grain lock for all address space.
>>  Introduce separated interface to allow mmio dispatcher called with/without big lock.
>>
>> Liu Ping Fan (8):
>>   atomic: introduce atomic operations
>>   qom: apply atomic on object's refcount
>>   hotplug: introduce qdev_unplug_complete() to remove device from views
>>   pci: remove pci device from mem view when unplug
>>   memory: introduce local lock for address space
>>   memory: make mmio dispatch able to be out of biglock
>>   memory: introduce tls context to trace nested mmio request issue
>>   vcpu: push mmio dispatcher out of big lock
>>
>>  cpu-common.h          |    3 +
>>  docs/memory.txt       |    4 +
>>  exec.c                |  219 +++++++++++++++++++++++++++++++++++++++++++++----
>>  hw/acpi_piix4.c       |    2 +-
>>  hw/pci.c              |   13 +++-
>>  hw/pci.h              |    1 +
>>  hw/qdev.c             |   26 ++++++
>>  hw/qdev.h             |    3 +-
>>  include/qemu/atomic.h |   63 ++++++++++++++
>>  include/qemu/object.h |    3 +-
>>  kvm-all.c             |    6 +-
>>  memory-internal.h     |    1 +
>>  memory.c              |    1 +
>>  memory.h              |    5 +
>>  qemu-thread.h         |    7 ++
>>  qom/object.c          |   11 +--
>>  16 files changed, 340 insertions(+), 28 deletions(-)
>>  create mode 100644 include/qemu/atomic.h
>>
>
> Very good! My feeling is we are getting closer.
>
> There are some minor style issues I'm not yet commenting on. We can go
> through this once everyone is happy with the design.
>
> Jan
>