On Mon, 2016-04-11 at 03:32 -0300, Edjunior Barbosa Machado wrote:

Hi Michael, Anshuman,

I've managed to implement the GDB support for the new regsets and test
on Power8 (BE and LE).

Great work thanks!

The following is an example of GDB 'info
registers all' partial output showing the new registers when inside a
suspended transaction on Power8 LE using this patchset. Please let me
know if you need any additional information or tests from GDB side.

What's the plan for merging the gdb changes?

(gdb) info registers all
...
dscr           0x0      0
ppr            0xc000000000000  3377699720527872
tar            0x0      0
ebbrr          <unavailable>
ebbhr          <unavailable>
bescr          <unavailable>
siar           <unavailable>
sdar           <unavailable>
sier           <unavailable>
mmcr2          <unavailable>
mmcr0          <unavailable>
tfhar          0x10002b30       268446512
texasr         0x110000098000001        76561196215435265
tfiar          0x10002ad9       268446425
cr0            0x10002b2c       268446508

Using 'c' as the prefix is a bit confusing here, as 'cr0' is usually used to
refer to the CR0 field of CR.

Speaking of which, I don't see CR here? Or is it somewhere above in the ... ?

Maybe the prefix could be 'c_' ?

Or do other arches already use 'c' as the prefix?

Otherwise looks good.

cheers