Re: [PATCH v6 0/1] MR4720: Implement test_continue() for amd64

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/ntdll/tests/exception.c:
> -
>    0x48, 0x8b, 0x54, 0x24, 0x58, /* mov 8*11(%rsp), %rdx; alertable */
> -
>    0x48, 0x83, 0xec, 0x70,       /* sub $0x70, %rsp; change stack   */
> -
> -
>    /* setup context to return to label 1 */
> -
>    0x48, 0x8d, 0x05, 0x18, 0x00, 0x00, 0x00, /* lea 1f(%rip) */
> -
>    0x48, 0x89, 0x81, 0xf8, 0x00, 0x00, 0x00, /* mov %rax, 0xf8(%rcx); context.Rip */
> -
> -
>    /* flip some EFLAGS */
> -
>    0x9c,                                           /* pushf */
> -
>    /*
> -
>       0x0001 Carry flag
> -
>       0x0004 Parity flag
> -
>       0x0010 Auxiliary Carry flag
> -
>       0x0040 Zero flag
> -
>       0x0080 Sign flag
> -
>       FIXME: 0x0400 Direction flag - not changing as it breaks Wine

Nested Task surely breaks Wine; however, I cannot reproduce such breakage with Direction Flag.

https://testbot.winehq.org/JobDetails.pl?Key=141583

Therefore, I suggest that DF be included in the mask as well.

(Did you make sure to test each flag separately?)