On Thu, Dec 14, 2017 at 03:32:46PM +0800, Eryu Guan wrote:
> On Thu, Dec 14, 2017 at 08:39:38AM +0200, Amir Goldstein wrote:
> > On Wed, Dec 13, 2017 at 8:03 AM, Darrick J. Wong
> > <darrick.wong@xxxxxxxxxx> wrote:
> > > From: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> > >
> > > Mix it up a bit by reflinking and deduping data blocks when possible.
> > >
> > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> > > ---
> > > ltp/fsstress.c | 440 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > > 1 file changed, 440 insertions(+)
> > >
> > >
> > > diff --git a/ltp/fsstress.c b/ltp/fsstress.c
> > > index 96f48b1..e2dfa5e 100644
> > > --- a/ltp/fsstress.c
> > > +++ b/ltp/fsstress.c
> > > @@ -68,7 +68,9 @@ typedef enum {
> > > OP_BULKSTAT,
> > > OP_BULKSTAT1,
> > > OP_CHOWN,
> > > + OP_CLONERANGE,
> > > OP_CREAT,
> > > + OP_DEDUPERANGE,
> > > OP_DREAD,
> > > OP_DWRITE,
> > > OP_FALLOCATE,
> > > @@ -174,7 +176,9 @@ void awrite_f(int, long);
> > > void bulkstat_f(int, long);
> > > void bulkstat1_f(int, long);
> > > void chown_f(int, long);
> > > +void clonerange_f(int, long);
> > > void creat_f(int, long);
> > > +void deduperange_f(int, long);
> > > void dread_f(int, long);
> > > void dwrite_f(int, long);
> > > void fallocate_f(int, long);
> > > @@ -221,7 +225,9 @@ opdesc_t ops[] = {
> > > { OP_BULKSTAT, "bulkstat", bulkstat_f, 1, 0 },
> > > { OP_BULKSTAT1, "bulkstat1", bulkstat1_f, 1, 0 },
> > > { OP_CHOWN, "chown", chown_f, 3, 1 },
> > > + { OP_CLONERANGE, "clonerange", clonerange_f, 4, 1 },
> > > { OP_CREAT, "creat", creat_f, 4, 1 },
> > > + { OP_DEDUPERANGE, "deduperange", deduperange_f, 4, 1},
> > > { OP_DREAD, "dread", dread_f, 4, 0 },
> > > { OP_DWRITE, "dwrite", dwrite_f, 4, 1 },
> > > { OP_FALLOCATE, "fallocate", fallocate_f, 1, 1 },
> > > @@ -1312,6 +1318,16 @@ make_freq_table(void)
> > > }
> > > }
> > >
> > > +void
> > > +free_freq_table(void)
> > > +{
> > > + if (!freq_table)
> > > + return;
> > > + free(freq_table);
> > > + freq_table = NULL;
> > > + freq_table_size = 0;
> > > +}
> > > +
> > > int
> > > mkdir_path(pathname_t *name, mode_t mode)
> > > {
> > > @@ -2189,6 +2205,430 @@ chown_f(int opno, long r)
> > > free_pathname(&f);
> > > }
> > >
> > > +static void
> > > +disable_op(opty_t opt)
> > > +{
> > > + opdesc_t *p;
> > > +
> > > + for (p = ops; p < ops_end; p++) {
> > > + if (opt == p->op) {
> > > + p->freq = 0;
> > > + free_freq_table();
> > > + make_freq_table();
> > > + return;
> > > + }
> > > + }
> > > +}
> > > +
> >
> > If we want to go down the path of runtime disable ops, the question is:
> > Why disable_op clonerange/deduperange and not disable_op
> > insert/collapse/zero/punch? there are probably other ops as well.
> > This is also inconsistent by the fact that build time disable of
> > neither FALLOC nor LONERANGE/DEDUPERANGE will not
> > change the random op sequence, while runtime disable will.
>
> That's a good point. I don't think we want to disable unsupported
> operations dynamically, fsstress just accepts/ignores whatever result
> the operation gives and just prints the errno to log in verbose mode.
Ok, I'll remove all the auto-disabling stuff & resubmit.
--D
> Thanks,
> Eryu
>
> >
> > This is related to the conversation of the golden output of xfs/068.
> > Will take my arguments there now...
> >
> > Amir.
> --
> To unsubscribe from this list: send the line "unsubscribe fstests" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
