Page MenuHomeFreeBSD
Differential D54771

pmu: avoid NULL ofw_get_bus_type(); handle pmu_send() calls w/ NULL recv buffers
Needs ReviewPublic
Actions

Authored by adrian on Jan 19 2026, 1:20 AM.
Tags
Referenced Files
Unknown Object (File)
Mon, Oct 19, 11:59 AM
Unknown Object (File)
Thu, Oct 15, 11:15 PM
Unknown Object (File)
Sun, Oct 11, 11:18 PM
Unknown Object (File)
Sun, Oct 11, 1:30 PM
Unknown Object (File)
Wed, Oct 7, 11:23 PM
Unknown Object (File)
Fri, Oct 2, 9:27 PM
Unknown Object (File)
Mon, Sep 28, 2:06 PM
Unknown Object (File)
Fri, Sep 25, 5:02 PM
View All 59 Files
Subscribers
alfredo
bdragon
imp
jhibbits
jrtc27
loos
luporl

Details

Reviewers
None
Group Reviewers
PowerPC
Summary

When booting in qemu with "-machine=mac99,via=pmu" I found two
things:

  • the "extint-gpio1" check returned NULL, so doing a strcmp() on it is wrong, and
  • a couple of places (notably the LED path) was calling pmu_send() with no receive buffer, leading to results being written to a NULL buffer.

Fix both of these, allowing boot to continue.

Locally tested:

  • qemu-system-ppc64 -machine=mac99,via=pmu -cpu 970fx -smp 1

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 69977
Build 66860: arc lint + arc unit

Event Timeline

adrian created this revision.Jan 19 2026, 1:20 AM
Herald added subscribers: jhibbits, loos, imp. · View Herald TranscriptJan 19 2026, 1:20 AM
adrian requested review of this revision.Jan 19 2026, 1:20 AM
Harbormaster completed remote builds in B69977: Diff 170027.Jan 19 2026, 1:20 AM
adrian added a reviewer: PowerPC.Jan 19 2026, 1:21 AM
adrian added a project: PowerPC.
Herald added subscribers: alfredo, luporl, bdragon. · View Herald TranscriptJan 19 2026, 1:21 AM
jhibbits added a comment.Jan 19 2026, 3:41 AM

The type NULL checks are good. I think the whole of the changes for pmu_send() are a bit overkill. Could you test with just the one line I pointed out, and the rest reverted?

sys/powerpc/powermac/pmu.c
634

I *think* this is the only change needed in this function. The expectation is that the callers (all in this file) know already that a response is needed or not. PMU_SET_SLEEPLED, 0xee, has a pmu_receive_cmd_type entry of -1,

jrtc27 added a subscriber: jrtc27.Jan 19 2026, 3:47 AM

the "extint-gpio1" check returned NULL

Passed NULL because ofw_bus_get_type returned NULL?

sys/powerpc/powermac/pmu.c
634

else after a goto is a bit confusing... may be worth making that less weird as a separate change? Then I'd probably swap this with the out_len read, groups the out_msg writes together rather than interleaving things.

641

Unnecessary change

Revision Contents
Changeset List

PathSize
sys/
powerpc/
powermac/
15 lines

Diff 170027

View Options

sys/powerpc/powermac/pmu.c

Loading...