lkml.org 
[lkml]   [2020]   [Oct]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.9 749/757] s390/qeth: dont let HW override the configured port role
    Date
    From: Julian Wiedmann <jwi@linux.ibm.com>

    [ Upstream commit a04f0ecacdb0639d416614619225a39de3927e22 ]

    The only time that our Bridgeport role should change is when we change
    the configuration ourselves. In which case we also adjust our internal
    state tracking, no need to do it again when we receive the corresponding
    event.

    Removing the locked section helps a subsequent patch that needs to flush
    the workqueue while under sbp_lock.

    It would be nice to raise a warning here in case HW does weird things
    after all, but this could end up generating false-positives when we
    change the configuration ourselves.

    Suggested-by: Alexandra Winter <wintera@linux.ibm.com>
    Signed-off-by: Julian Wiedmann <jwi@linux.ibm.com>
    Reviewed-by: Alexandra Winter <wintera@linux.ibm.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/s390/net/qeth_l2_main.c | 6 ------
    1 file changed, 6 deletions(-)

    diff --git a/drivers/s390/net/qeth_l2_main.c b/drivers/s390/net/qeth_l2_main.c
    index 9866d01b40fe7..4af7b5d57b4e4 100644
    --- a/drivers/s390/net/qeth_l2_main.c
    +++ b/drivers/s390/net/qeth_l2_main.c
    @@ -1120,12 +1120,6 @@ static void qeth_bridge_state_change_worker(struct work_struct *work)
    NULL
    };

    - /* Role should not change by itself, but if it did, */
    - /* information from the hardware is authoritative. */
    - mutex_lock(&data->card->sbp_lock);
    - data->card->options.sbp.role = entry->role;
    - mutex_unlock(&data->card->sbp_lock);
    -
    snprintf(env_locrem, sizeof(env_locrem), "BRIDGEPORT=statechange");
    snprintf(env_role, sizeof(env_role), "ROLE=%s",
    (entry->role == QETH_SBP_ROLE_NONE) ? "none" :
    --
    2.25.1


    \
     
     \ /
      Last update: 2020-10-27 17:14    [W:4.068 / U:0.640 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site