On Sat, 9 Mar 2019 08:50:58 +0000
Alexey Dokuchaev <da...@freebsd.org> wrote:

> On Sat, Mar 09, 2019 at 03:18:38AM +0000, Justin Hibbits wrote:
> > New Revision: 344960
> > URL: https://svnweb.freebsd.org/changeset/base/344960
> > 
> > Log:
> >   powerpc: Print trap frame address for fatal traps
> >   
> >   MFC after:        1 week
> > 
> > Modified:
> >   head/sys/powerpc/powerpc/trap.c
> > 
> > Modified: head/sys/powerpc/powerpc/trap.c
> > ==============================================================================
> > --- head/sys/powerpc/powerpc/trap.c Sat Mar  9 03:15:09
> > 2019        (r344959) +++ head/sys/powerpc/powerpc/trap.c
> > Sat Mar  9 03:18:37 2019    (r344960) @@ -546,6 +546,7 @@
> > printtrap(u_int vector, struct trapframe *frame, int i printf("
> > current msr     = 0x%" PRIxPTR "\n", mfmsr()); printf("
> > lr              = 0x%" PRIxPTR " (0x%" PRIxPTR ")\n", frame->lr,
> > frame->lr - (register_t)(__startkernel - KERNBASE));
> > +   printf("   frame           = %p\n", frame);
> >     printf("   curthread       = %p\n", curthread);  
> 
> Some of those are printed with %PRIxPTR, some with %p.  Perhaps this
> inconsistency can be avoided?
> 
> ./danfe

PRIxPTR is for printing a pointer-like value that's not a pointer.  The
other values printed are of type register_t, not void *.

- Justin
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to