Closed
Bug 2133
Opened 26 years ago
Closed 26 years ago
[PP]Jan25:Mac freezes waiting for data
Categories
(NSPR :: NSPR, defect, P1)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: mikepinkerton, Assigned: gordon)
References
()
Details
(This bug imported from BugSplat, Netscape's internal bugsystem. It
was known there as bug #142690
http://scopus.netscape.com/bugsplat/show_bug.cgi?id=142690
Imported into Bugzilla on 01/04/99 16:10)
Surf for a few pages, and the mac will freeze up in Delay(). What appears is
happening is NSPR is waiting for data (which never arrives), so it goes into an
infinite loop until something arrives, scheduling the main thread, sleeping,
waking up, and rescheduling. This starves out the UI so the mac appears frozen.
Comment 2•26 years ago
|
||
*** Bug 124235 has been marked as a duplicate of this bug. ***
Comment 3•26 years ago
|
||
Accepted the bug.
Bug 124235 was marked as duplicate of this bug.
Added sdagley, gagan, and freier to the Cc list.
I am studying the NSPR mac source code now. Is
there any area that you think is the suspect?
I suspect that PR_Poll may fail to report socket
connection close as a readable event, so we
don't know the download is done.
I also discovered that NSPR's implementation
of nonblocking socket I/O on the Mac is incorrect.
If we always call PR_Poll first and only call
I/O functions when the fd is readable or writable,
then this is probably not a problem. Gagan, can
you confirm that Netlib always calls PR_Poll
before calling read/recv/write/send on sockets?
I am not familiar with our user-level thread
scheduling code and with Mac, so please be
patient. I am doing my best to get up to
speed.
Comment 4•26 years ago
|
||
Bug #124235 is a test stopper and a duplicate
of this bug, so I guess this bug is a test
stopper too.
Assigned QA to glynn (same as in bug #124235).
I am adding Lou to the list. He did the NSPR20 upgrade and might know about
this. Lou?
jar and I have decided that a fix for the Mac NSPR2 problem is no longer a
requirement for ending the stability freeze so I'm moving the target fix version
of this bug to 5.0. But Wan-Teh, Gordon, and Joe should continue with fixing
this bug as their top priority.
*** Bug 299777 has been marked as a duplicate of this bug. ***
Reporter | ||
Comment 10•26 years ago
|
||
*** Bug 2303 has been marked as a duplicate of this bug. ***
Comment 11•26 years ago
|
||
This bug is slowing down our ability to do Mac testing greatly. What's the
status on a fix?
Assignee | ||
Comment 12•26 years ago
|
||
*** Bug 2430 has been marked as a duplicate of this bug. ***
Comment 13•26 years ago
|
||
gordon...whats the eta for some headway on this one? thanks!
URL: http://www.cnn.com
Comment 14•26 years ago
|
||
Adding URL info.
Summary: [PP] Mac freezes waiting for data → [PP]Jan25:Mac freezes waiting for data
Comment 15•26 years ago
|
||
I just played with Jan 25 Mac build and it looks better. What do you think
elig?
Assignee | ||
Comment 16•26 years ago
|
||
The flaw in the Mac NSPR socket implementation is still there. I've been out sick
for the past week, but this is my number one priority now.
Comment 17•26 years ago
|
||
Thanks, Gordon. <Purely anecodotally, Viewer doesn't seem to be crashingly nearly
as often as a result of this particular problem on the more recent builds,
believe it or not.>
Comment 18•26 years ago
|
||
Inserting Milestone info.
Comment 19•26 years ago
|
||
Setting all current Open Critical and Major to M3
Comment 20•26 years ago
|
||
per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser
Comment 21•26 years ago
|
||
Gordon status on this puppy?
Comment 22•26 years ago
|
||
elig, can you try this with the latest Apprunner Mac build please. Win32 Mar13
build loads this URL ok.
Comment 23•26 years ago
|
||
Resolved as FIXED.
Gordon checked in the stuff, so ... :-)
Comment 24•26 years ago
|
||
Whoo Hoo! Will check Monday morning :-) Changed to make myself, glynn as qa
contact.
Comment 25•26 years ago
|
||
As far as I can tell this puppy appears fixed. March 15 builds. Marking
verified, will reopen if necessary.
Reporter | ||
Updated•26 years ago
|
Status: VERIFIED → REOPENED
Reporter | ||
Comment 26•26 years ago
|
||
I got this just this morning on the tinderbox page (lockup in _Delay during
networking). I don't think it was waiting for DNS lookup (which is still
synchronous). Reopening bug. I really am terribly sorry, gordon.
Comment 27•26 years ago
|
||
Well ... crap.
Grodon, is Mike on something or is it really back?
Comment 28•26 years ago
|
||
Resolved bug as FIXED.
Since Pink or glynn can't reporduce the failure ...
Comment 29•26 years ago
|
||
Reverified until a reproducible example of new failure can be found.
Comment 30•25 years ago
|
||
NSPR now has its own Bugzilla product. Moving this bug to the NSPR product.
Updated•25 years ago
|
Target Milestone: M3 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•