Re: [V9fs-developer] [PATCH] [fs/9p] Remove the redundant rsize calculation in v9fs_file_write()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



If we are eliminating the check, we should just eliminate the
calculation higher up (and the rsize variable) from this function.
Please resubmit with more extensive cleanup.  Thanks.

         -eric


On Tue, Aug 17, 2010 at 12:20 PM, Venkateswararao Jujjuri (JV)
<jvrao@xxxxxxxxxxxxxxxxxx> wrote:
> The same calculation is done in p9_client_write().
>
> Signed-off-by: Venkateswararao Jujjuri <jvrao@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Badari Pulavarty <pbadari@xxxxxxxxxx>
> ---
>  fs/9p/vfs_file.c |    5 +----
>  1 files changed, 1 insertions(+), 4 deletions(-)
>
> diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
> index e97c92b..2695491 100644
> --- a/fs/9p/vfs_file.c
> +++ b/fs/9p/vfs_file.c
> @@ -235,11 +235,8 @@ v9fs_file_write(struct file *filp, const char __user * data,
>        rsize = fid->iounit ? fid->iounit : clnt->msize - P9_IOHDRSZ;
>
>        do {
> -               if (count < rsize)
> -                       rsize = count;
> -
>                n = p9_client_write(fid, NULL, data+total, origin+total,
> -                                                                       rsize);
> +                                                                       count);
>                if (n <= 0)
>                        break;
>                count -= n;
> --
> 1.6.5.2
>
>
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by
>
> Make an app they can't live without
> Enter the BlackBerry Developer Challenge
> http://p.sf.net/sfu/RIM-dev2dev
> _______________________________________________
> V9fs-developer mailing list
> V9fs-developer@xxxxxxxxxxxxxxxxxxxxx
> https://lists.sourceforge.net/lists/listinfo/v9fs-developer
>
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux