argo: suppress select logging messages
authorNicholas Tsirakis <niko.tsirakis@gmail.com>
Tue, 18 Jun 2019 20:10:29 +0000 (16:10 -0400)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Wed, 19 Jun 2019 20:14:05 +0000 (21:14 +0100)
Some logging messages made more sense as argo debug
logs rather than standard Xen logs. Use argo_dprintk
to only print this info if argo DEBUG is enabled.

Signed-off-by: Nicholas Tsirakis <tsirakisn@ainfosec.com>
Reviewed-by: Christopher Clark <christopher.w.clark@gmail.com>
xen/common/argo.c

index c8f4302963aae35c593a3c7e145f054eca1272af..2b91e944990b13e459b1f827d38adc7c70d313cf 100644 (file)
@@ -1456,10 +1456,10 @@ find_ring_mfns(struct domain *d, struct argo_ring_info *ring_info,
     if ( ring_info->mfns )
     {
         /* Ring already existed: drop the previous mapping. */
-        gprintk(XENLOG_INFO, "argo: 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);
+        argo_dprintk("argo: 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);
 
         ring_remove_mfns(d, ring_info);
         ASSERT(!ring_info->mfns);
@@ -1516,11 +1516,11 @@ find_ring_mfns(struct domain *d, struct argo_ring_info *ring_info,
     {
         ASSERT(ring_info->nmfns == NPAGES_RING(len));
 
-        gprintk(XENLOG_DEBUG, "argo: 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,
-                ring_info->mfn_mapping, ring_info->len, ring_info->nmfns);
+        argo_dprintk("argo: 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,
+                     ring_info->mfn_mapping, ring_info->len, ring_info->nmfns);
     }
 
     return ret;
@@ -1730,9 +1730,9 @@ register_ring(struct domain *currd,
         list_add(&ring_info->node,
                  &currd->argo->ring_hash[hash_index(&ring_info->id)]);
 
-        gprintk(XENLOG_DEBUG, "argo: vm%u registering ring (vm%u:%x vm%u)\n",
-                currd->domain_id, ring_id.domain_id, ring_id.aport,
-                ring_id.partner_id);
+        argo_dprintk("argo: vm%u registering ring (vm%u:%x vm%u)\n",
+                     currd->domain_id, ring_id.domain_id, ring_id.aport,
+                     ring_id.partner_id);
     }
     else if ( ring_info->len )
     {
@@ -1770,10 +1770,9 @@ register_ring(struct domain *currd,
             goto out_unlock2;
         }
 
-        gprintk(XENLOG_DEBUG,
-                "argo: 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);
+        argo_dprintk("argo: 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);
     }
 
     ret = find_ring_mfns(currd, ring_info, npage, gfn_hnd, reg.len);