From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juergen Gross Subject: Backport request Date: Wed, 1 Mar 2017 13:31:29 +0100 Message-ID: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="------------D47EBF3AC46830893172B307" Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cj3Po-0001TT-Lf for xen-devel@lists.xenproject.org; Wed, 01 Mar 2017 12:31:32 +0000 List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: xen-devel Cc: Ian Jackson , Jan Beulich List-Id: xen-devel@lists.xenproject.org This is a multi-part message in MIME format. --------------D47EBF3AC46830893172B307 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit I'd like to request backport of upstream commit bc6a499ebcf1 ("libxl: correct xenstore entry for empty cdrom") to at least 4.8 and 4.7. As libxl sources have been split in 4.9 the patch doesn't apply. Attached patch applies cleanly to 4.8 and with an offset to 4.7. Juergen --------------D47EBF3AC46830893172B307 Content-Type: text/x-patch; name="cdrom.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="cdrom.patch" >>From bc6a499ebcf102060987fb845c1450a38de441c7 Mon Sep 17 00:00:00 2001 From: Juergen Gross Date: Wed, 15 Feb 2017 12:11:12 +0100 Subject: [PATCH] libxl: correct xenstore entry for empty cdrom Specifying an empty cdrom device will result in a Xenstore entry params = aio:(null) as the physical device path isn't existing. This lets a domain booted via OVMF hang as OVMF is checking for "aio:" only in order to detect the empty cdrom case. Use an empty string for the physical device path in this case. As a cdrom device for HVM is always backed by qdisk we only need to cover this backend. Signed-off-by: Juergen Gross Acked-by: Wei Liu --- tools/libxl/libxl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index ac49df2..63de75c 100644 --- a/tools/libxl/libxl.c +++ b/tools/libxl/libxl.c @@ -2255,7 +2255,8 @@ static void device_disk_add(libxl__egc *egc, uint32_t domid, case LIBXL_DISK_BACKEND_QDISK: flexarray_append(back, "params"); flexarray_append(back, GCSPRINTF("%s:%s", - libxl__device_disk_string_of_format(disk->format), disk->pdev_path)); + libxl__device_disk_string_of_format(disk->format), + disk->pdev_path ? : "")); if (libxl_defbool_val(disk->colo_enable)) { flexarray_append(back, "colo-host"); flexarray_append(back, libxl__sprintf(gc, "%s", disk->colo_host)); -- 2.1.4 --------------D47EBF3AC46830893172B307 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWGVuLWRldmVs IG1haWxpbmcgbGlzdApYZW4tZGV2ZWxAbGlzdHMueGVuLm9yZwpodHRwczovL2xpc3RzLnhlbi5v cmcveGVuLWRldmVsCg== --------------D47EBF3AC46830893172B307--