On Wed, Sep 14, 2016 at 11:16:12AM -0500, Brijesh Singh wrote:
> Hi Eric,
>
> Thanks for feedback.
>
> > > # @present: true if KVM acceleration is built into this executable
> > > #
> > > +# @sev: true if SEV is active
> >
> > Worth expanding what the acronym stands for. Also needs a '(since 2.8)'
> > designator.
> >
> will fix in v2.
>
> > > +#
> > > # Since: 0.14.0
> > > ##
> > > -{ 'struct': 'KvmInfo', 'data': {'enabled': 'bool', 'present': 'bool'} }
> > > +{ 'struct': 'KvmInfo', 'data': {'enabled': 'bool', 'present': 'bool',
> > > 'sev' : 'bool'} }
> >
> > Long line; please wrap to keep it under 80 columns.
> >
> will fix in v2.
So memory-encryption : bool
Etc everywhere.
--
MST
