Re: [PATCH] generic/371: run write(2) and fallocate(2) in parallel

2016-07-25 Thread Wang Xiaoguang
hello, On 07/21/2016 06:41 PM, Eryu Guan wrote: On Thu, Jul 21, 2016 at 03:30:25PM +0800, Wang Xiaoguang wrote: Currently in btrfs, there is something wrong with fallocate(2)'s data space reservation, it'll temporarily occupy more data space thant it really needs, which in turn will impact

Re: [PATCH] generic/371: run write(2) and fallocate(2) in parallel

2016-07-21 Thread Dave Chinner
On Thu, Jul 21, 2016 at 06:41:00PM +0800, Eryu Guan wrote: > On Thu, Jul 21, 2016 at 03:30:25PM +0800, Wang Xiaoguang wrote: > > + > > +run_time=$((180 * $TIME_FACTOR)) > > 180s is too long time, I can reproduce it in around 10s on my test vm, > just loop for 100 times for each operation (pwrite

Re: [PATCH] generic/371: run write(2) and fallocate(2) in parallel

2016-07-21 Thread Eryu Guan
On Thu, Jul 21, 2016 at 03:30:25PM +0800, Wang Xiaoguang wrote: > Currently in btrfs, there is something wrong with fallocate(2)'s data > space reservation, it'll temporarily occupy more data space thant it > really needs, which in turn will impact other operations' data request. > > In this test

[PATCH] generic/371: run write(2) and fallocate(2) in parallel

2016-07-21 Thread Wang Xiaoguang
Currently in btrfs, there is something wrong with fallocate(2)'s data space reservation, it'll temporarily occupy more data space thant it really needs, which in turn will impact other operations' data request. In this test case, it runs write(2) and fallocate(2) in parallel and the total needed