Skip to content
This repository has been archived by the owner on Jan 21, 2021. It is now read-only.

Deadlock on stream seeking #363

Closed
ValdikSS opened this issue May 13, 2015 · 12 comments
Closed

Deadlock on stream seeking #363

ValdikSS opened this issue May 13, 2015 · 12 comments

Comments

@ValdikSS
Copy link
Contributor

Bomi deadlocks thread and consumes 100% CPU on stream seeking.

  1. Open any youtube video
  2. Seek it

Expected result:
Video continues to play from seek position

Actual result:
Video is not played and bomi consumes 100% cpu.

@bylee20
Copy link
Owner

bylee20 commented May 13, 2015

Have you tried non DASH format in Play > Streaming Format?

@ValdikSS
Copy link
Contributor Author

Yes, I never tried DASH. I use mp4 720.

@bylee20
Copy link
Owner

bylee20 commented May 18, 2015

I can't reproduce your problem.
However, I've just updated mpv source tree and it contains a fix for a high CPU usage situation.
Can you test with latest source?
And, if you still have problem, can you post any URL to reproduce your problem?

@ValdikSS
Copy link
Contributor Author

The issue is still here somehow. If I seek youtube stream, sometimes both video and audio are stuck or video only (audio plays fine). Bomi still consumes 100% CPU on seeking.
I believe this could be of not very high speed internet.

Please check "update mpv" commit, there is something with the audio volume.

@bylee20
Copy link
Owner

bylee20 commented May 18, 2015

Please check "update mpv" commit, there is something with the audio volume.

What is something?

@ValdikSS
Copy link
Contributor Author

It's very low.

@bylee20
Copy link
Owner

bylee20 commented May 18, 2015

I can't see any problem. Have you checked volume/amp in bomi and system volume?

@ValdikSS
Copy link
Contributor Author

Strangely enough, it works fine now. I've checked volume and amp levels 3 times when I wrote this, and now the volume level is the same across versions. Sorry about that.

@ValdikSS
Copy link
Contributor Author

Oh wait, it seems like with the mpv update, bomi now reads mpv config file. I have softvol-max 100 in it, and it clearly alter bomi volume level.

@bylee20
Copy link
Owner

bylee20 commented May 19, 2015

bomi now reads mpv config file.

Fixed.

@bylee20
Copy link
Owner

bylee20 commented May 20, 2015

I found a situation using 100% during seeking in low bandwidth and reported to mpv dev.
It has been fixed in upstream and I backported the fix, so please check if it works for you.

@ValdikSS
Copy link
Contributor Author

Just tested it. Don't see any deadlocks, and the CPU consumption is fixed. Thank you a lot!

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

No branches or pull requests

2 participants