Joe-
I think you're right and that there is some busy-waiting going
on. Though the behavior is legal API-wise, it looks sub-optimal.
Given that, disabling poll until there is a kernel without that
problem seems prudent. I'll go ahead and do that later today and do
that if no one objects.
-wsv
On May 25, 2005, at 2:36 PM, Joe Orton wrote:
> On Wed, May 25, 2005 at 11:49:25AM -0700, Wilfredo Sánchez Vega wrote:
>
>> I just fixed APR to retry on EAGAIN. We should not disable the
>> use of poll() on Darwin. It's behavior on 10.4 may be unusual, but
>> it has not been shown to be broken.
>>
>
> It's right for older versions still in any case; please fix it omit
> 10.4
> if you're really sure that's right, I can't guess a pattern to
> match the
> appropriate versions. Your change needs backporting to all the
> branches, too.
>
> The fact that Al is triggering this behaviour so easily leaves me
> entirely unconvinced that this is a happy poll implementation,
> however.
> Is APR now busy-waiting in the poll/send loop for Al, have you
> checked a
> system call trace? Have you run the APR test suite on it, too?
>
> joe
|