Re: [PATCH v2 3/3] argo: drop argo prefix from argo_dprintk() calls

[Mykola Kvach <xakep.amatop@xxxxxxxxx>]

On Fri, May 22, 2026 at 7:53 PM <dmukhin@xxxxxxxx> wrote:
>
> From: Denis Mukhin <dmukhin@xxxxxxxx>
>
> argo_dprintk() prefixes all log lines with "argo: " automatically.
>
> Remove duplicate prefixes from log messages in the Argo module where
> applicable.
>
> Signed-off-by: Denis Mukhin <dmukhin@xxxxxxxx>
> ---
> Changes since v1:
> - new patch
> ---
>  xen/common/argo.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/xen/common/argo.c b/xen/common/argo.c
> index 5da14c929e14..ffa1f43437ab 100644
> --- a/xen/common/argo.c
> +++ b/xen/common/argo.c
> @@ -1467,7 +1467,7 @@ find_ring_mfns(struct domain *d, struct argo_ring_info 
> *ring_info,
>      if ( ring_info->mfns )
>      {
>          /* Ring already existed: drop the previous mapping. */
> -        argo_dprintk("argo: vm%u re-register existing ring "
> +        argo_dprintk("vm%u re-register existing ring "
>                       "(vm%u:%x vm%u) clears mapping\n",
>                       d->domain_id, ring_info->id.domain_id,
>                       ring_info->id.aport, ring_info->id.partner_id);
> @@ -1527,7 +1527,7 @@ find_ring_mfns(struct domain *d, struct argo_ring_info 
> *ring_info,
>      {
>          ASSERT(ring_info->nmfns == NPAGES_RING(len));
>
> -        argo_dprintk("argo: vm%u ring (vm%u:%x vm%u) %p "
> +        argo_dprintk("vm%u ring (vm%u:%x vm%u) %p "
>                       "mfn_mapping %p len %u nmfns %u\n",
>                       d->domain_id, ring_info->id.domain_id,
>                       ring_info->id.aport, ring_info->id.partner_id, 
> ring_info,
> @@ -1741,7 +1741,7 @@ register_ring(struct domain *currd,
>          list_add(&ring_info->node,
>                   &currd->argo->ring_hash[hash_index(&ring_info->id)]);
>
> -        argo_dprintk("argo: vm%u registering ring (vm%u:%x vm%u)\n",
> +        argo_dprintk("vm%u registering ring (vm%u:%x vm%u)\n",
>                       currd->domain_id, ring_id.domain_id, ring_id.aport,
>                       ring_id.partner_id);
>      }
> @@ -1781,7 +1781,7 @@ register_ring(struct domain *currd,
>              goto out_unlock2;
>          }
>
> -        argo_dprintk("argo: vm%u re-registering existing ring (vm%u:%x 
> vm%u)\n",
> +        argo_dprintk("vm%u re-registering existing ring (vm%u:%x vm%u)\n",
>                       currd->domain_id, ring_id.domain_id, ring_id.aport,
>                       ring_id.partner_id);

Reviewed-by: Mykola Kvach <mykola_kvach@xxxxxxxx>

~Mykola