[Crash-utility] [PATCH v3 2/2] arm/arm64: read elf notes for qemu generated cores

Andrew Jones drjones at redhat.com
Mon Nov 23 19:57:14 UTC 2015


On Mon, Nov 23, 2015 at 02:44:21PM -0500, Dave Anderson wrote:
> 
> 
> ----- Original Message -----
> > > > Drew,
> > > > 
> > > > Can you sanity-test the attached patch?  If it works OK for you, I'll post
> > > > it as-is.
> > > 
> > > Works. I tested on both arm and aarch64. I still think we should commit
> > > it as two patches though. The qemu one, with its commit message focused
> > > on qemu, and then this one, with its commit message focused on handling
> > > missing prstatus for some cpus for unknown reasons. Actually, I don't
> > > think it should even be possible to be missing prstatus for some cpus in
> > > the qemu case (qemu will always populate them all), so the "dumpfiles
> > > created with qemu..." comment might need some tweaking as well, since the
> > > else-continue is being added to its if-block.
> > > 
> > > Thanks,
> > > drew
> > 
> > All right, I'll post your v3 patch as-is, and make a follow-up patch
> > with my additions (plus a tweak to the comment).
> > 
> > Thanks,
> 
> Your v3 patch, and then my subsequent update, are queued for crash-7.1.4:
>  
>   https://github.com/crash-utility/crash/commit/b2d8f20407dd575356f028de9f79d590c0fecf1d
>   https://github.com/crash-utility/crash/commit/d963c2c15278b72bab23bacdf3fad574c577b50f
> 
> Thanks for your help with this,
>   Dave

Thank you!

drew
>  
> 
> Thanks,
>   Dave




More information about the Crash-utility mailing list