diff options
| author | Toomas Soome <tsoome@me.com> | 2017-07-01 23:18:46 +0300 |
|---|---|---|
| committer | Hans Rosenfeld <hans.rosenfeld@joyent.com> | 2018-04-23 18:15:38 +0200 |
| commit | 59e9026779cf4a22e08153f0b7702f8167fb44c7 (patch) | |
| tree | 7f558e233810d609542cd173d36487ec28d072c2 | |
| parent | adb064afeb48ec687de5e8bd683ac74fbe19d656 (diff) | |
| download | illumos-joyent-59e9026779cf4a22e08153f0b7702f8167fb44c7.tar.gz | |
9221 cpqary3: this statement may fall through
9261 cpqary3: wrong value is used in panic message
Reviewed by: Yuri Pankov <yuripv@yuripv.net>
Reviewed by: Andy Fiddaman <omnios@citrus-it.co.uk>
Reviewed by: Gergő Mihály Doma <domag02@gmail.com>
Approved by: Hans Rosenfeld <rosenfeld@grumpf.hope-2000.org>
| -rw-r--r-- | usr/src/uts/common/io/cpqary3/cpqary3_transport.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/usr/src/uts/common/io/cpqary3/cpqary3_transport.c b/usr/src/uts/common/io/cpqary3/cpqary3_transport.c index 1880bba69b..ba1e0fa6c0 100644 --- a/usr/src/uts/common/io/cpqary3/cpqary3_transport.c +++ b/usr/src/uts/common/io/cpqary3/cpqary3_transport.c @@ -480,11 +480,12 @@ cpqary3_dma_alloc(cpqary3_t *cpqary3p, struct scsi_pkt *scsi_pktp, case DDI_DMA_INUSE: cmn_err(CE_PANIC, "CPQary3: Another I/O transaction " "is using the DMA handle"); + break; default: cmn_err(CE_PANIC, "CPQary3: Unexpected ERROR " "returned from Call to Bind Buffer " - "to Handle : 0x%X", i); + "to Handle : 0x%X", retvalue); } ddi_dma_free_handle(&cpqary3_pktp->cmd_dmahandle); |