Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minimize memory consumption #4

Open
4 tasks done
Kagami opened this issue Aug 17, 2015 · 4 comments
Open
4 tasks done

Minimize memory consumption #4

Kagami opened this issue Aug 17, 2015 · 4 comments
Labels

Comments

@Kagami
Copy link
Owner

Kagami commented Aug 17, 2015

Things currently broken:

  • Subtitles rendering, process hungs
  • Encoding the entire file, audio/last video processes hungs

Things still causing out of memory:

  • ~400M fails at info stage
  • ~200M fails while encoding

What can be done?

  • Small -max_alloc helps at info stage
  • -blocksize ?
  • Some advanced ffmpeg options/try to patch?
  • Compile with ALLOW_MEMORY_GROWTH, bench results?
Kagami added a commit that referenced this issue Aug 17, 2015
@Kagami
Copy link
Owner Author

Kagami commented Aug 17, 2015

TOTAL_MEMORY=256M for encoder and 128M for prober allows to successfully encode 600M+ files. MEMFS works with that values too but consumes much more memory.

ALLOW_MEMORY_GROWTH works as well and doesn't cause significant slowdown in nightly, though it may cause problems in v8 so let's keep it disabled for now.

It still may be worth to try advanced ffmpeg options and bigger files/different TOTAL_MEMORY values in order to decrease memory consumption even more.

@Kagami Kagami changed the title WORKERFS Minimalize memory consumption Aug 26, 2015
@Kagami Kagami changed the title Minimalize memory consumption Minimize memory consumption Aug 26, 2015
@uqee
Copy link

uqee commented Sep 27, 2017

Any updates on this?

@bvibber
Copy link

bvibber commented Sep 27, 2017

Switching build to WebAssembly should allow natural memory growth without the performance penalty from asm.js's growth mode...

@uqee
Copy link

uqee commented Sep 27, 2017

Thanks, @Brion, worth a try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants