Page MenuHomeFreeBSD

Add kevent support to drm
AbandonedPublic

Authored by johalun0_gmail.com on Aug 13 2016, 10:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 2:30 AM
Unknown Object (File)
Sun, Nov 3, 9:21 PM
Unknown Object (File)
Sun, Nov 3, 9:07 PM
Unknown Object (File)
Sun, Oct 27, 3:58 AM
Unknown Object (File)
Sun, Oct 27, 3:51 AM
Unknown Object (File)
Oct 1 2024, 7:55 PM
Unknown Object (File)
Sep 29 2024, 7:47 PM
Unknown Object (File)
Sep 28 2024, 11:30 PM

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 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.
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

I agree on both suggestions. Will test and see if works as expected.