[PATCH] libxl: support soundhw for hvm domains

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The xend driver and the parsing/formating code in src/xenconfig
have long supported soundhw.  Add support in the libxl driver too.

Signed-off-by: Jim Fehlig <jfehlig@xxxxxxxx>
---

This patch continues the tradition of silently ignoring unsupported
config in the libxl driver.  But it is an improvement; post-patch
only sound devices after the first are ignored :-).

 src/libxl/libxl_conf.c | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/libxl/libxl_conf.c b/src/libxl/libxl_conf.c
index 2a09190..fccada5 100644
--- a/src/libxl/libxl_conf.c
+++ b/src/libxl/libxl_conf.c
@@ -678,6 +678,19 @@ libxlMakeDomBuildInfo(virDomainDefPtr def,
                 libxl_defbool_set(&b_info->u.hvm.hpet, 1);
             }
         }
+
+        if (def->nsounds > 0) {
+            /*
+             * Use first sound device.  man xl.cfg(5) describes soundhw as
+             * a single device.  From the man page: soundhw=DEVICE
+             */
+            virDomainSoundDefPtr snd = def->sounds[0];
+
+            if (VIR_STRDUP(b_info->u.hvm.soundhw,
+                           virDomainSoundModelTypeToString(snd->model)) < 0)
+                return -1;
+        }
+
         for (i = 0; i < def->os.nBootDevs; i++) {
             switch (def->os.bootDevs[i]) {
                 case VIR_DOMAIN_BOOT_FLOPPY:
-- 
1.8.4.5

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]