FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Ubuntu Kernel Team

 
 
LinkBack Thread Tools
 
Old 02-11-2011, 08:31 PM
 
Default Maverick SRU, NFS: fix the return value of nfs_file_fsync()

The following changes since commit 92520197d26effb0e8ffb94c40ebabfe4c7bcab5:
Bruce Rogers (1):
virtio_net: Add schedule check to napi_enable call

are available in the git repository at:

git://kernel.ubuntu.com/rtg/ubuntu-maverick.git nfs-file-fsync-lp585657

Tim Gardner (1):
NFS: fix the return value of nfs_file_fsync()

fs/nfs/file.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
 
Old 02-14-2011, 09:48 AM
Stefan Bader
 
Default Maverick SRU, NFS: fix the return value of nfs_file_fsync()

On 02/11/2011 10:31 PM, Tim Gardner wrote:
> The following changes since commit 92520197d26effb0e8ffb94c40ebabfe4c7bcab5:
> Bruce Rogers (1):
> virtio_net: Add schedule check to napi_enable call
>
> are available in the git repository at:
>
> git://kernel.ubuntu.com/rtg/ubuntu-maverick.git nfs-file-fsync-lp585657
>
> Tim Gardner (1):
> NFS: fix the return value of nfs_file_fsync()
>
> fs/nfs/file.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> From 5c91fe497d7f593356315909b08126bc8c9357e0 Mon Sep 17 00:00:00 2001
> From: Tim Gardner <tim.gardner@canonical.com>
> Date: Fri, 11 Feb 2011 14:26:05 -0700
> Subject: [PATCH] NFS: fix the return value of nfs_file_fsync()
>
> BugLink: http://bugs.launchpad.net/bugs/585657
>
> Backport of 0702099bd86c33c2dcdbd3963433a61f3f503901
>
> By the commit af7fa16 2010-08-03 NFS: Fix up the fsync code
> close(2) became returning the non-zero value even if it went well.
> nfs_file_fsync() should return 0 when "status" is positive.
>
> Signed-off-by: J. R. Okajima <hooanon05@yahoo.co.jp>
> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> ---
> fs/nfs/file.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/nfs/file.c b/fs/nfs/file.c
> index 642c819..6fd4fba 100644
> --- a/fs/nfs/file.c
> +++ b/fs/nfs/file.c
> @@ -222,7 +222,7 @@ static int nfs_do_fsync(struct nfs_open_context *ctx, struct inode *inode)
> have_error |= test_bit(NFS_CONTEXT_ERROR_WRITE, &ctx->flags);
> if (have_error)
> ret = xchg(&ctx->error, 0);
> - if (!ret)
> + if (!ret && status < 0)
> ret = status;
> return ret;
> }
ACK with the same comments as for the Lucid version (we may need a Hardy patch too).

-Stefan

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 02-14-2011, 07:06 PM
John Johansen
 
Default Maverick SRU, NFS: fix the return value of nfs_file_fsync()

On 02/11/2011 01:31 PM, Tim Gardner wrote:
> The following changes since commit 92520197d26effb0e8ffb94c40ebabfe4c7bcab5:
> Bruce Rogers (1):
> virtio_net: Add schedule check to napi_enable call
>
> are available in the git repository at:
>
> git://kernel.ubuntu.com/rtg/ubuntu-maverick.git nfs-file-fsync-lp585657
>
> Tim Gardner (1):
> NFS: fix the return value of nfs_file_fsync()
>
> fs/nfs/file.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> From 5c91fe497d7f593356315909b08126bc8c9357e0 Mon Sep 17 00:00:00 2001
> From: Tim Gardner <tim.gardner@canonical.com>
> Date: Fri, 11 Feb 2011 14:26:05 -0700
> Subject: [PATCH] NFS: fix the return value of nfs_file_fsync()
>
> BugLink: http://bugs.launchpad.net/bugs/585657
>
> Backport of 0702099bd86c33c2dcdbd3963433a61f3f503901
>
> By the commit af7fa16 2010-08-03 NFS: Fix up the fsync code
> close(2) became returning the non-zero value even if it went well.
> nfs_file_fsync() should return 0 when "status" is positive.
>
> Signed-off-by: J. R. Okajima <hooanon05@yahoo.co.jp>
> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
> ---
> fs/nfs/file.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/fs/nfs/file.c b/fs/nfs/file.c
> index 642c819..6fd4fba 100644
> --- a/fs/nfs/file.c
> +++ b/fs/nfs/file.c
> @@ -222,7 +222,7 @@ static int nfs_do_fsync(struct nfs_open_context *ctx, struct inode *inode)
> have_error |= test_bit(NFS_CONTEXT_ERROR_WRITE, &ctx->flags);
> if (have_error)
> ret = xchg(&ctx->error, 0);
> - if (!ret)
> + if (!ret && status < 0)
> ret = status;
> return ret;
> }

Acked-by: John Johansen <john.johansen@canonical.com>


--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 
Old 02-14-2011, 07:26 PM
Tim Gardner
 
Default Maverick SRU, NFS: fix the return value of nfs_file_fsync()

applied and pushed

--
Tim Gardner tim.gardner@canonical.com

--
kernel-team mailing list
kernel-team@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/kernel-team
 

Thread Tools




All times are GMT. The time now is 07:19 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org