From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Volker Kuhlmann <list0570@paradise.net.nz>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
linux-ide@vger.kernel.org, Jens Axboe <axboe@suse.de>
Subject: Re: ide-cd turning off DMA when verifying DVD-R
Date: Fri, 13 Jan 2006 10:19:17 +0100 [thread overview]
Message-ID: <58cb370e0601130119g5c62b749r1bc5da59a0d4a56c@mail.gmail.com> (raw)
In-Reply-To: <20060113083009.GE12338@paradise.net.nz>
Hi,
On 1/13/06, Volker Kuhlmann <list0570@paradise.net.nz> wrote:
> On Fri 13 Jan 2006 17:40:33 NZDT +1300, Robert Hancock wrote:
>
> > I'm thinking the IDE code is too aggressive in assuming that the failure
> > is because of a DMA problem and disabling it.. Most likely all that's
> > happening is the drive is taking a long time to complete the current
> > command.
What actually happened is that normal command timed out
and because of that driver reset the device which caused
it to loose DMA:
->ide_atapi_error()
->ide_do_reset()
->pre_reset()
->check_dma_crc()
->__ide_dma_off()
Somebody needs to investigate why __ide_dma_off() is called
et all and if we need to restore DMA after reset (don't count ATM
on me, I'm buried by bugreports). Ondrej, could you fill the bug at
http://bugzilla.kernel.org so we don't lose it?
> Yes! Each time when inserting a faulty CD/DVD, or whenever the drive
> gives read errors for whatever reason, the kernel decides to turn DMA
> off and try again, fail (again) and leave DMA off. And this after having
> successfully used DMA before - so it's not that the device is
> DMA-incapable.
This is a separate issue and is related to retrying failed DMA
commands. Driver doesn't know what was the cause of the timeout
and it shouldn't disable DMA for the device (only the failed request
should be retried in PIO mode).
Any takers? :)
PS1 please don't trim cc: list
PS2 please use linux-ide@vger.kernel.org for ATA problem
Thanks,
Bartlomiej
next prev parent reply other threads:[~2006-01-13 9:19 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <5ujmU-1UQ-665@gated-at.bofh.it>
[not found] ` <5uoqr-Qq-7@gated-at.bofh.it>
2006-01-13 4:40 ` Robert Hancock
2006-01-13 8:30 ` Volker Kuhlmann
2006-01-13 9:19 ` Bartlomiej Zolnierkiewicz [this message]
2006-01-13 13:03 ` Ondrej Zary
[not found] ` <58cb370e0601130121s2f6c0a26jda00ff64df197342@mail.gmail.com>
[not found] ` <20060113093818.GA22984@sci.fi>
[not found] ` <58cb370e0601130149g32323b4axbf0ac55f83ac9148@mail.gmail.com>
[not found] ` <20060113112510.GA23264@sci.fi>
2006-01-13 13:33 ` Fwd: " Bartlomiej Zolnierkiewicz
2006-01-13 14:24 ` Alan Cox
[not found] ` <20060113140618.GB12360@tigershaunt.com>
2006-01-13 14:16 ` Bartlomiej Zolnierkiewicz
2006-01-12 21:35 Ondrej Zary
2006-01-13 4:04 ` Ville Syrjälä
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=58cb370e0601130119g5c62b749r1bc5da59a0d4a56c@mail.gmail.com \
--to=bzolnier@gmail.com \
--cc=axboe@suse.de \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=list0570@paradise.net.nz \
/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
all inboxes | Powered by JetHome®