[Crash-utility] Re: [PATCH v2] symbols: expand all kernel module symtable if not all expanded previously

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

 



On 2023/11/30 17:52, lijiang wrote:
> On Thu, Nov 30, 2023 at 3:56 PM HAGIO KAZUHITO(萩尾 一仁) <k-hagio-ab@xxxxxxx>
> wrote:
> 
>> On 2023/11/30 0:31, Tao Liu wrote:
>>
>>>> And I did not see the debuginfo again(only one time):
>>>> gdb_get_line_number 7101
>>>>
>>>> Did I miss anything else?
>>
>> Is there no case where sal.symtab == NULL after expansion?
>> I don't think of, but e.g. a wrong address is requested.
>>
>> personally I'm ok with the current patch for fail-safe, quite a simple one.
>>
>> also reverting is troublesome... this is my fault though.
>>
>>
> No worries, Kazu. If any risks are found in the future, we can try to fix
> them.
> 
> Let's leave it there for now. Just an idea, no other specific concerns.

I see, thanks.

Kazu
--
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