On Wed, Feb 01, 2017 at 04:48:28PM -0800, David Rientjes wrote: > On Thu, 2 Feb 2017, Tobin C. Harding wrote: > > > @@ -3696,8 +3695,8 @@ int handle_mm_fault(struct vm_area_struct *vma, unsigned long address, > > * VM_FAULT_OOM), there is no need to kill anything. > > * Just clean up the OOM state peacefully. > > */ > > - if (task_in_memcg_oom(current) && !(ret & VM_FAULT_OOM)) > > - mem_cgroup_oom_synchronize(false); > > + if (task_in_memcg_oom(current) && !(ret & VM_FAULT_OOM)) > > + mem_cgroup_oom_synchronize(false); > > } > > > > /* > > The comment suffers from the same problem. The comment is fixed in the next patch of the set. The fixes are in separate patches because one was a checkpatch warning and one was an error. Any comments on the structure of the patchset most appreciated. thanks, Tobin. -- 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>
