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

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




On 5/26/26 5:58 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>
Reviewed-by: Mykola Kvach <mykola_kvach@xxxxxxxx>
---
Changes since v2:
- n/a
---
  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: Daniel P. Smith <dpsmith@xxxxxxxxxxxxxxxxxxxx>



 


Rackspace

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