Bug #124932 for Net-DNS-Native: Conflict with epoll since 0.16

Mon Mar 26 21:44:56 2018 DBOOK [...] cpan.org - Ticket created

Subject:

Conflict with epoll since 0.16

I'm not able to diagnose or reduce further at the moment, but installing Net::DNS::Native 0.16 or above causes test failures in Mojo::Reactor::Epoll. I've disabled its use in the test in 0.007, but you can test with version 0.006 which is functionally the same. The error occurs during this section of the test: https://metacpan.org/source/DBOOK/Mojo-Reactor-Epoll-0.006/t/reactor_epoll.t#L258-284 and looks like (line referenced is when either ->add or ->modify is called on the Linux::Epoll object): Couldn't add filehandle from epoll set: Bad file descriptor at lib/Mojo/Reactor/Epoll.pm line 151.

Mon Mar 26 21:45:14 2018 DBOOK [...] cpan.org - Correspondence added

Reference issue: https://github.com/Grinnz/Mojo-Reactor-Epoll/issues/1

Mon Mar 26 21:53:36 2018 DBOOK [...] cpan.org - Correspondence added

This also seems to be the same issue, triggered when IO::Async::Loop::Epoll and Net::DNS::Native >= 0.16 are installed: https://github.com/Grinnz/Mojo-Reactor-IOAsync/issues/1

Tue Mar 27 00:24:09 2018 oleg [...] cpan.org - Correspondence added

Since 0.16 Net::DNS::Native uses tied handle instead of real one. But looks like Linux::Epoll doesn't handle tied handles correctly. Problem is here: https://metacpan.org/source/LEONT/Linux-Epoll-0.012/lib/Linux/Epoll.xs#L19 . This function returns -1 for tied handle. And here is minimal script to reproduce the issue: use strict; use Linux::Epoll; use Symbol; my $epoll = Linux::Epoll->new(); my $fh = Symbol::gensym(); # \*STDIN; tie *$fh, 'TiedHandle', 0; $epoll->add($fh, 'in', sub { my $events = shift; }); print "Ok\n"; package TiedHandle; sub TIEHANDLE { my ($class, $fd) = @_; bless { fd => $fd }, $class; } sub FILENO { return $_[0]->{fd}; }

Tue Mar 27 00:24:10 2018 The RT System itself - Status changed from 'new' to 'open'

Tue Mar 27 00:46:35 2018 DBOOK [...] cpan.org - Correspondence added

On Tue Mar 27 00:24:09 2018, OLEG wrote: Show quoted text

> Since 0.16 Net::DNS::Native uses tied handle instead of real one. But > looks like Linux::Epoll doesn't handle tied handles correctly. Problem > is here: https://metacpan.org/source/LEONT/Linux-Epoll- > 0.012/lib/Linux/Epoll.xs#L19 . This function returns -1 for tied > handle. And here is minimal script to reproduce the issue:

Thank you, I have reported this to Linux::Epoll at https://rt.cpan.org/Ticket/Display.html?id=124934

Tue Jan 15 12:23:16 2019 oleg [...] cpan.org - Status changed from 'open' to 'rejected'