Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 26 Oct 2017 17:04:23 -0600
From:      Alan Somers <asomers@freebsd.org>
To:        Warner Losh <imp@freebsd.org>
Cc:        "src-committers@freebsd.org" <src-committers@freebsd.org>,  "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>,  "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r325026 - head/sys/cam/ata
Message-ID:  <CAOtMX2gFAj-yQzG6pwNmjAWYaw8_o3RHRfiAVURfNd7ib-BFZg@mail.gmail.com>
In-Reply-To: <201710262253.v9QMrovZ009495@repo.freebsd.org>
References:  <201710262253.v9QMrovZ009495@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Oct 26, 2017 at 4:53 PM, Warner Losh <imp@freebsd.org> wrote:
> Author: imp
> Date: Thu Oct 26 22:53:49 2017
> New Revision: 325026
> URL: https://svnweb.freebsd.org/changeset/base/325026
>
> Log:
>   Always send STANDBY IMMEDIATE when shutting down
>
>   To save SMART data and for a drive to understand that it's been nicely
>   shutdown, we need to send a STANDBY IMMEDIATE. Modify adaspindown to
>   use a local CCB on the stack. When we're panicing, used
>   xpt_polled_action rather than cam_periph_runccb so that we can SEND
>   IMMEDIATE after we've shutdown the scheduler.
>
>   Sponsored by: Netflix
>   Reviewed by: scottl@, gallatin@
>   Differential Revision: https://reviews.freebsd.org/D12799
>
> Modified:
>   head/sys/cam/ata/ata_da.c

Will this put the drive into a standby state just prior to a warm
reboot?  That could cause lengthy delays on the new boot while the
drives spin up.  That behavior caused a problem when the mpr driver
did it to a JBOD full of 96 SATA drives.  On the new boot, each drive
spun up one at a time while they were being probed.  Eventually the
system paniced because run_interrupt_driven_hooks timed out.  With
mpr, I was able to fix the problem by setting hw.mpr.enable_ssu=0.

-Alan



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAOtMX2gFAj-yQzG6pwNmjAWYaw8_o3RHRfiAVURfNd7ib-BFZg>