conf: Move some members of virDomainSEVDef into virDomainSEVCommonDef

Some parts of SEV are to be shared with SEV SNP. In order to
reuse XML parsing / formatting code cleanly, let's move those
common bits into a new struct (virDomainSEVCommonDef) and adjust
rest of the code.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
This commit is contained in:
Michal Privoznik 2024-06-11 12:12:08 +02:00
parent 66efdfabd9
commit d2cad18ca3
7 changed files with 74 additions and 42 deletions

View File

@ -13621,8 +13621,8 @@ virDomainMemoryTargetDefParseXML(xmlNodePtr node,
static int
virDomainSEVDefParseXML(virDomainSEVDef *def,
xmlXPathContextPtr ctxt)
virDomainSEVCommonDefParseXML(virDomainSEVCommonDef *def,
xmlXPathContextPtr ctxt)
{
int rc;
@ -13630,12 +13630,6 @@ virDomainSEVDefParseXML(virDomainSEVDef *def,
&def->kernel_hashes) < 0)
return -1;
if (virXPathUIntBase("string(./policy)", ctxt, 16, &def->policy) < 0) {
virReportError(VIR_ERR_XML_ERROR, "%s",
_("failed to get launch security policy"));
return -1;
}
/* the following attributes are platform dependent and if missing, we can
* autofill them from domain capabilities later
*/
@ -13658,6 +13652,23 @@ virDomainSEVDefParseXML(virDomainSEVDef *def,
return -1;
}
return 0;
}
static int
virDomainSEVDefParseXML(virDomainSEVDef *def,
xmlXPathContextPtr ctxt)
{
if (virDomainSEVCommonDefParseXML(&def->common, ctxt) < 0)
return -1;
if (virXPathUIntBase("string(./policy)", ctxt, 16, &def->policy) < 0) {
virReportError(VIR_ERR_XML_ERROR, "%s",
_("failed to get launch security policy"));
return -1;
}
def->dh_cert = virXPathString("string(./dhCert)", ctxt);
def->session = virXPathString("string(./session)", ctxt);
@ -26641,6 +26652,24 @@ virDomainKeyWrapDefFormat(virBuffer *buf, virDomainKeyWrapDef *keywrap)
}
static void
virDomainSEVCommonDefFormat(virBuffer *attrBuf,
virBuffer *childBuf,
virDomainSEVCommonDef *def)
{
if (def->kernel_hashes != VIR_TRISTATE_BOOL_ABSENT)
virBufferAsprintf(attrBuf, " kernelHashes='%s'",
virTristateBoolTypeToString(def->kernel_hashes));
if (def->haveCbitpos)
virBufferAsprintf(childBuf, "<cbitpos>%d</cbitpos>\n", def->cbitpos);
if (def->haveReducedPhysBits)
virBufferAsprintf(childBuf, "<reducedPhysBits>%d</reducedPhysBits>\n",
def->reduced_phys_bits);
}
static void
virDomainSecDefFormat(virBuffer *buf, virDomainSecDef *sec)
{
@ -26657,16 +26686,8 @@ virDomainSecDefFormat(virBuffer *buf, virDomainSecDef *sec)
case VIR_DOMAIN_LAUNCH_SECURITY_SEV: {
virDomainSEVDef *sev = &sec->data.sev;
if (sev->kernel_hashes != VIR_TRISTATE_BOOL_ABSENT)
virBufferAsprintf(&attrBuf, " kernelHashes='%s'",
virTristateBoolTypeToString(sev->kernel_hashes));
virDomainSEVCommonDefFormat(&attrBuf, &childBuf, &sev->common);
if (sev->haveCbitpos)
virBufferAsprintf(&childBuf, "<cbitpos>%d</cbitpos>\n", sev->cbitpos);
if (sev->haveReducedPhysBits)
virBufferAsprintf(&childBuf, "<reducedPhysBits>%d</reducedPhysBits>\n",
sev->reduced_phys_bits);
virBufferAsprintf(&childBuf, "<policy>0x%04x</policy>\n", sev->policy);
virBufferEscapeString(&childBuf, "<dhCert>%s</dhCert>\n", sev->dh_cert);

View File

@ -2866,10 +2866,7 @@ typedef enum {
} virDomainLaunchSecurity;
struct _virDomainSEVDef {
char *dh_cert;
char *session;
unsigned int policy;
struct _virDomainSEVCommonDef {
bool haveCbitpos;
unsigned int cbitpos;
bool haveReducedPhysBits;
@ -2877,6 +2874,14 @@ struct _virDomainSEVDef {
virTristateBool kernel_hashes;
};
struct _virDomainSEVDef {
virDomainSEVCommonDef common;
char *dh_cert;
char *session;
unsigned int policy;
};
struct _virDomainSecDef {
virDomainLaunchSecurity sectype;
union {

View File

@ -524,6 +524,19 @@
</element>
</define>
<define name="launchSecuritySEVCommon">
<optional>
<element name="cbitpos">
<data type="unsignedInt"/>
</element>
</optional>
<optional>
<element name="reducedPhysBits">
<data type="unsignedInt"/>
</element>
</optional>
</define>
<define name="launchSecuritySEV">
<attribute name="type">
<value>sev</value>
@ -534,16 +547,7 @@
</attribute>
</optional>
<interleave>
<optional>
<element name="cbitpos">
<data type="unsignedInt"/>
</element>
</optional>
<optional>
<element name="reducedPhysBits">
<data type="unsignedInt"/>
</element>
</optional>
<ref name="launchSecuritySEVCommon"/>
<element name="policy">
<ref name="hexuint"/>
</element>

View File

@ -210,6 +210,8 @@ typedef struct _virDomainResctrlMonDef virDomainResctrlMonDef;
typedef struct _virDomainResourceDef virDomainResourceDef;
typedef struct _virDomainSEVCommonDef virDomainSEVCommonDef;
typedef struct _virDomainSEVDef virDomainSEVDef;
typedef struct _virDomainSecDef virDomainSecDef;

View File

@ -9728,7 +9728,7 @@ qemuBuildSEVCommandLine(virDomainObj *vm, virCommand *cmd,
g_autofree char *sessionpath = NULL;
VIR_DEBUG("policy=0x%x cbitpos=%d reduced_phys_bits=%d",
sev->policy, sev->cbitpos, sev->reduced_phys_bits);
sev->policy, sev->common.cbitpos, sev->common.reduced_phys_bits);
if (sev->dh_cert)
dhpath = g_strdup_printf("%s/dh_cert.base64", priv->libDir);
@ -9737,12 +9737,12 @@ qemuBuildSEVCommandLine(virDomainObj *vm, virCommand *cmd,
sessionpath = g_strdup_printf("%s/session.base64", priv->libDir);
if (qemuMonitorCreateObjectProps(&props, "sev-guest", "lsec0",
"u:cbitpos", sev->cbitpos,
"u:reduced-phys-bits", sev->reduced_phys_bits,
"u:cbitpos", sev->common.cbitpos,
"u:reduced-phys-bits", sev->common.reduced_phys_bits,
"u:policy", sev->policy,
"S:dh-cert-file", dhpath,
"S:session-file", sessionpath,
"T:kernel-hashes", sev->kernel_hashes,
"T:kernel-hashes", sev->common.kernel_hashes,
NULL) < 0)
return -1;

View File

@ -6569,14 +6569,14 @@ qemuProcessUpdateSEVInfo(virDomainObj *vm)
* mandatory on QEMU cmdline
*/
sevCaps = virQEMUCapsGetSEVCapabilities(qemuCaps);
if (!sev->haveCbitpos) {
sev->cbitpos = sevCaps->cbitpos;
sev->haveCbitpos = true;
if (!sev->common.haveCbitpos) {
sev->common.cbitpos = sevCaps->cbitpos;
sev->common.haveCbitpos = true;
}
if (!sev->haveReducedPhysBits) {
sev->reduced_phys_bits = sevCaps->reduced_phys_bits;
sev->haveReducedPhysBits = true;
if (!sev->common.haveReducedPhysBits) {
sev->common.reduced_phys_bits = sevCaps->reduced_phys_bits;
sev->common.haveReducedPhysBits = true;
}
return 0;

View File

@ -1318,7 +1318,7 @@ qemuValidateDomainDef(const virDomainDef *def,
return -1;
}
if (def->sec->data.sev.kernel_hashes != VIR_TRISTATE_BOOL_ABSENT &&
if (def->sec->data.sev.common.kernel_hashes != VIR_TRISTATE_BOOL_ABSENT &&
!virQEMUCapsGet(qemuCaps, QEMU_CAPS_SEV_GUEST_KERNEL_HASHES)) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
_("SEV measured direct kernel boot is not supported with this QEMU binary"));