Hlavní navigace

Názor ke zprávičce Ovládače od AMD majú skoro 2/3 DRM kódu jadra od pc2005 - Ta délka hrůza je. Tolik registrů v jednom...

  • 18. 4. 2017 23:22

    pc2005 (neregistrovaný) 2001:1ae9:ff1:----:----:----:----:----

    Ta délka hrůza je. Tolik registrů v jednom HW jsem nikde neviděl. Co jsem se koukal do git next, tak nejdelší řádek má přes 160 znalů (při více jak 80 znaků ti spousta maintainerů odmítne patch).

    V jednom 13MB headeru (! to je víc než má zkompilovaný kernel) je ta define hodnota zarovnaná na cca 110 znak a stejně jim to nestačilo, takže na pár řádkách jsou define hodnoty vyteklý ze zarovnání.

    Odsazování mezerama. "//" komentáře (no ty mě nevadí, ale už mě s nima odmítli pár patchů). "BIF_BX_DEV0_­EPF0_VF10_DOOR­BELL_SELFRING_GPA_A­PER_BASE_HIGH__DO­ORBELL_SELFRIN­G_GPA_APER_BA­SE_HIGH__SHIF­T" nechápu proč je tam ten string dvakrát.

    Kdyby dali každej ten blok definů do samostanýho souboru a zkrátili mu jméno, tak by ušetřili tak 50+% místa.

    Spousta těch definů je "shift" a "mask". Chci vidět jak ty 100+ znakový jména používaj pro vymaskování hodnoty z bitfieldu v nějakým registru :-D.

    Teďka kdyby všechny ty registry použili v driveru, tak by se objem AMD DRM prakticky zdvojnásobil.