On Sun, Sep 13, 2015 at 04:15:09PM -0400, Tejun Heo wrote: > __memcg_kmem_bypass() decides whether a kmem allocation should be > bypassed to the root memcg. Some conditions that it tests are valid > criteria regarding who should be held accountable; however, there are > a couple unnecessary tests for cold paths - __GFP_FAIL and > fatal_signal_pending(). > > The previous patch updated try_charge() to handle both __GFP_FAIL and > dying tasks correctly and the only thing these two tests are doing is > making accounting less accurate and sprinkling tests for cold path > conditions in the hot paths. There's nothing meaningful gained by > these extra tests. > > This patch removes the two unnecessary tests from > __memcg_kmem_bypass(). > > Signed-off-by: Tejun Heo <tj@xxxxxxxxxx> Reviewed-by: Vladimir Davydov <vdavydov@xxxxxxxxxxxxx> -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>