Hi Baolin, On Wed, 15 Feb 2023 18:39:33 +0800 Baolin Wang <baolin.wang@xxxxxxxxxxxxxxxxx> wrote: > Now the page isolation functions did not return a boolean to indicate > success or not, instead it will return a negative error when failed > to isolate a page. So below code used in most places seem a boolean > success/failure thing, which can confuse people whether the isolation > is successful. > > if (folio_isolate_lru(folio)) > continue; > > Moreover the page isolation functions only return 0 or -EBUSY, and > most users did not care about the negative error except for few users, > thus we can convert all page isolation functions to return a boolean > value, which can remove the confusion to make code more clear. > > No functional changes intended in this patch series. For the series, Reviewed-by: SeongJae Park <sj@xxxxxxxxxx> Thanks, SJ [...]
