[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[xen staging] tools/xendomains: Restrict domid pattern in LIST_GREP



commit 0c06760be3dc3f286015e18c4b1d1694e55da026
Author:     Peter Hoyes <Peter.Hoyes@xxxxxxx>
AuthorDate: Mon Oct 3 15:42:16 2022 +0100
Commit:     Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
CommitDate: Thu Oct 20 17:38:56 2022 +0100

    tools/xendomains: Restrict domid pattern in LIST_GREP
    
    The xendomains script uses the output of `xl list -l` to collect the
    id and name of each domain, which is used in the shutdown logic, amongst
    other purposes.
    
    The linked commit added a "domid" field to libxl_domain_create_info.
    This causes the output of `xl list -l` to contain two "domid"s per
    domain, which may not be equal. This in turn causes `xendomains stop` to
    issue two shutdown commands per domain, one of which is to a duplicate
    and/or invalid domid.
    
    To work around this, make the LIST_GREP pattern more restrictive for
    domid, so it only detects the domid at the top level and not the domid
    inside c_info.
    
    Fixes: 4a3a25678d92 ("libxl: allow creation of domains with a specified or 
random domid")
    Signed-off-by: Peter Hoyes <Peter.Hoyes@xxxxxxx>
    Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
    Release-acked-by: Henry Wang <Henry.Wang@xxxxxxx>
---
 tools/hotplug/Linux/xendomains.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/hotplug/Linux/xendomains.in 
b/tools/hotplug/Linux/xendomains.in
index 334d244882..70f4129ef4 100644
--- a/tools/hotplug/Linux/xendomains.in
+++ b/tools/hotplug/Linux/xendomains.in
@@ -211,7 +211,7 @@ get_xsdomid()
     fi
 }
 
-LIST_GREP='(domain\|(domid\|(name\|^    {$\|"name":\|"domid":'
+LIST_GREP='(domain\|(domid\|(name\|^    {$\|"name":\|^        "domid":'
 parseln()
 {
     if [[ "$1" =~ '(domain' ]] || [[ "$1" = "{" ]]; then
--
generated by git-patchbot for /home/xen/git/xen.git#staging



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.