* [PATCH] NFSD: Process time setting separately only for nfsv2
@ 2015-05-27 8:05 Kinglong Mee
2015-05-27 18:36 ` J. Bruce Fields
0 siblings, 1 reply; 3+ messages in thread
From: Kinglong Mee @ 2015-05-27 8:05 UTC (permalink / raw)
To: J. Bruce Fields; +Cc: linux-nfs@vger.kernel.org, kinglongmee
NFSv2 does not different between "set-[ac]time-to-now" which only
requires access, and "set-[ac]time-to-X" which requires ownership.
But, NFSv3/v4 are support the difference.
Just let those codes valid for NFSv2.
Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
---
fs/nfsd/vfs.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
index 84d770b..bf4a15b 100644
--- a/fs/nfsd/vfs.c
+++ b/fs/nfsd/vfs.c
@@ -300,7 +300,7 @@ commit_metadata(struct svc_fh *fhp)
* NFS semantics and what Linux expects.
*/
static void
-nfsd_sanitize_attrs(struct inode *inode, struct iattr *iap)
+nfsd_sanitize_attrs(struct svc_rqst *rqstp, struct inode *inode, struct iattr *iap)
{
/*
* NFSv2 does not differentiate between "set-[ac]time-to-now"
@@ -315,7 +315,8 @@ nfsd_sanitize_attrs(struct inode *inode, struct iattr *iap)
*/
#define BOTH_TIME_SET (ATTR_ATIME_SET | ATTR_MTIME_SET)
#define MAX_TOUCH_TIME_ERROR (30*60)
- if ((iap->ia_valid & BOTH_TIME_SET) == BOTH_TIME_SET &&
+ if (rqstp->rq_vers == 2 &&
+ (iap->ia_valid & BOTH_TIME_SET) == BOTH_TIME_SET &&
iap->ia_mtime.tv_sec == iap->ia_atime.tv_sec) {
/*
* Looks probable.
@@ -435,7 +436,7 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap,
if (!iap->ia_valid)
goto out;
- nfsd_sanitize_attrs(inode, iap);
+ nfsd_sanitize_attrs(rqstp, inode, iap);
/*
* The size case is special, it changes the file in addition to the
--
2.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] NFSD: Process time setting separately only for nfsv2
2015-05-27 8:05 [PATCH] NFSD: Process time setting separately only for nfsv2 Kinglong Mee
@ 2015-05-27 18:36 ` J. Bruce Fields
2015-05-28 0:09 ` Kinglong Mee
0 siblings, 1 reply; 3+ messages in thread
From: J. Bruce Fields @ 2015-05-27 18:36 UTC (permalink / raw)
To: Kinglong Mee; +Cc: linux-nfs@vger.kernel.org
On Wed, May 27, 2015 at 04:05:10PM +0800, Kinglong Mee wrote:
> NFSv2 does not different between "set-[ac]time-to-now" which only
> requires access, and "set-[ac]time-to-X" which requires ownership.
>
> But, NFSv3/v4 are support the difference.
> Just let those codes valid for NFSv2.
Looks like this is already done by Andreas Gruenbacher's "nfsd: Disable
NFSv2 timestamp workaround for NFSv3+" in my for-4.2-incoming branch.
Do you see any problem with that patch?
--b.
>
> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
> ---
> fs/nfsd/vfs.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
> index 84d770b..bf4a15b 100644
> --- a/fs/nfsd/vfs.c
> +++ b/fs/nfsd/vfs.c
> @@ -300,7 +300,7 @@ commit_metadata(struct svc_fh *fhp)
> * NFS semantics and what Linux expects.
> */
> static void
> -nfsd_sanitize_attrs(struct inode *inode, struct iattr *iap)
> +nfsd_sanitize_attrs(struct svc_rqst *rqstp, struct inode *inode, struct iattr *iap)
> {
> /*
> * NFSv2 does not differentiate between "set-[ac]time-to-now"
> @@ -315,7 +315,8 @@ nfsd_sanitize_attrs(struct inode *inode, struct iattr *iap)
> */
> #define BOTH_TIME_SET (ATTR_ATIME_SET | ATTR_MTIME_SET)
> #define MAX_TOUCH_TIME_ERROR (30*60)
> - if ((iap->ia_valid & BOTH_TIME_SET) == BOTH_TIME_SET &&
> + if (rqstp->rq_vers == 2 &&
> + (iap->ia_valid & BOTH_TIME_SET) == BOTH_TIME_SET &&
> iap->ia_mtime.tv_sec == iap->ia_atime.tv_sec) {
> /*
> * Looks probable.
> @@ -435,7 +436,7 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap,
> if (!iap->ia_valid)
> goto out;
>
> - nfsd_sanitize_attrs(inode, iap);
> + nfsd_sanitize_attrs(rqstp, inode, iap);
>
> /*
> * The size case is special, it changes the file in addition to the
> --
> 2.4.1
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] NFSD: Process time setting separately only for nfsv2
2015-05-27 18:36 ` J. Bruce Fields
@ 2015-05-28 0:09 ` Kinglong Mee
0 siblings, 0 replies; 3+ messages in thread
From: Kinglong Mee @ 2015-05-28 0:09 UTC (permalink / raw)
To: J. Bruce Fields; +Cc: linux-nfs@vger.kernel.org, kinglongmee
On 5/28/2015 2:36 AM, J. Bruce Fields wrote:
> On Wed, May 27, 2015 at 04:05:10PM +0800, Kinglong Mee wrote:
>> NFSv2 does not different between "set-[ac]time-to-now" which only
>> requires access, and "set-[ac]time-to-X" which requires ownership.
>>
>> But, NFSv3/v4 are support the difference.
>> Just let those codes valid for NFSv2.
>
> Looks like this is already done by Andreas Gruenbacher's "nfsd: Disable
> NFSv2 timestamp workaround for NFSv3+" in my for-4.2-incoming branch.
Sorry, I find that patch now.
Please ignore this.
> Do you see any problem with that patch?
No,
That patch is great!
thanks,
Kinglong Mee
>
> --b.
>
>>
>> Signed-off-by: Kinglong Mee <kinglongmee@gmail.com>
>> ---
>> fs/nfsd/vfs.c | 7 ++++---
>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>
>> diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c
>> index 84d770b..bf4a15b 100644
>> --- a/fs/nfsd/vfs.c
>> +++ b/fs/nfsd/vfs.c
>> @@ -300,7 +300,7 @@ commit_metadata(struct svc_fh *fhp)
>> * NFS semantics and what Linux expects.
>> */
>> static void
>> -nfsd_sanitize_attrs(struct inode *inode, struct iattr *iap)
>> +nfsd_sanitize_attrs(struct svc_rqst *rqstp, struct inode *inode, struct iattr *iap)
>> {
>> /*
>> * NFSv2 does not differentiate between "set-[ac]time-to-now"
>> @@ -315,7 +315,8 @@ nfsd_sanitize_attrs(struct inode *inode, struct iattr *iap)
>> */
>> #define BOTH_TIME_SET (ATTR_ATIME_SET | ATTR_MTIME_SET)
>> #define MAX_TOUCH_TIME_ERROR (30*60)
>> - if ((iap->ia_valid & BOTH_TIME_SET) == BOTH_TIME_SET &&
>> + if (rqstp->rq_vers == 2 &&
>> + (iap->ia_valid & BOTH_TIME_SET) == BOTH_TIME_SET &&
>> iap->ia_mtime.tv_sec == iap->ia_atime.tv_sec) {
>> /*
>> * Looks probable.
>> @@ -435,7 +436,7 @@ nfsd_setattr(struct svc_rqst *rqstp, struct svc_fh *fhp, struct iattr *iap,
>> if (!iap->ia_valid)
>> goto out;
>>
>> - nfsd_sanitize_attrs(inode, iap);
>> + nfsd_sanitize_attrs(rqstp, inode, iap);
>>
>> /*
>> * The size case is special, it changes the file in addition to the
>> --
>> 2.4.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-05-28 0:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-27 8:05 [PATCH] NFSD: Process time setting separately only for nfsv2 Kinglong Mee
2015-05-27 18:36 ` J. Bruce Fields
2015-05-28 0:09 ` Kinglong Mee
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).