diff --git a/qla_isp/linux/isp_scst.c b/qla_isp/linux/isp_scst.c
index 44be93a81..eef080453 100644
--- a/qla_isp/linux/isp_scst.c
+++ b/qla_isp/linux/isp_scst.c
@@ -1108,12 +1108,12 @@ isp_rdy_to_xfer(struct scst_cmd *scst_cmd)
         }
 
         BUS_DBG2(bp, "TMD[%llx] write nbytes %u\n", tmd->cd_tagval, scst_cmd_get_bufflen(scst_cmd));
-        up_read(&bc->disable_sem);
         (*bp->h.r_action)(QIN_TMD_CONT, xact);
+        up_read(&bc->disable_sem);
         /*
          * Did we have an error starting this particular transaction?
          */
-        if (unlikely(xact->td_lflags & (TDFL_ERROR|TDFL_SYNCERROR)) == (TDFL_ERROR|TDFL_SYNCERROR)) {
+        if (unlikely((xact->td_lflags & (TDFL_ERROR|TDFL_SYNCERROR)) == (TDFL_ERROR|TDFL_SYNCERROR))) {
             if (xact->td_error == -ENOMEM) {
                 return (SCST_TGT_RES_QUEUE_FULL);
             } else {
@@ -1224,7 +1224,7 @@ out:
     /*
      * Did we have an error starting this particular transaction?
      */
-    if (unlikely(xact->td_lflags & (TDFL_ERROR|TDFL_SYNCERROR)) == (TDFL_ERROR|TDFL_SYNCERROR)) {
+    if (unlikely((xact->td_lflags & (TDFL_ERROR|TDFL_SYNCERROR)) == (TDFL_ERROR|TDFL_SYNCERROR))) {
         if (xact->td_error == -ENOMEM) {
             return (SCST_TGT_RES_QUEUE_FULL);
         } else {