October 2013 - Wine-Devel - list.winehq.org

Re: [PATCH 5/6] ole32: Call IExternalConnection_ReleaseConnection with the correct final parameter.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://newtestbot.winehq.org/JobDetails.pl?Key=3009 Your paranoid android. === w864 (32 bit marshal) === marshal.c:3321: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3322: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3338: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3339: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3376: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3377: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3394: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3395: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3358: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3359: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3321: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3322: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3338: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3339: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3376: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3377: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3394: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3395: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3358: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3359: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3321: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3322: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3338: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3339: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3376: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3377: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3394: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3395: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 marshal.c:3358: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3359: Test failed: info->dwServerPid was 0xb60 instead of 0xa48 === w864 (64 bit marshal) === marshal.c:3321: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3322: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3338: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3339: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3376: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3377: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3394: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3395: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3358: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3359: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3321: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3322: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3338: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3339: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3376: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3377: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3394: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3395: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3358: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3359: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3321: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3322: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3338: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3339: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3376: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3377: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3394: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3395: Test failed: info->dwServerPid was 0x86c instead of 0xbb0 marshal.c:3358: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3359: Test failed: info->dwServerPid was 0x86c instead of 0xbb0

Oct. 24, 2013

Re: [PATCH 4/6] ole32: When releasing marshal data of a normal ref ignore any remaining weak refs.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://newtestbot.winehq.org/JobDetails.pl?Key=3008 Your paranoid android. === w864 (32 bit marshal) === marshal.c:3326: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3327: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3343: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3344: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3381: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3382: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3399: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3400: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3363: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3364: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3326: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3327: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3343: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3344: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3381: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3382: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3399: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3400: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3363: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3364: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3326: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3327: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3343: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3344: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3381: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3382: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3399: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3400: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 marshal.c:3363: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3364: Test failed: info->dwServerPid was 0x5bc instead of 0xac8 === w864 (64 bit marshal) === marshal.c:3326: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3327: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3343: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3344: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3381: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3382: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3399: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3400: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3363: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3364: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3326: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3327: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3343: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3344: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3381: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3382: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3399: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3400: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3363: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3364: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3326: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3327: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3343: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3344: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3381: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3382: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3399: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3400: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74 marshal.c:3363: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3364: Test failed: info->dwServerPid was 0x8e4 instead of 0xb74

Oct. 24, 2013

Re: [PATCH 3/6] ole32/tests: Add some tests for CoReleaseMarshalData with a combination of normal and weak marshals.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://newtestbot.winehq.org/JobDetails.pl?Key=3007 Your paranoid android. === w864 (32 bit marshal) === marshal.c:3326: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3327: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3343: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3344: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3381: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3382: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3399: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3400: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3363: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3364: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3326: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3327: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3343: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3344: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3381: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3382: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3399: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3400: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3363: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3364: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3326: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3327: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3343: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3344: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3381: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3382: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3399: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3400: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 marshal.c:3363: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3364: Test failed: info->dwServerPid was 0xaa4 instead of 0xb54 === w864 (64 bit marshal) === marshal.c:3326: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3327: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3343: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3344: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3381: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3382: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3399: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3400: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3363: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3364: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3326: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3327: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3343: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3344: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3381: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3382: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3399: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3400: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3363: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3364: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3326: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3327: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3343: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3344: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3381: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3382: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3399: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3400: Test failed: info->dwServerPid was 0x118 instead of 0xba4 marshal.c:3363: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3364: Test failed: info->dwServerPid was 0x118 instead of 0xba4

Oct. 24, 2013

Re: [PATCH 2/6] ole32/tests: Make the two reference marshalling thread more generic.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://newtestbot.winehq.org/JobDetails.pl?Key=3006 Your paranoid android. === w864 (32 bit marshal) === marshal.c:3230: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3231: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3247: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3248: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3285: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3286: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3303: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3304: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3267: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3268: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3230: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3231: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3247: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3248: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3285: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3286: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3303: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3304: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3267: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3268: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3230: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3231: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3247: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3248: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3285: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3286: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3303: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3304: Test failed: info->dwServerPid was 0x370 instead of 0xa48 marshal.c:3267: Test failed: info->cbSize was 1527466381 instead of 48 marshal.c:3268: Test failed: info->dwServerPid was 0x370 instead of 0xa48 === w864 (64 bit marshal) === marshal.c:3230: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3231: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3247: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3248: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3285: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3286: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3303: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3304: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3267: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3268: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3230: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3231: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3247: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3248: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3285: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3286: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3303: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3304: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3267: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3268: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3230: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3231: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3247: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3248: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3285: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3286: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3303: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3304: Test failed: info->dwServerPid was 0x738 instead of 0xbb8 marshal.c:3267: Test failed: info->cbSize was -1047396882 instead of 56 marshal.c:3268: Test failed: info->dwServerPid was 0x738 instead of 0xbb8

Oct. 24, 2013

Re: [PATCH 1/6] ole32/tests: Extend the lifecycle tests to objects which expose IExternalConnection.

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://newtestbot.winehq.org/JobDetails.pl?Key=3005 Your paranoid android. === w864 (32 bit marshal) === marshal.c:3226: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3227: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3243: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3244: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3281: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3282: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3299: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3300: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3263: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3264: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3226: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3227: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3243: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3244: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3281: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3282: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3299: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3300: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3263: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3264: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3226: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3227: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3243: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3244: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3281: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3282: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3299: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3300: Test failed: info->dwServerPid was 0x870 instead of 0xbac marshal.c:3263: Test failed: info->cbSize was -1846173327 instead of 48 marshal.c:3264: Test failed: info->dwServerPid was 0x870 instead of 0xbac === w864 (64 bit marshal) === marshal.c:3226: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3227: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3243: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3244: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3281: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3282: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3299: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3300: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3263: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3264: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3226: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3227: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3243: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3244: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3281: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3282: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3299: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3300: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3263: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3264: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3226: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3227: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3243: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3244: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3281: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3282: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3299: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3300: Test failed: info->dwServerPid was 0x18c instead of 0xa48 marshal.c:3263: Test failed: info->cbSize was -588349091 instead of 56 marshal.c:3264: Test failed: info->dwServerPid was 0x18c instead of 0xa48

Oct. 24, 2013

Re: server: Add support for pending write flag to the serial device. Take 2.

by Wolfgang Walter

Am Donnerstag, 24. Oktober 2013, 11:29:34 schrieb Dmitry Timoshkov: > Wolfgang Walter <wine(a)stwm.de> wrote: > > > > I have no intention to discuss that all again. Please think about the > > > > problem. Hint: writing twice as many bytes with 150 baud needs more > > > > time, > > > > significantly more time then TIMEOUT. > > > > > > Not really. Francois sent an investigation with pretty exhaustive > > > explanation of the behvaiour of COM-port UARTs regarding timings and > > > baud > > > rates. > > > > Have you read it? It says that with a standard UART windows may signals > > after 872ms that the tx queue is empty (though there are still 4 bytes to > > send) when you write 17 bytes. For 34 bytes he gets 1802ms > TIMEOUT. > > Where do you see 34 bytes to be written in that test? Sorry for that. I looked at the 1.7.2 codebase. There was SetLastError(0xdeadbeef); res = WriteFile(hcom, tbuf, sizeof(tbuf), &bytes, NULL); todo_wine ok(!res, "WriteFile on an overlapped handle without ovl structure should fail\n"); todo_wine ok(GetLastError() == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); In 1.7.4 there are no todo_wine's any more so this does not fail any more and there should be no pending bytes from that test. There should only be 17 bytes in the test in question. > > > And I really don't think that we can expect exactly that same behaviour > > when using the linux serial driver, anyway. > > At the time I was reworking the EV_TXEMPTY to use overlapped IO the > reasoning for using 900 ms as the designed time was simple: that was enough > to make the test pass under Windows with real hardware. How the patch we > are discussing makes the test pass for me under Linux with a USB-serial > adapter, which confirms that there is no reasons that it can't work under > Linux as well. > > I understand that there are cases of (really old) hardware where this limit > may not be enough, and Francois has already sent a patch which increases > timeout. The Francois' patch hasn't been accepted, but that's a different > matter. I built 1.7.4 here and I still get a timeout on my machine (with onboard device), though. The board has an Intel "Corporation 6 Series/C200 Series Chipset Family". I mesured the time needed to write with 150 baud on linux and it needs about 1000ms till ioctl(fd, TIOCOUTQ, &out) returns with zero. Sometimes up to 1100ms. When I mesure it I do not use select() or poll() but loop similar as wine does: call ioctl(), sleep for 1ms, ... Regards, -- Wolfgang Walter Studentenwerk München Anstalt des öffentlichen Rechts

Oct. 24, 2013

Re: [5/7] oleaut32/tests: Fix typelib tests compilation with __WINESRC__ defined

by Dmitry Timoshkov

André Hentschel <nerv(a)dawncrow.de> wrote: > - ok(lstrcmp(value, buf) == 0, #expr " expected \"%s\" got \"%s\"\n", value, buf); \ > + ok(lstrcmpA(value, buf) == 0, #expr " expected \"%s\" got \"%s\"\n", value, buf); \ Using simple strcmp() should be preferrable, leaving lstr* versions only for cases when an exception handler is really needed. P.S. Fixing compilation with __WINESRC__ defined gives an opportunity for reviewing the code, this should not be a simple search and replace job. -- Dmitry.

Oct. 24, 2013

Re: winspool.drv/tests: Fix tests compilation with __WINESRC__ defined

by Dmitry Timoshkov

André Hentschel <nerv(a)dawncrow.de> wrote: > @@ -3018,28 +3018,28 @@ START_TEST(info) > test_ConfigurePort(); > test_DeleteMonitor(); > test_DeletePort(); > - test_DeviceCapabilities(); > - test_DocumentProperties(); > + test_DeviceCapabilitiesA(); > + test_DocumentPropertiesA(); > test_EnumForms(NULL); > if (default_printer) test_EnumForms(default_printer); > test_EnumMonitors(); > - test_EnumPorts(); > + test_EnumPortsA(); > test_EnumPrinterDrivers(); > test_EnumPrinters(); > test_EnumPrintProcessors(); > test_GetDefaultPrinter(); > test_GetPrinterDriverDirectory(); > test_GetPrintProcessorDirectory(); > - test_OpenPrinter(); > - test_GetPrinter(); > + test_OpenPrinterA(); > + test_GetPrinterA(); > test_GetPrinterData(); > test_GetPrinterDataEx(); > - test_GetPrinterDriver(); > + test_GetPrinterDriverA(); > test_SetDefaultPrinter(); > test_XcvDataW_MonitorUI(); > test_XcvDataW_PortIsValid(); > test_IsValidDevmodeW(); > - test_OpenPrinter_defaults(); > + test_OpenPRINTER_DEFAULTSA(); Please leave the test names intact, that assumes that adding tests for W version of an API doesn't need a separate body. -- Dmitry.

Oct. 24, 2013

Re: server: Add support for pending write flag to the serial device. Take 2.

by Dmitry Timoshkov

Wolfgang Walter <wine(a)stwm.de> wrote: > > > I have no intention to discuss that all again. Please think about the > > > problem. Hint: writing twice as many bytes with 150 baud needs more time, > > > significantly more time then TIMEOUT. > > > > Not really. Francois sent an investigation with pretty exhaustive > > explanation of the behvaiour of COM-port UARTs regarding timings and baud > > rates. > > Have you read it? It says that with a standard UART windows may signals after > 872ms that the tx queue is empty (though there are still 4 bytes to send) when > you write 17 bytes. For 34 bytes he gets 1802ms > TIMEOUT. Where do you see 34 bytes to be written in that test? > And I really don't think that we can expect exactly that same behaviour when > using the linux serial driver, anyway. At the time I was reworking the EV_TXEMPTY to use overlapped IO the reasoning for using 900 ms as the designed time was simple: that was enough to make the test pass under Windows with real hardware. How the patch we are discussing makes the test pass for me under Linux with a USB-serial adapter, which confirms that there is no reasons that it can't work under Linux as well. I understand that there are cases of (really old) hardware where this limit may not be enough, and Francois has already sent a patch which increases timeout. The Francois' patch hasn't been accepted, but that's a different matter. -- Dmitry.

Oct. 24, 2013

Re: winspool.drv/tests: Fix tests compilation with __WINESRC__ defined

by Marvin

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://newtestbot.winehq.org/JobDetails.pl?Key=2988 Your paranoid android. === w864 (32 bit info) === info.c:2925: Test failed: got 0 info.c:2931: Test failed: got 0 info: unhandled exception c0000005 at 0040B246 === w864 (64 bit info) === info.c:2934: Test failed: got NULL DEVMODEA info: unhandled exception c0000005 at 00000000004099CD

Oct. 24, 2013