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

[Julien Grall <julien@xxxxxxx>]

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