Re: [PATCH] Adds check for numberio during verify phase.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 24 2014, Grant Grundler wrote:
> [dropping jcasse since this account was deleted after his internship ended]
> 
> On Fri, Jan 24, 2014 at 10:58 AM, Jens Axboe <axboe@xxxxxxxxx> wrote:
> > On Fri, Jan 24 2014, Grant Grundler wrote:
> >> Jens, Ping?
> >> You think you can still integrate the three patches from Juan?
> >
> > I think that would be manageable. But really a new feature (or feature
> > modification) like this should be accompanies by a job file example for
> > it. Care to provide one?
> 
> Yes. Do you mind cloning a git repo?

It was big :-)

> Specifically this file is what chromium.org is using:
> https://chromium.googlesource.com/chromiumos/third_party/autotest/+/master/client/site_tests/hardware_StorageFio/8k_async_randwrite
> 
> The git repo is hosted here:
>     https://chromium.googlesource.com/chromiumos/third_party/autotest/
> 
> Use git clone to pull the entire repo:
>     git clone https://chromium.googlesource.com/chromiumos/third_party/autotest
> 
> 
> BTW, Verification is failing on the 1m_stress control file...working
> on that now:
>     https://code.google.com/p/chromium/issues/detail?id=337651
> 
> I suspect it's a problem of the control file though since we are
> getting this warning:
>     "Multiple writers may overwrite blocks that belong to other jobs.
> This can cause verification failures."

Yes, with 8 jobs going, they are going to be stomping on each others
blocks potentially.

I queued up the 3 patches, but I killed the --verify-only command line
switch. Seems unneeded, might as well just use the job option for that.

-- 
Jens Axboe

--
To unsubscribe from this list: send the line "unsubscribe fio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Linux SCSI]     [Linux IDE]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux SCSI]

  Powered by Linux