]> xenbits.xensource.com Git - people/liuw/libxenctrl-split/xen.git/commitdiff
Revert "convert XSM_ENABLE to Kconfig"
authorJan Beulich <jbeulich@suse.com>
Fri, 8 Jan 2016 16:34:53 +0000 (17:34 +0100)
committerJan Beulich <jbeulich@suse.com>
Fri, 8 Jan 2016 16:34:53 +0000 (17:34 +0100)
This reverts commit 2b2ab5d88b2d2ab0155101a0a6922025064061af,
as osstest needs to be ready first.

Config.mk
INSTALL
docs/misc/xsm-flask.txt
xen/Rules.mk
xen/common/Kconfig
xen/include/asm-x86/config.h
xen/include/xen/sched.h
xen/include/xsm/dummy.h
xen/include/xsm/xsm.h
xen/xsm/Makefile

index 62f82099907ba56de6e14976d93f8bdbd2f37cfb..13159188cc73ebe9306d64b3855b81bd21ce0bed 100644 (file)
--- a/Config.mk
+++ b/Config.mk
@@ -212,6 +212,9 @@ APPEND_CFLAGS += $(foreach i, $(APPEND_INCLUDES), -I$(i))
 EMBEDDED_EXTRA_CFLAGS := -nopie -fno-stack-protector -fno-stack-protector-all
 EMBEDDED_EXTRA_CFLAGS += -fno-exceptions
 
+# Enable XSM security module (by default, Flask).
+XSM_ENABLE ?= n
+
 XEN_EXTFILES_URL ?= http://xenbits.xen.org/xen-extfiles
 # All the files at that location were downloaded from elsewhere on
 # the internet.  The original download URL is preserved as a comment
diff --git a/INSTALL b/INSTALL
index 3d2e86a2a0f67ac4cf91c601d21d364680a619be..c51447bdfea57fbf849c3748f282cdf1c1e023fb 100644 (file)
--- a/INSTALL
+++ b/INSTALL
@@ -275,10 +275,14 @@ Building the python tools may fail unless certain options are passed to
 setup.py. Config.mk contains additional info how to use this variable.
 PYTHON_PREFIX_ARG=
 
-he hypervisor may be build with XSM/Flask support, which can be changed
+The hypervisor may be build with XSM support, which can be changed with
+the following variables.
+XSM_ENABLE=y
+
+The hypervisor may be build with Flask support, which can be changed
 by running:
 make -C xen menuconfig
-and enabling XSM/Flask in the 'Common Features' menu.
+and enabling Flask in the 'Common Features' menu.
 
 Do a build for coverage.
 coverage=y
index fb2fe9f42fbd27c978a268ed8a7c5965a9a490d5..f2f0fd47d7ab0503a9e88781ee307c800983ee43 100644 (file)
@@ -172,9 +172,9 @@ Setting up FLASK
 ----------------
 
 Xen must be compiled with XSM and FLASK enabled; by default, the security
-framework is disabled. Running 'make -C xen menuconfig' and enabling XSM
-and FLASK inside 'Common Features'; this change requires a make clean and
-rebuild.
+framework is disabled. Edit Config.mk or the .config file to set XSM_ENABLE to
+"y" and running 'make -C xen menuconfig' and enabling FLASK inside 'Common
+Features'; this change requires a make clean and rebuild.
 
 FLASK uses only one domain configuration parameter (seclabel) defining the
 full security label of the newly created domain. If using the example policy,
index 8bd109825e6284c33a06951819d8fc904ea699c0..9e4e6ff5dc4e90126dc5123f2196fbcc2131a32f 100644 (file)
@@ -52,6 +52,7 @@ CFLAGS += -Werror -Wredundant-decls -Wno-pointer-arith
 CFLAGS += -pipe -g -D__XEN__ -include $(BASEDIR)/include/xen/config.h
 CFLAGS += '-D__OBJECT_FILE__="$@"'
 
+CFLAGS-$(XSM_ENABLE)    += -DXSM_ENABLE
 CFLAGS-$(verbose)       += -DVERBOSE
 CFLAGS-$(crash_debug)   += -DCRASH_DEBUG
 CFLAGS-$(perfc)         += -DPERF_COUNTERS
index eadfc3bbe284b3e01bcd9b2d814ace916af9de16..6373b7f3eb3a279fcd024de0081cd1d4bf6859ff 100644 (file)
@@ -10,9 +10,8 @@ config COMPAT
 
 config FLASK
        bool "FLux Advanced Security Kernel support"
-       default y
-       depends on XSM
-       ---help---
+       default n
+       --help---
          Enables the FLASK (FLux Advanced Security Kernel) support which
          provides a mandatory access control framework by which security
          enforcement, isolation, and auditing can be achieved with fine
@@ -63,38 +62,4 @@ config KEXEC
 
          If unsure, say Y.
 
-# Allows "late" initialization of the hardware domain
-config LATE_HWDOM
-       bool "dedicated hardware domain"
-       default n
-       depends on XSM && X86
-       ---help---
-         Allows the creation of a dedicated hardware domain distinct from
-         domain 0 that manages devices without needing access to other
-         privileged functionality such as the ability to manage domains.
-         This requires that the actual domain 0 be a stub domain that
-         constructs the actual hardware domain instead of initializing the
-         hardware itself.  Because the hardware domain needs access to
-         hypercalls not available to unprivileged guests, an XSM policy
-         is required to properly define the privilege of these domains.
-
-         This feature does nothing if the "hardware_dom" boot parameter is
-         not present.  If this feature is being used for security, it should
-         be combined with an IOMMU in strict mode.
-
-         If unsure, say N.
-
-# Enable/Disable XSM support
-config XSM
-       bool "Xen Security Modules support"
-       default n
-       ---help---
-         Enables the security framework known as Xen Security Modules which
-         allows administrators fine-grained control over a Xen domain and
-         its capabilities by defining permissible interactions between domains,
-         the hypervisor itself, and related resources such as memory and
-         devices.
-
-         If unsure, say N.
-
 endmenu
index 3305a75aea898370a798c68fed595e9175436982..f25d92ea383b9a7b1a65bc5ef4063d1455bc8ab4 100644 (file)
 
 #define CONFIG_MULTIBOOT 1
 
+#ifdef XSM_ENABLE
+#define CONFIG_LATE_HWDOM 1
+#endif
+
 #define HZ 100
 
 #define OPT_CONSOLE_STR "vga"
index e1428f7e68477b2dcf45c96ac83e3d3add771f54..6ea3cc7cc8541e9ed0e8808cc02ad85801cf532c 100644 (file)
@@ -110,7 +110,7 @@ struct evtchn
     u8 priority;
     u8 last_priority;
     u16 last_vcpu_id;
-#ifdef CONFIG_XSM
+#ifdef XSM_ENABLE
     union {
 #ifdef XSM_NEED_GENERIC_EVTCHN_SSID
         /*
index 55b84f0974683c4128896591994cca97f606d223..81fba406f3a26f74e8a757d674facb717e56c040 100644 (file)
@@ -27,9 +27,9 @@
 /* DO NOT implement this function; it is supposed to trigger link errors */
 void __xsm_action_mismatch_detected(void);
 
-#ifdef CONFIG_XSM
+#ifdef XSM_ENABLE
 
-/* In CONFIG_XSM builds, this header file is included from xsm/dummy.c, and
+/* In XSM_ENABLE builds, this header file is included from xsm/dummy.c, and
  * contains static (not inline) functions compiled to the dummy XSM module.
  * There is no xsm_default_t argument available, so the value from the assertion
  * is used to initialize the variable.
@@ -39,9 +39,9 @@ void __xsm_action_mismatch_detected(void);
 #define XSM_DEFAULT_VOID void
 #define XSM_ASSERT_ACTION(def) xsm_default_t action = def; (void)action
 
-#else /* CONFIG_XSM */
+#else /* XSM_ENABLE */
 
-/* In !CONFIG_XSM builds, this header file is included from xsm/xsm.h, and
+/* In !XSM_ENABLE builds, this header file is included from xsm/xsm.h, and
  * contains inline functions for each XSM hook. These functions also perform
  * compile-time checks on the xsm_default_t argument to ensure that the behavior
  * of the dummy XSM module is the same as the behavior with XSM disabled.
@@ -51,7 +51,7 @@ void __xsm_action_mismatch_detected(void);
 #define XSM_DEFAULT_VOID xsm_default_t action
 #define XSM_ASSERT_ACTION(def) LINKER_BUG_ON(def != action)
 
-#endif /* CONFIG_XSM */
+#endif /* XSM_ENABLE */
 
 static always_inline int xsm_default_action(
     xsm_default_t action, struct domain *src, struct domain *target)
index 2c365cdea6f64cf4357f685b4d5201e81663be7e..3fc3824cafbe2166764b39f629a81d70048720f2 100644 (file)
@@ -194,7 +194,7 @@ struct xsm_operations {
 #endif
 };
 
-#ifdef CONFIG_XSM
+#ifdef XSM_ENABLE
 
 extern struct xsm_operations *xsm_ops;
 
@@ -752,7 +752,7 @@ extern int unregister_xsm(struct xsm_operations *ops);
 extern struct xsm_operations dummy_xsm_ops;
 extern void xsm_fixup_ops(struct xsm_operations *ops);
 
-#else /* CONFIG_XSM */
+#else /* XSM_ENABLE */
 
 #include <xsm/dummy.h>
 
@@ -772,6 +772,6 @@ static inline int xsm_dt_init(void)
 }
 #endif
 
-#endif /* CONFIG_XSM */
+#endif /* XSM_ENABLE */
 
 #endif /* __XSM_H */
index 3252c46e64f5dd8c65cf5a132f45c47c7895d617..d29e71c3a48605898732297be7794fe9523d9265 100644 (file)
@@ -1,5 +1,7 @@
 obj-y += xsm_core.o
-obj-$(CONFIG_XSM) += xsm_policy.o
-obj-$(CONFIG_XSM) += dummy.o
+ifeq ($(XSM_ENABLE),y)
+obj-y += xsm_policy.o
+obj-y += dummy.o
+endif
 
 subdir-$(CONFIG_FLASK) += flask