Add kevent support to drm
AbandonedPublic
Actions

Authored by johalun0_gmail.com on Aug 13 2016, 10:00 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Aug 15, 1:53 PM

	Unknown Object (File)
	Sat, Aug 15, 8:58 AM

	Unknown Object (File)
	Sat, Aug 15, 8:58 AM

	Unknown Object (File)
	Sat, Aug 15, 8:58 AM

	Unknown Object (File)
	Thu, Aug 13, 10:38 AM

	Unknown Object (File)
	Thu, Aug 13, 10:38 AM

	Unknown Object (File)
	Thu, Aug 13, 10:38 AM

	Unknown Object (File)
	Sat, Aug 8, 3:13 PM

Subscribers

Contributor Reviews (src)

Details

Reviewers: None

Summary

Wayland requires kevent for drm devices. This patch adds that to the 3.8 drm branch (current HEAD).

Question:
Should there be some testing to see if kevent is being used in the wake up function so we don't call mtx_lock() and knote() when we don't need to?

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

johalun0_gmail.com updated this revision to Diff 19258.Aug 13 2016, 10:00 PM

johalun0_gmail.com retitled this revision from to Add kevent support to drm.

johalun0_gmail.com updated this object.

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

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

Herald added subscribers: Contributor Reviews (src), imp. · View Herald TranscriptAug 13 2016, 10:00 PM

wulf_cicgroup.ru added a subscriber: wulf_cicgroup.ru.Aug 19 2016, 10:42 PM

wulf_cicgroup.ru added inline comments.

sys/dev/drm2/drmP.h
422	struct drm_file has struct selinfo event_poll member already. Can it be used for kevents too?
423	Can dev->event_lock be reused to not introduce new mutex? As I see, both notify and kqfilter functions run with this mtx taken