[Vm-dev] aioDisable: epoll_ctl: Bad file descriptor

[stes@PANDORA.BE]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256


Hi,

Good news; I tried your changes related to epoll(7) support,
and in to my limited testing so far, it seems an improvement.

I tried on my Linux test server:

$ git branch
* (HEAD detached at origin/dtl/epoll-forksqueak)
  Cog

I built on Linux from that epoll-forksqueak branch, 

$ grep EPOLL config.h 
#define HAVE_EPOLL 1
#define HAVE_EPOLL_PWAIT 1

now for both the Cog and Stack Spur VM, when I launch it on Linux,
then quit it, I do *not* get the :

aioDisable: epoll_ctl: Bad file descriptor

However the odd thing now is that if I try the main Cog branch on Linux,
I'm not getting the error any longer, so it seems to be gone.

I may by coincidence have seen the message, while it may be
perhaps not 100% reproducible but happening only occasionaly.

Anyway when on OpenIndiana (not Linux), I tried:

 configure

configure decides for me that it wants to enable epoll on OpenIndiana:

$ uname
SunOS

$ grep EPOLL config.h 
#define HAVE_EPOLL 1
#define HAVE_EPOLL_PWAIT 1

so also on OpenIndiana (which is in the SunOS family) configure enables epoll.

$ git branch
* (HEAD detached at origin/dtl/epoll-forksqueak)
  Cog

so I compiled that version on OpenIndiana (with epoll as set by configure).

When I launch it, do something (open some browser), then quit it,
no error.

I tried sqstkspur64 and sqcogspur64.

Actually on OpenIndiana I have so far never seen the

aioDisable: epoll_ctl: Bad file descriptor

for the time epoll was enabled (a few months ago now).

But I have a feeling that these issues can be quite complex and subtle,
and that the issue may only be happening on some occasions.

In any case as far as I can see, your 'branch' seems an improvement.

David Stes

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQEcBAEBCAAGBQJgP7AKAAoJEAwpOKXMq1MaJXYH/ijJkk5Qg5K6gfC8C5F36kZp
EYr4yCfpibaYUgjg206MV9yN/RHUPnl+cnZtKzyv9xPgkdfDjZfA+V1IDahXnH/O
BmJNR3Y3bpJasDzlaTp/cCS9wJFrM54v1o92ZTERe9xIfGufTZSTLxRos/nh0gNy
a6jwj7GYnhORykXJAIrMCDA3Mnu9ziYPN5O/ycYfp2AfoLvQZHb8qsX5W4AD8JB+
0kIsnrLfShyTC8M8m3tWy6r2fKeMQc5FH8IK0hxZCqe19RSRnrBo+RhhP9GBV7sX
B6ZUNNGsoyE/CDgUK3l5yn5E7d7/Q2LixmqYcmAlbE96q79+hKXYVivvBTTM+2I=
=hhf5
-----END PGP SIGNATURE-----