On Fri, Jan 11, 2019 at 11:31:17AM +0800, Zorro Lang wrote:
> On Thu, Jan 10, 2019 at 10:01:13PM +0800, Eryu Guan wrote:
> > On Thu, Jan 10, 2019 at 04:25:15PM +0800, Zorro Lang wrote:
> > > Support the splice syscall in fsstress.
> > >
> > > Signed-off-by: Zorro Lang <zlang@xxxxxxxxxx>
> > > ---
> > >
<snap>
> > > + if (ret1 < 0 || ret2 < 0)
> > > + e = errno;
> > > + else
> > > + e = 0;
> > > + if (v1 || v2) {
> > > + printf("%d/%d: splice %s%s [%lld,%lld] -> %s%s [%lld,%lld]",
> >
> > Better to print out error number on success (0) too, to be consistent
> > with other operations.
>
> Hmm... it's not consistent for all, likes clonerange_f, copyrange_f and
> deduperange_f don't print out error number on success (0), do you need I change
> them to keep consistent too, as you're asking for that now?
That'd be great (but we can do that in a separate patch)! I think it's
easier to parse fsstress log when tracing down bugs if the log follows
similar formats.
Thanks,
Eryu
