This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [PATCH V7 0/8] Intel(R) MPX register support
- From: "Tedeschi, Walfred" <walfred dot tedeschi at intel dot com>
- To: "Tedeschi, Walfred" <walfred dot tedeschi at intel dot com>, Mark Kettenis <mark dot kettenis at xs4all dot nl>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Tue, 19 Nov 2013 08:31:19 +0000
- Subject: RE: [PATCH V7 0/8] Intel(R) MPX register support
- Authentication-results: sourceware.org; auth=none
- References: <1381320034-4092-1-git-send-email-walfred dot tedeschi at intel dot com> <201310201923 dot r9KJN3O2018018 at glazunov dot sibelius dot xs4all dot nl> <AC542571535E904D8E8ADAE745D60B191B1BC216 at IRSMSX104 dot ger dot corp dot intel dot com> <AC542571535E904D8E8ADAE745D60B191B1C36BE at IRSMSX104 dot ger dot corp dot intel dot com> <201311041029 dot rA4ATxrR007449 at glazunov dot sibelius dot xs4all dot nl> <AC542571535E904D8E8ADAE745D60B191B1C74E0 at IRSMSX104 dot ger dot corp dot intel dot com> <201311111008 dot rABA84oE032562 at glazunov dot sibelius dot xs4all dot nl> <5280B70D dot 6050009 at intel dot com>
Hello Mark,
Do you agree with this diff? Of course I will distribute it where it belongs to in the patch.
Thanks a lot for your support and best regards,
-Fred
-----Original Message-----
From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-owner@sourceware.org] On Behalf Of Walfred Tedeschi
Sent: Monday, November 11, 2013 11:53 AM
To: Mark Kettenis; gdb-patches@sourceware.org
Subject: Re: [PATCH V7 0/8] Intel(R) MPX register support
Hi Mark,
What I mean is the appended patch. This is what I added on top of v7.
1. It removes MPX support for x32.
2. Improves i386-xstate.h macros. (identation was weird) 3. Fix the regression on MPX initialization on i386-tdep.c
I considered that to be easy to review than having all the patch series again.
:)
Thanks a lot!
Best regards,
-Fred
Am 11/11/2013 11:08 AM, schrieb Mark Kettenis:
>> From: "Tedeschi, Walfred" <walfred.tedeschi@intel.com>
>> Date: Mon, 11 Nov 2013 08:15:36 +0000
>>
>> Hello Mark,
>>
>> I am not sure I got it right. Is this a ok to commit?
> Pretty much.
>
>> If so, I still would like to do some changes before proceeding.
>> Basically I want to remove the x32 support since it makes no sense
>> and a regression I caused. Those changes where mentioned here:
>> https://sourceware.org/ml/gdb-patches/2013-10/msg00513.html
> I'll await your patch then ;).
>
>> To avoid sending you a full patch again my Idea is to send an ad-hoc
>> patch to be added on top of my V7. While doing the commit then I
>> would join then on the respective place.
> Not sure what you mean by that. As long as all the intermediate
> states compile and you don't introduce regressions for non-MPX
> support, this should be fine.
Intel GmbH
Dornacher Strasse 1
85622 Feldkirchen/Muenchen, Deutschland
Sitz der Gesellschaft: Feldkirchen bei Muenchen
Geschaeftsfuehrer: Christian Lamprechter, Hannes Schwaderer, Douglas Lusk
Registergericht: Muenchen HRB 47456
Ust.-IdNr./VAT Registration No.: DE129385895 Citibank Frankfurt a.M. (BLZ 502 109 00) 600119052
Intel GmbH
Dornacher Strasse 1
85622 Feldkirchen/Muenchen, Deutschland
Sitz der Gesellschaft: Feldkirchen bei Muenchen
Geschaeftsfuehrer: Christian Lamprechter, Hannes Schwaderer, Douglas Lusk
Registergericht: Muenchen HRB 47456
Ust.-IdNr./VAT Registration No.: DE129385895
Citibank Frankfurt a.M. (BLZ 502 109 00) 600119052