DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] qede: fix 32-bit build with debug enabled
@ 2016-05-19 17:38 Thomas Monjalon
  2016-05-19 18:18 ` Harish Patil
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2016-05-19 17:38 UTC (permalink / raw)
  To: Harish Patil, Rasesh Mody, Sony Chacko; +Cc: dev

Some 64-bit variables are printed for debug.
%PRIx64 qualifier must be used because %lx is not long enough
on 32-bit systems

Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
 drivers/net/qede/base/bcm_osal.c    | 4 ++--
 drivers/net/qede/base/ecore_cxt.c   | 6 +++---
 drivers/net/qede/base/ecore_mcp.c   | 4 ++--
 drivers/net/qede/base/ecore_sriov.c | 8 ++++----
 drivers/net/qede/base/ecore_vf.c    | 6 +++---
 5 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/drivers/net/qede/base/bcm_osal.c b/drivers/net/qede/base/bcm_osal.c
index 9540c4b..ae5a8bc 100644
--- a/drivers/net/qede/base/bcm_osal.c
+++ b/drivers/net/qede/base/bcm_osal.c
@@ -115,7 +115,7 @@ void *osal_dma_alloc_coherent(struct ecore_dev *p_dev,
 	}
 	*phys = mz->phys_addr;
 	DP_VERBOSE(p_dev, ECORE_MSG_PROBE,
-		   "size=%zu phys=0x%lx virt=%p on socket=%u\n",
+		   "size=%zu phys=0x%" PRIx64 " virt=%p on socket=%u\n",
 		   mz->len, mz->phys_addr, mz->addr, socket_id);
 	return mz->addr;
 }
@@ -144,7 +144,7 @@ void *osal_dma_alloc_coherent_aligned(struct ecore_dev *p_dev,
 	}
 	*phys = mz->phys_addr;
 	DP_VERBOSE(p_dev, ECORE_MSG_PROBE,
-		   "aligned memory size=%zu phys=0x%lx virt=%p core=%d\n",
+		   "aligned memory size=%zu phys=0x%" PRIx64 " virt=%p core=%d\n",
 		   mz->len, mz->phys_addr, mz->addr, core_id);
 	return mz->addr;
 }
diff --git a/drivers/net/qede/base/ecore_cxt.c b/drivers/net/qede/base/ecore_cxt.c
index 8436621..1201c1a 100644
--- a/drivers/net/qede/base/ecore_cxt.c
+++ b/drivers/net/qede/base/ecore_cxt.c
@@ -876,8 +876,8 @@ ecore_ilt_blk_alloc(struct ecore_hwfn *p_hwfn,
 		ilt_shadow[line].size = size;
 
 		DP_VERBOSE(p_hwfn, ECORE_MSG_ILT,
-			   "ILT shadow: Line [%d] Physical 0x%lx "
-			   "Virtual %p Size %d\n",
+			   "ILT shadow: Line [%d] Physical 0x%" PRIx64
+			   " Virtual %p Size %d\n",
 			   line, (u64)p_phys, p_virt, size);
 
 		sz_left -= size;
@@ -1474,7 +1474,7 @@ static void ecore_ilt_init_pf(struct ecore_hwfn *p_hwfn)
 				DP_VERBOSE(p_hwfn, ECORE_MSG_ILT,
 					"Setting RT[0x%08x] from"
 					" ILT[0x%08x] [Client is %d] to"
-					" Physical addr: 0x%lx\n",
+					" Physical addr: 0x%" PRIx64 "\n",
 					rt_offst, line, i,
 					(u64)(p_shdw[line].p_phys >> 12));
 			}
diff --git a/drivers/net/qede/base/ecore_mcp.c b/drivers/net/qede/base/ecore_mcp.c
index bdc6a5e..9dd2eed 100644
--- a/drivers/net/qede/base/ecore_mcp.c
+++ b/drivers/net/qede/base/ecore_mcp.c
@@ -1197,8 +1197,8 @@ enum _ecore_status_t ecore_mcp_fill_shmem_func_info(struct ecore_hwfn *p_hwfn,
 	DP_VERBOSE(p_hwfn, (ECORE_MSG_SP | ECORE_MSG_IFUP),
 		   "Read configuration from shmem: pause_on_host %02x"
 		    " protocol %02x BW [%02x - %02x]"
-		    " MAC %02x:%02x:%02x:%02x:%02x:%02x wwn port %lx"
-		    " node %lx ovlan %04x\n",
+		    " MAC %02x:%02x:%02x:%02x:%02x:%02x wwn port %" PRIx64
+		    " node %" PRIx64 " ovlan %04x\n",
 		   info->pause_on_host, info->protocol,
 		   info->bandwidth_min, info->bandwidth_max,
 		   info->mac[0], info->mac[1], info->mac[2],
diff --git a/drivers/net/qede/base/ecore_sriov.c b/drivers/net/qede/base/ecore_sriov.c
index 7cd48ea..1b3119d 100644
--- a/drivers/net/qede/base/ecore_sriov.c
+++ b/drivers/net/qede/base/ecore_sriov.c
@@ -297,9 +297,9 @@ static enum _ecore_status_t ecore_iov_allocate_vfdb(struct ecore_hwfn *p_hwfn)
 		return ECORE_NOMEM;
 
 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
-		   "PF's Requests mailbox [%p virt 0x%lx phys],  Response"
-		   " mailbox [%p virt 0x%lx phys] Bulletins"
-		   " [%p virt 0x%lx phys]\n",
+		   "PF's Requests mailbox [%p virt 0x%" PRIx64 " phys], "
+		   "Response mailbox [%p virt 0x%" PRIx64 " phys] Bulletins"
+		   " [%p virt 0x%" PRIx64 " phys]\n",
 		   p_iov_info->mbx_msg_virt_addr,
 		   (u64)p_iov_info->mbx_msg_phys_addr,
 		   p_iov_info->mbx_reply_virt_addr,
@@ -1250,7 +1250,7 @@ static void ecore_iov_vf_mbx_acquire(struct ecore_hwfn *p_hwfn,
 
 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
 		   "VF[%d] ACQUIRE_RESPONSE: pfdev_info- chip_num=0x%x,"
-		   " db_size=%d, idx_per_sb=%d, pf_cap=0x%lx\n"
+		   " db_size=%d, idx_per_sb=%d, pf_cap=0x%" PRIx64 "\n"
 		   "resources- n_rxq-%d, n_txq-%d, n_sbs-%d, n_macs-%d,"
 		   " n_vlans-%d, n_mcs-%d\n",
 		   vf->abs_vf_id, resp->pfdev_info.chip_num,
diff --git a/drivers/net/qede/base/ecore_vf.c b/drivers/net/qede/base/ecore_vf.c
index 4073aaf..d32fb35 100644
--- a/drivers/net/qede/base/ecore_vf.c
+++ b/drivers/net/qede/base/ecore_vf.c
@@ -325,8 +325,8 @@ enum _ecore_status_t ecore_vf_hw_prepare(struct ecore_dev *p_dev)
 	}
 
 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
-		   "VF's Request mailbox [%p virt 0x%lx phys], Response"
-		   " mailbox [%p virt 0x%lx phys]\n",
+		   "VF's Request mailbox [%p virt 0x%" PRIx64 " phys], "
+		   "Response mailbox [%p virt 0x%" PRIx64 " phys]\n",
 		   p_sriov->vf2pf_request,
 		   (u64)p_sriov->vf2pf_request_phys,
 		   p_sriov->pf2vf_reply, (u64)p_sriov->pf2vf_reply_phys);
@@ -339,7 +339,7 @@ enum _ecore_status_t ecore_vf_hw_prepare(struct ecore_dev *p_dev)
 							   p_sriov->bulletin.
 							   size);
 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
-		   "VF's bulletin Board [%p virt 0x%lx phys 0x%08x bytes]\n",
+		   "VF's bulletin Board [%p virt 0x%" PRIx64 " phys 0x%08x bytes]\n",
 		   p_sriov->bulletin.p_virt, (u64)p_sriov->bulletin.phys,
 		   p_sriov->bulletin.size);
 
-- 
2.7.0

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] qede: fix 32-bit build with debug enabled
  2016-05-19 17:38 [dpdk-dev] [PATCH] qede: fix 32-bit build with debug enabled Thomas Monjalon
@ 2016-05-19 18:18 ` Harish Patil
  2016-05-24 14:11   ` Thomas Monjalon
  0 siblings, 1 reply; 3+ messages in thread
From: Harish Patil @ 2016-05-19 18:18 UTC (permalink / raw)
  To: Thomas Monjalon, Rasesh Mody, Sony Chacko; +Cc: dev

>
>Some 64-bit variables are printed for debug.
>%PRIx64 qualifier must be used because %lx is not long enough
>on 32-bit systems
>
>Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
>---
> drivers/net/qede/base/bcm_osal.c    | 4 ++--
> drivers/net/qede/base/ecore_cxt.c   | 6 +++---
> drivers/net/qede/base/ecore_mcp.c   | 4 ++--
> drivers/net/qede/base/ecore_sriov.c | 8 ++++----
> drivers/net/qede/base/ecore_vf.c    | 6 +++---
> 5 files changed, 14 insertions(+), 14 deletions(-)
>
>diff --git a/drivers/net/qede/base/bcm_osal.c
>b/drivers/net/qede/base/bcm_osal.c
>index 9540c4b..ae5a8bc 100644
>--- a/drivers/net/qede/base/bcm_osal.c
>+++ b/drivers/net/qede/base/bcm_osal.c
>@@ -115,7 +115,7 @@ void *osal_dma_alloc_coherent(struct ecore_dev *p_dev,
> 	}
> 	*phys = mz->phys_addr;
> 	DP_VERBOSE(p_dev, ECORE_MSG_PROBE,
>-		   "size=%zu phys=0x%lx virt=%p on socket=%u\n",
>+		   "size=%zu phys=0x%" PRIx64 " virt=%p on socket=%u\n",
> 		   mz->len, mz->phys_addr, mz->addr, socket_id);
> 	return mz->addr;
> }
>@@ -144,7 +144,7 @@ void *osal_dma_alloc_coherent_aligned(struct
>ecore_dev *p_dev,
> 	}
> 	*phys = mz->phys_addr;
> 	DP_VERBOSE(p_dev, ECORE_MSG_PROBE,
>-		   "aligned memory size=%zu phys=0x%lx virt=%p core=%d\n",
>+		   "aligned memory size=%zu phys=0x%" PRIx64 " virt=%p core=%d\n",
> 		   mz->len, mz->phys_addr, mz->addr, core_id);
> 	return mz->addr;
> }
>diff --git a/drivers/net/qede/base/ecore_cxt.c
>b/drivers/net/qede/base/ecore_cxt.c
>index 8436621..1201c1a 100644
>--- a/drivers/net/qede/base/ecore_cxt.c
>+++ b/drivers/net/qede/base/ecore_cxt.c
>@@ -876,8 +876,8 @@ ecore_ilt_blk_alloc(struct ecore_hwfn *p_hwfn,
> 		ilt_shadow[line].size = size;
> 
> 		DP_VERBOSE(p_hwfn, ECORE_MSG_ILT,
>-			   "ILT shadow: Line [%d] Physical 0x%lx "
>-			   "Virtual %p Size %d\n",
>+			   "ILT shadow: Line [%d] Physical 0x%" PRIx64
>+			   " Virtual %p Size %d\n",
> 			   line, (u64)p_phys, p_virt, size);
> 
> 		sz_left -= size;
>@@ -1474,7 +1474,7 @@ static void ecore_ilt_init_pf(struct ecore_hwfn
>*p_hwfn)
> 				DP_VERBOSE(p_hwfn, ECORE_MSG_ILT,
> 					"Setting RT[0x%08x] from"
> 					" ILT[0x%08x] [Client is %d] to"
>-					" Physical addr: 0x%lx\n",
>+					" Physical addr: 0x%" PRIx64 "\n",
> 					rt_offst, line, i,
> 					(u64)(p_shdw[line].p_phys >> 12));
> 			}
>diff --git a/drivers/net/qede/base/ecore_mcp.c
>b/drivers/net/qede/base/ecore_mcp.c
>index bdc6a5e..9dd2eed 100644
>--- a/drivers/net/qede/base/ecore_mcp.c
>+++ b/drivers/net/qede/base/ecore_mcp.c
>@@ -1197,8 +1197,8 @@ enum _ecore_status_t
>ecore_mcp_fill_shmem_func_info(struct ecore_hwfn *p_hwfn,
> 	DP_VERBOSE(p_hwfn, (ECORE_MSG_SP | ECORE_MSG_IFUP),
> 		   "Read configuration from shmem: pause_on_host %02x"
> 		    " protocol %02x BW [%02x - %02x]"
>-		    " MAC %02x:%02x:%02x:%02x:%02x:%02x wwn port %lx"
>-		    " node %lx ovlan %04x\n",
>+		    " MAC %02x:%02x:%02x:%02x:%02x:%02x wwn port %" PRIx64
>+		    " node %" PRIx64 " ovlan %04x\n",
> 		   info->pause_on_host, info->protocol,
> 		   info->bandwidth_min, info->bandwidth_max,
> 		   info->mac[0], info->mac[1], info->mac[2],
>diff --git a/drivers/net/qede/base/ecore_sriov.c
>b/drivers/net/qede/base/ecore_sriov.c
>index 7cd48ea..1b3119d 100644
>--- a/drivers/net/qede/base/ecore_sriov.c
>+++ b/drivers/net/qede/base/ecore_sriov.c
>@@ -297,9 +297,9 @@ static enum _ecore_status_t
>ecore_iov_allocate_vfdb(struct ecore_hwfn *p_hwfn)
> 		return ECORE_NOMEM;
> 
> 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
>-		   "PF's Requests mailbox [%p virt 0x%lx phys],  Response"
>-		   " mailbox [%p virt 0x%lx phys] Bulletins"
>-		   " [%p virt 0x%lx phys]\n",
>+		   "PF's Requests mailbox [%p virt 0x%" PRIx64 " phys], "
>+		   "Response mailbox [%p virt 0x%" PRIx64 " phys] Bulletins"
>+		   " [%p virt 0x%" PRIx64 " phys]\n",
> 		   p_iov_info->mbx_msg_virt_addr,
> 		   (u64)p_iov_info->mbx_msg_phys_addr,
> 		   p_iov_info->mbx_reply_virt_addr,
>@@ -1250,7 +1250,7 @@ static void ecore_iov_vf_mbx_acquire(struct
>ecore_hwfn *p_hwfn,
> 
> 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
> 		   "VF[%d] ACQUIRE_RESPONSE: pfdev_info- chip_num=0x%x,"
>-		   " db_size=%d, idx_per_sb=%d, pf_cap=0x%lx\n"
>+		   " db_size=%d, idx_per_sb=%d, pf_cap=0x%" PRIx64 "\n"
> 		   "resources- n_rxq-%d, n_txq-%d, n_sbs-%d, n_macs-%d,"
> 		   " n_vlans-%d, n_mcs-%d\n",
> 		   vf->abs_vf_id, resp->pfdev_info.chip_num,
>diff --git a/drivers/net/qede/base/ecore_vf.c
>b/drivers/net/qede/base/ecore_vf.c
>index 4073aaf..d32fb35 100644
>--- a/drivers/net/qede/base/ecore_vf.c
>+++ b/drivers/net/qede/base/ecore_vf.c
>@@ -325,8 +325,8 @@ enum _ecore_status_t ecore_vf_hw_prepare(struct
>ecore_dev *p_dev)
> 	}
> 
> 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
>-		   "VF's Request mailbox [%p virt 0x%lx phys], Response"
>-		   " mailbox [%p virt 0x%lx phys]\n",
>+		   "VF's Request mailbox [%p virt 0x%" PRIx64 " phys], "
>+		   "Response mailbox [%p virt 0x%" PRIx64 " phys]\n",
> 		   p_sriov->vf2pf_request,
> 		   (u64)p_sriov->vf2pf_request_phys,
> 		   p_sriov->pf2vf_reply, (u64)p_sriov->pf2vf_reply_phys);
>@@ -339,7 +339,7 @@ enum _ecore_status_t ecore_vf_hw_prepare(struct
>ecore_dev *p_dev)
> 							   p_sriov->bulletin.
> 							   size);
> 	DP_VERBOSE(p_hwfn, ECORE_MSG_IOV,
>-		   "VF's bulletin Board [%p virt 0x%lx phys 0x%08x bytes]\n",
>+		   "VF's bulletin Board [%p virt 0x%" PRIx64 " phys 0x%08x bytes]\n",
> 		   p_sriov->bulletin.p_virt, (u64)p_sriov->bulletin.phys,
> 		   p_sriov->bulletin.size);
> 
>-- 
>2.7.0
>
>

Looks good, thanks.

Acked-by: Harish Patil <harish.patil@qlogic.com>


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] qede: fix 32-bit build with debug enabled
  2016-05-19 18:18 ` Harish Patil
@ 2016-05-24 14:11   ` Thomas Monjalon
  0 siblings, 0 replies; 3+ messages in thread
From: Thomas Monjalon @ 2016-05-24 14:11 UTC (permalink / raw)
  To: Harish Patil; +Cc: Rasesh Mody, Sony Chacko, dev

2016-05-19 18:18, Harish Patil:
> >
> >Some 64-bit variables are printed for debug.
> >%PRIx64 qualifier must be used because %lx is not long enough
> >on 32-bit systems
> >
> >Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> 
> Looks good, thanks.
> 
> Acked-by: Harish Patil <harish.patil@qlogic.com>

Applied

PS: please remove useless context when ack'ing, thanks

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-05-24 14:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-19 17:38 [dpdk-dev] [PATCH] qede: fix 32-bit build with debug enabled Thomas Monjalon
2016-05-19 18:18 ` Harish Patil
2016-05-24 14:11   ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).