Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 16 Apr 2004 13:41:46 +0200
From:      =?UTF-8?B?U8O4cmVuIFNjaG1pZHQ=?= <sos@DeepCore.dk>
To:        Roman Kurakin <rik@cronyx.ru>
Cc:        Nate Lawson <nate@root.org>
Subject:   Re: cvs commit: src/sys/dev/ata ata-all.c ata-all.h ata-card.c   ata-lowlevel.c ata-pci.c ata-pci.h ata-queue.c
Message-ID:  <407FC67A.7090306@DeepCore.dk>
In-Reply-To: <407FC333.6000901@cronyx.ru>
References:  <20040413094519.7230616A4DB@hub.freebsd.org> <20040415162313.S92883@root.org> <407F9CB7.3020407@DeepCore.dk> <407FC333.6000901@cronyx.ru>

next in thread | previous in thread | raw e-mail | index | archive | help
Roman Kurakin wrote:
> Søren Schmidt wrote:
> 
>>> Includes fixes to avoid checking the ATA_FLAG_54_58 bit, fixing LBA
>>> detection:
>>> Submitted by:    rk
>>
>>
>> not true, rk submitted the problem not the code.
> 
> 
> Probably you are talking about me "rik" :-)
> I realy doesn't submit any code. I just tell my suggestions that 
> checking this bit
> is bad idea according to ata spec, and according last ata specs it 
> shouldn't be
> used at all since it obsoleted. So I didn't submit any fixes about it. 
> Only idea.
> So this may be "Asked by" or "Pointed by" :-)

Right, it may break ATA on old devices though, but I guess none of these 
drives are in use any more.
The real problem here is that I cannot just use the latest ATA specs (it 
would make my life *alot* simpler though :) ), I need to make things 
work that are compatible to at least back to ATA rev3 so just relying on 
things be obsoleted in the latest spec revision doesn't cut it..

> Code I submitted was to make ata driver more verbose about CHS mode.
> They still are not there and should be revised to make them possible to
> apply to current code.

I'll think about it, but since CHS mode is almost no longer used anyways
and the info can pretty easily be calculated from the LBA's printed..

> By the way, may be it would be a nice idea to warn users that they have
> ATA_FLAG_54_58 bit off, and device will work in LBA mode not
> in CHS as was before?

Users wont notice at all, from users perspective the disk is just a 
bunch of sectors, the interface method used to talk to the disk is 
nowhere to be seen...

-- 
-Søren



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?407FC67A.7090306>