[Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Hui Zhu]

Hi,

I found that *$esp@300 cannot work on GDB,  for example:
actions
Enter actions for tracepoint 4, one per line.
End with a line saying just "end".
>collect *$esp@300
Attempt to dereference a generic pointer.

But *(unsigned char *)$esp@300 can work:
actions
Enter actions for tracepoint 4, one per line.
End with a line saying just "end".
>collect *(unsigned char *)$esp@300
>end

I make a patch for it.  Please help me review it.

Thanks,
Hui

2010-08-24  Hui Zhu  <teawater@gmail.com>

	* gdb.texinfo (Tracepoint Restrictions): Change *$esp@300
	to *(unsigned char *)$esp@300.

---
 doc/gdb.texinfo |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/doc/gdb.texinfo
+++ b/doc/gdb.texinfo
@@ -10390,7 +10390,7 @@ by @code{ptr}.
 @item
 It is not possible to collect a complete stack backtrace at a
 tracepoint.  Instead, you may collect the registers and a few hundred
-bytes from the stack pointer with something like @code{*$esp@@300}
+bytes from the stack pointer with something like @code{*(unsigned
char *)$esp@@300}
 (adjust to use the name of the actual stack pointer register on your
 target architecture, and the amount of stack you wish to capture).
 Then the @code{backtrace} command will show a partial backtrace when

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Eli Zaretskii]

> From: Hui Zhu <teawater@gmail.com>
> Date: Tue, 24 Aug 2010 10:10:19 +0800
> Cc: Eli Zaretskii <eliz@gnu.org>
> 
> But *(unsigned char *)$esp@300 can work:

What about *(int *)$esp@300 ?

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Hui Zhu]

On Tue, Aug 24, 2010 at 11:07, Eli Zaretskii <eliz@gnu.org> wrote:
>> From: Hui Zhu <teawater@gmail.com>
>> Date: Tue, 24 Aug 2010 10:10:19 +0800
>> Cc: Eli Zaretskii <eliz@gnu.org>
>>
>> But *(unsigned char *)$esp@300 can work:
>
> What about *(int *)$esp@300 ?
>

It will make trace collect 1200.
If "unsigned char" is too big, what about *(char *)$esp@300?

Thanks,
Hui

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Eli Zaretskii]

> From: Hui Zhu <teawater@gmail.com>
> Date: Tue, 24 Aug 2010 11:30:03 +0800
> Cc: gdb-patches@sourceware.org
> 
> If "unsigned char" is too big, what about *(char *)$esp@300?

"char" is better, but not because of the length.

However, what I really meant is to ask whether *$esp should indeed
refuse to work, or is that a bug?

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Michael Snyder]

Hui Zhu wrote:
> On Tue, Aug 24, 2010 at 11:07, Eli Zaretskii <eliz@gnu.org> wrote:
>>> From: Hui Zhu <teawater@gmail.com>
>>> Date: Tue, 24 Aug 2010 10:10:19 +0800
>>> Cc: Eli Zaretskii <eliz@gnu.org>
>>>
>>> But *(unsigned char *)$esp@300 can work:
>> What about *(int *)$esp@300 ?
>>
> 
> It will make trace collect 1200.
> If "unsigned char" is too big, what about *(char *)$esp@300?

But 300 words might be good too.  What about:

+bytes from the stack pointer with something like @code{*
(char *)$esp@@300}, or 300 words with @code{*(int *)$esp@@300}.

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Michael Snyder]

Eli Zaretskii wrote:
>> From: Hui Zhu <teawater@gmail.com>
>> Date: Tue, 24 Aug 2010 11:30:03 +0800
>> Cc: gdb-patches@sourceware.org
>>
>> If "unsigned char" is too big, what about *(char *)$esp@300?
> 
> "char" is better, but not because of the length.
> 
> However, what I really meant is to ask whether *$esp should indeed
> refuse to work, or is that a bug?

I imagine it must have worked at one time...

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Eli Zaretskii]

> Date: Tue, 24 Aug 2010 10:32:58 -0700
> From: Michael Snyder <msnyder@vmware.com>
> CC: Eli Zaretskii <eliz@gnu.org>, 
>  "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
> 
> Hui Zhu wrote:
> > On Tue, Aug 24, 2010 at 11:07, Eli Zaretskii <eliz@gnu.org> wrote:
> >>> From: Hui Zhu <teawater@gmail.com>
> >>> Date: Tue, 24 Aug 2010 10:10:19 +0800
> >>> Cc: Eli Zaretskii <eliz@gnu.org>
> >>>
> >>> But *(unsigned char *)$esp@300 can work:
> >> What about *(int *)$esp@300 ?
> >>
> > 
> > It will make trace collect 1200.
> > If "unsigned char" is too big, what about *(char *)$esp@300?
> 
> But 300 words might be good too.  What about:
> 
> +bytes from the stack pointer with something like @code{*
> (char *)$esp@@300}, or 300 words with @code{*(int *)$esp@@300}.

Fine with me.

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Eli Zaretskii]

> Date: Tue, 24 Aug 2010 10:35:34 -0700
> From: Michael Snyder <msnyder@vmware.com>
> CC: Hui Zhu <teawater@gmail.com>, 
>  "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
> 
> Eli Zaretskii wrote:
> >> From: Hui Zhu <teawater@gmail.com>
> >> Date: Tue, 24 Aug 2010 11:30:03 +0800
> >> Cc: gdb-patches@sourceware.org
> >>
> >> If "unsigned char" is too big, what about *(char *)$esp@300?
> > 
> > "char" is better, but not because of the length.
> > 
> > However, what I really meant is to ask whether *$esp should indeed
> > refuse to work, or is that a bug?
> 
> I imagine it must have worked at one time...

If we-all think now that it's okay for *$esp not to work, then let's
fix the docs and be done.

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Michael Snyder]

Eli Zaretskii wrote:
>> Date: Tue, 24 Aug 2010 10:35:34 -0700
>> From: Michael Snyder <msnyder@vmware.com>
>> CC: Hui Zhu <teawater@gmail.com>, 
>>  "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
>>
>> Eli Zaretskii wrote:
>>>> From: Hui Zhu <teawater@gmail.com>
>>>> Date: Tue, 24 Aug 2010 11:30:03 +0800
>>>> Cc: gdb-patches@sourceware.org
>>>>
>>>> If "unsigned char" is too big, what about *(char *)$esp@300?
>>> "char" is better, but not because of the length.
>>>
>>> However, what I really meant is to ask whether *$esp should indeed
>>> refuse to work, or is that a bug?
>> I imagine it must have worked at one time...
> 
> If we-all think now that it's okay for *$esp not to work, then let's
> fix the docs and be done.

I think the reason it doesn't work now is that it's typed as a void pointer.

(gdb) pt $esp
type = void *

Probably it used to be typed as a char * or something.

Re: [Doc]tracepoint: change *$esp@300 to *(unsigned char *)$esp@300

[Pedro Alves]

On Tuesday 24 August 2010 19:07:17, Michael Snyder wrote:
> I think the reason it doesn't work now is that it's typed as a void pointer.
> 
> (gdb) pt $esp
> type = void *
> 
> Probably it used to be typed as a char * or something.

Dereferencing void* used to "work" in agent expressions,
but since it is forbidden in normal expressions, 

 (gdb) p *$sp
 Attempt to dereference a generic pointer.

the agent expression code was fixed to behave the same:

2010-02-11  Pedro Alves  <pedro@codesourcery.com>

        * ax-gdb.c (gen_deref): Don't allow dereferencing void pointers.

Let's just fix the docs.

-- 
Pedro Alves