nitrram / tcpserv Goto Github PK

Tento glob matchne i hlavičky pro daemon_c. Ale dívám se že se ta proměnná v Makefile stejně.

Bylo by dobré alespoň něco zalogovat pokud pid < 0.

Použil bych místo 1024 konstantu nebo sizeof, ať se délka toho pole neobjevuje na třech různých místech.

Pokud by měl démon asynchroně zvládat i zprávy rozložené do více TCP paketů tak by tady bylo potřeba zatím načtenou část zprávy uložit do bufferu a vrátit se do hlavní smyčky s epoll_wait. break způsobí že se částečná zpráva zpracuje jako by byla úplná.

Tady je nutné ošetřit případ -1 < n < response_len a případ EAGAIN/EWOULDBLOCK. Pokud by server měl být úplně asynchronní tak by musel ukládat neodeslanou část odpovědi do nějakého bufferu asociovaného s tímto FD.

Tady bych preferoval buď dalším argumentem předávat maximální délku pole prvního argumentu, nebo alespoň přidat komentář s předpoklady pro volání této funkce.

V této aplikaci to není takový problém, ale obecně není blokující spánek v asynchronním serveru žádoucí. Optimální by to bylo řešit to pomocí timeoutu epoll_wait, což bohužel znamená docela dost kódu navíc. Nebo vyčítat /proc/stat periodicky a v print_cpu_stats použít poslední hodnotu.

Tady bych taky ocenil dokumentaci o požadavcích na pole map.

Místo 64 by měl být počet prvků pole events.

Opět to v této aplikaci není problém, ale obecně by se každý otevřený filedescriptor měl i někde zavřít. A neměl by leaknout ani v případech kdy funkce server_work vrací chybu.

Je flag EPOLLET potřeba?

Bylo by dobré chyby alespoň logovat. A není potřeba ošetřit případ kdy druhá strana zavřela TCP spojení, a zavřít FD na naší straně, a případně na něj předtím zavolat epoll_ctl(EPOLL_CTL_DEL)?

Přepíše předchozí hodnotu. Ideální by bylo mít seznam všech FD klientů + příslušející data (e.g. výše zmíněné buffery).

Oceňuju nepoužití threadů:)