lkml.org 
[lkml]   [2014]   [Jan]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/3] fs: factor out common code in fget_light() and fget_raw_light()
On Mon, Jan 13, 2014 at 04:48:40PM +0100, Oleg Nesterov wrote:
> Apart from FMODE_PATH check fget_light() and fget_raw_light() are
> identical, shift the code into the new helper, __fget_light(fd, mask).
> Saves 208 bytes.
>
> Signed-off-by: Oleg Nesterov <oleg@redhat.com>

Reviewed-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>

> ---
> fs/file.c | 33 +++++++++------------------------
> 1 files changed, 9 insertions(+), 24 deletions(-)
>
> diff --git a/fs/file.c b/fs/file.c
> index 4ed58a3..50c1208 100644
> --- a/fs/file.c
> +++ b/fs/file.c
> @@ -683,21 +683,21 @@ EXPORT_SYMBOL(fget_raw);
> * The fput_needed flag returned by fget_light should be passed to the
> * corresponding fput_light.
> */
> -struct file *fget_light(unsigned int fd, int *fput_needed)
> +struct file *__fget_light(unsigned int fd, fmode_t mask, int *fput_needed)
> {
> - struct file *file;
> struct files_struct *files = current->files;
> + struct file *file;
>
> *fput_needed = 0;
> if (atomic_read(&files->count) == 1) {
> file = __fcheck_files(files, fd);
> - if (file && (file->f_mode & FMODE_PATH))
> + if (file && (file->f_mode & mask))
> file = NULL;
> } else {
> rcu_read_lock();
> file = fcheck_files(files, fd);
> if (file) {
> - if (!(file->f_mode & FMODE_PATH) &&
> + if (!(file->f_mode & mask) &&
> atomic_long_inc_not_zero(&file->f_count))
> *fput_needed = 1;
> else
> @@ -709,30 +709,15 @@ struct file *fget_light(unsigned int fd, int *fput_needed)
>
> return file;
> }
> +struct file *fget_light(unsigned int fd, int *fput_needed)
> +{
> + return __fget_light(fd, FMODE_PATH, fput_needed);
> +}
> EXPORT_SYMBOL(fget_light);
>
> struct file *fget_raw_light(unsigned int fd, int *fput_needed)
> {
> - struct file *file;
> - struct files_struct *files = current->files;
> -
> - *fput_needed = 0;
> - if (atomic_read(&files->count) == 1) {
> - file = __fcheck_files(files, fd);
> - } else {
> - rcu_read_lock();
> - file = fcheck_files(files, fd);
> - if (file) {
> - if (atomic_long_inc_not_zero(&file->f_count))
> - *fput_needed = 1;
> - else
> - /* Didn't get the reference, someone's freed */
> - file = NULL;
> - }
> - rcu_read_unlock();
> - }
> -
> - return file;
> + return __fget_light(fd, 0, fput_needed);
> }
>
> void set_close_on_exec(unsigned int fd, int flag)
> --
> 1.5.5.1
>
>



\
 
 \ /
  Last update: 2014-01-14 01:01    [W:0.121 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site