[Bug 10573] New: dlls/kernel32/tests/comm. c test fails on systems with serial ports

http://bugs.winehq.org/show_bug.cgi?id=10573

Summary: dlls/kernel32/tests/comm.c test fails on systems with
                    serial ports
           Product: Wine
           Version: CVS/GIT
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Keywords: regression
          Severity: minor
          Priority: P2
         Component: wine-kernel
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

I only noticed this after enabling the serial port on my laptop...

for dll/kernel32/tests/comm.c, I get the following errors:

comm.c:677:Found Com port COM1. Connected devices may disturbe results
comm.c:705:GetCommModemStatus returned 0x00000000->
comm.c:739: Test failed: A timed-out read should return TRUE
comm.c:740: Test failed: err=109
comm.c:742: Test failed: Unexpected TimeOut 0, expected 1000

For many people, this test gets skipped, ie:

http://test.winehq.org/data/200711221000/wine_2000_0.9.49-331-g7a8eb4c/kerne...

the regression is caused by:

commit 539d5863e6fae78943fe59a60b151433126a9481
Author: Alexandre Julliard 
Date:   Fri Apr 6 16:05:53 2007 +0200

ntdll: Reimplement NtReadFile.

Always try a read() before queuing an async I/O.
    Handle timeout waits for synchronous I/O entirely on the client side.
    Queue the final APC as a proper user APC.

[Bug 10573] dlls/kernel32/tests/comm. c test fails on systems with serial ports

http://bugs.winehq.org/show_bug.cgi?id=10573

Timo-Heikki Mäkelä  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail],
                   |                            |[e-mail]

--- Comment #1 from Timo-Heikki Mäkelä   2008-03-07 10:40:10 ---
Correct component is ntdll (that was changed), not kernel32, see the result of
the regression test.

[Bug 10573] dlls/kernel32/tests/comm. c test fails on systems with serial ports

http://bugs.winehq.org/show_bug.cgi?id=10573

Lei Zhang  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from Lei Zhang   2008-03-21 19:26:37 ---
*** Bug 12141 has been marked as a duplicate of this bug. ***

[Bug 10573] dlls/kernel32/tests/comm. c test fails on systems with serial ports

http://bugs.winehq.org/show_bug.cgi?id=10573

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Alexandre Julliard   2008-05-07 13:29:17 ---
This should work now.

[Bug 10573] dlls/kernel32/tests/comm. c test fails on systems with serial ports

http://bugs.winehq.org/show_bug.cgi?id=10573

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #4 from Alexandre Julliard   2008-05-09 12:54:47 ---
Closing bugs fixed in 1.0-rc1.

[Bug 10573] dlls/kernel32/tests/comm.c test fails on systems with serial ports

http://bugs.winehq.org/show_bug.cgi?id=10573

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|CVS/GIT                     |unspecified