|
@@ -1351,24 +1351,27 @@ static int msm_ioctl_register_event(struct drm_device *dev, void *data,
|
|
|
* calls add to client list and return.
|
|
|
*/
|
|
|
count = msm_event_client_count(dev, req_event, false);
|
|
|
- /* Add current client to list */
|
|
|
- spin_lock_irqsave(&dev->event_lock, flag);
|
|
|
- list_add_tail(&client->base.link, &priv->client_event_list);
|
|
|
- spin_unlock_irqrestore(&dev->event_lock, flag);
|
|
|
-
|
|
|
- if (count)
|
|
|
+ if (count) {
|
|
|
+ /* Add current client to list */
|
|
|
+ spin_lock_irqsave(&dev->event_lock, flag);
|
|
|
+ list_add_tail(&client->base.link, &priv->client_event_list);
|
|
|
+ spin_unlock_irqrestore(&dev->event_lock, flag);
|
|
|
return 0;
|
|
|
+ }
|
|
|
|
|
|
ret = msm_register_event(dev, req_event, file, true);
|
|
|
if (ret) {
|
|
|
DRM_ERROR("failed to enable event %x object %x object id %d\n",
|
|
|
req_event->event, req_event->object_type,
|
|
|
req_event->object_id);
|
|
|
+ kfree(client);
|
|
|
+ } else {
|
|
|
+ /* Add current client to list */
|
|
|
spin_lock_irqsave(&dev->event_lock, flag);
|
|
|
- list_del(&client->base.link);
|
|
|
+ list_add_tail(&client->base.link, &priv->client_event_list);
|
|
|
spin_unlock_irqrestore(&dev->event_lock, flag);
|
|
|
- kfree(client);
|
|
|
}
|
|
|
+
|
|
|
return ret;
|
|
|
}
|
|
|
|