lkml.org 
[lkml]   [2016]   [Nov]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.12 035/127] scsi: megaraid_sas: Fix data integrity failure for JBOD (passthrough) devices
    Date
    From: Kashyap Desai <kashyap.desai@broadcom.com>

    3.12-stable review patch. If anyone has any objections, please let me know.

    ===============

    commit 1e793f6fc0db920400574211c48f9157a37e3945 upstream.

    Commit 02b01e010afe ("megaraid_sas: return sync cache call with
    success") modified the driver to successfully complete SYNCHRONIZE_CACHE
    commands without passing them to the controller. Disk drive caches are
    only explicitly managed by controller firmware when operating in RAID
    mode. So this commit effectively disabled writeback cache flushing for
    any drives used in JBOD mode, leading to data integrity failures.

    [mkp: clarified patch description]

    Fixes: 02b01e010afeeb49328d35650d70721d2ca3fd59
    Signed-off-by: Kashyap Desai <kashyap.desai@broadcom.com>
    Signed-off-by: Sumit Saxena <sumit.saxena@broadcom.com>
    Reviewed-by: Tomas Henzl <thenzl@redhat.com>
    Reviewed-by: Hannes Reinecke <hare@suse.com>
    Reviewed-by: Ewan D. Milne <emilne@redhat.com>
    Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
    Signed-off-by: Jiri Slaby <jslaby@suse.cz>
    ---
    drivers/scsi/megaraid/megaraid_sas_base.c | 13 +++++--------
    1 file changed, 5 insertions(+), 8 deletions(-)

    diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c b/drivers/scsi/megaraid/megaraid_sas_base.c
    index 8c3270c809c8..11eafc3f4ca0 100644
    --- a/drivers/scsi/megaraid/megaraid_sas_base.c
    +++ b/drivers/scsi/megaraid/megaraid_sas_base.c
    @@ -1537,16 +1537,13 @@ megasas_queue_command_lck(struct scsi_cmnd *scmd, void (*done) (struct scsi_cmnd
    goto out_done;
    }

    - switch (scmd->cmnd[0]) {
    - case SYNCHRONIZE_CACHE:
    - /*
    - * FW takes care of flush cache on its own
    - * No need to send it down
    - */
    + /*
    + * FW takes care of flush cache on its own for Virtual Disk.
    + * No need to send it down for VD. For JBOD send SYNCHRONIZE_CACHE to FW.
    + */
    + if ((scmd->cmnd[0] == SYNCHRONIZE_CACHE) && MEGASAS_IS_LOGICAL(scmd)) {
    scmd->result = DID_OK << 16;
    goto out_done;
    - default:
    - break;
    }

    if (instance->instancet->build_and_issue_cmd(instance, scmd)) {
    --
    2.10.2
    \
     
     \ /
      Last update: 2016-11-25 10:08    [W:5.519 / U:0.140 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site