diff mbox

[16/27] drm/vgem: Use lockless gem BO free callback

Message ID 1464630800-30786-17-git-send-email-daniel.vetter@ffwll.ch (mailing list archive)
State New, archived
Headers show

Commit Message

Daniel Vetter May 30, 2016, 5:53 p.m. UTC 
No dev->struct_mutex anywhere to be seen.

Cc: seanpaul@chromium.org
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
 drivers/gpu/drm/vgem/vgem_drv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sean Paul May 31, 2016, 1:20 p.m. UTC | #1 
On Mon, May 30, 2016 at 1:53 PM, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> No dev->struct_mutex anywhere to be seen.
>
> Cc: seanpaul@chromium.org
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>

Reviewed-by: Sean Paul <seanpaul@chromium.org>

> ---
>  drivers/gpu/drm/vgem/vgem_drv.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c
> index 341f9be3dde6..1b4cc8b27080 100644
> --- a/drivers/gpu/drm/vgem/vgem_drv.c
> +++ b/drivers/gpu/drm/vgem/vgem_drv.c
> @@ -235,7 +235,7 @@ static const struct file_operations vgem_driver_fops = {
>
>  static struct drm_driver vgem_driver = {
>         .driver_features                = DRIVER_GEM,
> -       .gem_free_object                = vgem_gem_free_object,
> +       .gem_free_object_unlocked       = vgem_gem_free_object,
>         .gem_vm_ops                     = &vgem_gem_vm_ops,
>         .ioctls                         = vgem_ioctls,
>         .fops                           = &vgem_driver_fops,
> --
> 2.8.1
>
Daniel Vetter May 31, 2016, 1:29 p.m. UTC | #2 
On Tue, May 31, 2016 at 09:20:07AM -0400, Sean Paul wrote:
> On Mon, May 30, 2016 at 1:53 PM, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> > No dev->struct_mutex anywhere to be seen.
> >
> > Cc: seanpaul@chromium.org
> > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> 
> Reviewed-by: Sean Paul <seanpaul@chromium.org>

Applied to drm-misc, thanks for the review.
-Daniel

> 
> > ---
> >  drivers/gpu/drm/vgem/vgem_drv.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c
> > index 341f9be3dde6..1b4cc8b27080 100644
> > --- a/drivers/gpu/drm/vgem/vgem_drv.c
> > +++ b/drivers/gpu/drm/vgem/vgem_drv.c
> > @@ -235,7 +235,7 @@ static const struct file_operations vgem_driver_fops = {
> >
> >  static struct drm_driver vgem_driver = {
> >         .driver_features                = DRIVER_GEM,
> > -       .gem_free_object                = vgem_gem_free_object,
> > +       .gem_free_object_unlocked       = vgem_gem_free_object,
> >         .gem_vm_ops                     = &vgem_gem_vm_ops,
> >         .ioctls                         = vgem_ioctls,
> >         .fops                           = &vgem_driver_fops,
> > --
> > 2.8.1
> >
diff mbox

Patch

diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c
index 341f9be3dde6..1b4cc8b27080 100644
--- a/drivers/gpu/drm/vgem/vgem_drv.c
+++ b/drivers/gpu/drm/vgem/vgem_drv.c
@@ -235,7 +235,7 @@  static const struct file_operations vgem_driver_fops = {
 
 static struct drm_driver vgem_driver = {
 	.driver_features		= DRIVER_GEM,
-	.gem_free_object		= vgem_gem_free_object,
+	.gem_free_object_unlocked	= vgem_gem_free_object,
 	.gem_vm_ops			= &vgem_gem_vm_ops,
 	.ioctls				= vgem_ioctls,
 	.fops				= &vgem_driver_fops,