[1/2] net/bnxt: silence iova warnings

Message ID 20190314213214.26957-2-stephen@networkplumber.org
State Accepted, archived
Delegated to: Ferruh Yigit
Headers show
Series
  • net/bnxt: supress warnings
Related show

Checks

Context Check Description
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/intel-Performance-Testing success Performance Testing PASS
ci/Intel-compilation success Compilation OK
ci/checkpatch success coding style OK

Commit Message

Stephen Hemminger March 14, 2019, 9:32 p.m.
When using bnxt on bare-metal with vfio-pci, the driver logs an
unnecessary warning. Hardware works fine, message is not urgent.
Change it to INFO level.

Fixes: 62196f4e0941 ("mem: rename address mapping function to IOVA")
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/bnxt/bnxt_ethdev.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

Comments

Rami Rosen March 15, 2019, 1:03 p.m. | #1
Stephen Hemminger ‏<stephen@networkplumber.org>:

> When using bnxt on bare-metal with vfio-pci, the driver logs an
> unnecessary warning. Hardware works fine, message is not urgent.
> Change it to INFO level.
>
> Fixes: 62196f4e0941 ("mem: rename address mapping function to IOVA")
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>   Reviewd-by: Rami Rosen <ramirose@gmail.com>
Ajit Khaparde March 15, 2019, 6:44 p.m. | #2
On Fri, Mar 15, 2019 at 6:03 AM Rami Rosen <ramirose@gmail.com> wrote:

>
>  Stephen Hemminger ‏<stephen@networkplumber.org>:
>
>> When using bnxt on bare-metal with vfio-pci, the driver logs an
>> unnecessary warning. Hardware works fine, message is not urgent.
>> Change it to INFO level.
>>
>> Fixes: 62196f4e0941 ("mem: rename address mapping function to IOVA")
>> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
>> ---
>>   Reviewd-by: Rami Rosen <ramirose@gmail.com>
>
> Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>

Patch

diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index 189527207f9a..bdc6a1267a52 100644
--- a/drivers/net/bnxt/bnxt_ethdev.c
+++ b/drivers/net/bnxt/bnxt_ethdev.c
@@ -3242,10 +3242,8 @@  bnxt_dev_init(struct rte_eth_dev *eth_dev)
 		memset(mz->addr, 0, mz->len);
 		mz_phys_addr = mz->iova;
 		if ((unsigned long)mz->addr == mz_phys_addr) {
-			PMD_DRV_LOG(WARNING,
-				"Memzone physical address same as virtual.\n");
-			PMD_DRV_LOG(WARNING,
-				"Using rte_mem_virt2iova()\n");
+			PMD_DRV_LOG(INFO,
+				"Memzone physical address same as virtual using rte_mem_virt2iova()\n");
 			mz_phys_addr = rte_mem_virt2iova(mz->addr);
 			if (mz_phys_addr == 0) {
 				PMD_DRV_LOG(ERR,