Re: [PATCH v3 1/3] qapi: Moved architecture agnostic data types to `machine`

[Markus Armbruster]

Dinah B <dinahbaum123@gmail.com> writes:

> Hi,
>
> I noticed that qapi now has a machine-common category - do you think these
> changes would be more appropriate in that file
> rather than "machine" for the revision?

You're right.

We created machine-common.json for stuff needed by both machine.json and
machine-target.json.  The alternative was to have machine-target.json
include machine.json instead.  When we discussed which alternative to
pick, I didn't remember your patch adds the include.

Picking both makes no sense.  Please look into adding stuff in a way
that avoids the include.

Thanks for paying attention!

> Thanks and sorry for the delay,

I'm not in a hurry here :)