Log errors for absent LUNs too.

MFC after:	1 week
This commit is contained in:
Alexander Motin 2014-11-21 08:24:37 +00:00
parent 23b30f5600
commit c82bf2659b

View File

@ -13710,7 +13710,7 @@ static int
ctl_process_done(union ctl_io *io) ctl_process_done(union ctl_io *io)
{ {
struct ctl_lun *lun; struct ctl_lun *lun;
struct ctl_softc *ctl_softc; struct ctl_softc *ctl_softc = control_softc;
void (*fe_done)(union ctl_io *io); void (*fe_done)(union ctl_io *io);
uint32_t targ_port = ctl_port_idx(io->io_hdr.nexus.targ_port); uint32_t targ_port = ctl_port_idx(io->io_hdr.nexus.targ_port);
@ -13776,10 +13776,8 @@ ctl_process_done(union ctl_io *io)
if (lun == NULL) { if (lun == NULL) {
CTL_DEBUG_PRINT(("NULL LUN for lun %d\n", CTL_DEBUG_PRINT(("NULL LUN for lun %d\n",
io->io_hdr.nexus.targ_mapped_lun)); io->io_hdr.nexus.targ_mapped_lun));
fe_done(io);
goto bailout; goto bailout;
} }
ctl_softc = lun->ctl_softc;
mtx_lock(&lun->lun_lock); mtx_lock(&lun->lun_lock);
@ -13851,6 +13849,8 @@ ctl_process_done(union ctl_io *io)
} else } else
mtx_unlock(&lun->lun_lock); mtx_unlock(&lun->lun_lock);
bailout:
/* /*
* If this command has been aborted, make sure we set the status * If this command has been aborted, make sure we set the status
* properly. The FETD is responsible for freeing the I/O and doing * properly. The FETD is responsible for freeing the I/O and doing
@ -13933,8 +13933,6 @@ ctl_process_done(union ctl_io *io)
} else } else
fe_done(io); fe_done(io);
bailout:
return (CTL_RETVAL_COMPLETE); return (CTL_RETVAL_COMPLETE);
} }