This is the mail archive of the mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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,

Am 11/11/2013 11:08 AM, schrieb Mark Kettenis:
From: "Tedeschi, Walfred" <>
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:
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

Attachment: mpx_no_x32.diff
Description: Text document

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]