[Xen-changelog] [xen master] tools/xenstored: Make gnttab interface mandatory

[patchbot@xxxxxxx]

commit ecdeb7f8835398a1a4816d8cf68b7313674f84fe
Author:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
AuthorDate: Fri May 17 11:06:16 2019 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Thu Jul 11 09:21:21 2019 -0500

    tools/xenstored: Make gnttab interface mandatory
    
    xenstored currently requires an libxc and evtchn interface, but leaves
    the gnttab interface as optional.
    
    gnttab is ubiquitous these days, and in practice mandatory in all cases
    where xenstored isn't running as root in dom0 (due to the inability to
    foreign map by MFN).
    
    The toolstack has unconditionally set up grant details for many years
    now, and longterm it would be good to phase out the use of libxc.  This
    requires that xenstored map the store ring by grant map, rather than
    foreign map.
    
    No practical change.
    
    Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
    Reviewed-by: Juergen Gross <jgross@xxxxxxxx>
---
 tools/xenstore/xenstored_domain.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/xenstore/xenstored_domain.c 
b/tools/xenstore/xenstored_domain.c
index fa6655033a..463ebba643 100644
--- a/tools/xenstore/xenstored_domain.c
+++ b/tools/xenstore/xenstored_domain.c
@@ -641,9 +641,9 @@ void domain_init(void)
 
        *xgt_handle = xengnttab_open(NULL, 0);
        if (*xgt_handle == NULL)
-               xprintf("WARNING: Failed to open connection to gnttab\n");
-       else
-               talloc_set_destructor(xgt_handle, close_xgt_handle);
+               barf_perror("Failed to open connection to gnttab");
+
+       talloc_set_destructor(xgt_handle, close_xgt_handle);
 
        xce_handle = xenevtchn_open(NULL, 0);
 
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog