Re: [PATCH v2 1/3] monitor: Add HMP and QMP interfaces

[Paolo Bonzini]

On 13/09/21 12:37, Yang Zhong wrote:
> > > +{ 'struct': 'SGXInfo',
> > > +  'data': { 'sgx': 'bool',
> > > +            'sgx1': 'bool',
> > > +            'sgx2': 'bool',
> > > +            'flc': 'bool',
> > > +            'section-size': 'uint64'},
> > > +   'if': 'TARGET_I386' }
> >
> > Is it possible to restrict it to KVM? Maybe:
> >
> >       'if': { 'all': ['TARGET_I386', 'CONFIG_KVM'] } },
> >
> > ? (I'm not sure).
>
>    Philippe, i tried this definition, which is feasible.
>    This seems more accurate for sgx in the kvm of i386 platform. thanks!

The definition is needed in the stubs as well (cross-compilation 
currently fails due to missing sgx_get_{info,capabilities}), so I think 
this doesn't work.

Paolo