From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3347 invoked by alias); 4 Nov 2013 10:30:31 -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 3338 invoked by uid 89); 4 Nov 2013 10:30:31 -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; Mon, 04 Nov 2013 10:30:29 +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 rA4AU0Ug024068; Mon, 4 Nov 2013 11:30:00 +0100 (CET) Received: (from kettenis@localhost) by glazunov.sibelius.xs4all.nl (8.14.5/8.14.3/Submit) id rA4ATxrR007449; Mon, 4 Nov 2013 11:29:59 +0100 (CET) Date: Mon, 04 Nov 2013 10:30:00 -0000 Message-Id: <201311041029.rA4ATxrR007449@glazunov.sibelius.xs4all.nl> From: Mark Kettenis To: walfred.tedeschi@intel.com CC: walfred.tedeschi@intel.com, tromey@redhat.com, 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> X-SW-Source: 2013-11/txt/msg00052.txt.bz2 > From: "Tedeschi, Walfred" > Date: Mon, 4 Nov 2013 09:06:06 +0000 > > Mark, > > Do you think the proposal makes sense? Sorry. No I don't think that makes sense. So I think you should go ahead and commit your diff as-is (i.e. with both bndN and bndNraw register names). Cheers, Mark > -----Original Message----- > From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-owner@sourceware.org] On Behalf Of Tedeschi, Walfred > Sent: Monday, October 21, 2013 1:33 PM > To: Mark Kettenis > Cc: tromey@redhat.com; gdb-patches@sourceware.org > Subject: RE: [PATCH V7 0/8] Intel(R) MPX register support > > Mark, > > Our internal users of MPX found it useful to see the raw value, processed values and size together. > They found out that it was really handy to avoid doing the complement of one for every bound manipulation. > (I our first patch we changed the type system to add a complement of one type, but this was considered overkill) > > In this sense we would like still to display the bounds on a friendly manner to the user. When I say friendly I mean displaying values that are meaningful to the user. > ->Lower and upper bound should be presented as address format. > ->No additional manipulation needed to calculate the upper bound limit. > > On the other hand we could add the nice display on the python script and eliminate the pseudo register set, having only "bndN". > > Would you be ok with such a change? > (Having the pretty print only on the python side and eliminating the bndraws) > > > Thanks and regards, > -Fred. > > PS: You might use SDE to have an idea how it looks like. > http://software.intel.com/en-us/articles/using-intel-mpx-with-the-intel-software-development-emulator > http://software.intel.com/en-us/articles/intel-software-development-emulator > > > > -----Original Message----- > From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-owner@sourceware.org] On Behalf Of Mark Kettenis > Sent: Sunday, October 20, 2013 9:23 PM > To: Tedeschi, Walfred > Cc: tromey@redhat.com; gdb-patches@sourceware.org; Tedeschi, Walfred > Subject: Re: [PATCH V7 0/8] Intel(R) MPX register support > > > From: Walfred Tedeschi > > Cc: gdb-patches@sourceware.org, Walfred Tedeschi > > > > > > Mark and all, > > > > I have noticed no feedback on this patch series. > > Is there a major change that you would like to see in here? > > > > Thanks a lot for your support, > > Best regards, > > -Fred > > Walfred, > > The only thing I'm still somewhat unhappy about the fact that this introduces the "bndNraw" register names in addition to the "bndN" > names. I think having both the "raw" and "cooked" variants present in GDB's user interface will be confusing. But it is hard to decide what the right interface is for a feature that isn't available in hardware yet and people don't really have any experience with debugging code that uses MPX. > > 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 > > >