[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 11/15] qtest: add a QOM object for qtest
From: |
Igor Mammedov |
Subject: |
Re: [PATCH 11/15] qtest: add a QOM object for qtest |
Date: |
Mon, 7 Dec 2020 17:57:39 +0100 |
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
>
- Re: [PATCH 09/15] machine: record whether nvdimm= was set, (continued)
[PATCH 06/15] vl: move all generic initialization out of vl.c, Paolo Bonzini, 2020/12/02
[PATCH 10/15] vl: make qemu_get_machine_opts static, Paolo Bonzini, 2020/12/02