Re: [PATCH] scripts/gdb: Fix loading of modules information

2016-03-30 Thread Daniel Wagner
On 03/29/2016 09:22 AM, Jan Kiszka wrote: > No problem. At least we now know another frequent user. Let's say sometimes :) > 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. Good idea. I a

Re: [PATCH] scripts/gdb: Fix loading of modules information

2016-03-29 Thread Jan Kiszka
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_cor

Re: [PATCH] scripts/gdb: Fix loading of modules information

2016-03-29 Thread Daniel Wagner
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_layo

Re: [PATCH] scripts/gdb: Fix loading of modules information

2016-03-29 Thread Jan Kiszka
On 2016-03-29 08:41, Daniel Wagner wrote: > From: Daniel Wagner > > The module_core symbols is not available due to > > 7523e4dc5057 ("module: use a structure to encapsulate layout.") > > The base address of the module is now stored inside > struct module_layout. > > Signed-off-by: Daniel Wagn

[PATCH] scripts/gdb: Fix loading of modules information

2016-03-28 Thread Daniel Wagner
From: Daniel Wagner The module_core symbols is not available due to 7523e4dc5057 ("module: use a structure to encapsulate layout.") The base address of the module is now stored inside struct module_layout. Signed-off-by: Daniel Wagner Cc: Jiri Kosina Cc: Rusty Russell --- scripts/gdb/linux