On Fri, 20 May 2011 13:00:53 +0400 Pavel Shilovsky <[email protected]> wrote:
> 2011/5/19 Jeff Layton <[email protected]>: > > 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: > > > > ./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. > > > > I figured it out. > > Steve, I attach two diffs: 1) diff for this version of the patch and > 2) new fixed version of the whole patch - you can use either one that > is more comfortable to you. > Confirmed -- the patch seems to fix the issue. With that, all the connectathon tests seem to now pass. -- 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
