This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Don't use SSE4_2 instructions on Intel Silvermont Micro Architecture.


On 06/27/2013 03:24 AM, Liubov Dmitrieva wrote:
> I think for this particular patch we don't need super accurate
> benchmarks to see that it is better because we talk not about 20-60%
> of boost but about several times asymptotically boost as current
> benchmarks showed. It was a server machine, nobody runs Firefox there.

Agreed, but we still need some kind of reproducible result that shows
your patch improved performance. I'm not happy with performance patches
going into glibc without some proof that they made things better.

If you can't show that with the benchmark, and have some other benchmark
numbers, then I might be convinced, but I'd like you to update our
benchmarking results page for 2.18 to describe what you did and what
numbers you achieved.

http://sourceware.org/glibc/wiki/benchmarking/results_2_18

Thanks.

Cheers,
Carlos.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]