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

moires runs very slow #193

Closed
nightpool opened this issue Feb 20, 2016 · 3 comments
Closed

moires runs very slow #193

nightpool opened this issue Feb 20, 2016 · 3 comments

Comments

@nightpool
Copy link

@nightpool nightpool commented Feb 20, 2016

really amazing demo! I tried cloning servo and compiling it myself, but I'm pretty consistently getting 5 fps on the moire demo (on a pretty recent macbook pro).

  • I compiled with ./mach build --release
  • I cloned moire.html from https://github.com/pcwalton/webrender-demos
  • I ran servo with target/release/servo -Z wr-stats -w --resolution=1024x768 --pref dom.mozbrowser.enabled ../webrender-demos/moire.htm (I copied it straight from the demo stream)

this is what I'm seeing: (note the 4 fps)

Interestingly, this seems to be limited to moires. Both spheres and transparent_rects run at ~60fps.

Any thoughts?

@glennw
Copy link
Member

@glennw glennw commented Feb 21, 2016

I think the moire demo relies on some fixes to the tesselator code landing (which pcwalton has locally). Without this fix the moire demo suffers from a huge amount of needless overdraw. Is that right @pcwalton ?

@dw
Copy link

@dw dw commented Feb 25, 2016

I observed this too -- disabling WebRender causes the moire demo to run much more smoothly on a 2011 era Macbook (HD graphics 3000).

@glennw
Copy link
Member

@glennw glennw commented Feb 25, 2016

I checked with @pcwalton and this is due to the tesselator being disabled on master at the moment due to a bug - he is working on a fix for this which we should be able to merge soon :)

bors-servo added a commit that referenced this issue Feb 26, 2016
Start texture atlases off small, and allow them to grow.

Includes #199.

Closes #193.

r? @glennw
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.