Messages in this thread | | | From | Qian Cai <> | Subject | Re: [v2 PATCH] mm: move_pages: return valid node id in status if the page is already on the target node | Date | Thu, 5 Dec 2019 13:11:51 -0500 |
| |
> On Dec 5, 2019, at 12:39 PM, Yang Shi <yang.shi@linux.alibaba.com> wrote: > > There are definitely a few inconsistencies, but I think the manpage is quite clear for this specific case, which says "status is an array of integers that return the status of each page. The array contains valid values only if move_pages() did not return an error." And, it looks kernel just misbehaved since 4.17 due to the fixes commit, so it sounds like a regression too.
“Only if” in strictly logical term does not necessarily mean it must contain valid values if move_pages() succeed.
| |