Differential D9187

Ensure arswitch is always locked when calling arswitch_modifyreg()
ClosedPublic
Actions

Authored by kp on Jan 14 2017, 11:44 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mar 19 2024, 4:26 AM

	Unknown Object (File)
	Mar 19 2024, 2:49 AM

	Unknown Object (File)
	Dec 28 2023, 11:32 PM

	Unknown Object (File)
	Dec 20 2023, 4:58 AM

	Unknown Object (File)
	Oct 29 2023, 4:38 AM

	Unknown Object (File)
	Aug 31 2023, 2:26 PM

	Unknown Object (File)
	Jul 10 2023, 10:14 AM

	Unknown Object (File)
	Jul 4 2023, 4:26 PM

View All 26 Files

Subscribers

Details

Reviewers

kan
adrian

Commits

rS312224: arswitch: Ensure the lock is always held when calling arswitch_modifyreg()

Summary

While upgrading my TPLink 3600 to a less ancient FreeBSD version I encountered a panic (panic: mutex arswitch not owned at .../sys/dev/etherswitch/arswitch/arswitch_reg.c:236).
This happened because arswitch_modifyreg() can be called without having the arswitch lock held.

Test Plan

Boot TPLink 3600.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kp updated this revision to Diff 24012.Jan 14 2017, 11:44 PM

kp retitled this revision from to Ensure arswitch is always locked when calling arswitch_modifyreg().

kp updated this object.

kp edited the test plan for this revision. (Show Details)

kp added a reviewer: adrian.

kp set the repository for this revision to rS FreeBSD src repository - subversion.

kp added a subscriber: adrian.

Herald added a subscriber: imp. · View Herald TranscriptJan 14 2017, 11:44 PM

yup! good catch!

This revision is now accepted and ready to land.Jan 14 2017, 11:46 PM

Yup

Closed by commit rS312224: arswitch: Ensure the lock is always held when calling arswitch_modifyreg() (authored by kp). · Explain WhyJan 15 2017, 10:21 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

etherswitch/

arswitch/

8 lines

arswitch_7240.c

4 lines

arswitch_8316.c

3 lines

arswitch_8327.c

3 lines

arswitch_9340.c

4 lines

Diff 24031

head/sys/dev/etherswitch/arswitch/arswitch.c

Loading...

head/sys/dev/etherswitch/arswitch/arswitch_7240.c

Loading...

head/sys/dev/etherswitch/arswitch/arswitch_8316.c

Loading...

head/sys/dev/etherswitch/arswitch/arswitch_8327.c

Loading...

head/sys/dev/etherswitch/arswitch/arswitch_9340.c

Loading...