[libvirt] [libvirt-go PATCH 2/2] Add support for AMD SEV platform info
Erik Skultety
eskultet at redhat.com
Mon Jun 18 06:26:03 UTC 2018
On Thu, Jun 14, 2018 at 04:52:04PM +0100, Daniel P. Berrangé wrote:
> On Thu, Jun 14, 2018 at 04:30:01PM +0200, Erik Skultety wrote:
> > Signed-off-by: Erik Skultety <eskultet at redhat.com>
> > ---
> > connect.go | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > connect_compat.go | 12 +++++++++++
> > connect_compat.h | 7 +++++++
> > 3 files changed, 78 insertions(+)
> >
> > diff --git a/connect.go b/connect.go
> > index e3e643e..8bb5fe6 100644
> > --- a/connect.go
> > +++ b/connect.go
> > @@ -2765,3 +2765,62 @@ func (c *Connect) GetAllDomainStats(doms []*Domain, statsTypes DomainStatsTypes,
> >
> > return stats, nil
> > }
> > +
> > +type NodeSEVParameters struct {
> > + PdhSet bool
> > + Pdh string
>
> Lets s/Pdh/PDH/ since its an acronym
>
> > + CertChainSet bool
> > + CertChain string
> > + CbitposSet bool
> > + Cbitpos uint
>
> and s/Cbitpos/CBitPos/
>
> > + ReducedPhysBitsSet bool
> > + ReducedPhysBits uint
> > +}
> > +
> > +func getNodeSEVFieldInfo(params *NodeSEVParameters) map[string]typedParamsFieldInfo {
> > + return map[string]typedParamsFieldInfo{
> > + C.VIR_NODE_SEV_PDH: typedParamsFieldInfo{
> > + set: ¶ms.PdhSet,
> > + s: ¶ms.Pdh,
> > + },
> > + C.VIR_NODE_SEV_CERT_CHAIN: typedParamsFieldInfo{
> > + set: ¶ms.CertChainSet,
> > + s: ¶ms.CertChain,
> > + },
> > + C.VIR_NODE_SEV_CBITPOS: typedParamsFieldInfo{
> > + set: ¶ms.CbitposSet,
> > + ui: ¶ms.Cbitpos,
> > + },
> > + C.VIR_NODE_SEV_REDUCED_PHYS_BITS: typedParamsFieldInfo{
> > + set: ¶ms.ReducedPhysBitsSet,
> > + ui: ¶ms.ReducedPhysBits,
> > + },
>
> Miinor nitpick on indentation - just run gofmt over it
>
>
>
> > diff --git a/connect_compat.go b/connect_compat.go
> > index 617bc4a..544def2 100644
> > --- a/connect_compat.go
> > +++ b/connect_compat.go
> > @@ -157,5 +157,17 @@ int virConnectCompareHypervisorCPUCompat(virConnectPtr conn,
> > #endif
> > }
> >
> > +int virNodeGetSEVInfoCompat(virConnectPtr conn,
> > + virTypedParameterPtr *params,
> > + int *nparams,
> > + unsigned int flags)
>
> Indent here too, though gofmt probably won't fix this since it is C code layer
>
> > +{
> > +#if LIBVIR_VERSION_NUMBER < 4005000
> > + assert(0); // Caller should have checked version
> > +#else
> > + return virNodeGetSEVInfo(conn, params, nparams, flags);
> > +#endif
> > +}
> > +
> > */
> > import "C"
>
> With the few nitpicks fixed
>
> Reviewed-by: Daniel P. Berrangé <berrange at redhat.com>
Fixed and pushed, thanks.
Erik
More information about the libvir-list
mailing list