Page MenuHomeFreeBSD
Differential D30018

powerpc: Rework IPI message processing
ClosedPublic
Actions

Authored by jhibbits on Apr 28 2021, 1:36 AM.
Tags
None
Referenced Files
F164275221: D30018.id.diff
Thu, Jul 30, 7:08 AM
F164236571: D30018.diff
Wed, Jul 29, 11:15 PM
Unknown Object (File)
Mon, Jul 27, 5:18 AM
Unknown Object (File)
Fri, Jul 24, 2:36 AM
Unknown Object (File)
Mon, Jul 20, 11:22 PM
Unknown Object (File)
Fri, Jul 10, 12:16 AM
Unknown Object (File)
Tue, Jul 7, 3:59 PM
Unknown Object (File)
Sun, Jul 5, 1:29 PM
View All Files
Subscribers
imp

Details

Reviewers
bdragon
nwhitehorn
Group Reviewers
PowerPC
Commits
rG7ed09a6778d3: powerpc: Rework IPI message processing
Summary

There's no need to use a while loop in the IPI handler, the message list
is cached once and processed. Instead, since the existing code calls
ffs(), sort the handlers, and use a simple 'if' sequence.

Test Plan

Boot a SMP PowerPC system

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 38897
Build 35786: arc lint + arc unit

Event Timeline

jhibbits created this revision.Apr 28 2021, 1:36 AM
Herald added a subscriber: imp. · View Herald TranscriptApr 28 2021, 1:36 AM
jhibbits requested review of this revision.Apr 28 2021, 1:36 AM
Harbormaster completed remote builds in B38897: Diff 88287.Apr 28 2021, 1:36 AM
nwhitehorn accepted this revision.May 7 2021, 2:08 PM

Looks good to me, assuming it works anyway.

sys/powerpc/powerpc/mp_machdep.c
324

There's no switch anymore here.

This revision is now accepted and ready to land.May 7 2021, 2:08 PM
Closed by commit rG7ed09a6778d3: powerpc: Rework IPI message processing (authored by jhibbits). · Explain WhyMay 17 2021, 1:48 PM
This revision was automatically updated to reflect the committed changes.
jhibbits added a commit: rG7ed09a6778d3: powerpc: Rework IPI message processing.

Revision Contents
Changeset List

PathSize
sys/
powerpc/
powerpc/
68 lines

Diff 88287

View Options

sys/powerpc/powerpc/mp_machdep.c

Loading...