From d4a11d6a22cf73ac7441750e5e8113779348885e Mon Sep 17 00:00:00 2001 From: Jason Andryuk Date: Mon, 31 Oct 2022 13:21:31 +0100 Subject: [PATCH] argo: Remove reachable ASSERT_UNREACHABLE I observed this ASSERT_UNREACHABLE in partner_rings_remove consistently trip. It was in OpenXT with the viptables patch applied. dom10 shuts down. dom7 is REJECTED sending to dom10. dom7 shuts down and this ASSERT trips for dom10. The argo_send_info has a domid, but there is no refcount taken on the domain. Therefore it's not appropriate to ASSERT that the domain can be looked up via domid. Replace with a debug message. Signed-off-by: Jason Andryuk Reviewed-by: Christopher Clark master commit: 197f612b77c5afe04e60df2100a855370d720ad7 master date: 2022-10-14 14:45:41 +0100 --- xen/common/argo.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/xen/common/argo.c b/xen/common/argo.c index eaea7ba888..80f3275092 100644 --- a/xen/common/argo.c +++ b/xen/common/argo.c @@ -1298,7 +1298,8 @@ partner_rings_remove(struct domain *src_d) ASSERT_UNREACHABLE(); } else - ASSERT_UNREACHABLE(); + argo_dprintk("%pd has entry for stale partner d%u\n", + src_d, send_info->id.domain_id); if ( dst_d ) rcu_unlock_domain(dst_d); -- 2.30.2