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.
