On Thu, Jun 14, 2018 at 04:30:01PM +0200, Erik Skultety wrote:
Signed-off-by: Erik Skultety <eskultet(a)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(a)redhat.com>
Regards,
Daniel
--
|:
https://berrange.com -o-
https://www.flickr.com/photos/dberrange :|
|:
https://libvirt.org -o-
https://fstop138.berrange.com :|
|:
https://entangle-photo.org -o-
https://www.instagram.com/dberrange :|