Re: [PATCH 11/15] qtest: add a QOM object for qtest

[Igor Mammedov <imammedo@redhat.com>]

On Mon, 7 Dec 2020 17:43:16 +0100
Paolo Bonzini <pbonzini@redhat.com> wrote:

> On 07/12/20 17:24, Igor Mammedov wrote:
> >> +void qtest_server_init(const char *qtest_chrdev, const char *qtest_log, Error **errp)
> >> +{
> >> +    Chardev *chr;
> >> +
> >> +    chr = qemu_chr_new("qtest", qtest_chrdev, NULL);
> >> +
> >> +    if (chr == NULL) {
> >> +        error_setg(errp, "Failed to initialize device for qtest: \"%s\"",
> >> +                   qtest_chrdev);
> >> +        return;
> >> +    }
> >> +
> >> +    qtest_server_start(chr, qtest_log, errp);  
> > why not create qtest object here instead of trying to preserve old way,
> > or create it directly at the place that calls qtest_server_init()?  
> 
> Because I wasn't sure of where to put it in the QOM object tree.  So I 
> punted and left it for later.

but you implicitly decided where it should be (with -object qtest),
it goes to /objects.
So I'd wouldn't put anywhere else to be consistent.

> 
> Paolo
>