[Crash-utility] Re: [PATCH v4 01/16] Remove 'frame' from prohibited commands list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Lijiang,

On Mon, Jun 3, 2024 at 9:26 AM lijiang <lijiang@xxxxxxxxxx> wrote:
>
> Hi, Tao
> Thank you for the update.
>
> I will look at the patchset later, currently I'm working on other issues.
>
Sure, no problem.

Thanks,
Tao Liu

> Thanks.
> Lianbo
>
> On Fri, May 31, 2024 at 5:20 PM Tao Liu <ltao@xxxxxxxxxx> wrote:
>>
>> From: Aditya Gupta <adityag@xxxxxxxxxxxxx>
>>
>> Currently, 'frame' is prohibited on every architecture, even in gdb mode.
>> But with gdb stack unwinding working correctly on some architectures (eg. ppc64,
>> x86_64, arm64), so remove it from prohibited list, so that 'frame' command
>> can be used
>>
>> This has an implication on all architectures, that 'frame' now outputs
>> this in default mode:
>>
>> ```
>> crash> frame
>>  #0  <unavailable> in ?? ()
>> ```
>>
>> instead of the before prohibited message
>>
>> ```
>> crash> frame
>> crash: prohibited gdb command: frame
>> ```
>>
>> Though, 'frame' should work in gdb mode for architectures that implement
>> 'machdep->get_cpu_reg'
>>
>> Cc: Sourabh Jain <sourabhjain@xxxxxxxxxxxxx>
>> Cc: Hari Bathini <hbathini@xxxxxxxxxxxxx>
>> Cc: Mahesh J Salgaonkar <mahesh@xxxxxxxxxxxxx>
>> Cc: Naveen N. Rao <naveen.n.rao@xxxxxxxxxxxxxxxxxx>
>> Cc: Lianbo Jiang <lijiang@xxxxxxxxxx>
>> Cc: HAGIO KAZUHITO(萩尾 一仁) <k-hagio-ab@xxxxxxx>
>> Cc: Tao Liu <ltao@xxxxxxxxxx>
>> Cc: Alexey Makhalov <alexey.makhalov@xxxxxxxxxxxx>
>> Signed-off-by: Aditya Gupta <adityag@xxxxxxxxxxxxx>
>> ---
>>  gdb_interface.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/gdb_interface.c b/gdb_interface.c
>> index b14319c..8f99a0d 100644
>> --- a/gdb_interface.c
>> +++ b/gdb_interface.c
>> @@ -711,7 +711,7 @@ static char *prohibited_list[] = {
>>         "watch", "rwatch", "awatch", "attach", "continue", "c", "fg", "detach",
>>         "finish", "handle", "interrupt", "jump", "kill", "next", "nexti",
>>         "signal", "step", "s", "stepi", "target", "until", "delete",
>> -       "clear", "disable", "enable", "condition", "ignore", "frame", "catch",
>> +       "clear", "disable", "enable", "condition", "ignore", "catch",
>>         "tcatch", "return", "file", "exec-file", "core-file", "symbol-file",
>>         "load", "si", "ni", "shell", "sy",
>>         NULL  /* must be last */
>> --
>> 2.40.1
>>
--
Crash-utility mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxxxxxx
https://${domain_name}/admin/lists/devel.lists.crash-utility.osci.io/
Contribution Guidelines: https://github.com/crash-utility/crash/wiki




[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux