Re: [PATCH 1/2] CRED: Rename cred_exec_mutex to reflect that it'sa guard against ptrace

From: Ingo Molnar
Date: Mon May 11 2009 - 07:35:57 EST



* Roland McGrath <roland@xxxxxxxxxx> wrote:

> > Hmm. Ingo's "rename ptrace_may_access => ptrace_access_check" conflicts
> > with my patch too.
>
> Andrew seemed to want a different name choice too, so that will have to be
> resolved before we worry about patch conflicts.
>
> > Andrew, Roland, I guess I should re-send
> >
> > ptrace-ptrace_attach-check-pf_kthread-exit_state-instead-of-mm.patch
> > ptrace-cleanup-check-set-of-pt_ptraced-during-attach.patch
> > ptrace-do-not-use-task_lock-for-attach.patch
> >
> > patches?
>
> I guess so too. Your series of changes is more substantial and
> potentially controversial or problematic than the various
> renamings, so I think it makes sense to settle and merge the
> renamings first.

yeah, cleanups first is generally the better strategy. Not only does
it make reverts easier, it also makes it easier to review (and
potentially fix) later patches.

Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/