linux-kernel - Re: [PATCH] [7/11] Convert DRM to unlocked

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.0.82.0805210735590.25986@skynet.skynet.ie>
Date:	Wed, 21 May 2008 07:36:31 +0100 (IST)
From:	Dave Airlie <airlied@...ux.ie>
To:	Andi Kleen <andi@...stfloor.org>
cc:	corbet@....net, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] [7/11] Convert DRM to unlocked_fasync


> Doesn't need BKL because it just uses fasync_helper and minor->dev is 
> protected by the file reference count.
> 
> Cc: airlied@...ux.ie
> 
> Signed-off-by: Andi Kleen <ak@...ux.intel.com>

Acked-by: Dave Airlie <airlied@...ux.ie>

unless you want me to take it via my tree.

Dave.

> 
> ---
>  drivers/char/drm/i810_dma.c   |    2 +-
>  drivers/char/drm/i810_drv.c   |    2 +-
>  drivers/char/drm/i830_dma.c   |    2 +-
>  drivers/char/drm/i830_drv.c   |    2 +-
>  drivers/char/drm/i915_drv.c   |    2 +-
>  drivers/char/drm/mga_drv.c    |    2 +-
>  drivers/char/drm/r128_drv.c   |    2 +-
>  drivers/char/drm/radeon_drv.c |    2 +-
>  drivers/char/drm/savage_drv.c |    2 +-
>  drivers/char/drm/sis_drv.c    |    2 +-
>  drivers/char/drm/tdfx_drv.c   |    2 +-
>  drivers/char/drm/via_drv.c    |    2 +-
>  12 files changed, 12 insertions(+), 12 deletions(-)
> 
> Index: linux/drivers/char/drm/i810_dma.c
> ===================================================================
> --- linux.orig/drivers/char/drm/i810_dma.c
> +++ linux/drivers/char/drm/i810_dma.c
> @@ -117,7 +117,7 @@ static const struct file_operations i810
>  	.release = drm_release,
>  	.ioctl = drm_ioctl,
>  	.mmap = i810_mmap_buffers,
> -	.fasync = drm_fasync,
> +	.unlocked_fasync = drm_fasync,
>  };
>  
>  static int i810_map_buffer(struct drm_buf * buf, struct drm_file *file_priv)
> Index: linux/drivers/char/drm/i810_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/i810_drv.c
> +++ linux/drivers/char/drm/i810_drv.c
> @@ -62,7 +62,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  	},
>  
>  	.pci_driver = {
> Index: linux/drivers/char/drm/i830_dma.c
> ===================================================================
> --- linux.orig/drivers/char/drm/i830_dma.c
> +++ linux/drivers/char/drm/i830_dma.c
> @@ -119,7 +119,7 @@ static const struct file_operations i830
>  	.release = drm_release,
>  	.ioctl = drm_ioctl,
>  	.mmap = i830_mmap_buffers,
> -	.fasync = drm_fasync,
> +	.unlocked_fasync = drm_fasync,
>  };
>  
>  static int i830_map_buffer(struct drm_buf * buf, struct drm_file *file_priv)
> Index: linux/drivers/char/drm/i830_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/i830_drv.c
> +++ linux/drivers/char/drm/i830_drv.c
> @@ -73,7 +73,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  	},
>  
>  	.pci_driver = {
> Index: linux/drivers/char/drm/i915_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/i915_drv.c
> +++ linux/drivers/char/drm/i915_drv.c
> @@ -559,7 +559,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  #ifdef CONFIG_COMPAT
>  		 .compat_ioctl = i915_compat_ioctl,
>  #endif
> Index: linux/drivers/char/drm/mga_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/mga_drv.c
> +++ linux/drivers/char/drm/mga_drv.c
> @@ -71,7 +71,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  #ifdef CONFIG_COMPAT
>  		 .compat_ioctl = mga_compat_ioctl,
>  #endif
> Index: linux/drivers/char/drm/r128_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/r128_drv.c
> +++ linux/drivers/char/drm/r128_drv.c
> @@ -66,7 +66,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  #ifdef CONFIG_COMPAT
>  		 .compat_ioctl = r128_compat_ioctl,
>  #endif
> Index: linux/drivers/char/drm/radeon_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/radeon_drv.c
> +++ linux/drivers/char/drm/radeon_drv.c
> @@ -88,7 +88,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  #ifdef CONFIG_COMPAT
>  		 .compat_ioctl = radeon_compat_ioctl,
>  #endif
> Index: linux/drivers/char/drm/savage_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/savage_drv.c
> +++ linux/drivers/char/drm/savage_drv.c
> @@ -53,7 +53,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  	},
>  
>  	.pci_driver = {
> Index: linux/drivers/char/drm/sis_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/sis_drv.c
> +++ linux/drivers/char/drm/sis_drv.c
> @@ -83,7 +83,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  	},
>  	.pci_driver = {
>  		 .name = DRIVER_NAME,
> Index: linux/drivers/char/drm/tdfx_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/tdfx_drv.c
> +++ linux/drivers/char/drm/tdfx_drv.c
> @@ -51,7 +51,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  	},
>  	.pci_driver = {
>  		 .name = DRIVER_NAME,
> Index: linux/drivers/char/drm/via_drv.c
> ===================================================================
> --- linux.orig/drivers/char/drm/via_drv.c
> +++ linux/drivers/char/drm/via_drv.c
> @@ -67,7 +67,7 @@ static struct drm_driver driver = {
>  		 .ioctl = drm_ioctl,
>  		 .mmap = drm_mmap,
>  		 .poll = drm_poll,
> -		 .fasync = drm_fasync,
> +		 .unlocked_fasync = drm_fasync,
>  	},
>  	.pci_driver = {
>  		 .name = DRIVER_NAME,
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/