google / brotli

Brotli compression format
MIT License
13.6k stars 1.24k forks source link

Performance regressions found in Brotli v1.1.0 #1202

Open carlossanlop opened 2 months ago

carlossanlop commented 2 months ago

The dotnet/runtime repo recently updated its in-tree copy of its Brotli dependency from v1.0.9 to v1.1.0.

We ran our performance benchmarks, and we noticed regressions in Linux-x64 and Windows-x64.

Most of our regression results were under 15%, but 6 results were between 39% an 49%.

Here's a list of the tests that showed regressions:

Windows x64

Linux x64

Some more info:

Unfortunately, the v1.1.0 tag has no mention of known perf regressions, so we were wondering if you were aware of these results and if you could help us figure out the possible root causes.

eustas commented 3 days ago

Hello. Thanks for the report. I'll investigate soon.

We thoroughly test and polish changes until they are at least as performant as before. Likely regression happens on arch / compiler / OS combination we do not track.

carlossanlop commented 3 days ago

Likely regression happens on arch / compiler / OS combination we do not track.

We found them in Linux-x64 and Windows-x64.