<div dir="ltr"><div>Thank you for the suggestions. I will look into it.</div><div>However, I'm wondering why the test fails on that two 32bit systems...<br></div></div><br><div class="gmail_quote"><div dir="ltr">Am So., 9. Sep. 2018 um 21:10 Uhr schrieb Chris Robinson <<a href="mailto:chris.kcat@gmail.com">chris.kcat@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 09/09/2018 11:16 AM, Roger Zoellner wrote:<br>
> +static DWORD WINAPI thread_main(LPVOID lpParam)<br>
> +{<br>
> +    /* Thread is created with CREATE_SUSPENDED flag set and isn't doing anything */<br>
> +}<br>
<br>
You should still probably do<br>
<br>
(void)lpParam;<br>
return 0;<br>
<br>
to avoid compiler warnings about unused parameters and lacking a return <br>
value.<br>
<br>
> +    /* Test required affinity mask, all bits set */<br>
> +    req_aff = ~0u;<br>
<br>
This isn't all bits set for 64-bit targets. Try<br>
<br>
req_aff = ~(ULONG_PTR)0;<br>
<br>
for that.<br>
<br>
> +    status = pNtSetInformationThread(thread, class, &req_aff, req_aff_length);<br>
> +    ok(status == STATUS_SUCCESS, "NtSetInformationThread failed. Expected STATUS_SUCCESS, got %08x (class=ThreadAffinityMask, case 2, req_aff=%08x, process_aff=%08x)\n", status, req_aff, process_aff);<br>
> ...<br>
> +    /* Test required affinity mask = process affinity mask with a prepended bit set */<br>
> +    req_aff = (process_aff << 1) | process_aff;<br>
> +    status = pNtSetInformationThread(thread, class, &req_aff, req_aff_length);<br>
> +    ok(status == STATUS_SUCCESS, "NtSetInformationThread failed. Expected STATUS_SUCCESS, got %08x (class=ThreadAffinityMask, case 4, req_aff=%08x, process_aff=%08x)\n", status, req_aff, process_aff);<br>
<br>
Might it be worth checking to see what's returned from <br>
NtQueryInformationThread after trying to set extraneous mask bits?<br>
<br>
Also, the tests that currently fail under Wine should be marked with <br>
todo_wine until a proper fix is made, and such a fix would remove the <br>
rodo_wines (unless the tests are part of a series that come after the fix).<br>
<br>
<br>
</blockquote></div>