From 7e39e1e3b438168e80a8ebbfcb4dc41184b0a634 Mon Sep 17 00:00:00 2001 From: Stefan Richter Date: Thu, 2 Nov 2006 21:16:08 +0100 Subject: [PATCH] --- yaml --- r: 43452 b: refs/heads/master c: 5796aa7b11a6ba9d28a523fc82dddbc96c1a175e h: refs/heads/master v: v3 --- [refs] | 2 +- trunk/drivers/ieee1394/sbp2.c | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/[refs] b/[refs] index 5ed67653a208..3fbd569fec0d 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: e8ca5668f1546dacc85abcc06511ed113fd929c3 +refs/heads/master: 5796aa7b11a6ba9d28a523fc82dddbc96c1a175e diff --git a/trunk/drivers/ieee1394/sbp2.c b/trunk/drivers/ieee1394/sbp2.c index 625f1eb80698..96a743d9b5d5 100644 --- a/trunk/drivers/ieee1394/sbp2.c +++ b/trunk/drivers/ieee1394/sbp2.c @@ -1910,12 +1910,12 @@ static int sbp2scsi_queuecommand(struct scsi_cmnd *SCpnt, struct sbp2scsi_host_info *hi; int result = DID_NO_CONNECT << 16; - if (!sbp2util_node_is_available(scsi_id)) + if (unlikely(!sbp2util_node_is_available(scsi_id))) goto done; hi = scsi_id->hi; - if (!hi) { + if (unlikely(!hi)) { SBP2_ERR("sbp2scsi_host_info is NULL - this is bad!"); goto done; } @@ -1923,7 +1923,7 @@ static int sbp2scsi_queuecommand(struct scsi_cmnd *SCpnt, /* Multiple units are currently represented to the SCSI core as separate * targets, not as one target with multiple LUs. Therefore return * selection time-out to any IO directed at non-zero LUNs. */ - if (SCpnt->device->lun) + if (unlikely(SCpnt->device->lun)) goto done; /* handle the request sense command here (auto-request sense) */ @@ -1934,7 +1934,7 @@ static int sbp2scsi_queuecommand(struct scsi_cmnd *SCpnt, return 0; } - if (!hpsb_node_entry_valid(scsi_id->ne)) { + if (unlikely(!hpsb_node_entry_valid(scsi_id->ne))) { SBP2_ERR("Bus reset in progress - rejecting command"); result = DID_BUS_BUSY << 16; goto done; @@ -1942,7 +1942,7 @@ static int sbp2scsi_queuecommand(struct scsi_cmnd *SCpnt, /* Bidirectional commands are not yet implemented, * and unknown transfer direction not handled. */ - if (SCpnt->sc_data_direction == DMA_BIDIRECTIONAL) { + if (unlikely(SCpnt->sc_data_direction == DMA_BIDIRECTIONAL)) { SBP2_ERR("Cannot handle DMA_BIDIRECTIONAL - rejecting command"); result = DID_ERROR << 16; goto done;