Re: [PATCH v2] bio: limit bio max size

2021-01-24 Thread Changheun Lee
> On Thu, Jan 21, 2021 at 09:58:03AM +0900, Changheun Lee wrote: > > bio size can grow up to 4GB when muli-page bvec is enabled. > > but sometimes it would lead to inefficient behaviors. > > in case of large chunk direct I/O, - 32MB chunk read in user space - > > all pages for 32MB would be merged

Re: [PATCH v2] bio: limit bio max size.

2021-01-21 Thread Ming Lei
On Thu, Jan 21, 2021 at 09:58:03AM +0900, Changheun Lee wrote: > bio size can grow up to 4GB when muli-page bvec is enabled. > but sometimes it would lead to inefficient behaviors. > in case of large chunk direct I/O, - 32MB chunk read in user space - > all pages for 32MB would be merged to a bio s

Re: [PATCH v2] bio: limit bio max size

2021-01-21 Thread Changheun Lee
> On Thu, 2021-01-21 at 18:36 +0900, Changheun Lee wrote: > > > Please drop the "." at the end of the patch title. > > > > > > > bio size can grow up to 4GB when muli-page bvec is enabled. > > > > but sometimes it would lead to inefficient behaviors. > > > > in case of large chunk direct I/O, - 32

Re: [PATCH v2] bio: limit bio max size

2021-01-21 Thread Damien Le Moal
On Thu, 2021-01-21 at 18:36 +0900, Changheun Lee wrote: > > Please drop the "." at the end of the patch title. > > > > > bio size can grow up to 4GB when muli-page bvec is enabled. > > > but sometimes it would lead to inefficient behaviors. > > > in case of large chunk direct I/O, - 32MB chunk rea

Re: [PATCH v2] bio: limit bio max size

2021-01-21 Thread Changheun Lee
>Please drop the "." at the end of the patch title. > >> bio size can grow up to 4GB when muli-page bvec is enabled. >> but sometimes it would lead to inefficient behaviors. >> in case of large chunk direct I/O, - 32MB chunk read in user space - >> all pages for 32MB would be merged to a bio struct

Re: [PATCH v2] bio: limit bio max size.

2021-01-20 Thread kernel test robot
Hi Changheun, Thank you for the patch! Yet something to improve: [auto build test ERROR on block/for-next] [also build test ERROR on linux/master linus/master v5.11-rc4 next-20210120] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to

Re: [PATCH v2] bio: limit bio max size.

2021-01-20 Thread Damien Le Moal
On 2021/01/21 10:13, Changheun Lee wrote: Please drop the "." at the end of the patch title. > bio size can grow up to 4GB when muli-page bvec is enabled. > but sometimes it would lead to inefficient behaviors. > in case of large chunk direct I/O, - 32MB chunk read in user space - > all pages for

[PATCH v2] bio: limit bio max size.

2021-01-20 Thread Changheun Lee
bio size can grow up to 4GB when muli-page bvec is enabled. but sometimes it would lead to inefficient behaviors. in case of large chunk direct I/O, - 32MB chunk read in user space - all pages for 32MB would be merged to a bio structure if memory address is continued phsycally. it makes some delay