> Date: Sat, 4 Nov 2017 18:51:34 +0100
> From: Martin Pieuchot <m...@openbsd.org>
> 
> On 04/11/17(Sat) 17:20, Paul Irofti wrote:
> > Hi,
> > 
> > Someone, somewhere (perhaps on bugs@?) said they would like to see
> > something useful when typing show panic on a page fault'd kernel. That
> > struck very close to home so I went ahead and did it.
> > 
> > The following diff shows the uvm_fault and the first trace entry in an
> > attempt to mimic a real show panic. The uvm_fault entry is extra.
> > 
> > Currently I only wrote support for amd64 but I can add support for
> > other architectures if you guys like it.
> > 
> > Thoughts?
> 
> I think you should just change the printf("uvm_fault...) into a panic.

No.  That makes it a lot harder to investigate the faulting instruction.

I think Paul's idea makes sense; might need a bit of polishing though.

Reply via email to