Re: [PATCH] perf tools: avoid possible race condition in copyfile

From: Milos Vyletel
Date: Mon Jun 08 2015 - 10:01:48 EST


On Mon, Jun 08, 2015 at 10:39:57AM -0300, Arnaldo Carvalho de Melo wrote:

> Em Thu, Jun 04, 2015 at 11:11:00AM +0200, Ingo Molnar escreveu:
> > * Milos Vyletel <milos@xxxxxxxxxx> wrote:
> > > On top of that slow_copyfile no longer needs to deal with file mode when
> > > creating file since temporary file is already created and mode is set.
> > >
> > > Signed-off-by: Milos Vyletel <milos@xxxxxxxxxx>
> >
> > Ok, that looks nice!
> >
> > Assuming it passes testing you can add my ack to it:
> >
> > Acked-by: Ingo Molnar <mingo@xxxxxxxxxx>
>
> Can you please try applying it on tip/perf/core instead? I think it is
> clashing with:
>
> commit 9c9f5a2f1944e8b6bf2b618d04b31e1c1760637e
> Author: Namhyung Kim <namhyung@xxxxxxxxxx>
> Date: Mon May 18 09:30:18 2015 +0900
>
> perf tools: Introduce copyfile_offset() function
>

And also with

commit 0b1de0be1eac7b23e89cb43c17b02d38ead6b6c8
Author: Namhyung Kim <namhyung@xxxxxxxxxx>
Date: Mon May 18 09:30:17 2015 +0900

perf tools: Add rm_rf() utility function

Will resend version based on tip/perf/core after I rerun my testing.

Milos
--
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/