[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [PATCH v4 1/9] qom: API to get instance_size of a type
From: |
David Gibson |
Subject: |
Re: [Qemu-ppc] [PATCH v4 1/9] qom: API to get instance_size of a type |
Date: |
Wed, 15 Jun 2016 15:48:23 +1000 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Fri, Jun 10, 2016 at 09:38:24AM +0200, Igor Mammedov wrote:
> On Fri, 10 Jun 2016 14:04:41 +1000
> David Gibson <address@hidden> wrote:
>
> > On Fri, Jun 10, 2016 at 06:29:00AM +0530, Bharata B Rao wrote:
> > > Add an API object_type_get_size(const char *typename) that returns the
> > > instance_size of the give typename.
> I'd rename it to
> object_type_get_instance_size()
> so it'd apparent what size is returned from name.
I'll make that change in my tree.
>
> > >
> > > Signed-off-by: Bharata B Rao <address@hidden>
> >
> > Reviewed-by: David Gibson <address@hidden>
> >
> > This looks sensible to me, it would be nice to have an ack or two from
> > the qemu community at large.
> >
> > > ---
> > > include/qom/object.h | 8 +++++++-
> > > qom/object.c | 8 ++++++++
> > > 2 files changed, 15 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/include/qom/object.h b/include/qom/object.h
> > > index 21bb5ff..460ddfc 100644
> > > --- a/include/qom/object.h
> > > +++ b/include/qom/object.h
> > > @@ -1608,5 +1608,11 @@ int object_child_foreach_recursive(Object *obj,
> > > */
> > > Object *container_get(Object *root, const char *path);
> > >
> > > -
> > > +/**
> > > + * object_type_get_size:
> > > + * @typename: Name of the Type whose instance_size is required
> > > + *
> > > + * Returns the instance_size of the given @typename.
> > > + */
> > > +size_t object_type_get_size(const char *typename);
> > > #endif
> > > diff --git a/qom/object.c b/qom/object.c
> > > index 3bc8a00..0e75877 100644
> > > --- a/qom/object.c
> > > +++ b/qom/object.c
> > > @@ -202,6 +202,14 @@ static size_t type_object_get_size(TypeImpl *ti)
> > > return 0;
> > > }
> > >
> > > +size_t object_type_get_size(const char *typename)
> > > +{
> > > + TypeImpl *type = type_get_by_name(typename);
> > > +
> > > + g_assert(type != NULL);
> > > + return type_object_get_size(type);
> > > +}
> > > +
> > > static bool type_is_ancestor(TypeImpl *type, TypeImpl *target_type)
> > > {
> > > assert(target_type);
> >
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- [Qemu-ppc] [PATCH v4 0/9] Core based CPU hotplug for PowerPC sPAPR, Bharata B Rao, 2016/06/09
- [Qemu-ppc] [PATCH v4 2/9] spapr: Abstract CPU core device and type specific core devices, Bharata B Rao, 2016/06/09
- [Qemu-ppc] [PATCH v4 4/9] spapr: convert boot CPUs into CPU core devices, Bharata B Rao, 2016/06/09
- [Qemu-ppc] [PATCH v4 3/9] spapr: Move spapr_cpu_init() to spapr_cpu_core.c, Bharata B Rao, 2016/06/09
- [Qemu-ppc] [PATCH v4 6/9] spapr: CPU hot unplug support, Bharata B Rao, 2016/06/09
- [Qemu-ppc] [PATCH v4 5/9] spapr: CPU hotplug support, Bharata B Rao, 2016/06/09
- [Qemu-ppc] [PATCH v4 8/9] hmp: Add 'info hotpluggable-cpus' HMP command, Bharata B Rao, 2016/06/09
- [Qemu-ppc] [PATCH v4 9/9] spapr: implement query-hotpluggable-cpus callback, Bharata B Rao, 2016/06/09