Re: loading new envstat properties reboots system

[Nicolas Joly <njoly%pasteur.fr@localhost>]

On Tue, Dec 14, 2010 at 05:10:38AM -0800, Paul Goyette wrote:
> Hmmm.
> 
> I only have one machine remaining with an amdtemp sensor.  I tried to 
> reproduce this problem, but was unsuccessful.  My kernel is a bit newer 
> than yours - 5.99.39 from 2010-10-28 16:51:58 UTC.  (The Aug. 11 kernel 
> no longer exists on nyftp.netbsd.org as far as I can see.)
> 
> There was only one change to envsys code between your kernel and mine:
> 
>    Module Name:    src
>    Committed By:   pgoyette
>    Date:           Wed Sep 22 00:13:55 UTC 2010
> 
>    Modified Files:
>         src/sys/dev/sysmon: sysmon_envsys_events.c
> 
>    Log Message:
>    1. Don't try to call the driver's get_limits routine if it doesn't
>       have one.  Fixes a panic reported by njoly@ in private Email

I confirm that this revision should fix your problem.

> >uvm_fault(0xffff80003eb3eba0, 0x0, 4) -> e
> >fatal page fault in supervisor mode
> >trap type 6 code 10 rip 0 cs 8 flags 10246 cr2 0 cp 0 rsp 
> >ffff8000003e1598c8
> >kernel: page fault trap, code=0
> >Stopped in pid 6850.1 (envstat) at     0:  invalid address

Mine was :

Stopped in pid 755.1 (envstat) at       0:      invalid address
db{1}> bt
kernel: page fault trap, code=0
Faulted in DDB; continuing...

Where it was trying to call unset sme_get_limits function (e.g.
NULL). Kaboom.

-- 
Nicolas Joly

Biological Software and Databanks.
Institut Pasteur, Paris.