Nick Kew wrote:
>
> OK, I've just tested APR variants with Event MPM.
>
> It is indeed that fix that breaks it. When I revert
> the fix, the problem goes away.
>
> The original patch posted to fix the same bug works fine.
> Can I suggest you apply the following patch to 1.3.x
> (should work on 1.3.0 or 1.3.2 too) and let us know if
> it fixes the problem you're seeing?
>
> Index: poll/unix/port.c
> ===================================================================
> --- poll/unix/port.c (revision 662044)
> +++ poll/unix/port.c (working copy)
> @@ -315,7 +315,15 @@
>
> if (ret == -1) {
> (*num) = 0;
> - rv = apr_get_netos_error();
> + if (errno == EINTR) {
> + rv = APR_EINTR;
> + }
> + else if (errno == ETIME) {
> + rv = APR_TIMEUP;
> + }
> + else {
> + rv = APR_EGENERAL;
> + }
> }
> else if (nget == 0) {
> rv = APR_TIMEUP;
>
We have now applied this patch to 1.3.2 and it successfully solved the problem,
ie no more errors about "Critical
poller failure" in the log file. Thanks!
By the way, does it look like this patch (or some variant, as mentioned in
other postings in this thread) will be included in a future release (like
1.3.3)?
Regards
/Jimi
