Re: [PATCH v2 1/3] argo: lower level of noisy connection-refused log

[Mykola Kvach <xakep.amatop@xxxxxxxxx>]

On Fri, May 22, 2026 at 7:53 PM <dmukhin@xxxxxxxx> wrote:
>
> From: Denis Mukhin <dmukhin@xxxxxxxx>
>
> Lower the log level of the "connection refused" log line, as it can
> spam the logs when a dom0 service using the Argo hypercall tries to
> communicate with a domain that is still starting up.
>
> Signed-off-by: Denis Mukhin <dmukhin@xxxxxxxx>
> ---
> Changes since v1:
> - dropped duplicate "argo: " prefix
> ---
>  xen/common/argo.c | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/xen/common/argo.c b/xen/common/argo.c
> index 28626e00a8cb..98a3db7fd070 100644
> --- a/xen/common/argo.c
> +++ b/xen/common/argo.c
> @@ -2034,10 +2034,9 @@ sendv(struct domain *src_d, xen_argo_addr_t *src_addr,
>                                          src_id.domain_id);
>      if ( !ring_info )
>      {
> -        gprintk(XENLOG_ERR,
> -                "argo: vm%u connection refused, src (vm%u:%x) dst 
> (vm%u:%x)\n",
> -                current->domain->domain_id, src_id.domain_id, src_id.aport,
> -                dst_addr->domain_id, dst_addr->aport);
> +        argo_dprintk("vm%u connection refused, src (vm%u:%x) dst 
> (vm%u:%x)\n",
> +                     current->domain->domain_id, src_id.domain_id, 
> src_id.aport,
> +                     dst_addr->domain_id, dst_addr->aport);
>
>          ret = -ECONNREFUSED;
>      }
> --
> 2.54.0
>
>

Reviewed-by: Mykola Kvach <mykola_kvach@xxxxxxxx>

~Mykola