Differential D50029

e6000sw: unlock the driver lock in the error path during attach
ClosedPublic
Actions

Authored by adrian on Apr 25 2025, 6:39 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Jul 6, 7:11 AM

	Unknown Object (File)
	Sat, Jul 5, 11:42 AM

	Unknown Object (File)
	Fri, Jul 4, 9:28 PM

	Unknown Object (File)
	Fri, Jul 4, 3:51 PM

	Unknown Object (File)
	Thu, Jul 3, 9:45 AM

	Unknown Object (File)
	Sat, Jun 28, 1:07 AM

	Unknown Object (File)
	Fri, Jun 27, 10:15 AM

	Unknown Object (File)
	Wed, Jun 18, 5:22 PM

View All 17 Files

Subscribers

Details

Reviewers

Commits

rGaf298663e9ff: e6000sw: unlock the driver lock in the error path during attach

Summary

The driver sleep lock was being held during most of the error paths,
and not unlocking it will panic the kernel during detach. So, fix it.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

adrian created this revision.Apr 25 2025, 6:39 PM

Herald added a subscriber: imp. · View Herald TranscriptApr 25 2025, 6:39 PM

adrian requested review of this revision.Apr 25 2025, 6:39 PM

Harbormaster completed remote builds in B63734: Diff 154298.Apr 25 2025, 6:39 PM

adrian added a parent revision: D50028: e6000sw: always compile it.Apr 25 2025, 6:47 PM

adrian added a child revision: D50030: e6000sw: stop / drain the taskqueue (and tick) during detach.

imp accepted this revision.Apr 25 2025, 7:37 PM

This revision is now accepted and ready to land.Apr 25 2025, 7:37 PM

Closed by commit rGaf298663e9ff: e6000sw: unlock the driver lock in the error path during attach (authored by adrian). · Explain WhyApr 27 2025, 6:39 PM

This revision was automatically updated to reflect the committed changes.

adrian added a commit: rGaf298663e9ff: e6000sw: unlock the driver lock in the error path during attach.

Revision Contents
Changeset List

Path

Size

sys/

dev/

etherswitch/

e6000sw/

1 line

Diff 154348

sys/dev/etherswitch/e6000sw/e6000sw.c

Loading...