linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bill Davidsen <davidsen@tmr.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: lkml <linux-kernel@vger.kernel.org>, linux-ide@vger.kernel.org
Subject: Re: IDE CDROM tail read errors
Date: Tue, 04 Apr 2006 15:09:20 -0400	[thread overview]
Message-ID: <4432C460.3080606@tmr.com> (raw)
In-Reply-To: <1143717489.29388.32.camel@localhost.localdomain>

Alan Cox wrote:
> On Mer, 2006-03-29 at 21:11 +0200, Krzysztof Halasa wrote:
>> # ls -l
>> -rw-r--r-- 1 root root 687177728 Mar 29 15:51 img-15.iso
>> -rw-r--r-- 1 root root 687177728 Mar 29 15:58 img-15a.iso
>>
>> The files are just truncated FC5d1 images (57344 bytes missing).
> 
> The final partial read is dropped rather than partially completed.
> 
>> # cat /sys/block/sr0/size
>> 1342264 (i.e., the same as with 2.6.15 + drivers/ide)
>>
>> # cat /dev/cdrw > img-16a.iso
>> cat: /dev/cdrw: Input/output error
>>
>> # cat /sys/block/sr0/size
>> 1342256 (looks like it has been adjusted to .iso image size / 512 when
>>          the first I/O error occured)
> 
> The SCSI layer does this bit for everyone. Its actually not libata or
> the PATA drivers that have done the work here. You should find ide-scsi
> does the same.
> 
> I patched the old IDE driver a bit to try and deal with this and if you
> want the patch to hack on and tidy up further feel free. 
> 
> 
Any hope in hell of getting this fixed? It's been broken for too long to 
remember. IIRC the problem is that on a read which hits EOF, instead of 
returning partial data read and no error, the status returned is just 
some "didn't work" variant and the partial read is lost. I also seem to 
remember that if the length of the ISO was a multiple of a magic number, 
32k, or the metric hexadecimal phase of the moon, you got a clean EOF.

As you say, ide-scsi fixes this, if the people who don't like it would 
make the alternatives work right it wouldn't be an issue...

Thanks for the info.

  parent reply	other threads:[~2006-04-04 19:09 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-29 19:11 IDE CDROM tail read errors Krzysztof Halasa
2006-03-30  8:19 ` Jan Engelhardt
2006-04-02 10:55   ` Krzysztof Halasa
2006-04-02 18:13     ` Jan Engelhardt
2006-03-30 11:18 ` Alan Cox
2006-04-02 10:50   ` Krzysztof Halasa
2006-04-04 19:09   ` Bill Davidsen [this message]
2006-04-12  8:57     ` Alan Cox

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=4432C460.3080606@tmr.com \
    --to=davidsen@tmr.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /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;
as well as URLs for NNTP newsgroup(s).