From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:38955) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UY60U-0000v2-6u for qemu-devel@nongnu.org; Thu, 02 May 2013 22:45:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UY60T-0002MD-Ce for qemu-devel@nongnu.org; Thu, 02 May 2013 22:45:58 -0400 Received: from mail-qa0-x22b.google.com ([2607:f8b0:400d:c00::22b]:58883) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UY60T-0002M5-6I for qemu-devel@nongnu.org; Thu, 02 May 2013 22:45:57 -0400 Received: by mail-qa0-f43.google.com with SMTP id bs12so142676qab.2 for ; Thu, 02 May 2013 19:45:56 -0700 (PDT) From: Liu Ping Fan Date: Fri, 3 May 2013 10:45:19 +0800 Message-Id: <1367549122-2948-4-git-send-email-qemulist@gmail.com> In-Reply-To: <1367549122-2948-1-git-send-email-qemulist@gmail.com> References: <1367549122-2948-1-git-send-email-qemulist@gmail.com> Subject: [Qemu-devel] [PATCH v2 3/6] memory: add ref/unref interface for MemroyRegionOps List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Peter Maydell , Anthony Liguori , Jan Kiszka , Vasilis Liaskovitis , Stefan Hajnoczi , Paolo Bonzini From: Liu Ping Fan This pair of interface are optinal, except for those device which is used outside the biglock's protection for hot unplug. Currently, HostMem used by virtio-blk dataplane is outside biglock, so the RAM device should implement this. Signed-off-by: Liu Ping Fan --- include/exec/memory.h | 10 ++++++++++ memory.c | 18 ++++++++++++++++++ 2 files changed, 28 insertions(+), 0 deletions(-) diff --git a/include/exec/memory.h b/include/exec/memory.h index 2761668..edeb480 100644 --- a/include/exec/memory.h +++ b/include/exec/memory.h @@ -54,6 +54,12 @@ struct MemoryRegionIORange { * Memory region callbacks */ struct MemoryRegionOps { + + /* ref/unref pair is optional; ref. + * inc refcnt of object who store MemoryRegion + */ + void (*ref)(void); + void (*unref)(void); /* Read from the memory region. @addr is relative to @mr; @size is * in bytes. */ uint64_t (*read)(void *opaque, @@ -223,6 +229,10 @@ struct MemoryListener { QTAILQ_ENTRY(MemoryListener) link; }; +/**/ +bool memory_region_ref(MemoryRegion *mr); +bool memory_region_unref(MemoryRegion *mr); + /** * memory_region_init: Initialize a memory region * diff --git a/memory.c b/memory.c index 75ca281..c29998d 100644 --- a/memory.c +++ b/memory.c @@ -786,6 +786,24 @@ static bool memory_region_wrong_endianness(MemoryRegion *mr) #endif } +bool memory_region_ref(MemoryRegion *mr) +{ + if (mr->ops && mr->ops->ref) { + mr->ops->ref(); + return true; + } + return false; +} + +bool memory_region_unref(MemoryRegion *mr) +{ + if (mr->ops && mr->ops->unref) { + mr->ops->unref(); + return true; + } + return false; +} + void memory_region_init(MemoryRegion *mr, const char *name, uint64_t size) -- 1.7.4.4