Re: [Qemu-devel] [PATCH 1/1] Added address_space_init2().

2013-01-08 Thread Alexander Barabash
Hi, On 01/07/2013 03:05 PM, Andreas Färber wrote: A memory: prefix in the subject would've been nice for filtering. OK Am 07.01.2013 13:07, schrieb Alexander Barabash: address_space_init2: initializes a named address space. What for? There are no users in this patch that justify its

Re: [Qemu-devel] [PATCH 1/1] Support abstract socket namespace in AF_UNIX socket family.

2013-01-08 Thread Alexander Barabash
Hi, On 01/07/2013 04:07 PM, Anthony Liguori wrote: Alexander Barabash alexander_barab...@mentor.com writes: The abstract socket namespace is a nonportable Linux extension. The sockets' names in this namespace have no connection with file system pathnames. To specify a named AF_UNIX socket

[Qemu-devel] [PATCH 1/1] memory: add name in AddressSpace initialization.

2013-01-08 Thread Alexander Barabash
Pass the AddressSpace's name (to be used for debugging) to address_space_init(). If NULL is passed, the name of root memory region is used instead. Signed-off-by: Alexander Barabash alexander_barab...@mentor.com --- exec.c|6 ++ hw/pci/pci.c |3 ++- include

Re: [Qemu-devel] [PATCH 1/1] memory: add name in AddressSpace initialization.

2013-01-08 Thread Alexander Barabash
On 01/08/2013 04:00 PM, Anthony Liguori wrote: Alexander Barabash alexander_barab...@mentor.com writes: Pass the AddressSpace's name (to be used for debugging) to address_space_init(). If NULL is passed, the name of root memory region is used instead. @@ -1784,13 +1784,11 @@ static void

[Qemu-devel] [PATCH v2 1/1] chardev: Support abstract socket namespace in AF_UNIX socket family.

2013-01-08 Thread Alexander Barabash
=NAME_OF_THE_SOCKET,abstract=on Signed-off-by: Alexander Barabash alexander_barab...@mentor.com --- qemu-config.c |3 +++ qemu-sockets.c | 35 ++- 2 files changed, 33 insertions(+), 5 deletions(-) diff --git a/qemu-config.c b/qemu-config.c index 2188c3e

[Qemu-devel] [PATCH v3 1/1] memory: add name in AddressSpace initialization.

2013-01-08 Thread Alexander Barabash
Pass the AddressSpace's name (to be used for debugging) to address_space_init(). If NULL is passed, the name of root memory region is used instead. Signed-off-by: Alexander Barabash alexander_barab...@mentor.com --- exec.c|6 ++ hw/pci/pci.c |3 ++- include

[Qemu-devel] [PATCH 1/1] Added address_space_init2().

2013-01-07 Thread Alexander Barabash
address_space_init2: initializes a named address space. Signed-off-by: Alexander Barabash alexander_barab...@mentor.com --- include/exec/memory.h |9 + memory.c |6 ++ 2 files changed, 15 insertions(+) diff --git a/include/exec/memory.h b/include/exec/memory.h

[Qemu-devel] [PATCH 1/1] Support 2**64 bytes memory regions.

2013-01-07 Thread Alexander Barabash
Memory regions created with size UINT64_MAX are currently treated as regions of size 2**64 bytes. This patch adds full support for such regions. Signed-off-by: Alexander Barabash alexander_barab...@mentor.com --- exec.c |4 memory.c | 72

[Qemu-devel] [PATCH 1/1] Support abstract socket namespace in AF_UNIX socket family.

2013-01-07 Thread Alexander Barabash
-by: Alexander Barabash alexander_barab...@mentor.com --- qemu-sockets.c | 58 +--- 1 file changed, 55 insertions(+), 3 deletions(-) diff --git a/qemu-sockets.c b/qemu-sockets.c index 3537bf3..70c8ad5 100644 --- a/qemu-sockets.c +++ b/qemu

[Qemu-devel] [PATCH 1/1] Support gdbstub qXfer:spaces features.

2013-01-07 Thread Alexander Barabash
Support qXfer:spaces:read and qXfer:spaces:write. These gdbstub features allow GDB to access target's hardware registers. Signed-off-by: Alexander Barabash alexander_barab...@mentor.com --- Makefile.target |1 + gdbstub.c | 176 ++- xfer-spaces.c | 419

Re: [Qemu-devel] [PATCH] qom: Make object_unref() free the object's memory when refcount goes to 0.

2012-02-26 Thread Alexander Barabash
On 02/24/2012 05:11 PM, Anthony Liguori wrote: On 02/23/2012 10:21 AM, Alexander Barabash wrote: On 02/22/2012 09:12 PM, Anthony Liguori wrote: On 02/22/2012 12:00 PM, alexander_barab...@mentor.com wrote: From: Alexander Barabashalexander_barab...@mentor.com Why do you want to have a delete

Re: [Qemu-devel] [PATCH] qom: Make object_unref() free the object's memory when refcount goes to 0.

2012-02-23 Thread Alexander Barabash
On 02/22/2012 09:12 PM, Anthony Liguori wrote: On 02/22/2012 12:00 PM, alexander_barab...@mentor.com wrote: From: Alexander Barabashalexander_barab...@mentor.com In the existing implementation, object_delete() calls object_unref(), then frees the object's storage. Running object_delete() on an

Re: [Qemu-devel] [PATCH] qom: In function object_set_link_property(), first call object_ref(), then object_unref().

2012-02-22 Thread Alexander Barabash
On 02/22/2012 07:17 PM, Paolo Bonzini wrote: On 02/22/2012 06:13 PM, alexander_barab...@mentor.com wrote: From: Alexander Barabashalexander_barab...@mentor.com In the old implementation, if the new value of the property links to the same object, as the old value, that object is first unref-ed,

Re: [Qemu-devel] [PATCH] qom: In function object_set_link_property(), first call object_ref(), then object_unref().

2012-02-22 Thread Alexander Barabash
On 02/22/2012 07:24 PM, Paolo Bonzini wrote: On 02/22/2012 06:22 PM, alexander_barab...@mentor.com wrote: From: Alexander Barabashalexander_barab...@mentor.com In the old implementation, if the new value of the property links to the same object, as the old value, that object is first unref-ed,

Re: [Qemu-devel] [PATCH] Add object_property_get_child().

2012-02-20 Thread Alexander Barabash
On 02/20/2012 11:11 AM, Paolo Bonzini wrote: On 02/19/2012 01:36 PM, Alexander Barabash wrote: The proposed object_property_get_child() may return either the direct child with the specified name in the composition tree, or the value of the link with the specified name

Re: [Qemu-devel] [PATCH v2] Revised: Add object_property_get_child().

2012-02-20 Thread Alexander Barabash
On 02/19/2012 06:04 PM, Alexander Barabash wrote: Add object_property_get_child(). Please disregard this patch. object_property_get_link() works for this purpose.

Re: [Qemu-devel] [PATCH v2] Revised: Add object_property_get_child().

2012-02-20 Thread Alexander Barabash
On 02/19/2012 07:14 PM, Andreas Färber wrote: Am 19.02.2012 17:04, schrieb Alexander Barabash: ... Signed-off-by: Alexander Barabashalexander_barab...@mentor.com Please use git-send-email to submit your patches: The commit message is unnecessarily indented and the first line is duplicated

Re: [Qemu-devel] [PATCH] Add object_property_get_child().

2012-02-19 Thread Alexander Barabash
On 02/17/2012 12:17 PM, Paolo Bonzini wrote: Besides the non-triviality of the patch, how is this different from object_property_get_link? Perhaps we should just rename that one to object_property_get_obj, or add a function that is just a synonym. As for the patch's (non-)triviality, I just

[Qemu-devel] [PATCH v2] Revised: Add object_property_get_child().

2012-02-19 Thread Alexander Barabash
, QERR_OBJECT_PROPERTY_NOT_FOUND and QERR_OBJECT_PROPERTY_INVALID_TYPE were added. Also, in the new implementation, some common sense refactoring was done in the file 'qom/object.c' in the code extracting child and link properties. Signed-off-by: Alexander Barabash alexander_barab...@mentor.com

[Qemu-devel] [PATCH] Add object_property_get_child().

2012-02-16 Thread Alexander Barabash
, some common sense refactoring was done in the file 'qom/object.c' in the code extracting child and link properties. Signed-off-by: Alexander Barabash alexander_barab...@mentor.com diff --git a/include/qemu/object.h b/include/qemu/object.h index e7e32fe..190f422 100644 --- a/include/qemu

[Qemu-devel] [PATCH] [TRIVIAL] Removed unused arm_sysctl_init().

2012-02-16 Thread Alexander Barabash
Removed unused arm_sysctl_init(). Renamed arm_sysctl_init1() into arm_sysctl_init(). Signed-off-by: Alexander Barabash alexander_barab...@mentor.com diff --git a/hw/arm_sysctl.c b/hw/arm_sysctl.c index 149c639..5f1237b 100644 --- a/hw/arm_sysctl.c +++ b/hw/arm_sysctl.c @@ -378,7

[Qemu-devel] [PATCH] [TRIVIAL] Removed unused pl080_init().

2012-02-16 Thread Alexander Barabash
Removed unused pl080_init(). Signed-off-by: Alexander Barabash alexander_barab...@mentor.com diff --git a/hw/primecell.h b/hw/primecell.h index de7d6f2..ded0446 100644 --- a/hw/primecell.h +++ b/hw/primecell.h @@ -5,9 +5,6 @@ /* Also includes some devices that are currently only used

[Qemu-devel] [PATCH] [TRIVIAL] Replace object_delete() with object_unref().

2012-02-16 Thread Alexander Barabash
(). Signed-off-by: Alexander Barabash alexander_barab...@mentor.com diff --git a/hw/qdev.c b/hw/qdev.c index f0eb3a7..891981a 100644 --- a/hw/qdev.c +++ b/hw/qdev.c @@ -247,7 +247,7 @@ void qdev_init_nofail(DeviceState *dev) /* Unlink device from bus and free the structure. */ void qdev_free