[LU-1402] cfs_alloc_flags_to_gfp should translate CFS_ALLOC_HIGH to __GFP_HIGHMEM instead of __GFP_HIGH

• Type: Bug
• Resolution: Fixed
• Priority: Major
• Fix Version/s: Lustre 2.3.0
• Severity: 3
• Bugzilla ID: 22,135
• Assignee: Di Wang (Inactive)
• Reporter: Isaac Huang (Inactive)

Description

According to the comment above CFS_ALLOC_HIGH, the flag allows allocator to return memory in the highmen zone:
/* allow to return page beyond KVM. It has to be mapped into KVM by

• cfs_page_map(); */

But cfs_alloc_flags_to_gfp() converts it to __GFP_HIGH, which according to kernel header:
#define __GFP_HIGH 0x20 /* Should access emergency pools? */

I think __GFP_HIGHMEM should be used instead. Also it makes sense to rename CFS_ALLOC_HIGH to CFS_ALLOC_HIGHMEM to avoid further confusion.

Issue Links

• is related to: LU-1513 Test failure on test lustre-initialization-1, sanity, others: "kernel BUG at mm/slab.c:2833"