Thema: CDI: AHCI-Treiber

[kevin]

Ich bin mir nicht sicher, ob alle, die CDI benutzen, auch auf der Mailingliste sind, deswegen auch hier nochmal ein kurzer Hinweis: Ich habe auf cdi-devel einen AHCI-Treiber gepostet, der gerne erstens ein Review bekommen und zweitens auch in anderen Betriebssystemen getestet werden würde.

Hier der Link zum Cover Letter: http://list.tyndur.org/pipermail/cdi-devel/2014-December/000264.html

Für alle anderen, die hier nur aus Interesse an týndur mitlesen: Die OS-spezifischen Patches dafür liegen auch schon bereit, dauert also hoffentlich nicht mehr lang.

[OsDevNewbie]

Eine Frage: wie kommt man auf die Mailingliste?

[kevin]

Man kann sich hier für cdi-devel und hier für tyndur-devel registrieren.

[OsDevNewbie]

Noch eine weitere Frage: wird der Treiber dann auch ins cdi repository aufgenommen?

[kevin]

Ja, natürlich. Wenn niemand mehr Kommentare hat, werde ich den Treiber demnächst ins Repository pushen.

[OsDevNewbie]

Wirsz du denn auch noch CD-ROM Unterstützung implementieren?
Ich würde ihn nämlich gerne in mein os aufnehmen.

[kevin]

CD-Unterstützung ist doch in v2 (was ich oben verlinkt habe) schon drin.

Oder vielleicht sollte ich besser SATAPI-Unterstützung sagen. Das bedeutet, dass du ein CDI-SCSI-Gerät bekommst, an das du SCSI-Pakete schicken kannst. Die Pakete musst du noch selber bauen. tyndur hat in seiner OS-spezifischen CDI-Lib noch einen Hack drin, der dann ein CDI-Storage-Gerät obendrauf setzt. Falls du SCSI nicht direkt unterstützen willst, könnte das einen Blick wert sein.

[OsDevNewbie]

CD-Unterstützung ist doch in v2 (was ich oben verlinkt habe) schon drin.

Oh, entschuldigung ich habe da irgendwie die nachricht der vorherigen Version gelesen.

Was ist mit den TODOs, die im Quellcode sind? Ich habe dort etwas gesehen mit

Code: [Auswählen]

//TODO 64-bit Unterstützung
Ist dieser Treiber denn nicht 64-bit tauglich?

[kevin]

Er macht kein 64-Bit-DMA, d.h. du brauchst physische Adressen < 4 GB für deine DMA-Puffer. Wenn du cdi_mem_alloc() korrekt implementiert hast, dann sollte das auch auf einem 64-Bit-System funktionieren, weil der Treiber CDI_MEM_DMA_4G benutzt, um diese Einschränkung anzuzeigen.

[OsDevNewbie]

Aber theoretisch würde es auch mit 64-bit funktionieren.

[kevin]

Theoretisch ja, ausprobiert habe ich es mangels 64-Bit-OS mit CDI nicht.

[kevin]

Der Treiber ist jetzt übrigens sowohl im CDI- als auch im tyndur-Repository committet.

Ein Problem mit tyndur ist im Moment noch, dass die Bootkonfiguration für IDE und AHCI unterschiedlich sein muss. Gerade die Live-CD sollte eigentlich einfach das richtige nehmen. Das sollte man vermutlich bis zum nächsten Release noch verbessern. Also falls sich jemand berufen fühlt...?

[OsDevNewbie]

Eine Frage habe ich noch: fwas macht das Macro CDI_BUILD_BUG_ON()?

[kevin]

Es produziert einen Compilerfehler, wenn die Bedingung falsch ist. Quasi wie ein assert() zur Compilezeit.

[OsDevNewbie]

Und wo wird das verwendet?
Und für was kann man das verwenden?

P.S: Vielleicht werde ich jetzt ein bisschen Offtopic. Wenn ja kann ich ja einen eigenen Thread öffnen für dieses Thema.

[Svenska]

Du kannst da im Prinzip jede Bedingung eintragen, die der Compiler zur Compilezeit auflösen kann. Einen Teil kannst du auch mit dem Präprozessor erschlagen (#ifdef usw.), aber bestimmte Dinge wie sizeof() oder offsetof() kann man dort nicht zuverlässig verwenden. Dafür gibt es dann solche BUILD_BUG-Makros.

Ich benutze eins z.B., um sicherzustellen, dass ein Array nicht mehr Speicher benutzt, als da ist: "BUILD_BUG(SIZE_X * SIZE_Y * sizeof(node_t) > 24*1024)", wobei SIZE_X, SIZE_Y und node_t einstellbar sind. Die kann man im Prinzip beliebig einstellen, aber wenn das Ergebnis größer als 24 KB wird, funktioniert es halt nicht. Das fange ich damit ab.