On Fri, May 19, 2017 at 12:37:38PM -0400, Zi Yan wrote: > > As you say, there is no functional change but the helper name is vague > > and gives no hint to what's it's checking for. It's somewhat tolerable as > > it is as it's obvious what is being checked but the same is not true with > > the helper name. > > > > Does queue_pages_invert_nodemask_check() work? I can change the helper name > in the next version. >
Not particularly, maybe queue_pages_required and invert the check with a comment above it explaining what it's checking for would be ok. -- Mel Gorman SUSE Labs