vt(4): Fix a LOR in vt_allocate()
ClosedPublic
Actions

Authored by dumbbell on Sep 15 2014, 9:38 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Mar 28, 4:31 PM

	Unknown Object (File)
	Thu, Mar 28, 3:31 PM

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

	Unknown Object (File)
	Dec 19 2023, 10:11 PM

	Unknown Object (File)
	Dec 19 2023, 10:11 PM

	Unknown Object (File)
	Nov 30 2023, 2:51 PM

	Unknown Object (File)
	Nov 30 2023, 1:14 PM

	Unknown Object (File)
	Aug 28 2023, 6:50 AM

View All 58 Files

Subscribers

None

Details

Reviewers

ray
nwhitehorn
emaste

Summary

When we load a KMS driver, the following LOR is reported:

lock order reversal:
 1st 0xffffffff8084bab8 vtdev (vtdev) @ /usr/home/kostik/work/build/bsd/DEV/src/sys/dev/vt/vt_core.c:2322
 2nd 0xfffff8000d69ac08 ttymtx (ttymtx) @ /usr/home/kostik/work/build/bsd/DEV/src/sys/kern/subr_terminal.c:236

See Bugzilla: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=193597

This patch moves VT_LOCK(vd) after the call to terminal_mute(). The call to vt_suspend_flush_timer() is now outside the lock, but this function already uses an atomic cmpset to protect itself.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

dumbbell updated this revision to Diff 1630.Sep 15 2014, 9:38 AM

dumbbell retitled this revision from to vt(4): Fix a LOR in vt_allocate().

dumbbell updated this object.

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

dumbbell added reviewers: emaste, ray, nwhitehorn.

Looks good.

This revision is now accepted and ready to land.Sep 15 2014, 1:38 PM

Committed in r271682. Thanks!

Revision Contents
Changeset List

Path

Size

sys/

dev/

vt/

vt_core.c

2 lines

Diff 1630

View Options

vt(4): Fix a LOR in vt_allocate()ClosedPublicActions