Re: epoll.c bug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxxxxxxx>:
>
> 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@xxxxxxxxxx
> 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


[Index of Archives]     [Gluster Users]     [Ceph Users]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux