[Gluster-devel] epoll.c bug

Anand Avati avati at zresearch.com
Thu Nov 1 18:16:22 UTC 2007


Thierry,
 can you please elaborate on the bug you faced? the change suggested below
is redundant. Such a check is generally needed in a poll/select kind of
construct, while not so using epoll.

avati

2007/11/1, Thierry EXCOFFIER <exco at bat710.univ-lyon1.fr>:
>
> Hello.
>
> My 'glusterfsd 1.3.7' crashs if 2 clients use it at the same time.
>
> After a small change in the sources, it seems to work :
>
> In 'epoll.c' / 'sys_epoll_iteration'
>
>   for (i=0; i < ret; i++) {
>    if ( ectx->evs[i].events )     /**** This test was missing ****/
>     epoll_notify (ectx->evs[i].events,
>                   ectx->evs[i].data.ptr);
>   }
>
>
> Sincerely yours.
>
> PS : My server is a Buffalo Linkstation with original firmware :
>      linux 2.6.12.6 on an Arm processor with 128MB of RAM.
> --
> Thierry.EXCOFFIER@@liris.univ-lyon1.fr//   GO: Oriental Strategy Game
> LIRIS/UFR Informatique, bat710       //DYLAN: Programming Language
> Universite Claude Bernard LYON 1  __// UNIX: Standard Operating System
> 43 Bd du 11 novembre 1918       //
> 69622 VILLEURBANNE, FRANCE     //WWW: http://www710.univ-lyon1.fr/~exco
>
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel at nongnu.org
> http://lists.nongnu.org/mailman/listinfo/gluster-devel
>



-- 
It always takes longer than you expect, even when you take into account
Hofstadter's Law.

-- Hofstadter's Law



More information about the Gluster-devel mailing list