Re: PR/58201 CVS commit: src/sys/dev/acpi

[Malte Dehling <mdehling%gmail.com@localhost>]

The following reply was made to PR kern/58201; it has been noted by GNATS.

From: Malte Dehling <mdehling%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: PR/58201 CVS commit: src/sys/dev/acpi
Date: Fri, 26 Apr 2024 15:30:39 -0500

 On Fri, Apr 26, 2024 at 06:20:01PM +0000, Christos Zoulas wrote:
 > The following reply was made to PR kern/58201; it has been noted by GNATS.
 >
 > From: "Christos Zoulas" <christos%netbsd.org@localhost>
 > To: gnats-bugs%gnats.NetBSD.org@localhost
 > Cc:
 > Subject: PR/58201 CVS commit: src/sys/dev/acpi
 > Date: Fri, 26 Apr 2024 14:19:18 -0400
 >
 >  Module Name:	src
 >  Committed By:	christos
 >  Date:		Fri Apr 26 18:19:18 UTC 2024
 >
 >  Modified Files:
 >  	src/sys/dev/acpi: acpi_bat.c
 >
 >  Log Message:
 >  PR/58201: Malte Dehling: re-order sysmon initialization before acpi
 >  registration, to avoid needing to call to acpi_deregister_notify on sysmon
 >  failure.
 >
 >
 >  To generate a diff of this commit:
 >  cvs rdiff -u -r1.121 -r1.122 src/sys/dev/acpi/acpi_bat.c
 >
 >  Please note that diffs are not public domain; they are subject to the
 >  copyright notices on the relevant files.
 >
 
 The call to sysmon_envsys_register() will trigger an initial refresh and
 the refresh function looks at sc->sc_last which is uninitialized since
 acpibat_update_status() hasn't been called yet.
 
 -- 
 Malte Dehling