[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2] qom: simplify object_find_property / object_class_find_pr
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2] qom: simplify object_find_property / object_class_find_property |
Date: |
Tue, 15 Sep 2020 09:53:10 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Daniel P. Berrangé <berrange@redhat.com> writes:
> When debugging QEMU it is often useful to put a breakpoint on the
> error_setg_internal method impl.
>
> Unfortunately the object_property_add / object_class_property_add
> methods call object_property_find / object_class_property_find methods
> to check if a property exists already before adding the new property.
>
> As a result there are a huge number of calls to error_setg_internal
> on startup of most QEMU commands, making it very painful to set a
> breakpoint on this method.
Work-around: make the breakpoint conditional on errp != NULL.
Use of error_propagate() can defeat the work-around, but doesn't here,
as far as I can tell.
> Most callers of object_find_property and object_class_find_property,
> however, pass in a NULL for the Error parameter. This simplifies the
> methods to remove the Error parameter entirely, and then adds some
> new wrapper methods that are able to raise an Error when needed.
I don't mind. Up to the QOM maintainers.
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>