Ovládače od AMD majú skoro 2/3 DRM kódu jadra

.... jj, kometáře v hlavičkových souborech :-)

https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/plain/drivers/gpu/drm/amd/include/asic_reg/vega10/GC/gc_9_0_sh_mask.h?id=refs/tags/next-20170413
To je hrůza! Ten hlavičkovej soubor má přes 3 mega :-O

Ne, to opravdu není hrůza. Hrůza je, jak to pitomci psali.

Copak je to normální, mít x bloků po y makrech, kde se liší jenom číslovka v názvu makra?
- GRBM_STATUS_SE2, GRBM_STATUS_SE3
- CP_ROQ1_THRESHOLDS, CP_ROQ2_THRESHOLDS
- CP_ROQ_RB_STAT, CP_ROQ_RB1_STAT, CP_ROQ_RB2_STAT
- PA_SC_BINNER_E­VENT_CNTL_0, PA_SC_BINNER_E­VENT_CNTL_2, PA_SC_BINNER_E­VENT_CNTL_3
- SX_DEBUG_BUSY, SX_DEBUG_BUSY[2..5] - 65LOC každý blok
- ...

Jednak to vypadá hnusně a je to nepřehledný, ale za další je jedna hodnota pro různý instance uvedená pokaždý jinde. A jak udělají funkci, která přistupuje k N kanálům na základě indexu, kde oba registry mají stejnou definci? Udělají tabulku konstant a používají pointerovou arimetiku, nebo na přiřazení čtyř instancí jednoho registru padne switch na 15LOC? Nebo switch neznají a narvou tam if-else, aby to ztížili kompilátoru?

Takhle prasácky se píše soft jenom ve dvou případech. Pokud je práce zadaná do Indie, nebo pokud je programátor placený za LOC (ale v tom případě by šlo ještě nahnat řádky vložením maker do maker a jejich dokumentací). V obou případech je to na převrtání konečníku špičatou botou.

Jeste je tu moznost, ze to vykuchali z nejake dokumentace, ktera je pod NDA a nadale chteji aby ty zdrojaky odpovidali svou strukturou te dokumentaci.

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.