Dotaz: pthreads - predani promenne

[mik]

zdravim snazim se naucit pracovat s pthreads, ale bohuzel se mi nedari rozjet ani vzorovy priklad z jednoho manualu. Poradite mi nekdo co s tim?

prekladam to pomoci:

gcc test.cpp -o test -Wall -pedantic -pthread

a haze mi to chybu:

test.cpp: In function ‘void* PrintHello(void*)’: test.cpp:8: error: cast from ‘void*’ to ‘int’ loses precision

a zdrojak je tento:

void *PrintHello(void *threadid)
{
   int tid=(int)threadid;
   printf("Hello World! It's me, thread #%d!\n", tid);
   pthread_exit(NULL);
}


int main (int argc, char *argv[])
{
   pthread_t   threads[5];
   int         rc, t;
   for(t=0; t<5; t++)
   {
      printf("In main: creating thread %d\n", t);
      rc = pthread_create(&threads[t], NULL, PrintHello, (void *)t);
      if(rc)
      {
         printf("ERROR; return code from pthread_create() is %d\n", rc);
         return -1;
      }
   }
   pthread_exit(NULL);

}

omlouvam se ze na vas vyrukuji asi s trivialnim dotazem, ale opravdu netusim co s tim.....

Odpovědi

[Luboš Doležel (Doli)]

int tid=(int)(long)threadid;

by mělo jít.

[mik]

bohuzel nepomohlo.....

/tmp/ccsYEnqT.o:(.eh_frame+0x12): undefined reference to `__gxx_personality_v0' collect2: ld returned 1 exit status

-pedantic odstranen

[Andrej Herceg]

Problém je prípona .cpp (buď sa to musí kompilovať s g++, alebo sa musí prípona zmeniť na .c).

[Roger]

Prekladej to s g++ misto gcc.

[depka]

jeste prilinkovat knihovny: -lstdc++ a zrejme asi jeste pthread,a le ta nevim jak se menuje

[Andrej Herceg]

O tú knižnicu pthread sa postará parameter -pthread (ale je pravda, že ak sa tam pridá -lstdc++, tak sa to skompiluje správne).

[Michal Kubeček]

Trochu mi vrtá hlavou, co vede člověka k použití '-pedantic' při překladu zdrojáku, kde se přetypovává void* na int… :-)

Buď to překládejte s méně striktními parametry nebo to předání proveďte čistším způsobem.

[mik]

ze by spatne zdroje a nezkusenost zacatecnika? ani po odstraneni -pedantic tak to nepomohlo

[Ash]

V tomto případě to pedantic/nepedantic není relevantní, protože gcc hlásí error (a -pedantic-error použit není). K hlášení error místo warning asi nějké důvody vedly, ale to bych nespekuloval.

[Michal Kubeček]

Tak za to nemůže pedantic, ale čtyřkové GCC, které je pedantické samo od sebe. Důvod je jednoduchý: pokud je pointer delší než int (64-bitová platforma?), pak tam prostě ke ztrátě informace dochází. Překladač nemůže vědět, že to nevadí, protože to stejně vždycky voláte s pointerem, který sám vznikl přetypováním z int.

[Ash]

Důvod není jednoduchý, důvod je spíš dost spekulacíhodný.

Jde o to že gcc u tohoto konkrátního přiřazení, které se masivně objevuje na 64bitových platformách (tedy přiřazení 64bitový pointer na 32bitový integer) hlásí error místo warning, přestože u jiných přiřazení podobného charakteru hlásí jen tradiční warning.

Asi (...) proto, že je z 32bitů příliš zažité že velikosti pointru a integeru jsou stejné. Ale to není jednoduchý důvod. To je z mé strany opravdu jen domněnka... :)

Překladač opravdu nemůže vědět, že to nevadí, nakonec je to dobře, že se to nesnaží zjišťovat, protože on by to ani 100% zjistit nemohl, ale to je stále jen důvod zahlásit -- jak je v obdobných případech zvykem -- warning, ne error.

[Messa]

Přetypováváš z pointeru na nějaký úplně jiný typ. To je docela ošklivé, a když překládáš s tak pedantickými volbami, nemůžeš se divit, že ti to neprojde Na x86 je šířka intu stejná nebo menší než šířka pointeru, ale jinde to tak být nemusí.

Já osobně bych to ale řešil tak, že bych předávat pointer na int (nebo rovnou na nějakou strukturu, pokud bude potřeba více dat). Ale pozor aby ses nespálil (nepřepsal si něco pod rukama dříve, než to to vlákno přečte), taky bys měl dbát na uvolňování paměti.

[mik]

mam x64, staci mi predat jenom tuhle jednu promennou......nehodil bys mi kus funkcniho kodu?

[Ash]

To je tím, že to nejspíš kompilujete na 64-bitovém systému, a gcc se rozhodlo toto přiřazení hlásit jako error, ne jen jako warning. Možná proto, že je lepší když program nejde přeložit, než aby dělal něco úplně jiného než na 32bit systému :)

Můžete použít třeba nějakou 64bit proměnnou, většinou třeba unsigned long long

unsigned long long tid=(unsigned long long)threadid;
printf("Hello World! It's me, thread #%llu!\n", tid);

i když by se jistě našlo něco lepšího či přenositelnějšího.

P.S. A je docela možné, že ten zdroják v c++ budete muset kompilovat třeba g++, ne jen gcc.

[Bluebear]

Stálo by za to označit, která z řádek programu je řádka 8, na které to tu chybu hlásí

Jinak, potenciální problémy tam vidím tyto:

• Funkce PrintHello() nemá stanovenou výchozí hodnotu, přestože je deklarovaná jako vracející void *; vím, že ve skutečnosti se ta funkce nikdy nevrátí, ale překladač to IMHO neví a může ho to mást. Doplnil bych fiktivní return NULL. To samé platí pro main().
• Štábní kultura: funkce, která se jmenuje PrintHello(), by měla pouze napsat pozdrav; rozhodně ne vypsat pozdrav a pak ukončit thread! Buď přejmenovat na PrintHelloAndExit, nebo dát ukončení threadu někam jinam. Kdyby se taková funkce vyskytovala ve skutečné aplikaci, bude to ošklivá nášlapná mina na každého, kdo bude ten program udržovat nebo upravovat.
• Obě dvě volání pthread_exit() jsou v daném kódu úplně zbytečná; hlavní funkce threadu může svůj výstupní ukazatel prostě vrátit (proto je deklarována jako vracející void *; hlavní funkce programu taktéž.
• Štábní kultura: pojmenování parametru tid, je matoucí, protože skutečný TID (ten, který se získá voláním funkce gettid()) je něco úplně jiného. Opět ošklivá nášlapná mina, zvlášť proto, že skutečný TID je také celé číslo.

[Luboš Doležel (Doli)]

Funkce PrintHello() nemá stanovenou výchozí hodnotu, přestože je deklarovaná jako vracející void *; vím, že ve skutečnosti se ta funkce nikdy nevrátí, ale překladač to IMHO neví a může ho to mást. Doplnil bych fiktivní return NULL. To samé platí pro main().

Překladač to ví.

Štábní kultura: funkce, která se jmenuje PrintHello(), by měla pouze napsat pozdrav; rozhodně ne vypsat pozdrav a pak ukončit thread! Buď přejmenovat na PrintHelloAndExit, nebo dát ukončení threadu někam jinam. Kdyby se taková funkce vyskytovala ve skutečné aplikaci, bude to ošklivá nášlapná mina na každého, kdo bude ten program udržovat nebo upravovat.

To by pak mělo na konci jména mít AndExit nějakých 90 % funkcí pro vlákna, co se v programech kdy vyskytují

A pthread_exit v main() se mi nelíbí ani trochu.

[Bluebear]

To by pak mělo na konci jména mít AndExit nějakých 90 % funkcí pro vlákna, co se v programech kdy vyskytují

Ne, pouze hlavní funkce vláken, a ty by měly být tak pojmenované - z názvu PrintHello nelze vyčíst, že je to hlavní funkce vlákna. Kdyby se jmenovala, dejme tomu, HelloThreadMain, bylo by hned jasno. Kdyby to byla knihovní funkce, která dejme tomu vykreslí nějaké logo, tak se může klidně stát, že ji někdo vyvolá ze singlethreadového programu, a žuch.

[Andrej Herceg]

A pthread_exit v main() se mi nelíbí ani trochu.

Niekedy to volanie môže myť zmysel. Ak tam je, tak sa počká, kým sa všetky vlákna ukončia (a až potom sa ukončí aj hlavné vlákno). Práve v tom príklade to zmysel má, kedže tam nie je žiadne čakanie na to, až urobia tie vytvorené vlákna to, čo urobiť majú.

[mik]

tak jo vyreseno, je to uz funkcni....

funguje jak:

unsigned long long tid=(unsigned long long)threadid;
printf("Hello World! It's me, thread #%llu!\n", tid

tak:

int tid=(int)(long)threadid;

nutno kompilovat s g++

dekuji vsem kteri prispeli k vyreseni meho problemu, dekuji i tem kteri me poucili o štábní kultuře, v budoucnu az ziskam urcite zkusenosti se ji budu urcite ridit....