core/configs: Rename cell flag "Unmanaged Exit" to "Passive Comm Region"

The purpose of this flag is to enable simple cells that do not actively
participate in the Communication Region protocols. The fact that we can
be shut down without prior consultation is one result of this but will
not be the only one in the future. Moreover, we may once use the Comm
Region to pass read-only information to the cells - without requiring
every cell to use the region actively or even map it.

Rename the flag to reflect its refined semantic.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>

diff --git a/Documentation/hypervisor-interfaces.txt b/Documentation/hypervisor-interfaces.txt
index 056cd4ba445f9e674cd153bb3f6fa7eeefea25c6..abbae72886c29868eea35d27cdc441bfa3d8c4c8 100644 (file)
--- a/Documentation/hypervisor-interfaces.txt
+++ b/Documentation/hypervisor-interfaces.txt
@@ -237,8 +237,9 @@ The following messages and corresponding replies are defined:
         2 - Shutdown OK
 
 Note: The hypervisor does not request shutdown permission from a cell if that
-      cell has the "Unmanaged Exit" flag set in its configuration (see also
-      [2]) or if the cell state is set to "Shut Down" or "Failed" (see below).
+      cell has the "Passive Communication Region" flag set in its configuration
+      (see also [2]) or if the cell state is set to "Shut Down" or "Failed"
+      (see below).
 
 
 Logical Channel "Cell State"

diff --git a/configs/tiny-demo.c b/configs/tiny-demo.c
index 926b6a2c912a729b423c34b0cce770dfeb373749..9d368748e86b7efe1b6238030e43ca1f0fc7c1e1 100644 (file)
--- a/configs/tiny-demo.c
+++ b/configs/tiny-demo.c
@@ -26,7 +26,7 @@ struct {
 } ALIGN config = {
        .cell = {
                .name = "tiny-demo",
-               .flags = JAILHOUSE_CELL_UNMANAGED_EXIT,
+               .flags = JAILHOUSE_CELL_PASSIVE_COMMREG,
 
                .cpu_set_size = sizeof(config.cpus),
                .num_memory_regions = ARRAY_SIZE(config.mem_regions),

diff --git a/hypervisor/control.c b/hypervisor/control.c
index ce96438727d04682648fb1819780d75bb40f5564..081b0363c0d21df3783842220f2eebf56748fdd6 100644 (file)
--- a/hypervisor/control.c
+++ b/hypervisor/control.c
@@ -331,7 +331,7 @@ static bool cell_shutdown_ok(struct cell *cell)
        volatile u32 *reply = &cell->comm_page.comm_region.reply_from_cell;
        volatile u32 *cell_state = &cell->comm_page.comm_region.cell_state;
 
-       if (cell->config->flags & JAILHOUSE_CELL_UNMANAGED_EXIT)
+       if (cell->config->flags & JAILHOUSE_CELL_PASSIVE_COMMREG)
                return true;
 
        jailhouse_send_msg_to_cell(&cell->comm_page.comm_region,

diff --git a/hypervisor/include/jailhouse/cell-config.h b/hypervisor/include/jailhouse/cell-config.h
index 3cc9d6fbb110a765fadf14196536eb85a666e96a..4b691ef2cf20d555206510eb363a52cb86f72efa 100644 (file)
--- a/hypervisor/include/jailhouse/cell-config.h
+++ b/hypervisor/include/jailhouse/cell-config.h
@@ -15,7 +15,7 @@
 
 #define JAILHOUSE_CELL_NAME_MAXLEN     31
 
-#define JAILHOUSE_CELL_UNMANAGED_EXIT  0x00000001
+#define JAILHOUSE_CELL_PASSIVE_COMMREG 0x00000001
 
 struct jailhouse_cell_desc {
        char name[JAILHOUSE_CELL_NAME_MAXLEN+1];