Portál AbcLinuxu, 1. května 2025 12:31

Nástroje: Začni sledovat (0) ?Zašle upozornění na váš email při vložení nového komentáře.

Vložit další komentář
20.5.2008 16:41 zde | skóre: 9 | blog: Linuch | Brno
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
Odpovědět | Sbalit | Link | Blokovat | Admin
> with all the quirky metadata to parse

Co blbneš? Jde o parsování triviální tabulky, na pár set bajtů kódu. Když se to dá do userspace binárky, tak 1) se to díky ELF hlavičkám, zarovnání stránek linkerem apod natáhne na nějakých 16kB+ 2) se zavede další zbytečné API. Podle mě jde o dosti blbý nápad.
Táto, ty de byl? V práci, já debil.
michich avatar 20.5.2008 22:08 michich | skóre: 51 | blog: ohrivane_parky
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
Je to trochu víc:
$ wc -l fs/partitions/*.[ch]
...
 5463 celkem
$ size fs/partitions/*.o | perl -e 'while(<>){if(/^((\s*\d+){4})/){$_=$1;@line=split;($text,$data,$bss,$dec)=($text+$line[0],$data+$line[1],$bss+$line[2],$dec+$line[3]);}} print "text:$text, data:$data, bss:$bss, dec:$dec\n";'
text:27707, data:420, bss:132, dec:28259
21.5.2008 10:44 zde | skóre: 9 | blog: Linuch | Brno
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
Tak to jsi první člověk, který na PC používá disk s partition table z Amigy, odpoledne to vymění za disk z Acorna, na noc tam dá disk z Atárka, který ještě o půlnoci vymění za další disk z MACa. .-) Navíc jsi zřejmě do součtu zahrnul i built-in.o, takže máš všechno dvakrát. Vanilla 2.6.25, defconfig + CONFIG_PARTITION_ADVANCED=y
$ rm fs/partitions/built-in.o
$ size -t fs/partitions/*.o
   text    data     bss     dec     hex filename
   2891     144       0    3035     bdb fs/partitions/check.o
   1488       0       0    1488     5d0 fs/partitions/msdos.o
   4379     144       0    4523    11ab (TOTALS)
check.o by v jádru z velké části musel stejně zůstat, takže jde jen o msdos.o, který má necelý 1.5k. Fakt si myslíš že má cenu dělat userspace subsystém kvůli takové drobnosti?
Táto, ty de byl? V práci, já debil.
michich avatar 20.5.2008 22:11 michich | skóre: 51 | blog: ohrivane_parky
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
A kde je to další API? Device mapper snad umí všechno potřebné.
21.5.2008 10:50 zde | skóre: 9 | blog: Linuch | Brno
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
DM je tím dalším api. Když ho bude používat každý disk, bude se to z jádra odstraňovat mnohem hůř :-)
Táto, ty de byl? V práci, já debil.
22.5.2008 22:50 Petr Ročkai
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
To bude, no. Na druhé straně, jestli to vůbec někdy projde, tak to taky hnedka nepovede k zrušení původního kódu pro parsování tabulek, spíš se to zatím jen vyseká z konfigurace distribučního jádra. Takže radikální minimalisté tam ten kód můžou nechat, nadále používat statický /dev a případně i monolitické jádro : - ). Dát něco pryč z Linuxu je na docela dost dlouho...
21.5.2008 09:22 Marek 'marx' Grác | skóre: 21 | blog: Paralelný blog | Brno / Bratislava
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
Autor príspevku je (plateným) vývojárom LVM. Takže o tom ako to okolo toho vyzerá by istý prehľad mohol mať :) Či je dobrý, alebo blbý nápad sa vášnivo debatovalo aj na Fedora liste.
22.5.2008 22:45 Petr Ročkai
Rozbalit Rozbalit vše Re: ATOL: Moving partition code to userspace
1) Ne, neblbnu.

2) Strawman -- nikde jsem neargumentoval, že toho kódu je nějak mega moc nebo že je nějak hodně složitý. Těch formátů je poměrně hodně a podpora pro ně se nemůže jen tak zahodit -- ale stačí, aby existoval jeden balík kódu, který je parsuje (a z existujících bych spíš zachovával ten v userspace než ten v kernelu).

3) Ten kód už v userspace existuje (viz článek a kpartx).

4) Množství rozhraní (API) půjde spíš dolů než nahoru (všechno co návrh použivá už existuje a používá se jinde a má potenciál odstranit existující rozhraní v kernelu).

Ještě něco?

Založit nové vláknoNahoru

Tiskni Sdílej: Linkuj Jaggni to Vybrali.sme.sk Google Del.icio.us Facebook

ISSN 1214-1267, (c) 1999-2007 Stickfish s.r.o.