Re: ubifs: fix page_count in ->ubifs_migrate_page()

2018-12-14 Thread Gao Xiang
Hi, On 2018/12/14 14:15, zhangjun wrote: > On 2018/12/14 上午6:57, Dave Chinner wrote: >> On Thu, Dec 13, 2018 at 03:23:37PM +0100, Richard Weinberger wrote: >>> Hello zhangjun, >>> >>> thanks a lot for bringing this up! >>> >>> Am Mittwoch, 12. Dezember 2018, 15:13:57 CET schrieb zhangjun: Bec

Re: ubifs: fix page_count in ->ubifs_migrate_page()

2018-12-13 Thread zhangjun
On 2018/12/14 上午6:57, Dave Chinner wrote: On Thu, Dec 13, 2018 at 03:23:37PM +0100, Richard Weinberger wrote: Hello zhangjun, thanks a lot for bringing this up! Am Mittwoch, 12. Dezember 2018, 15:13:57 CET schrieb zhangjun: Because the PagePrivate() in UBIFS is different meanings, alloc_cma()

Re: ubifs: fix page_count in ->ubifs_migrate_page()

2018-12-13 Thread Dave Chinner
On Thu, Dec 13, 2018 at 03:23:37PM +0100, Richard Weinberger wrote: > Hello zhangjun, > > thanks a lot for bringing this up! > > Am Mittwoch, 12. Dezember 2018, 15:13:57 CET schrieb zhangjun: > > Because the PagePrivate() in UBIFS is different meanings, > > alloc_cma() will fail when one dirty pa

Re: ubifs: fix page_count in ->ubifs_migrate_page()

2018-12-13 Thread zhangjun
On 2018/12/13 下午10:23, Richard Weinberger wrote: Hello zhangjun, thanks a lot for bringing this up! Am Mittwoch, 12. Dezember 2018, 15:13:57 CET schrieb zhangjun: Because the PagePrivate() in UBIFS is different meanings, alloc_cma() will fail when one dirty page cache located in the type of

Re: ubifs: fix page_count in ->ubifs_migrate_page()

2018-12-13 Thread Richard Weinberger
Hello zhangjun, thanks a lot for bringing this up! Am Mittwoch, 12. Dezember 2018, 15:13:57 CET schrieb zhangjun: > Because the PagePrivate() in UBIFS is different meanings, > alloc_cma() will fail when one dirty page cache located in > the type of MIGRATE_CMA > > If not adjust the 'extra_count'