Re: [PATCH 01/10] perf, tools: Factor out scale conversion code

From: Arnaldo Carvalho de Melo
Date: Fri Oct 14 2016 - 12:16:47 EST


Em Fri, Oct 14, 2016 at 08:45:15AM -0700, Andi Kleen escreveu:
> > So if we can't convert the scale because of an allocation failure
> > related to locale issues we silently trow it away and do no scale at
> > all?

> That is right. If your machine is thrashing to death in a OOM this
> is your smallest problem.

Please keep it was before, i.e. return an error value, and bail out. It
was like that before, why introduce these kinds of silent "do something
else in an unlikely case" handling?

- Arnaldo