On 25/07/2019 20.42, Pengfei Li wrote:
> Because "order" will never be negative in __rmqueue_fallback(),
> so just make "order" unsigned int.
> And modify trace_mm_page_alloc_extfrag() accordingly.
>
> diff --git a/mm/page_alloc.c b/mm/page_alloc.c
> index 75c18f4fd66a..1432cbcd87cd 100644
> --- a/mm/page_alloc.c
> +++ b/mm/page_alloc.c
> @@ -2631,8 +2631,8 @@ static bool unreserve_highatomic_pageblock(const struct
> alloc_context *ac,
> * condition simpler.
> */
> static __always_inline bool
> -__rmqueue_fallback(struct zone *zone, int order, int start_migratetype,
> - unsigned int alloc_flags)
> +__rmqueue_fallback(struct zone *zone, unsigned int order,
> + int start_migratetype, unsigned int alloc_flags)
> {
Please read the last paragraph of the comment above this function, run
git blame to figure out when that was introduced, and then read the full
commit description. Here be dragons. At the very least, this patch is
wrong in that it makes that comment inaccurate.
Rasmus
