[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v2 2/5] vl: Make object_create() public
From: |
Jason Wang |
Subject: |
Re: [Qemu-devel] [PATCH RFC v2 2/5] vl: Make object_create() public |
Date: |
Mon, 1 Feb 2016 11:05:45 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 |
On 01/27/2016 04:29 PM, zhanghailiang wrote:
> Make the helper object_create() public and fix its first
> parameter to accept NULL value.
Looks not very nice. Maybe pass a new predicate func for sanity check it
better.
>
> Signed-off-by: zhanghailiang <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> ---
> v2:
> - New patch
> ---
> include/qemu-common.h | 2 ++
> vl.c | 4 ++--
> 2 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/include/qemu-common.h b/include/qemu-common.h
> index 22b010c..52cf4fd 100644
> --- a/include/qemu-common.h
> +++ b/include/qemu-common.h
> @@ -500,4 +500,6 @@ int parse_debug_env(const char *name, int max, int
> initial);
> const char *qemu_ether_ntoa(const MACAddr *mac);
> void page_size_init(void);
>
> +int object_create(void *opaque, QemuOpts *opts, Error **errp);
> +
> #endif
> diff --git a/vl.c b/vl.c
> index f043009..b21335e 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2819,7 +2819,7 @@ static bool object_create_delayed(const char *type)
> }
>
>
> -static int object_create(void *opaque, QemuOpts *opts, Error **errp)
> +int object_create(void *opaque, QemuOpts *opts, Error **errp)
> {
> Error *err = NULL;
> char *type = NULL;
> @@ -2842,7 +2842,7 @@ static int object_create(void *opaque, QemuOpts *opts,
> Error **errp)
> if (err) {
> goto out;
> }
> - if (!type_predicate(type)) {
> + if (type_predicate && !type_predicate(type)) {
> goto out;
> }
>
- [Qemu-devel] [PATCH RFC v2 0/5] Netfilter: Add each netdev a default filter, zhanghailiang, 2016/01/27
- [Qemu-devel] [PATCH RFC v2 4/5] filter-buffer: Accept zero interval, zhanghailiang, 2016/01/27
- [Qemu-devel] [PATCH RFC v2 1/5] net/filter: Add a 'status' property for filter object, zhanghailiang, 2016/01/27
- [Qemu-devel] [PATCH RFC v2 2/5] vl: Make object_create() public, zhanghailiang, 2016/01/27
- Re: [Qemu-devel] [PATCH RFC v2 2/5] vl: Make object_create() public,
Jason Wang <=
- [Qemu-devel] [PATCH RFC v2 3/5] net/filter: Introduce a helper to add a filter to the netdev, zhanghailiang, 2016/01/27
- [Qemu-devel] [PATCH RFC v2 5/5] net/filter: Add a default filter to each netdev, zhanghailiang, 2016/01/27