RE: [PATCH V7 0/8] Intel(R) MPX register support

["Tedeschi, Walfred" <walfred dot tedeschi at intel dot com>]

Mark,

Do you think the proposal makes sense? 

Regards,
-Fred

-----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 <walfred.tedeschi@intel.com>
> Cc: gdb-patches@sourceware.org, Walfred Tedeschi 
> <walfred.tedeschi@intel.com>
> 
> 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