Current-Users archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: IDENTIFY failed



It's also possible that 2 full seconds of delays are unnecessary. Do those delays really need to be 500ms each?

On Thu, 4 Nov 2021, Rin Okuyama wrote:

Yeah, I know that. But, we already have two problem reports. What I am concerned about is similar problems will occur for a lot of machines.

(Thinking again...) But, yes, by this way, innocent people will be punished
forever by extra seconds per boot...

Hmm, if affected hardware is somehow limited, we can just introduce something like AHCI_QUIRK_EXTRADELAY. Otherwise, we can reconsider, for example, before
NetBSD 10 is released.

Jun, Patrick, can you please provide full dmesg for your machines?

Thanks,
rin

On 2021/11/04 19:58, Jared McNeill wrote:
 From the commit message:

  There are a handful of inexplicable 500ms delays introduced to the drive   detect path in this driver, slowing boot. They can be re-enabled with   options AHCISATA_EXTRA_DELAY, but should not be enabled for normal
   kernels.
  If a delay does need to be introduced in these places, the value should   either be more carefully selected or the scope limited to hardware that
   requires the extra delay.

I don't have any hardware that has problems with the delays removed, so go ahead and revert this commit if you're happy with that as a solution. It would be better to fix the problem properly though as this costs multiple seconds per drive at boot.

Take care,
Jared


On Thu, 4 Nov 2021, Rin Okuyama wrote:

Can't we put back AHCISATA_EXTRA_DELAY by default?

IIUC, the option affects only probe/reset; no bad effects for
I/O performance.

Thanks,
rin

On 2021/11/01 21:19, Patrick Welche wrote:
On Fri, Oct 29, 2021 at 01:05:26PM +0900, Jun Ebihara wrote:
From: matthew green <mrg%eterna.com.au@localhost>
Subject: re: IDENTIFY failed
Date: Fri, 29 Oct 2021 07:18:09 +1100

autoconfiguration error: ahcisata0 port 1: setting WDCTL_RST failed for drive 0
https://mail-index.netbsd.org/current-users/2021/10/27/msg041615.html
this one has reduced timeframe, too:
between
NetBSD 9.99.91 (GENERIC) #0: Tue Oct 12 19:57:53 UTC 2021 OK
NetBSD 9.99.92 (GENERIC) #0: Mon Oct 25 20:32:38 UTC 2021 Failed
which changed how some interrupt handling works, and:
    http://mail-index.netbsd.org/source-changes/2021/10/11/msg132941.html which removed some delays in the probe path.  possibly this one
is more likely to be at fault since it touches the probe path
directly.

add
/usr/src/sys/arch/amd64/conf/GENERIC.local
options         AHCISATA_EXTRA_DELAY

compile kernel

That did the trick - thanks! (Wanted to be near the box before trying it)


Cheers,

Patrick




Home | Main Index | Thread Index | Old Index