From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8957 invoked by alias); 19 Nov 2013 10:28:04 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 8948 invoked by uid 89); 19 Nov 2013 10:28:03 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.6 required=5.0 tests=AWL,BAYES_50,RDNS_NONE,URIBL_BLOCKED autolearn=no version=3.3.2 X-HELO: glazunov.sibelius.xs4all.nl Received: from Unknown (HELO glazunov.sibelius.xs4all.nl) (83.163.83.176) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 19 Nov 2013 10:28:02 +0000 Received: from glazunov.sibelius.xs4all.nl (kettenis@localhost [127.0.0.1]) by glazunov.sibelius.xs4all.nl (8.14.5/8.14.3) with ESMTP id rAJARnWn001242; Tue, 19 Nov 2013 11:27:49 +0100 (CET) Received: (from kettenis@localhost) by glazunov.sibelius.xs4all.nl (8.14.5/8.14.3/Submit) id rAJARmfT015742; Tue, 19 Nov 2013 11:27:48 +0100 (CET) Date: Tue, 19 Nov 2013 11:13:00 -0000 Message-Id: <201311191027.rAJARmfT015742@glazunov.sibelius.xs4all.nl> From: Mark Kettenis To: walfred.tedeschi@intel.com CC: gdb-patches@sourceware.org In-reply-to: (walfred.tedeschi@intel.com) Subject: Re: [PATCH V7 0/8] Intel(R) MPX register support References: <1381320034-4092-1-git-send-email-walfred.tedeschi@intel.com> <201310201923.r9KJN3O2018018@glazunov.sibelius.xs4all.nl> <201311041029.rA4ATxrR007449@glazunov.sibelius.xs4all.nl> <201311111008.rABA84oE032562@glazunov.sibelius.xs4all.nl> <5280B70D.6050009@intel.com> X-SW-Source: 2013-11/txt/msg00521.txt.bz2 > From: "Tedeschi, Walfred" > Date: Tue, 19 Nov 2013 08:31:19 +0000 > > 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 Sorry, thought I already replied to your message. It's not enitrely clear to me why MPX for x32 can't work, but since I don't particularly care about x32 (and even think it's a silly idea and its implementation on Linux is actually broken) I'm not going to let that stand in the way of getting this stuff in. I also think that this whole splitting up of diffs is going too far. Why don't you just squash this one together with all the other ones and commit them in one go? Cheers, Mark > -----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" > >> 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 > > >