search for: ga9352

On Wed, May 13, 2020 at 01:30:11PM +0200, Joerg Roedel wrote: > Yeah, I had this this way in v2, but changed it upon you request[1] :) Yeah, I was wondering why this isn't a separate function - you like them so much. :-P > [1] https://lore.kernel.org/lkml/20200402114941.GA9352 at zn.tnic/ But that one didn't have the ghcb_fault check. Maybe it was being added later... :) Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette

On Tue, Apr 28, 2020 at 05:16:34PM +0200, Joerg Roedel wrote: > @@ -302,9 +313,13 @@ void do_boot_page_fault(struct pt_regs *regs, unsigned long error_code) > * - User faults > * - Reserved bits set > */ > - if (error_code & (X86_PF_PROT | X86_PF_USER | X86_PF_RSVD)) { > + if (ghcb_fault || > + error_code & (X86_PF_PROT | X86_PF_USER | X86_PF_RSVD)) { >

On Tue, Apr 28, 2020 at 05:16:34PM +0200, Joerg Roedel wrote: > @@ -302,9 +313,13 @@ void do_boot_page_fault(struct pt_regs *regs, unsigned long error_code) > * - User faults > * - Reserved bits set > */ > - if (error_code & (X86_PF_PROT | X86_PF_USER | X86_PF_RSVD)) { > + if (ghcb_fault || > + error_code & (X86_PF_PROT | X86_PF_USER | X86_PF_RSVD)) { >

...or_putstr("Unexpected page-fault:"); > > You could carve out the info dumping into a separate function to > unclutter this if-statement (diff ontop): Yeah, I had this this way in v2, but changed it upon you request[1] :) Joerg [1] https://lore.kernel.org/lkml/20200402114941.GA9352 at zn.tnic/