Re: [PATCH 2/8] staging: erofs: separate erofs_get_meta_page

2018-08-13 Thread Chao Yu
On 2018/8/13 19:04, Dan Carpenter wrote: > On Sun, Aug 12, 2018 at 10:01:44PM +0800, Chao Yu wrote: >> --- a/drivers/staging/erofs/data.c >> +++ b/drivers/staging/erofs/data.c >> @@ -39,31 +39,44 @@ static inline void read_endio(struct bio *bio) >> } >> >> /* prio -- true is used for dir */ >>

Re: [PATCH 2/8] staging: erofs: separate erofs_get_meta_page

2018-08-13 Thread Gao Xiang
Hi Dan, On 2018/8/13 19:04, Dan Carpenter wrote: > On Sun, Aug 12, 2018 at 10:01:44PM +0800, Chao Yu wrote: >> --- a/drivers/staging/erofs/data.c >> +++ b/drivers/staging/erofs/data.c >> @@ -39,31 +39,44 @@ static inline void read_endio(struct bio *bio) >> } >> >> /* prio -- true is used for

Re: [PATCH 2/8] staging: erofs: separate erofs_get_meta_page

2018-08-13 Thread Dan Carpenter
On Sun, Aug 12, 2018 at 10:01:44PM +0800, Chao Yu wrote: > --- a/drivers/staging/erofs/data.c > +++ b/drivers/staging/erofs/data.c > @@ -39,31 +39,44 @@ static inline void read_endio(struct bio *bio) > } > > /* prio -- true is used for dir */ > -struct page *erofs_get_meta_page(struct

[PATCH 2/8] staging: erofs: separate erofs_get_meta_page

2018-08-12 Thread Chao Yu
From: Gao Xiang This patch separates 'erofs_get_meta_page' into 'erofs_get_meta_page' and 'erofs_get_meta_page_nofail'. The second one ensures that it should not fail under memory pressure and should make best efforts if IO errors occur. It also adds auxiliary variables in order to fulfill 80