Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 27 Jul 2002 17:09:02 -0700
From:      Kent Stewart <kstewart@owt.com>
To:        "Dan O'Brien" <dmobrien_2001@yahoo.com>
Cc:        Jesse Gross <jesse_gross@yahoo.com>, bmah@FreeBSD.ORG, FreeBSD-Stable Mailing List <freebsd-stable@FreeBSD.ORG>, Eric Chet <ejc@bazzle.com>
Subject:   Re: ATA CDROM Problems 4.6 (and 4.6.1)
Message-ID:  <3D43361E.3060204@owt.com>
References:  <20020727221358.37195.qmail@web9901.mail.yahoo.com>

next in thread | previous in thread | raw e-mail | index | archive | help


Dan O'Brien wrote:

> This patch is golden! 
> 
> I tested it using the former failing scenario:
> 
> mount /cdrom
> cd /cdrom/ports
> cat ports.tgz > /dev/null
> 
> Then I hammered it doing:
> 
> cd /cdrom
> tar cvf /dev/null .
> 
> It hangs together, no errors, no panics.
> 
> Put this patch up on the 4.6 Errata, please!
> 


Sos has two patches to atapi-cd.c that have been added to -current. 
The first one changed the 252 to 126 (instead of 64) and the second 
one changed some parameter calculations. You may still need the other 
patch. I imagine they will be MFCed soon.

Kent


> Thanks.
> 
> --- Jesse Gross <jesse_gross@yahoo.com> wrote:
> 
>>--- Dan O'Brien <dmobrien_2001@yahoo.com> wrote:
>>
>>>--- "Bruce A. Mah" <bmah@FreeBSD.ORG> wrote:
>>>
>>>>If memory serves me right, "Dan O'Brien" wrote:
>>>>
>>>>>I just upgraded from 4.5 (off the CD Release) to 4.6 (off the
>>>>>
>>CD
>>
>>>>>release) and yadda yadda yadda ATA CDROM fails per the Errata
>>>>>
>>>(big
>>>
>>>>read
>>>>
>>>>>timeout, etc).  
>>>>>
> 
>>Actually there is a newer patch that people (including myself) have
>>had
>>luck with.
>>
>>Try the patch below.
>>
>>Jesse Gross
>>
>>Index: atapi-cd.c
>>===================================================================
>>RCS file: /home/iedowse/CVS/src/sys/dev/ata/atapi-cd.c,v
>>retrieving revision 1.48.2.16
>>diff -u -r1.48.2.16 atapi-cd.c
>>--- atapi-cd.c	27 Mar 2002 19:48:37 -0000	1.48.2.16
>>+++ atapi-cd.c	26 Jul 2002 14:22:37 -0000
>>@@ -266,7 +266,7 @@
>>     dev = make_dev(&acd_cdevsw, dkmakeminor(cdp->lun, 0, 0),
>> 		   UID_ROOT, GID_OPERATOR, 0644, "acd%d", cdp->lun);
>>     dev->si_drv1 = cdp;
>>-    dev->si_iosize_max = 252 * DEV_BSIZE;
>>+    dev->si_iosize_max = 64 * DEV_BSIZE;
>>     dev->si_bsize_phys = 2048; /* XXX SOS */
>>     cdp->dev = dev;
>>     cdp->device->flags |= ATA_D_MEDIA_CHANGED;
>>
> 
> 
> =====
> Dan O'Brien, dmobrien_2001@yahoo.com
> Cell: 614-783-4859  Work: 614-476-8473
> Home: 740-927-2178  Pataskala, OH  USA
> '02 Suzuki DL1000 #00082 Grey
> 
> __________________________________________________
> Do You Yahoo!?
> Yahoo! Health - Feel better, live better
> http://health.yahoo.com
> 
> To Unsubscribe: send mail to majordomo@FreeBSD.org
> with "unsubscribe freebsd-stable" in the body of the message
> 
> .
> 
> 


-- 
Kent Stewart
Richland, WA

http://users.owt.com/kstewart/index.html


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-stable" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3D43361E.3060204>