Attachment 100789 Details for Bug 77271 – patch update 3

[patch] patch update 3

0001-systemd-detect-virt-only-discover-Xen-domU.patch (text/plain), 2.46 KB, created by Thomas Blume on 2014-06-10 06:48:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Thomas Blume

Created: 2014-06-10 06:48:24 UTC

Size: 2.46 KB

patch

obsolete

>From 377c167c2e9cd171419e955ad86d98eeadc2839e Mon Sep 17 00:00:00 2001
>From: Thomas Blume <Thomas.Blume@suse.com>
>Date: Fri, 6 Jun 2014 16:36:45 +0200
>Subject: [PATCH] systemd-detect-virt: only discover Xen domU
>
>The current vm detection lacks the distinction between Xen dom0 and Xen domU.
>Both, dom0 and domU are running inside the hypervisor.
>Therefore systemd-detect-virt and the ConditionVirtualization directive detect
>dom0 as a virtual machine.
>
>dom0 is not using virtual devices but is accessing the real hardware.
>Therefore dom0 should be considered the virtualisation host and not a virtual
>machine.
>---
> src/shared/virt.c | 31 ++++++++++++++++++++++---------
> 1 file changed, 22 insertions(+), 9 deletions(-)
>
>diff --git a/src/shared/virt.c b/src/shared/virt.c
>index 1e227c5..e0de9a8 100644
>--- a/src/shared/virt.c
>+++ b/src/shared/virt.c
>@@ -162,17 +162,30 @@ int detect_vm(const char **id) {
>                 return cached_found;
>         }
> 
>-        /* Try high-level hypervisor sysfs file first:
>+        /* Try xen capabilities file first, if not found try high-level hypervisor sysfs file:
>          *
>-         * https://bugs.freedesktop.org/show_bug.cgi?id=61491 */
>-        r = read_one_line_file("/sys/hypervisor/type", &hvtype);
>+         * https://bugs.freedesktop.org/show_bug.cgi?id=77271 */
>+        char *domcap = NULL;
>+        r = read_one_line_file("/proc/xen/capabilities", &domcap);
>         if (r >= 0) {
>-                if (streq(hvtype, "xen")) {
>-                        _id = "xen";
>-                        r = 1;
>-                        goto finish;
>-                }
>-        } else if (r != -ENOENT)
>+                char *cap = NULL;
>+                while ((cap = strsep(&domcap, ",")) && !streq(cap, "control_d")) 
>+                        if (domcap == NULL) {
>+                                r = 1;
>+                                _id = "xen";
>+                        }
>+                goto finish;
>+        } else if (r == -ENOENT) {
>+                r = read_one_line_file("/sys/hypervisor/type", &hvtype);
>+                if (r >= 0) {
>+                        if (streq(hvtype, "xen")) {
>+                                r = 1;
>+                                _id = "xen";
>+                                goto finish;
>+                        }
>+                } else if (r != -ENOENT)
>+                        return r;
>+        } else 
>                 return r;
> 
>         /* this will set _id to "other" and return 0 for unknown hypervisors */
>-- 
>1.8.4.5
>

Actions: View | Splinter Review

Attachments on bug 77271: 97166 | 97822 | 100447 | 100513 | 100789 | 102974

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.