Re: [Debug ] fs : export debugfs_file_operations symbol

[Greg KH]

On Sat, Aug 04, 2012 at 03:30:44AM +0530, Akhilesh Kumar wrote:
> Hi Hartman
> 
> This patch just exports debugfs_file_operations which improve debugfs
> interface .
> Please review below patch for main line and share your review comments.
> 
> Thanks,
> Akhilesh
> 
> 
> 
> 
> From fd68900cd32f220beee70b55c20b3c74a38d7df8 Mon Sep 17 00:00:00 2001
> From: Akhilesh Kumar <akhilesh.lxr@xxxxxxxxx>
> Date: Tue, 31 Jul 2012 17:17:12 +0530
> Subject:[Debug ] fs : export debugfs_file_operations symbol
>  
>  debugfs_file_operations is usefull for file system
>  debugging debugfs_file_operations is not an exported
>  symbol.
> 
> Any kernel module (http://lwn.net/Articles/371208/) using
> 
> debugfs_file_operations will result in build failures
> because of this.
> 
> This patch just exports debugfs_file_operations to fix such problems in
> future
> 
> Signed-off-by: Akhilesh Kumar <akhilesh.lxr@xxxxxxxxx>
> ---
>  fs/debugfs/file.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/fs/debugfs/file.c b/fs/debugfs/file.c
> index 2340f69..a5150d8 100644
> --- a/fs/debugfs/file.c
> +++ b/fs/debugfs/file.c
> @@ -40,6 +40,7 @@ const struct file_operations debugfs_file_operations = {
>   .open = simple_open,
>   .llseek = noop_llseek,
>  };
> +EXPORT_SYMBOL(debugfs_file_operations);

I fail to understand how this will help you out at all.  You should not
be using this symbol, why would you?

Please show the in-kernel code that needs it, and I will be glad to
reconsider this patch, after it it fixed to use EXPORT_SYMBOL_GPL() as
well, to be consistant with the other debugfs symbols.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/