From: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
To: Thierry Reding <thierry.reding@gmail.com>,
dri-devel@lists.freedesktop.org
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Subject: Re: [PATCH] drm: Implement drm_modeset_{,un}lock_all_ctx()
Date: Thu, 10 Sep 2015 07:38:25 +0200 [thread overview]
Message-ID: <55F11751.7080003@linux.intel.com> (raw)
In-Reply-To: <1441718807-13206-1-git-send-email-thierry.reding@gmail.com>
Op 08-09-15 om 15:26 schreef Thierry Reding:
> From: Thierry Reding <treding@nvidia.com>
>
> These functions are like drm_modeset_{,un}lock_all(), but they take the
> lock acquisition context as a parameter rather than storing it in the
> DRM device's drm_mode_config structure.
>
> Implement drm_modeset_{,un}lock_all() in terms of the new function for
> better code reuse, and add a note to the kerneldoc that new code should
> use the new functions.
Looks good, maybe in a release or 2 we'll be able to nuke the old functions so devs have time to convert things over. :-)
This also needs an interruptible counterpart, I think the current non-interruptible locking is a bug.
~Maarten
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2015-09-10 5:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-08 13:26 [PATCH] drm: Implement drm_modeset_{,un}lock_all_ctx() Thierry Reding
2015-09-10 5:38 ` Maarten Lankhorst [this message]
2015-09-10 9:51 ` Daniel Vetter
2015-09-10 13:35 ` Maarten Lankhorst
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55F11751.7080003@linux.intel.com \
--to=maarten.lankhorst@linux.intel.com \
--cc=daniel.vetter@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=thierry.reding@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.