Bug 239372

Summary: r350207 panic on ppc64 (G5Quad) in moea64_pvo_remove_from_page
Product: Base System Reporter: Francis Little <oggy>
Component: kernAssignee: freebsd-ppc (Nobody) <powerpc>
Status: Closed FIXED    
Severity: Affects Only Me CC: cem, jhibbits, tuexen
Priority: ---    
Version: CURRENT   
Hardware: powerpc   
OS: Any   
Attachments:
Description Flags
panic screen shot none

Description Francis Little 2019-07-22 07:35:12 UTC 
Hi, updated to r350207 on PowerMac G5 Quad.

On Reboot, the Kernel panics instantly in moea64_pvo_remove_from_page, screen shot attached.
Comment 1 Francis Little 2019-07-22 07:38:05 UTC 
Created attachment 205983 [details]
panic screen shot
Comment 2 Justin Hibbits freebsd_committer freebsd_triage 2019-07-22 15:42:47 UTC 
Oops, sorry.  Apparently I hadn't tested the same patch I committed.  On line 1458, the 'if (oldpvo != NULL)' should be 'if (error == EEXIST)'.  I'll commit the fix tonight.
Comment 3 commit-hook freebsd_committer freebsd_triage 2019-07-22 23:00:44 UTC 
A commit references this bug:

Author: jhibbits
Date: Mon Jul 22 22:59:51 UTC 2019
New revision: 350232
URL: https://svnweb.freebsd.org/changeset/base/350232

Log:
  powerpc: Unbreak 64-bit pmap from 350206

  oldpvo is never explicitly NULL'd by moea64_pvo_enter(), so don't check for
  NULL to do anything, only check error.

  PR:		239372
  Reported by:	Francis Little

Changes:
  head/sys/powerpc/aim/mmu_oea64.c
Comment 4 Francis Little 2019-07-23 20:18:04 UTC 
Yep, that's fixed it!