IB/iser: Use proper debug level value for info prints
Commit 4f36388261
("IB/iser: Move informational messages from error
to info level") set info prints to be emitted at a lower debug level
than warning prints, which is a bit odd. Fix that.
Also move the prints on unaligned SG from warning to debug level.
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
This commit is contained in:
parent
c095ba7224
commit
f91424cf5b
|
@ -78,14 +78,14 @@
|
|||
|
||||
#define iser_warn(fmt, arg...) \
|
||||
do { \
|
||||
if (iser_debug_level > 1) \
|
||||
if (iser_debug_level > 0) \
|
||||
pr_warn(PFX "%s:" fmt, \
|
||||
__func__ , ## arg); \
|
||||
} while (0)
|
||||
|
||||
#define iser_info(fmt, arg...) \
|
||||
do { \
|
||||
if (iser_debug_level > 0) \
|
||||
if (iser_debug_level > 1) \
|
||||
pr_info(PFX "%s:" fmt, \
|
||||
__func__ , ## arg); \
|
||||
} while (0)
|
||||
|
|
|
@ -267,11 +267,8 @@ static void iser_data_buf_dump(struct iser_data_buf *data,
|
|||
struct scatterlist *sg;
|
||||
int i;
|
||||
|
||||
if (iser_debug_level == 0)
|
||||
return;
|
||||
|
||||
for_each_sg(sgl, sg, data->dma_nents, i)
|
||||
iser_warn("sg[%d] dma_addr:0x%lX page:0x%p "
|
||||
iser_dbg("sg[%d] dma_addr:0x%lX page:0x%p "
|
||||
"off:0x%x sz:0x%x dma_len:0x%x\n",
|
||||
i, (unsigned long)ib_sg_dma_address(ibdev, sg),
|
||||
sg_page(sg), sg->offset,
|
||||
|
@ -373,9 +370,11 @@ int iser_reg_rdma_mem(struct iscsi_iser_task *iser_task,
|
|||
if (aligned_len != mem->dma_nents ||
|
||||
(!ib_conn->fmr_pool && mem->dma_nents > 1)) {
|
||||
iscsi_conn->fmr_unalign_cnt++;
|
||||
iser_warn("rdma alignment violation (%d/%d aligned) or FMR not supported\n",
|
||||
aligned_len, mem->size);
|
||||
iser_data_buf_dump(mem, ibdev);
|
||||
iser_dbg("rdma alignment violation (%d/%d aligned) or FMR not supported\n",
|
||||
aligned_len, mem->size);
|
||||
|
||||
if (iser_debug_level > 0)
|
||||
iser_data_buf_dump(mem, ibdev);
|
||||
|
||||
/* unmap the command data before accessing it */
|
||||
iser_dma_unmap_task_data(iser_task);
|
||||
|
|
Loading…
Reference in a new issue