On Thu, 19 May 2011 14:01:32 -0400
Jeff Layton <[email protected]> wrote:

> On Sat, 23 Apr 2011 17:30:17 +0400
> Pavel Shilovsky <[email protected]> wrote:
> 
> > Simplify many places when we call cifs_revalidate/invalidate to make
> > it do what it exactly needs.
> > 
> > Signed-off-by: Pavel Shilovsky <[email protected]>
> > ---
> >  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 <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to