From cae93578a9737e29bd473434260342429b2b2693 Mon Sep 17 00:00:00 2001
From: Gagandeep Singh <g.singh@nxp.com>
Date: Mon, 12 Jul 2021 17:10:59 +0530
Subject: [PATCH] fsl_qbman: usdpaa: change a print to info from critical
Resources release function is printing an information
level print as critical level which basically tells
how many USDPAA owned resources have been released.
Also it is printing information like "USDPAA process
leaks count" which may give wrong impression to user that
some resources are leaked.
By default, all critical level prints are visible on console
and this information level print is not required to be visible
on console.
In this patch, changing this print to "info" which we can
see using dmesg command.
Signed-off-by: Gagandeep Singh <g.singh@nxp.com>
DPDK-2888
---
drivers/staging/fsl_qbman/fsl_usdpaa.c | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/fsl_qbman/fsl_usdpaa.c b/drivers/staging/fsl_qbman/fsl_usdpaa.c
index 5eb53a731a1c44..2bdd51fe378331 100644
--- a/drivers/staging/fsl_qbman/fsl_usdpaa.c
+++ b/drivers/staging/fsl_qbman/fsl_usdpaa.c
@@ -680,7 +680,7 @@ static int usdpaa_release(struct inode *inode, struct file *filp)
qm_check_and_destroy_fqs(qm_cleanup_portal, ctx, check_channel_device);
while (backend->id_type != usdpaa_id_max) {
- int leaks = 0;
+ int res_count = 0;
list_for_each_entry(res, &ctx->resources[backend->id_type],
list) {
if (backend->id_type == usdpaa_id_fqid) {
@@ -692,12 +692,14 @@ static int usdpaa_release(struct inode *inode, struct file *filp)
res->id + i);
}
}
- leaks += res->num;
+ res_count += res->num;
+
backend->release(res->id, res->num);
}
- if (leaks)
- pr_crit("USDPAA process leaking %d %s%s\n", leaks,
- backend->acronym, (leaks > 1) ? "s" : "");
+ if (res_count)
+ pr_info("USDPAA release: %d %s%s\n",
+ res_count, backend->acronym,
+ (res_count > 1) ? "s" : "");
backend++;
}
/* Release any DMA regions */
--
GitLab