Hi, 2011/12/16 Måns Rullgård <[email protected]>
> Måns Rullgård <[email protected]> writes: > > > Martin Storsjö <[email protected]> writes: > > > >> On Fri, 16 Dec 2011, Mans Rullgard wrote: > >> > >>> diff --git a/configure b/configure > >>> index 974e75f..0349862 100755 > >>> --- a/configure > >>> +++ b/configure > >>> @@ -113,6 +113,7 @@ Configuration options: > >>> --disable-dxva2 disable DXVA2 code > >>> --enable-runtime-cpudetect detect cpu capabilities at runtime > (bigger binary) > >>> --enable-hardcoded-tables use hardcoded tables instead of runtime > generation > >>> + --disable-safe-bitstream-reader disable buffer boundary checking in > bitreaders (faster, but may crash) > >>> --enable-memalign-hack emulate memalign, interferes with memory > debuggers > >>> --disable-everything disable all components listed below > >>> --disable-encoder=NAME disable encoder NAME > >>> @@ -976,6 +977,7 @@ CONFIG_LIST=" > >>> rdft > >>> rtpdec > >>> runtime_cpudetect > >>> + safe_bitstream_reader > >>> shared > >>> sinewin > >>> small > >> > >> Doesn't this still need to be enabled somewhere, or the help text > >> changed into --enable? > > > > Yes, I'm running some benchmarks now to determine whether to enable this > > by default. We'll update the patch depending on the outcome. > > The performance penalty on ARM (both Cortex-A8 and A9) is 0-2% for most > files, 4% in extreme cases. In many cases, the change is within the > error margin of the measurement. > > I will not object to turning this on by default. > Awesome, if you can add the proper line to configure, patch is OK with me. Feel free to push directly. Ronald
_______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
