[mdb-discuss] New option for mdb's dis command: Link updated

[m...@bruningsystems.com]

Mike Shapiro wrote:
> On Wed, Mar 12, 2008 at 03:42:41PM -0700, Shesha Sreenivasamurthy SCA wrote:
>   
>> Hello All,
>>Sorry the link was messed up. Here is the new link: 
>> http://lightside.sfbay/~ss161897/webrevs/solaris/mdb_dis_b_v1
>>
>> If there are any audience external to Sun please bear with me until I 
>> open a RFE for this.
>>
>> 0
>> Thanks, (_-/>
>> Shesha; /\ \
>>  (*)--(*)
>> 
>
> Looks reasonable to me.
>
>   
Hi Shesha,
Doesn't "address,count/aI"  do this already?  So, your adding this to 
"::dis"?
Never mind.  Sounds good to me.
max

> -Mike
>
>   

[mdb-discuss] New option for mdb's dis command: Link updated

[Mike Shapiro]

On Wed, Mar 12, 2008 at 03:42:41PM -0700, Shesha Sreenivasamurthy SCA wrote:
> Hello All,
>Sorry the link was messed up. Here is the new link: 
> http://lightside.sfbay/~ss161897/webrevs/solaris/mdb_dis_b_v1
> 
> If there are any audience external to Sun please bear with me until I 
> open a RFE for this.
> 
> 0
> Thanks, (_-/>
> Shesha; /\ \
>  (*)--(*)

Looks reasonable to me.

-Mike

-- 
Mike Shapiro, Sun Microsystems Fishworks. blogs.sun.com/mws/

[mdb-discuss] New option for mdb's dis command: Link updated

[Shesha Sreenivasamurthy SCA]

Hello All,
Sorry the link was messed up. Here is the new link: 
http://lightside.sfbay/~ss161897/webrevs/solaris/mdb_dis_b_v1

If there are any audience external to Sun please bear with me until I 
open a RFE for this.

 0
Thanks, (_-/>
Shesha; /\ \
  (*)--(*)

  G O   G R E E N



Mike Shapiro wrote:
> On Wed, Mar 12, 2008 at 02:02:06PM -0700, Shesha Sreenivasamurthy SCA wrote:
>   
>> Hello Everyone,
>>
>>I feel the lack of getting addresses and symbols side by side from 
>> mdb as a serious short coming of mdb. Therefore, I propose the following 
>> change and request your opinion on this before I can file an RFE for 
>> integrating to the on10-feature-patch gate. I think this is the right 
>> gate, but let me know if I'm wrong.
>>
>> My plan is to do add a new option '-b', which stands for "both" to the 
>> dis command of mdb, so that the addresses can be dis-assembled as follows:
>>
>> 10fcf34   do_relocate+0x29c:lduh  [%i4 + 0x6], %i0
>> 10fcf38   do_relocate+0x2a0:ldx   [%i4 + 0x8], %g2
>> 10fcf3c   do_relocate+0x2a4:ba+0xd4 
>> 
>> 10fcf40   do_relocate+0x2a8:ldx   [%fp + 0x7df], %g5
>> 10fcf44   do_relocate+0x2ac:cmp   %i0, 0x0
>>
>> Corresponding code change is here: http://lightside.sfbay/~ss161897/snon10
>> 
>
> Can you resend the right webrev?  That doesn't look like mdb
> changes, it is something different.
>
> -Mike
>
>   
-- next part --
An HTML attachment was scrubbed...
URL: