Re: [PATCH] mm: convert scan_control.priority int => byte

2018-05-29 Thread Greg Thelen
Matthew Wilcox wrote: > On Mon, May 28, 2018 at 07:40:25PM -0700, Greg Thelen wrote: >> Reclaim priorities range from 0..12(DEF_PRIORITY). >> scan_control.priority is a 4 byte int, which is overkill. >> >> Since commit 6538b8ea886e ("x86_64: expand kernel stack to 16K") x86_64 >> stack overflows

Re: [PATCH] mm: convert scan_control.priority int => byte

2018-05-28 Thread Matthew Wilcox
On Mon, May 28, 2018 at 07:40:25PM -0700, Greg Thelen wrote: > Reclaim priorities range from 0..12(DEF_PRIORITY). > scan_control.priority is a 4 byte int, which is overkill. > > Since commit 6538b8ea886e ("x86_64: expand kernel stack to 16K") x86_64 > stack overflows are not an issue. But it's in

[PATCH] mm: convert scan_control.priority int => byte

2018-05-28 Thread Greg Thelen
Reclaim priorities range from 0..12(DEF_PRIORITY). scan_control.priority is a 4 byte int, which is overkill. Since commit 6538b8ea886e ("x86_64: expand kernel stack to 16K") x86_64 stack overflows are not an issue. But it's inefficient to use 4 bytes for priority. Use s8 (signed byte) rather tha