Re: [PATCH 1/8] mm: page_counter: let page_counter_try_charge() return bool

2015-10-23 Thread Michal Hocko
On Thu 22-10-15 00:21:29, Johannes Weiner wrote: > page_counter_try_charge() currently returns 0 on success and -ENOMEM > on failure, which is surprising behavior given the function name. > > Make it follow the expected pattern of try_stuff() functions that > return a boolean true to indicate

Re: [PATCH 1/8] mm: page_counter: let page_counter_try_charge() return bool

2015-10-23 Thread Michal Hocko
On Thu 22-10-15 00:21:29, Johannes Weiner wrote: > page_counter_try_charge() currently returns 0 on success and -ENOMEM > on failure, which is surprising behavior given the function name. > > Make it follow the expected pattern of try_stuff() functions that > return a boolean true to indicate

[PATCH 1/8] mm: page_counter: let page_counter_try_charge() return bool

2015-10-21 Thread Johannes Weiner
page_counter_try_charge() currently returns 0 on success and -ENOMEM on failure, which is surprising behavior given the function name. Make it follow the expected pattern of try_stuff() functions that return a boolean true to indicate success, or false for failure. Signed-off-by: Johannes Weiner

[PATCH 1/8] mm: page_counter: let page_counter_try_charge() return bool

2015-10-21 Thread Johannes Weiner
page_counter_try_charge() currently returns 0 on success and -ENOMEM on failure, which is surprising behavior given the function name. Make it follow the expected pattern of try_stuff() functions that return a boolean true to indicate success, or false for failure. Signed-off-by: Johannes Weiner