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

Re: [PATCH v2 04/19] tools/xenstore: remove all watches when a domain has stopped



Hi Juergen,

On 13/12/2022 16:00, Juergen Gross wrote:
When a domain has been released by Xen tools, remove all its
registered watches. This avoids sending watch events to the dead domain
when all the nodes related to it are being removed by the Xen tools.

AFAICT, the only user of the command in the tree is softreset. Would you be able to check this is still working as expected?


Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
---
V2:
- move call to do_release() (Julien Grall)
---
  tools/xenstore/xenstored_domain.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/tools/xenstore/xenstored_domain.c 
b/tools/xenstore/xenstored_domain.c
index aa86892fed..e669c89e94 100644
--- a/tools/xenstore/xenstored_domain.c
+++ b/tools/xenstore/xenstored_domain.c
@@ -740,6 +740,9 @@ int do_release(const void *ctx, struct connection *conn,
        if (IS_ERR(domain))
                return -PTR_ERR(domain);
+ /* Avoid triggering watch events when the domain's nodes are deleted. */
+       conn_delete_all_watches(domain->conn);
+
        talloc_free(domain->conn);
send_ack(conn, XS_RELEASE);

Cheers,

--
Julien Grall



 


Rackspace

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