Re: [Lost] [Patch] cdi: DMA und ein paar kleinere Dinge

[Antoine Kaufmann <toni@xxxxxxxxxxxxxxxx>]

Am Samstag, 22. Dezember 2007 11.37:01 schrieb Kevin Wolf:
> Antoine Kaufmann schrieb:
> > Am Samstag, 22. Dezember 2007 10.41:38 schrieb Kevin Wolf:
> >> Wäre es nicht sinnvoller, die Länge nicht ins open, sondern ins read zu
> >> packen? Puffer übergeben ohne Länge verursacht bei mir immer so ein
> >> leicht ungutes Gefühl. ;-)
> >
> > Das kann ich schon machen, die Konsequenz wäre aber, dass der
> > DMA-Controller erst hier im Read initialisiert würde, und das würde wohl
> > bei keinem Treiber funktionieren, da man ja die Daten meist erst lesen
> > will, wenn sie da sind. ;-)
>
> Den Puffer dann vielleicht einfach schon dem open mitgeben? Einfach
> damit er räumlich nicht so sehr von seiner Größe getrennt ist.
>
> > Naja, ich kann es auch so machen, dass der erste Treiber, der ein
> > LostIO-Interface hat, LostIO allgemein für den Treiber initialisiert und
> > ihn registiert. Aber gibt es bei uns Treiber, die das nicht brauchen?
> > Okay, im Moment vielleicht noch deine Netzwerkkarten, aber das wird doch
> > auch ändern, oder?
>
> Gute Frage. Mir fällt im Moment nicht viel ein, was ich dort über LostIO
> machen würde, außer vielleicht ein Konfigurationsinterface. Laß es halt
> mal drin.

In dieser 2. Version des Patches sind die Kommentare wie gewünscht ergänzt, 
und der Puffer wird jetzt bei den DMA-Funktionen beim open übergeben.