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

[Jan Kiszka]

On 2012-11-05 06:38, Liu Ping Fan wrote:
> From: Liu Ping Fan <address@hidden>
> 
> 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

signature.asc
Description: OpenPGP digital signature