This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] x86-64: fix ZMM register state tracking
- From: "Jan Beulich" <jbeulich at suse dot com>
- To: <simon dot marchi at ericsson dot com>,<simon dot marchi at polymtl dot ca>
- Cc: <markus dot t dot metzger at intel dot com>,<tim dot wiederhake at intel dot com>, <gdb-patches at sourceware dot org>
- Date: Wed, 03 Oct 2018 08:30:16 -0600
- Subject: Re: [PATCH] x86-64: fix ZMM register state tracking
- References: <5B8FD8B302000078001E5940@prv1-mh.provo.novell.com> <369f9b84-77e3-3fa4-f363-a89ee503cff4@ericsson.com> <5B960E3D020000780013C85E@prv1-mh.provo.novell.com> <f5a4adb4d3f4a0aa84b1666e7967a9fc@polymtl.ca> <5BA0FFB102000078001E97B0@prv1-mh.provo.novell.com> <0e0d9f23-2cbf-eb5a-64fa-6cda3392053a@ericsson.com> <5BAA4E7302000078001EBC73@prv1-mh.provo.novell.com> <580ba4ea-93ae-ffd0-7bce-bd1f75cf9ff9@ericsson.com>
>>> Simon Marchi <simon.marchi@ericsson.com> 10/02/18 9:20 PM >>>
>On 2018-09-25 11:04 AM, Jan Beulich wrote:
>>>>> On 25.09.18 at 05:28, <simon.marchi@ericsson.com> wrote:
>>> On 2018-09-18 09:37 AM, Jan Beulich wrote:
>>> Thanks for the instructions. There is already a test covering AVX512
>>> instructions, so I figured I would add it there. However, I don't
>>> have a processor that supports AVX512, so I'm unable to run the test.
>>>
>>> Here's a patch, can you try to confirm that the test fails without the
>>> fix and passes with the fix? I probably screwed up somewhere, but it
>>> should be pretty close.
>>
>> There are two issues here: First of all, unrelated to this patch, the
>> construct around line 95 in i386-avx512.exp should look like
>>
>> if [is_amd64_regs_target] {
>> set nr_regs 32
>> } else {
>> set nr_regs 8
>> }
>>
>> Of course this also affects other tests in here, but without this correction
>> the loop you add does nothing at all.
>
>Thanks, this has now been fixed in master.
Ah, good to know.
>Here's the revised version with this fixed. I am not sure about the output
>for zmm0 though.
I'll give this a go and adjust if need be, but it'll likely take me a couple of
days to get to it. I take it that ...
>--- a/gdb/testsuite/gdb.arch/i386-avx512.c
>+++ b/gdb/testsuite/gdb.arch/i386-avx512.c
>@@ -249,6 +249,13 @@ main (int argc, char **argv)
>move back to array and check values. */
>move_zmm_data_to_memory ();
>asm ("nop"); /* sixth breakpoint here */
>+
>+ asm ("vpternlogd $0xff, %zmm0, %zmm0, %zmm0");
>+#ifdef __x86_64__s
... the trailing s here simply is a typo.
Jan