122433 - The cvar2 and socket tests hang on dual-processor Windows 2000.

Status: RESOLVED WONTFIX

(NSPR :: NSPR, defect, P1)

Description

[Wan-Teh Chang]

I discovered this problem while testing NSPR 4.2 Beta 1, and
to my surprise, found that this problem also exists in NSPR
4.1.2.

I have only seen this problem with the WINNT optimized build
on my dual-processor Windows 2000 box.  The symtom is that the
cvar2 and socket tests hang intermittently.  It is not hard to
make these two tests hang.  If you run these tests repeatedly,
they will usually hang in fewer than 10 tries.

It is difficult to debug this problem because they only occur
with the optimized build.  I found that by disabling "global
optimization" (the -Og- compiler flag) on ntio.c, I can make
the problem go away.  So the problem is probably in that file.

Comment 1

[Wan-Teh Chang]

Attachment: Workaround: disables global optimization on ntio.c

I found that if I disable global optimization (-Og-) on ntio.c,
and only ntio.c, the problem is gone.

I've checked this workaround into the tip of NSPR.

Comment 2

[Matt]

Was this with VC++ 7.0 or 7.1? This seems like a pretty serious bug, not sure why it's still open :(

Comment 3

[Wan-Teh Chang]

This was with MSVC 6.  I don't remember which service pack
level it was at.  Most likely SP4 or later.

Comment 4

[Mike Moening]

I have a Win2000 dual processor box and have seen hanging behavior with VC++ 6.0.  Apparently its still happening. I'll try to get a fresh build and run the tests.

Comment 5

[Christophe Ravel]

The target milestone is already released. Resetting target milestone.

Comment 6

[Dibyendu Ghosh]

Does anybody have the stack trace for the hanging thread?
Also, is this issue fixed on 4.2.2?

Comment 7

[Nelson Bolyard (seldom reads bugmail)]

I propose that we consider this bug to have been fixed by the workaround
that was committed 5+ years ago.  

Comment 8

[Nelson Bolyard (seldom reads bugmail)]

The workaround was committed for NSPR 4.2, and has been present in 
that version and all higher numbered versions.