On Tue, Aug 05, 2014 at 09:47:07PM +0200, Oleg Nesterov wrote: > 1. There is no reason to reset ->tail_vma in m_start(), if we return > IS_ERR_OR_NULL() it won't be used. > > 2. m_start() also clears priv->task to ensure that m_stop() won't use > the stale pointer if we fail before get_task_struct(). But this is > ugly and confusing, move this initialization in m_stop(). > > Signed-off-by: Oleg Nesterov <o...@redhat.com>
Acked-by: Kirill A. Shutemov <kirill.shute...@linux.intel.com> -- Kirill A. Shutemov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/