[Eisfair_dev] sata_inic162x broken?

[Marcus Roeckrath]

Hallo Kay,

Kay Martinen wrote:

> "Possible Data Corruption" ist ein guter Grund. Meine Frage: Wieso ist
> so ein Treiber überhaupt im Kernel, speziell: Im Eisfair-Kernel. Wenn er
> komplett nutzlos ist, wäre es nicht besser ihr schmeist den Raus? Oder
> geht das bei Eisfair nicht so einfach?

Das muss Thomas oder das Team entscheiden.

> Ich weiß ehrlich gesagt nicht 
> woher ihr den Kernel nehmt, und ob/wie er von euch bearbeitet wird für
> Eisfair.

Es ist ein originaler Kernel von kernel.org mit einigen wenigen Patches.

>> http://lxr.free-electrons.com/source/drivers/ata/sata_inic162x.c
> 
> Und was bedeutet das hier (unten) nun?
> 
>>  39  * Then, I picked up their changes again and here's the usable driver
>>  40  * which uses IDMA for everything.  Everything works now including
>>  41  * LBA48, CD/DVD burning, suspend/resume and hotplug.  There are some
>>  42  * issues tho.  Result Tf is not resported properly, NCQ isn't
>>  43  * supported yet and CD/DVD writing works with DMA assisted PIO
>>  44  * protocol (which, for native SATA devices, shouldn't cause any
>>  45  * noticeable difference).
>>  46  *
>>  47  * Anyways, so, here's finally a working driver for inic162x.  Enjoy!

Ich weiß es nicht; das ist alles sehr zweischneidig formuliert.

> Heißt: Mit Linux Kernel 4.7 würde dieser Treiber; mit normalen
> SATA-Platten dann funktionieren??? Also alle neueren Distri's, außer
> Eisfair-1.

Das, was ich als Links zitiert habe, ist von 2006; im 4.xer-Kernel wird das
Modul wohl auch nicht besser funktionieren.

Und im aktuellsten 4.8er ist scheinbar immer noch der gleiche Treiber drin,
wie ich gerade in der Source nachgesehen habe und auch dort wird er laut

http://cateee.net/lkddb/web-lkddb/SATA_INIC162X.html

als VERY EXPERIMENTICAL bezeichnet.

> Generell sehe ich dahinter aber wieder mal eine art ausufernde
> Programmierer-chuzpe. Stürzen sich alle lieber auf neuen Code, als mal
> vorhandenen Besser zu machen.

Dazu müsste man genauer recherchieren, worum es da keine Entwicklung gibt.
Denkbar ist auch, dass der Hersteller nicht wirklich alle notwendigen
Informationen rausrückt.

-- 
Gruss Marcus