[PATCH 01/12] hyperv: XML parsing of serial ports
Michal Privoznik
mprivozn at redhat.com
Mon Feb 1 16:16:39 UTC 2021
On 1/22/21 9:18 PM, Matt Coleman wrote:
> Co-authored-by: Sri Ramanujam <sramanujam at datto.com>
> Signed-off-by: Matt Coleman <matt at datto.com>
> ---
> src/hyperv/hyperv_driver.c | 68 +++++++++++++++++++++++++++
> src/hyperv/hyperv_wmi.c | 10 ++++
> src/hyperv/hyperv_wmi.h | 4 ++
> src/hyperv/hyperv_wmi_classes.h | 1 +
> src/hyperv/hyperv_wmi_generator.input | 5 ++
> 5 files changed, 88 insertions(+)
>
> diff --git a/src/hyperv/hyperv_driver.c b/src/hyperv/hyperv_driver.c
> index bdc084790a..9bbbcfc88c 100644
> --- a/src/hyperv/hyperv_driver.c
> +++ b/src/hyperv/hyperv_driver.c
> @@ -1253,6 +1253,61 @@ hypervDomainDefParseStorage(hypervPrivate *priv,
> }
>
>
> +static int
> +hypervDomainDefParseSerial(virDomainDefPtr def, Msvm_ResourceAllocationSettingData *rasd)
> +{
> + int port_num = 0;
> + char **conn = NULL;
> + const char *srcPath = NULL;
> + Msvm_ResourceAllocationSettingData *entry = rasd;
> + virDomainChrDefPtr serial = NULL;
> +
> + while (entry) {
> + if (entry->data->ResourceType == MSVM_RASD_RESOURCETYPE_SERIAL_PORT) {
> + /* clear some vars */
> + serial = NULL;
> + port_num = 0;
> + conn = NULL;
> + srcPath = NULL;
Well, you don't need to clear them if they'd be declared inside the loop :-)
> +
> + /* get port number */
> + port_num = entry->data->ElementName[4] - '0';
> + if (port_num < 1) {
> + entry = entry->next;
> + continue;
> + }
> +
> + serial = virDomainChrDefNew(NULL);
> +
> + serial->deviceType = VIR_DOMAIN_CHR_DEVICE_TYPE_SERIAL;
> + serial->source->type = VIR_DOMAIN_CHR_TYPE_PIPE;
> + serial->target.port = port_num;
> +
> + /* set up source */
> + if (entry->data->Connection.count < 1) {
> + srcPath = "-1";
> + } else {
> + conn = entry->data->Connection.data;
> + if (!*conn)
> + srcPath = "-1";
> + else
> + srcPath = *conn;
> + }
> +
> + serial->source->data.file.path = g_strdup(srcPath);
> +
> + if (VIR_APPEND_ELEMENT(def->serials, def->nserials, serial) < 0) {
> + virDomainChrDefFree(serial);
> + return -1;
> + }
> + }
> +
> + entry = entry->next;
> + }
> +
> + return 0;
> +}
> +
>
Michal
More information about the libvir-list
mailing list