Date: Thu, 6 May 2004 21:36:31 +1000 (EST) From: Bruce Evans <bde@zeta.org.au> To: =?ISO-8859-1?Q?S=F8ren_Schmidt?= <sos@DeepCore.dk> Cc: current@FreeBSD.org Subject: Re: ATA_FLUSHCACHE failing Message-ID: <20040506212202.O19882@gamplex.bde.org> In-Reply-To: <409948A7.1040100@DeepCore.dk> References: <20040505195425.GA2559@laptop.6bone.nl> <409948A7.1040100@DeepCore.dk>
next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 5 May 2004, [ISO-8859-1] S=F8ren Schmidt wrote:
> Mark Santcroos wrote:
> > Hi,
> >
> > I finally took the time to find out why I can't get a coredump on panic=
:
> >
> > ata_controlcmd(&ch->device[MASTER], ATA_FLUSHCACHE, 0, 0, 0);
> >
> > never returns in ata_shutdown() on my system (dmesg attached).
> >
> > Commenting it out gives me back my coredumps! ;)
> >
> > Have you seen this before?
>
> Nope, but I have seen a few disks that claims to support flush and then
> do wierd things when asked to...
>
> > If not, should we maybe create some blacklist of chips that report that
> > they can flush but in reality can't ...
>
> That would be blacklist of disks then, hmm, I'm not a fan of blacklists
> actually, they tend to always be as incorrect and incomplete as not
> having any. We should find out why it does not return, my guess is that
> it doesn't interrupt and the timeout doesn't fire because we are on the
> way down...
Panic dumps cannot use either device interrupts or timeouts, not to
mention normal locks, since panics may occur in almost any context;
in particular they may occur with all interrupts masked and important
locks in a bad state. Immediate mode must be used, but it doesn't
seem to be, and a comment says that it is only used for other things:
% void
% ata_queue_request(struct ata_request *request)
% {
% /* mark request as virgin */
% request->result =3D request->status =3D request->error =3D 0;
% if (!request->callback && !(request->flags & ATA_R_REQUEUE))
% =09sema_init(&request->done, 0, "ATA request done");
%
% /* in IMMEDIATE_MODE we dont queue but call HW directly */
% /* used only during reinit for getparm and config */
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
% if ((request->device->channel->flags & ATA_IMMEDIATE_MODE) &&
% =09(request->flags & (ATA_R_CONTROL | ATA_R_IMMEDIATE))) {
Bruce
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040506212202.O19882>