Received: by 2002:ac0:a581:0:0:0:0:0 with SMTP id m1-v6csp3766599imm; Mon, 25 Jun 2018 04:23:03 -0700 (PDT) X-Google-Smtp-Source: ADUXVKKTj4poOAb50FdbDyVDdiE4Ygn/uCAxwhrfkBmG/vPhL4kyvbOPV9ZB6UIqxDNpPMftA160 X-Received: by 2002:a63:735d:: with SMTP id d29-v6mr6958878pgn.156.1529925783710; Mon, 25 Jun 2018 04:23:03 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1529925783; cv=none; d=google.com; s=arc-20160816; b=IJZtTp935z2Zpw1ClsIFG//4g6IFq1C8JF0lOERym97J2tKHwI6S4HsaZOI2gfSN1M 2y6fZ+ciAXiPJKrpMtyjQ1WCJhTYzalQu+ylj1GIRA8s+oDvgY0eHPE5SWWFZKZ4JXBZ g+XYCA5DaRbyq/VuHz6eiNSfTlLFyvxAX+4gL24FNPpiyTfX/XwW8qcjJ2zuhQpy/PsJ DO1tp+ezZxWAc3uJQI5s+hIOWU1GBMApaQu39OovskwBPeVcw9AUqL4VHuXxuqL/Y341 kvpaW09tVaV5SPyi7AJkbFQS+ccIHR7n3vz2LjIdTBgFftBCLcM3SP9je/Q88IUUo0uA FPaw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:arc-authentication-results; bh=zaiDTcbXGjKXfTnUUgBxBYgDFbkFiqtyCCDW579+MuQ=; b=sNC5pveCozJ8dC64cMwQTlPlgKsUHtPRp2901fvfjwE8N2LqTTR2i8KwOUgaPB5tRo qvD22jKH8EWEoElyN3xESoHU8gRt20bF2Ercag7fDyAaGPh/Ui3VRbeJ1gg8SBLYd+By MdD0AQeLc5vXgHBza9/3bWf8gasaLK7syBmzPh9sSdGuhO3fWDjUBZJw69PAVsERrrA6 n57vwCzoP5yCOwpQjojSOMGuIvbe+nNTuXYJfmuR2TAw0mB9zz8nHOFY2jbEPwELtvin WzgdrzBpiqI9LE9L37p3DcUJs543fL7FZ+yET1s1K5ccQE6zrpC1BkdRFWshLBcZoTxJ btXQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id e14-v6si11549069pgt.185.2018.06.25.04.22.36; Mon, 25 Jun 2018 04:23:03 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755050AbeFYLVJ (ORCPT + 99 others); Mon, 25 Jun 2018 07:21:09 -0400 Received: from mx2.suse.de ([195.135.220.15]:37416 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752711AbeFYLVG (ORCPT ); Mon, 25 Jun 2018 07:21:06 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay1.suse.de (charybdis-ext-too.suse.de [195.135.220.254]) by mx2.suse.de (Postfix) with ESMTP id 6C19DAE55; Mon, 25 Jun 2018 11:21:05 +0000 (UTC) From: Johannes Thumshirn To: "Martin K . Petersen" Cc: Linux Kernel Mailinglist , Linux SCSI Mailinglist , Johannes Thumshirn Subject: [PATCH v3 2/2] scsi: don't add scsi command result bytes Date: Mon, 25 Jun 2018 13:20:59 +0200 Message-Id: <20180625112059.12872-3-jthumshirn@suse.de> X-Mailer: git-send-email 2.16.4 In-Reply-To: <20180625112059.12872-1-jthumshirn@suse.de> References: <20180625112059.12872-1-jthumshirn@suse.de> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Some drivers are ADDing the scsi command's result bytes instead of ORing them. While this can produce correct results it has unexpected side effects. Signed-off-by: Johannes Thumshirn Reviewed-by: Christoph Hellwig Reviewed-by: Bart Van Assche --- Changes since v2: - Re-added braces around shift operators (Bart, Christoph) - Add Reviewed-bys Changes since v1: - Fix kbuild robot warnings --- drivers/scsi/imm.c | 2 +- drivers/scsi/mesh.c | 4 ++-- drivers/scsi/sym53c8xx_2/sym_glue.c | 2 +- drivers/scsi/sym53c8xx_2/sym_glue.h | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/imm.c b/drivers/scsi/imm.c index 87c94191033b..8c6627bc8a39 100644 --- a/drivers/scsi/imm.c +++ b/drivers/scsi/imm.c @@ -892,7 +892,7 @@ static int imm_engine(imm_struct *dev, struct scsi_cmnd *cmd) /* Check for optional message byte */ if (imm_wait(dev) == (unsigned char) 0xb8) imm_in(dev, &h, 1); - cmd->result = (DID_OK << 16) + (l & STATUS_MASK); + cmd->result = (DID_OK << 16) | (l & STATUS_MASK); } if ((dev->mode == IMM_NIBBLE) || (dev->mode == IMM_PS2)) { w_ctr(ppb, 0x4); diff --git a/drivers/scsi/mesh.c b/drivers/scsi/mesh.c index 1753e42826dd..82e01dbe90af 100644 --- a/drivers/scsi/mesh.c +++ b/drivers/scsi/mesh.c @@ -594,9 +594,9 @@ static void mesh_done(struct mesh_state *ms, int start_next) ms->current_req = NULL; tp->current_req = NULL; if (cmd) { - cmd->result = (ms->stat << 16) + cmd->SCp.Status; + cmd->result = (ms->stat << 16) | cmd->SCp.Status; if (ms->stat == DID_OK) - cmd->result += (cmd->SCp.Message << 8); + cmd->result |= cmd->SCp.Message << 8; if (DEBUG_TARGET(cmd)) { printk(KERN_DEBUG "mesh_done: result = %x, data_ptr=%d, buflen=%d\n", cmd->result, ms->data_ptr, scsi_bufflen(cmd)); diff --git a/drivers/scsi/sym53c8xx_2/sym_glue.c b/drivers/scsi/sym53c8xx_2/sym_glue.c index 7320d5fe4cbc..5f10aa9bad9b 100644 --- a/drivers/scsi/sym53c8xx_2/sym_glue.c +++ b/drivers/scsi/sym53c8xx_2/sym_glue.c @@ -252,7 +252,7 @@ void sym_set_cam_result_error(struct sym_hcb *np, struct sym_ccb *cp, int resid) cam_status = sym_xerr_cam_status(DID_ERROR, cp->xerr_status); } scsi_set_resid(cmd, resid); - cmd->result = (drv_status << 24) + (cam_status << 16) + scsi_status; + cmd->result = (drv_status << 24) | (cam_status << 16) | scsi_status; } static int sym_scatter(struct sym_hcb *np, struct sym_ccb *cp, struct scsi_cmnd *cmd) diff --git a/drivers/scsi/sym53c8xx_2/sym_glue.h b/drivers/scsi/sym53c8xx_2/sym_glue.h index 805369521df8..e34801ae5d69 100644 --- a/drivers/scsi/sym53c8xx_2/sym_glue.h +++ b/drivers/scsi/sym53c8xx_2/sym_glue.h @@ -256,7 +256,7 @@ sym_get_cam_status(struct scsi_cmnd *cmd) static inline void sym_set_cam_result_ok(struct sym_ccb *cp, struct scsi_cmnd *cmd, int resid) { scsi_set_resid(cmd, resid); - cmd->result = (((DID_OK) << 16) + ((cp->ssss_status) & 0x7f)); + cmd->result = (DID_OK << 16) | (cp->ssss_status & 0x7f); } void sym_set_cam_result_error(struct sym_hcb *np, struct sym_ccb *cp, int resid); -- 2.16.4