Re: [Xen-devel] [PATCH] tools/gdbsx: fix 'g' packet response for 64bit guests

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Tue, Jul 31, 2018 at 06:10:07PM +0200, Marek Marczykowski-Górecki wrote:
> On Tue, Jul 31, 2018 at 10:00:24AM +0100, Wei Liu wrote:
> > On Tue, Jul 31, 2018 at 04:30:42AM +0200, Marek Marczykowski-Górecki wrote:
> > > --- a/tools/debugger/gdbsx/gx/gx_local.c
> > > +++ b/tools/debugger/gdbsx/gx/gx_local.c
> > > @@ -45,8 +45,8 @@ prnt_32regs(struct xg_gdb_regs32 *r32p)
> > >  static void
> > >  prnt_64regs(struct xg_gdb_regs64 *r64p)
> > >  {
> > > -    printf("rip:"XGF64" rsp:"XGF64" flags:"XGF64"\n", r64p->rip, 
> > > r64p->rsp,
> > > -           r64p->rflags);
> > > +    printf("rip:"XGF64" rsp:"XGF64" flags:%08x\n", r64p->rip, r64p->rsp,
> > > +           r64p->eflags);
> > 
> > I think it would be better to introduce XGF32 and XGFM32 in the header.
> 
> I was inspired by prnt_32regs function, which use explicit %08x. Should
> it be changed there too (to PRIx32)? Otherwise that would be
> inconsistent (and maybe warrant a totally separate cleanup).

See Andrew's suggestion to switch to PRIx*.

Wei.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel