On Aug 27, 2009 15:46 +0530, Subrata Modak wrote: > >On Tue, 2009-08-25 at 23:25 -0600, Andreas Dilger wrote: > >I've done some work to merge some of the existing fsx.c mods into a > > single version. Over & above the version that is in the LTP, I've > > included AKPM's O_DIRECT fixes (with a twist), the BSD mmap page and > > segfault handling, and per-write fsync. > > > > The twist for the O_DIRECT feature is that it will randomly open file > > descriptors with O_DIRECT, and if you use the Lustre-inspired multi-fd > > support fsx will be testing concurrent buffered and O_DIRECT and mmap > > IO on the same file. > > The following patch will integrate your new fsx.c program in LTP, by replacing > (http://ltp.cvs.sourceforge.net/viewvc/ltp/ltp/testcases/kernel/fs/fsx-linux/fsx-linux.c) > the existing one. Would you mind providing a Sign-off for the below Patch ? > > Patch-prepared-for-ltp-by: Subrata Modak <subrata@xxxxxxxxxxxxxxxxxx> Signed-off-by: Andreas Dilger <adilger@xxxxxxx> > @@ -1,5 +1,4 @@ > /* > - * Copyright (C) 1991, NeXT Computer, Inc. All Rights Reserverd. > * Copyright (c) 1998-2001 Apple Computer, Inc. All rights reserved. > * > * @APPLE_LICENSE_HEADER_START@ This hunk should be removed from this patch. This copyright was recently re-added into LTP from the very original fsx implementation, from looking at the commit logs. > +static const char *my_basename(const char *path) > +{ > + char *c = strrchr(path, '/'); > + > + return c ? c++ : path; > +} I think you can also remove this function for the LTP version. We need to be able to run fsx on very old distros that our customers still use, where basename() isn't declared properly, but this was recently removed in the LTP version (presumably because it is used only on modern distros). > +int do_fallocate(int fd, int flags, loff_t offset, loff_t maxlen) > +{ > +#ifdef FALLOC_FL_KEEP_SIZE > + return fallocate(fd, flags, offset, maxlen); > +#else > +#define FALLOC_FL_KEEP_SIZE 0x01 > +#ifdef __i386__ > +#define __NR_fallocate 324 > + return syscall(__NR_fallocate, fd, flags, offset, maxlen); > +#elif defined (__powerpc__) > +#define __NR_fallocate 309 > + return syscall(__NR_fallocate, fd, flags, offset >> 32, > + offset & 0xffffffff, maxlen >> 32, maxlen & 0xffffffff); > +#else /* !__i386__ && !__powerpc__ */ > + errno = ENOSYS; > + return -1; > +#endif /* __i386__ */ > +#endif /* FALLOC_FL_KEEP_SIZE */ > +} As an FYI (largely to Eric Sandeen) the fallocate support is only for the initial file data. I was thinking of adding the ability to add OP_FALLOC operations (both keep size and extend size) to give better stress testing of falloc. Sadly, I ran out of time for implementing that. Did you ever do something similar? > +void > +segv(int sig) > +{ > + if (jmpbuf_good) { > + jmpbuf_good = 0; > + longjmp(jmpbuf, 1); Can you please fix the indenting here to use tabs... Cheers, Andreas -- Andreas Dilger Sr. Staff Engineer, Lustre Group Sun Microsystems of Canada, Inc. -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
