From: "Sturm, Michael" <michael.sturm@intel.com>
To: Pedro Alves <palves@redhat.com>, mark.kettenis@xs4all.nl, eliz@gnu.org
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH v3 3/5] Rename target descriptions to reflect actual content of description.
Date: Thu, 26 Jan 2017 15:55:00 -0000 [thread overview]
Message-ID: <588A1BE9.10408@intel.com> (raw)
In-Reply-To: <aee6eb17-e617-23bc-219a-b2333915c195@redhat.com>
Hello Pedro,
thank you for pointing this out - this change is not required. I'll
revert that part for the next version. How about the remainder of the patch?
Thanks and Regards,
Michael
On 26/01/2017 15:50, Pedro Alves wrote:
> On 12/06/2016 10:58 AM, Michael Sturm wrote:
>> # Record which registers should be sent to GDB by default after stop.
>> aarch64-expedite = x29,sp,pc
>> arm-expedite = r11,sp,pc
>> -i386-expedite = ebp,esp,eip
>> -amd64-expedite = rbp,rsp,rip
>> -x32-expedite = rbp,rsp,rip
>> +i386/i386-expedite = ebp,esp,eip
>> +i386/i386-linux-expedite = ebp,esp,eip
>> +i386/amd64-expedite = rbp,rsp,rip
>> +i386/amd64-linux-expedite = rbp,rsp,rip
>> +i386/i386-avx-expedite = ebp,esp,eip
>> +i386/i386-avx-linux-expedite = ebp,esp,eip
>> +i386/i386-mpx-expedite = ebp,esp,eip
>> +i386/i386-avx-mpx-expedite = ebp,esp,eip
>> +i386/i386-mpx-linux-expedite = ebp,esp,eip
>> +i386/i386-avx-mpx-linux-expedite = ebp,esp,eip
>> +i386/i386-avx-mpx-avx512-expedite = ebp,esp,eip
>> +i386/i386-avx-mpx-avx512-linux-expedite = ebp,esp,eip
>> +i386/i386-mmx-expedite = ebp,esp,eip
>> +i386/i386-mmx-linux-expedite = ebp,esp,eip
>> +i386/amd64-avx-expedite = rbp,rsp,rip
>> +i386/amd64-avx-linux-expedite = rbp,rsp,rip
>> +i386/amd64-mpx-expedite = rbp,rsp,rip
>> +i386/amd64-avx-mpx-expedite = rbp,rsp,rip
>> +i386/amd64-mpx-linux-expedite = rbp,rsp,rip
>> +i386/amd64-avx-mpx-linux-expedite = rbp,rsp,rip
>> +i386/amd64-avx-mpx-avx512-expedite = rbp,rsp,rip
>> +i386/amd64-avx-mpx-avx512-linux-expedite = rbp,rsp,rip
>> +i386/x32-expedite = rbp,rsp,rip
>> +i386/x32-linux-expedite = rbp,rsp,rip
>> +i386/x32-avx-expedite = rbp,rsp,rip
>> +i386/x32-avx-linux-expedite = rbp,rsp,rip
>> +i386/x32-avx-mpx-avx512-expedite = rbp,rsp,rip
>> +i386/x32-avx-mpx-avx512-linux-expedite = rbp,rsp,rip
> I didn't see a rationale for this change.
>
> Thanks,
> Pedro Alves
>
Intel Deutschland GmbH
Registered Address: Am Campeon 10-12, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Christin Eisenschmid, Christian Lamprechter
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928
next prev parent reply other threads:[~2017-01-26 15:55 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-12-06 10:58 [PATCH v3 0/5] Add support for PKRU register to GDB and GDBServer Michael Sturm
2016-12-06 10:58 ` [PATCH v3 2/5] Change xstate_bv handling to use 8 bytes of data Michael Sturm
2017-01-26 14:46 ` Pedro Alves
2016-12-06 10:58 ` [PATCH v3 5/5] Add support for Intel PKRU register to GDB and GDBserver Michael Sturm
2016-12-08 15:35 ` Luis Machado
2017-01-26 15:38 ` Pedro Alves
2017-01-27 15:00 ` Sturm, Michael
2017-01-27 15:22 ` Pedro Alves
2017-02-01 12:34 ` Sturm, Michael
2016-12-06 10:58 ` [PATCH v3 1/5] Sync up x86-gcc-cpuid.h with cpuid.h from gcc-6 branch Michael Sturm
2016-12-07 18:28 ` Luis Machado
2016-12-08 12:16 ` Sturm, Michael
2016-12-08 15:35 ` Luis Machado
2017-01-26 14:29 ` Pedro Alves
2016-12-06 10:58 ` [PATCH v3 3/5] Rename target descriptions to reflect actual content of description Michael Sturm
2017-01-26 14:50 ` Pedro Alves
2017-01-26 15:55 ` Sturm, Michael [this message]
2017-01-26 16:17 ` Pedro Alves
2016-12-06 10:59 ` [PATCH v3 4/5] Add target description for avx-avx512 Michael Sturm
2017-01-26 14:57 ` Pedro Alves
2017-01-26 16:02 ` Sturm, Michael
[not found] ` <DB865139DDA33A4DA36CC4A460C112734740AE54@irsmsx105.ger.corp.intel.com>
2017-01-26 14:07 ` [PING][PATCH v3 0/5] Add support for PKRU register to GDB and GDBServer Sturm, Michael
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=588A1BE9.10408@intel.com \
--to=michael.sturm@intel.com \
--cc=eliz@gnu.org \
--cc=gdb-patches@sourceware.org \
--cc=mark.kettenis@xs4all.nl \
--cc=palves@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox