On 2016-03-29 09:13, Daniel Wagner wrote:
> On 03/29/2016 08:59 AM, Jan Kiszka wrote:
>> On 2016-03-29 08:41, Daniel Wagner wrote:
>>> for module in module_list():
>>> gdb.write("{address} {name:<19} {size:>8} {ref}".format(
>>> - address=str(module['module_core']).split()[0],
>>> + module_layout=module['core_layout']
>>> + addressr=str(module_layout['base'])
>>
>> This looks untested.
>
> Yep, I send out the untested version. Need more coffee...
>
>>> name=module['name'].string(),
>>> size=str(module['core_size']),
>>> ref=str(module['refcnt']['counter'])))
>>> diff --git a/scripts/gdb/linux/symbols.py b/scripts/gdb/linux/symbols.py
>>> index 627750c..8650ede 100644
>>> --- a/scripts/gdb/linux/symbols.py
>>> +++ b/scripts/gdb/linux/symbols.py
>>> @@ -108,7 +108,8 @@ lx-symbols command."""
>>>
>>> def load_module_symbols(self, module):
>>> module_name = module['name'].string()
>>> - module_addr = str(module['module_core']).split()[0]
>>> + module_layout = module['core_layout']
>>> + module_addr = str(module_layout['base'])
>>>
>>> module_file = self._get_module_file(module_name)
>>> if not module_file and not self.module_files_updated:
>>>
>>
>> Thanks, but ad4db3b24a should already be on its way into stable.
>
> Ah, I haven't got that fix yet because I am still on a older branch.
> Also my google-foo didn't help.
>
> Sorry about that noise.
No problem. At least we now know another frequent user.
BTW, Kieran and some other people at Linaro started to think about how
to auto-test for such regressions - probably the only reliable way to
catch them.
Jan
--
Siemens AG, Corporate Technology, CT RDA ITP SES-DE
Corporate Competence Center Embedded Linux
