Re: [PATCH -tip 1/3] Pass mm->flags to binfmt core_dump for bitflagconsistency

From: Ingo Molnar
Date: Fri Nov 13 2009 - 18:25:20 EST



* Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:

> On Fri, 13 Nov 2009 17:52:27 -0500
> Masami Hiramatsu <mhiramat@xxxxxxxxxx> wrote:
>
> > Pass mm->flags to binfmt core_dump for bitflag consistency.
> > Since mm->flags bit flags is not protected by locks, it will be
> > changed while dumping core. This patch copies mm->flags to a
> > mm_flags local variable at the beginning of do_coredump() and
> > use it while dumping. mm_flags also includes dump_filter which
> > filters elf sections from core file in elf_core_dump().
> > So, this patch also passes mm_flags to each binfmt->core_dump().
>
> I can kind-of guess the answer, but it would be much more reliable if
> we were to hear this from yourself:
>
> Why did you write this patch? What problem is being observed?

i'm not Masami so i'm only guessing that while writing the tracepoint a
race got noticed but that otherwise there's no big practical effect,
'just' a cleanliness problem fixed.

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/