>> "*_memcg = memcg" should be executed when "memcg_charge_kmem" is success. >> "memcg_charge_kmem" return 0 if success in charging. >> Therefore, I think this code is wrong. >> If I am right, it is a serious bug that affect behavior of all the patchset. > > Which is precisely what it does. ret is a boolean, that will be true > when charge succeeded (== 0 test)
Ahh...Okay! I didn't see (== 0 test) _______________________________________________ Devel mailing list Devel@openvz.org https://openvz.org/mailman/listinfo/devel