Since all of epoll code and its multithreading is under ifdefs, netbsd should just continue working as single threaded poll unaffected by the patch. If netbsd kqueue supports single shot event delivery and edge triggered notification, we could have an equivalent implantation on netbsd too. Even if kqueue does not support these features, it might well be worth implementing a single threaded level triggered kqueue based event handler, and promote netbsd from suffering from sucky vanilla poll.<br><br>Thanks<br><br><div class="gmail_quote">On Fri, Jan 23, 2015, 19:29 Emmanuel Dreyfus <<a href="mailto:manu@netbsd.org">manu@netbsd.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ben England <<a href="mailto:bengland@redhat.com" target="_blank">bengland@redhat.com</a>> wrote:<br>
<br>
> NetBSD may be useful for exposing race conditions, but it's not clear to<br>
> me that all of these race conditions would happen in a non-NetBSD<br>
> environment,<br>
<br>
In many times, NetBSD exhibited cases where non specified,<br>
Linux-specific behaviors were assumed. I recall my very first finding in<br>
Glusterfs: Linux lets you use a mutex without calling<br>
pthread_mutex_init() first. That broke on NetBSD, as expected.<br>
<br>
Fixing this kind of issues is interesting beyond NetBSD support, since<br>
you cannot take for granted that an unspecified behavior will not be<br>
altered in the future.<br>
<br>
That said, I am fine if you let NetBSD run without fixing the underlying<br>
issue, but you have been warned :-)<br>
<br>
--<br>
Emmanuel Dreyfus<br>
<a href="http://hcpnet.free.fr/pubz" target="_blank">http://hcpnet.free.fr/pubz</a><br>
<a href="mailto:manu@netbsd.org" target="_blank">manu@netbsd.org</a><br>
______________________________<u></u>_________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="http://www.gluster.org/mailman/listinfo/gluster-devel" target="_blank">http://www.gluster.org/<u></u>mailman/listinfo/gluster-devel</a><br>
</blockquote></div>