Skip to content

Commit d684a7a

Browse files
snitsmartinkpetersen
authored andcommitted
scsi: mpt3sas: Don't print sense pool info twice
_base_allocate_sense_dma_pool() already prints out the sense pool information, so don't print it a second time after calling it in _base_allocate_memory_pools(). In addition the version in _base_allocate_memory_pools() was using the wrong size value, sz, which was last assigned when doing some nvme calculations instead of sense_sz to determine the pool size in kilobytes. Cc: Sathya Prakash <[email protected]> Cc: Sreekanth Reddy <[email protected]> Cc: Suganath Prabu Subramani <[email protected]> Cc: [email protected] Cc: "Martin K. Petersen" <[email protected]> Cc: "James E.J. Bottomley" <[email protected]> Fixes: 970ac2b ("scsi: mpt3sas: Force sense buffer allocations to be within same 4 GB region") Signed-off-by: Jerry Snitselaar <[email protected]> Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Martin K. Petersen <[email protected]>
1 parent f0aa59a commit d684a7a

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

drivers/scsi/mpt3sas/mpt3sas_base.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6616,11 +6616,6 @@ _base_allocate_memory_pools(struct MPT3SAS_ADAPTER *ioc)
66166616
else if (rc == -EAGAIN)
66176617
goto try_32bit_dma;
66186618
total_sz += sense_sz;
6619-
ioc_info(ioc,
6620-
"sense pool(0x%p)- dma(0x%llx): depth(%d),"
6621-
"element_size(%d), pool_size(%d kB)\n",
6622-
ioc->sense, (unsigned long long)ioc->sense_dma, ioc->scsiio_depth,
6623-
SCSI_SENSE_BUFFERSIZE, sz / 1024);
66246619
/* reply pool, 4 byte align */
66256620
sz = ioc->reply_free_queue_depth * ioc->reply_sz;
66266621
rc = _base_allocate_reply_pool(ioc, sz);

0 commit comments

Comments
 (0)