lkml.org 
[lkml]   [2013]   [Jul]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[ 042/103] libata: skip SRST for all SIMG [34]7x port-multipliers
    Date
    3.10-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Tejun Heo <tj@kernel.org>

    commit 7a87718d92760fc688628ad6a430643dafa16f1f upstream.

    For some reason, a lot of port-multipliers have issues with softreset.
    SIMG [34]7x series port-multipliers have been quite erratic in this
    regard. I recall that it was better with some firmware revisions and
    the current list of quirks worked fine for a while. I think it got
    worse with later firmwares or maybe my test coverage wasn't good
    enough. Anyways, HPA is reporting that his 3726 setup suffers SRST
    failures and then the PMP gets confused and fails to probe the last
    port.

    The hope was that we try to stick to the standard as much as possible
    and soonish the PMPs and their firmwares will improve in quality, so
    the quirk list was kept to minimum. Well, it seems like that's never
    gonna happen.

    Let's set NO_SRST for all [34]7x PMPs so that whatever remaining
    userbase of the device suffer the least. Maybe we should do the same
    for 57xx's but unfortunately I don't have any device left to test and
    I'm not even sure 57xx's have ever been made widely available, so
    let's leave those alone for now.

    Signed-off-by: Tejun Heo <tj@kernel.org>
    Reported-by: "H. Peter Anvin" <hpa@zytor.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/ata/libata-pmp.c | 33 +++++++++++++++++----------------
    1 file changed, 17 insertions(+), 16 deletions(-)

    --- a/drivers/ata/libata-pmp.c
    +++ b/drivers/ata/libata-pmp.c
    @@ -389,9 +389,13 @@ static void sata_pmp_quirks(struct ata_p
    /* link reports offline after LPM */
    link->flags |= ATA_LFLAG_NO_LPM;

    - /* Class code report is unreliable. */
    + /*
    + * Class code report is unreliable and SRST times
    + * out under certain configurations.
    + */
    if (link->pmp < 5)
    - link->flags |= ATA_LFLAG_ASSUME_ATA;
    + link->flags |= ATA_LFLAG_NO_SRST |
    + ATA_LFLAG_ASSUME_ATA;

    /* port 5 is for SEMB device and it doesn't like SRST */
    if (link->pmp == 5)
    @@ -399,20 +403,17 @@ static void sata_pmp_quirks(struct ata_p
    ATA_LFLAG_ASSUME_SEMB;
    }
    } else if (vendor == 0x1095 && devid == 0x4723) {
    - /* sil4723 quirks */
    - ata_for_each_link(link, ap, EDGE) {
    - /* link reports offline after LPM */
    - link->flags |= ATA_LFLAG_NO_LPM;
    -
    - /* class code report is unreliable */
    - if (link->pmp < 2)
    - link->flags |= ATA_LFLAG_ASSUME_ATA;
    -
    - /* the config device at port 2 locks up on SRST */
    - if (link->pmp == 2)
    - link->flags |= ATA_LFLAG_NO_SRST |
    - ATA_LFLAG_ASSUME_ATA;
    - }
    + /*
    + * sil4723 quirks
    + *
    + * Link reports offline after LPM. Class code report is
    + * unreliable. SIMG PMPs never got SRST reliable and the
    + * config device at port 2 locks up on SRST.
    + */
    + ata_for_each_link(link, ap, EDGE)
    + link->flags |= ATA_LFLAG_NO_LPM |
    + ATA_LFLAG_NO_SRST |
    + ATA_LFLAG_ASSUME_ATA;
    } else if (vendor == 0x1095 && devid == 0x4726) {
    /* sil4726 quirks */
    ata_for_each_link(link, ap, EDGE) {



    \
     
     \ /
      Last update: 2013-07-24 01:41    [W:4.135 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site