Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 16 Sep 2005 15:22:29 -0500
From:      Patrick Bowen <pbowen@fastmail.fm>
To:        Freebsd-current@FreeBSD.org
Subject:   Re: ATA error on 6.0-BETA4
Message-ID:  <432B2985.50302@fastmail.fm>
In-Reply-To: <2B22BF98-3FF4-4BE5-B8FF-56BB07CDB4D9@FreeBSD.org>
References:  <1126047871.687.1.camel@localhost> <E52963F6-A762-4719-A6AD-4412C0011C17@FreeBSD.org> <4328D0F7.2090501@fastmail.fm> <2B22BF98-3FF4-4BE5-B8FF-56BB07CDB4D9@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Søren Schmidt wrote:

> On 15/09/2005, at 3:40, Patrick Bowen wrote:
>
>> I've notice the same behaviour on my Dell C600. I've attached  
>> verbose dmesg's for -current and 5.4. When I boot 5.4 the CD-RW is  
>> found and I have no problems using it for either reading or  writing. 
>> I hope they're useful for you.
>
>
> OK, please try the below patch and let me know if that helps any.
>
>> I'd like to say that I have the greatest respect for *all* of you  
>> that work at making FreeBSD the great OS that it is. I just wonder  
>> where you find the time to work on it, what with your day jobs,  
>> family responsibilities and all...
>
>
> I guess we have understanding environments and need for less sleep  
> than usual :)
>
> Index: ata-lowlevel.c
> ===================================================================
> RCS file: /home/ncvs/src/sys/dev/ata/ata-lowlevel.c,v
> retrieving revision 1.71
> diff -u -r1.71 ata-lowlevel.c
> --- ata-lowlevel.c      14 Sep 2005 12:45:06 -0000      1.71
> +++ ata-lowlevel.c      15 Sep 2005 07:35:41 -0000
> @@ -278,7 +278,7 @@
>             /* if read data get it */
>             if (request->flags & ATA_R_READ) {
> -               if (ata_wait(ch, atadev, (ATA_S_READY | ATA_S_DRQ)) <  
> 0) {
> +               if (ata_wait(ch, atadev, ATA_S_DRQ) < 0) {
>                     device_printf(request->dev,
>                                   "timeout waiting for read DRQ\n");
>                     request->result = EIO;
>
>
> Søren Schmidt
> sos@FreeBSD.org
>
>
>
>
Søren:

Outstanding! The drive is now recognized at boot, I can read and write 
CD's, and even listen to audio CD's. It all seems to work! Such a small 
change had a huge impact.

If you don't mind, what exactly was the purpose for making the change 
(other than to make things work)?

Thanks for everything,
Patrick



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?432B2985.50302>