From: Damien Le Moal <damien.lemoal@opensource.wdc.com>
To: marius@psihoexpert.ro
Cc: linux-ide@vger.kernel.org
Subject: Re: Bug report for ahci-mvebu driver
Date: Thu, 8 Dec 2022 06:54:00 +0900 [thread overview]
Message-ID: <ddb8cb6d-96b9-9eaf-632b-f535c7536d87@opensource.wdc.com> (raw)
In-Reply-To: <1b74a7e40419b6f99870c21ca3be7834@psihoexpert.ro>
On 12/8/22 03:27, marius@psihoexpert.ro wrote:
> December 6, 2022 8:08 AM, "Damien Le Moal" <damien.lemoal@opensource.wdc.com> wrote:
>
>> By the way, what is the maximum speed of you pmp box ? Is it really 1.5
>> Gbps (sata gen 1) ?
>
> No. It's 3 Gbps.
Hmmm... OK. Given that when it works (with the revert), the speed
reduction is to 1.5 Gbps. So it seems that the revert changes the error
return into checking the lowest speed because we do not already have a
previous speed... See below.
>
>> Without reverting the above patch, could you please try the following patch ?
>
> I did more tests, with and without the patch, with and without the commit reverted.
> All logs start when eSATA cable is connected. I didn't include boot logs.
>
[...]
> [ 153.028797][ T30] resume, after do loop
> [ 153.088796][ T30] debounce, SCR=0x113
> [ 153.148796][ T30] debounce, SCR=0x113
> [ 153.208796][ T30] debounce, SCR=0x113
> [ 153.268796][ T30] debounce, SCR=0x113
> [ 153.328796][ T30] debounce, SCR=0x113
> [ 153.388795][ T30] debounce, SCR=0x113
> [ 153.448796][ T30] debounce, SCR=0x113
> [ 153.508796][ T30] debounce, SCR=0x113
> [ 153.568796][ T30] debounce, SCR=0x113
> [ 153.572654][ T30] resume, return at end of function
> [ 153.576511][ T30] hardreset, Offline=>Online
> [ 153.586069][ T30] hardreset, sata_pmp_supported, before check_ready
> [ 153.758800][ T30] ata2: SATA link up 1.5 Gbps (SStatus 113 SControl 310)
> [ 153.772539][ T30] ata2.15: Port Multiplier 1.2, 0x197b:0x0325 r193, 8 ports, feat 0xf/0x1f
>
> So, is this a quirk of the mvebu controller or the Areca DAS? BTW, Areca uses some mvebu SOC too.
> I'm going to try the Areca DAS with a few other controllers. I want to see what happens.
Can you try with no patches and no revert adding the following options to
your kernel command line ?
libata.force=2:3Gbps
and
libata.force=2:1.5Gbps
One at a time. Do not try them together :)
This will force the initial link speed to be set for ata2 port.
>
> Marius Dinu
>
>
--
Damien Le Moal
Western Digital Research
next prev parent reply other threads:[~2022-12-07 21:54 UTC|newest]
Thread overview: 67+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-11-01 19:21 Bug report for ahci-mvebu driver Dinu Marius
2022-11-06 7:05 ` Damien Le Moal
2022-11-07 9:22 ` marius
2022-11-08 6:27 ` Damien Le Moal
2022-11-09 19:52 ` marius
2022-11-09 19:55 ` marius
2022-11-10 2:05 ` Damien Le Moal
2022-11-11 22:32 ` marius
2022-11-12 1:57 ` Damien Le Moal
2022-11-14 20:52 ` marius
2022-11-15 3:02 ` Damien Le Moal
2022-11-15 7:28 ` marius
2022-11-15 8:10 ` Damien Le Moal
2022-11-18 18:24 ` Dinu Marius
2022-11-24 2:40 ` Damien Le Moal
2022-12-04 1:41 ` marius
2022-12-04 10:22 ` Pali Rohár
2022-12-04 21:14 ` marius
2022-12-04 21:46 ` Pali Rohár
2022-12-05 2:02 ` Damien Le Moal
2022-12-06 6:08 ` Damien Le Moal
2022-12-07 18:27 ` marius
2022-12-07 21:54 ` Damien Le Moal [this message]
2022-12-07 22:51 ` Damien Le Moal
2022-12-08 18:06 ` marius
2022-12-09 0:34 ` Damien Le Moal
2022-12-09 2:58 ` Damien Le Moal
2022-12-09 7:31 ` marius
2022-12-09 9:28 ` Damien Le Moal
2022-12-09 18:30 ` marius
2023-01-14 18:01 ` marius
2023-01-15 23:37 ` Damien Le Moal
2023-01-17 8:26 ` Damien Le Moal
2023-01-18 19:43 ` marius
2023-01-19 0:29 ` Damien Le Moal
2023-01-19 18:46 ` marius
2023-01-23 7:02 ` Damien Le Moal
2023-01-23 20:00 ` marius
2023-01-23 23:00 ` Damien Le Moal
2023-01-24 8:04 ` marius
2023-01-24 9:53 ` Damien Le Moal
2023-01-24 17:02 ` marius
2023-01-27 5:13 ` Damien Le Moal
2023-01-27 6:28 ` Damien Le Moal
2023-01-28 17:08 ` marius
2023-01-29 2:23 ` Damien Le Moal
2023-01-29 10:24 ` marius
2023-01-30 1:16 ` Damien Le Moal
2023-01-30 2:38 ` Damien Le Moal
2023-01-30 7:23 ` marius
2023-01-30 7:37 ` Damien Le Moal
2023-01-30 23:22 ` Damien Le Moal
2023-01-31 7:20 ` marius
2023-01-31 7:28 ` Damien Le Moal
2023-02-07 18:40 ` marius
2023-02-07 23:39 ` Damien Le Moal
2023-02-08 12:17 ` marius
2023-02-08 23:00 ` Damien Le Moal
2023-02-12 13:14 ` marius
2023-02-12 23:11 ` Damien Le Moal
2023-01-24 17:06 ` Bug report for sata_via driver marius
2023-01-25 1:26 ` Damien Le Moal
2022-12-08 21:26 ` Bug report for ahci-mvebu driver Pali Rohár
2022-12-09 1:54 ` Damien Le Moal
2022-11-26 14:26 ` Pali Rohár
2023-01-23 9:10 ` Hajo Noerenberg
2023-01-23 12:13 ` Damien Le Moal
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ddb8cb6d-96b9-9eaf-632b-f535c7536d87@opensource.wdc.com \
--to=damien.lemoal@opensource.wdc.com \
--cc=linux-ide@vger.kernel.org \
--cc=marius@psihoexpert.ro \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox