linux-kernel - Re: [2.6 patch] select ATA

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <480DF9BD.1080104@rtr.ca>
Date:	Tue, 22 Apr 2008 10:44:13 -0400
From:	Mark Lord <liml@....ca>
To:	Tejun Heo <htejun@...il.com>
Cc:	Sergei Shtylyov <sshtylyov@...mvista.com>,
	Jeff Garzik <jeff@...zik.org>, Adrian Bunk <bunk@...nel.org>,
	linux-kernel@...r.kernel.org, linux-ide@...r.kernel.org,
	Alan Cox <alan@...rguk.ukuu.org.uk>
Subject: Re: [2.6 patch] select ATA_SFF

Tejun Heo wrote:
> Mark Lord wrote:
>> Tejun Heo wrote:
>>> Sergei Shtylyov wrote:
>>>>>>    Jeff, Tejun, what "sff" in the file name actually means? Isn't 
>>>>>> it strange that the drivers lacking DMA support or not really 
>>>>>> compliant with SFF-8038i have to link with this file?
>>>>
>>>>> Maybe it should be libata-tf and libata-bmdma, but sff (sans bmdma)
>>>>> and bmdma is acceptable, hopefully, right?
>>>>
>>>>    What's sff sans bmdma?
>>>
>>> Supposed to be TF interface.  IIRC, the SFF term was first from Alan 
>>> although it's entirely possible that I misunderstood it and used it 
>>> in the wrong way.  Alan, can you please clear up the confusion?
>> ..
>>
>> SFF stands for "Small Form Factor", as in the "SFF Committee Information
>> Specification for Bus Master Programming Interface for IDE ATA 
>> Controllers Rev 1.0"
>> from May 16, 1994.
>>
>> That document is basically a committee rubber-stamp of the earlier Intel
>> "PCI IDE Controller Specification Revision 1.0" dated 3/4/94.
> 
> Ah.. that means we have the whole naming thing wrong.  Yewwww....
..

I suppose.   If we were needlessly pedantic, then perhaps
the libata-bmdma.c should be renamed to libata-sff.c,
and the libata-sff.c should be renamed libata-tf.c

But there's probably been a document or two since then,
where the SFF folks have documented the TF interfaces, too.

So.. whatever.  :)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/