lkml.org 
[lkml]   [2019]   [Nov]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH AUTOSEL 4.19 210/237] scsi: megaraid_sas: Fix msleep granularity
    Date
    From: Shivasharan S <shivasharan.srikanteshwara@broadcom.com>

    [ Upstream commit 9155cf30a3c4ef97e225d6daddf9bd4b173267e8 ]

    In megasas_transition_to_ready() driver waits 180seconds for controller to
    change FW state. Here we are calling msleep(1) in a loop for this. As
    explained in timers-howto.txt, msleep(1) will actually sleep longer than
    1ms. If a faulty controller is connected, we will end up waiting for much
    more than 180 seconds causing unnecessary delays during load.

    Change the granularity of msleep() call from 1ms to 1000ms.

    Signed-off-by: Shivasharan S <shivasharan.srikanteshwara@broadcom.com>
    Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/scsi/megaraid/megaraid_sas_base.c | 4 ++--
    1 file changed, 2 insertions(+), 2 deletions(-)

    diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c b/drivers/scsi/megaraid/megaraid_sas_base.c
    index bc37666f998e6..2f94ab9c23540 100644
    --- a/drivers/scsi/megaraid/megaraid_sas_base.c
    +++ b/drivers/scsi/megaraid/megaraid_sas_base.c
    @@ -3894,12 +3894,12 @@ megasas_transition_to_ready(struct megasas_instance *instance, int ocr)
    /*
    * The cur_state should not last for more than max_wait secs
    */
    - for (i = 0; i < (max_wait * 1000); i++) {
    + for (i = 0; i < max_wait; i++) {
    curr_abs_state = instance->instancet->
    read_fw_status_reg(instance->reg_set);

    if (abs_state == curr_abs_state) {
    - msleep(1);
    + msleep(1000);
    } else
    break;
    }
    --
    2.20.1
    \
     
     \ /
      Last update: 2019-11-16 16:48    [W:4.210 / U:0.132 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site