On Fri, May 08, 2026 at 07:18:29PM +0400, [email protected] wrote:
> On Fri, 08 May 2026 12:24:20 +0100, Daniel P. Berrangé <[email protected]> 
> wrote:
> > diff --git a/qom/trace-events b/qom/trace-events
> > index b2e9f4a7127..9aabbae4f73 100644
> > --- a/qom/trace-events
> > +++ b/qom/trace-events
> > @@ -1,5 +1,13 @@
> >  # See docs/devel/tracing.rst for syntax documentation.
> >  
> >  # object.c
> > -object_dynamic_cast_assert(const char *type, const char *target, const 
> > char *file, int line, const char *func) "%s->%s (%s:%d:%s)"
> > -object_class_dynamic_cast_assert(const char *type, const char *target, 
> > const char *file, int line, const char *func) "%s->%s (%s:%d:%s)"
> > +object_dynamic_cast_assert(void *obj, const char *type, const char 
> > *target, const char *file, int line, const char *func) "obj=%p type=%s->%s 
> > (%s:%d:%s)"
> > +object_finalize(void *obj, const char *type) "obj=%p type=%s"
> > +object_new(void *obj, const char *type) "obj=%p type=%s"
> > +object_property_add(void *obj, const char *type, const char *name, void 
> > *value) "obj=%p type=%s name=%s value=%p"
> > +object_property_add_child(void *obj, const char *type, const char *name, 
> > void *parent, const char *parenttype) "obj=%p type=%s name=%s parent=%p 
> > parent-type=%s"
> 
> The 4th and 5th parameters are the child being added, not a parent.
> In object_property_try_add_child the call is:
> 
>   trace_object_property_add_child(obj, ..., child, child->class->type->name)
> 
> So the parameter names should be "child"/"childtype" and the format
> string should use "child=" / "child-type=".
> 
> > +object_property_del(void *obj, const char *type, const char *name, void 
> > *value) "obj=%p type=%s name=%s value=%p"
> > +object_property_del_child(void *obj, const char *type, void *parent, const 
> > char *parenttype) "obj=%p type=%s parent=%p parent-type=%s"
> 
> Same here

Opps, yes, I don't know what I was thinking there !

With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|


Reply via email to