Re: [PATCH] CIFS: Simplify invalidate part (try #5)

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

 



On Thu, 19 May 2011 14:01:32 -0400
Jeff Layton <jlayton@xxxxxxxxxx> wrote:

> On Sat, 23 Apr 2011 17:30:17 +0400
> Pavel Shilovsky <piastry@xxxxxxxxxxx> wrote:
> 
> > Simplify many places when we call cifs_revalidate/invalidate to make
> > it do what it exactly needs.
> > 
> > Signed-off-by: Pavel Shilovsky <piastry@xxxxxxxxxxx>
> > ---
> >  fs/cifs/cifsfs.c |   33 +++++++++++-----
> >  fs/cifs/cifsfs.h |    4 +-
> >  fs/cifs/file.c   |   16 ++++++--
> >  fs/cifs/inode.c  |  117 ++++++++++++++++++++++++++++++++++-------------------
> >  4 files changed, 113 insertions(+), 57 deletions(-)
> > 
> 
> This patch is causing a regression in tests5:
> 

Sorry, that should be "connectathon basic test5"...

> ./test5: read and write
>         ./test5: (/mnt/salusa/rawhide.test) 'bigfile' has size -2130066000, should be 1048576
> basic tests failed
> 
> Pavel, can you track this down?
> 
> Steve, in the meantime I suggest that this patch not be pushed when
> the merge window opens.
>  

-- 
Jeff Layton <jlayton@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux