UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

drm: reduce GETCLIENT to a minimum 

The *only* known user of GETCLIENT is libva, which uses it to check
whether its own context is authenticated. It used to iterate all clients,
look for one that matches its own pid and then check its state.

The entire purpose for us to still have a GETCLIENT implementation is to
serve libva. So lets not pretend we do anything else: Make this function
return information on the caller's context only, fake the PID to the
caller's pid so they always match, and just fill in the "authenticated"
bit, nothing else.

This patch reduces the complexity of GETCLIENT to a bare minimum, avoids
any dependency on priv->uid or priv->pid (allows us to get rid of them),
and makes libva happy by always *exactly* returning the information it
wants.

Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20160825143505.7447-1-dh.herrmann@gmail.com
This commit is contained in:
David Herrmann 2016-08-25 16:35:05 +02:00 committed by Daniel Vetter
parent d10571fc4f
commit a8c798186b
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
drivers/gpu/drm/drm_ioctl.c
View File

@ -189,9 +189,8 @@ static int drm_getclient(struct drm_device *dev, void *data,
*/ */
if (client->idx == 0) { if (client->idx == 0) {
client->auth = file_priv->authenticated; client->auth = file_priv->authenticated;
client->pid = pid_vnr(file_priv->pid); client->pid = task_pid_vnr(current);
client->uid = from_kuid_munged(current_user_ns(), client->uid = overflowuid;
file_priv->uid);
client->magic = 0; client->magic = 0;
client->iocs = 0; client->iocs = 0;