Page MenuHomeFreeBSD
Differential D13839

Fix minor locking issues in the PowerMac Uninorth PCI driver
ClosedPublic
Actions

Authored by landonf on Jan 10 2018, 9:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 4, 11:41 AM
Unknown Object (File)
Oct 1 2024, 1:44 PM
Unknown Object (File)
Sep 28 2024, 12:39 AM
Unknown Object (File)
Sep 17 2024, 8:09 PM
Unknown Object (File)
Sep 16 2024, 2:22 AM
Unknown Object (File)
Sep 14 2024, 10:14 PM
Unknown Object (File)
Sep 4 2024, 8:55 AM
Unknown Object (File)
Aug 30 2024, 11:17 AM
View All 51 Files
Subscribers
imp

Details

Reviewers
jhibbits
Commits
rS327798: Fix minor locking issues in the Power Mac Uninorth PCI bridge driver.
Summary
  • Call resource_int_value() once during attach, rather than within the pci_(read|write)_config() code path; this avoids taking a blocking mutex to read kenv variables.
  • Use a spin lock to protect non-atomic config space accesses (this matches the behavior of the Darwin AppleMacRiscPCI driver).
Test Plan

Tested for regressions on a PowerBook5,8.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 14272
Build 14432: arc lint + arc unit

Event Timeline

landonf created this revision.Jan 10 2018, 9:26 PM
Harbormaster completed remote builds in B14269: Diff 37757.Jan 10 2018, 9:26 PM
landonf updated this revision to Diff 37760.Jan 10 2018, 9:37 PM
  • Add mtx_assert(MA_OWNED) to uninorth_enable_config() for good measure.
Harbormaster completed remote builds in B14272: Diff 37760.Jan 10 2018, 9:37 PM
jhibbits accepted this revision.Jan 10 2018, 9:41 PM
jhibbits added inline comments.
sys/powerpc/powermac/uninorthvar.h
43

Match whitespace? Might be phabricator messing with it, but just double check.

This revision is now accepted and ready to land.Jan 10 2018, 9:41 PM
landonf marked an inline comment as done.Jan 10 2018, 9:42 PM
landonf added inline comments.
sys/powerpc/powermac/uninorthvar.h
43

Yeah, looks like phabricator behaving badly.

Closed by commit rS327798: Fix minor locking issues in the Power Mac Uninorth PCI bridge driver. (authored by landonf). · Explain WhyJan 10 2018, 10:19 PM
This revision was automatically updated to reflect the committed changes.
landonf marked an inline comment as done.
Herald added a subscriber: imp. · View Herald TranscriptJan 10 2018, 10:19 PM

Revision Contents
Changeset List

PathSize
sys/
powerpc/
powermac/
35 lines
2 lines

Diff 37760

View Options

sys/powerpc/powermac/uninorthpci.c

Loading...
View Options

sys/powerpc/powermac/uninorthvar.h

Loading...