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

Flatten display list transfer into a separately-transferred byte array, and consolidate the multitude of display list messages into one. #231

Merged
merged 1 commit into from Mar 21, 2016

Conversation

@pcwalton
Copy link
Collaborator

pcwalton commented Mar 10, 2016

Along with the corresponding patches to other Servo repositories, this
improves display list building performance on small display lists by
about 3x.

r? @glennw

@pcwalton pcwalton force-pushed the pcwalton:faster-display-list-transfer branch from ad585aa to 9c52f9c Mar 11, 2016
and consolidate the multitude of display list messages into one.

Along with the corresponding patches to other Servo repositories, this
improves display list building performance on small display lists by
about 3x.
@glennw
Copy link
Member

glennw commented Mar 15, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Mar 15, 2016

📌 Commit 9c52f9c has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Mar 15, 2016

Testing commit 9c52f9c with merge f487bf5...

bors-servo added a commit that referenced this pull request Mar 15, 2016
Flatten display list transfer into a separately-transferred byte array, and consolidate the multitude of display list messages into one.

Along with the corresponding patches to other Servo repositories, this
improves display list building performance on small display lists by
about 3x.

r? @glennw
@bors-servo
Copy link
Contributor

bors-servo commented Mar 15, 2016

💔 Test failed - travis

@jdm
Copy link
Member

jdm commented Mar 18, 2016

@bors-servo: retry

@bors-servo
Copy link
Contributor

bors-servo commented Mar 18, 2016

Testing commit 9c52f9c with merge 5c9a65a...

bors-servo added a commit that referenced this pull request Mar 18, 2016
Flatten display list transfer into a separately-transferred byte array, and consolidate the multitude of display list messages into one.

Along with the corresponding patches to other Servo repositories, this
improves display list building performance on small display lists by
about 3x.

r? @glennw
@bors-servo
Copy link
Contributor

bors-servo commented Mar 18, 2016

💔 Test failed - travis

@jdm
Copy link
Member

jdm commented Mar 18, 2016

Hmm, we pinned the nightly that travis uses but continue to use latest serde, apparently.

@jdm
Copy link
Member

jdm commented Mar 21, 2016

@bors-servo: retry

@bors-servo
Copy link
Contributor

bors-servo commented Mar 21, 2016

Testing commit 9c52f9c with merge 9942c2d...

bors-servo added a commit that referenced this pull request Mar 21, 2016
Flatten display list transfer into a separately-transferred byte array, and consolidate the multitude of display list messages into one.

Along with the corresponding patches to other Servo repositories, this
improves display list building performance on small display lists by
about 3x.

r? @glennw
@bors-servo
Copy link
Contributor

bors-servo commented Mar 21, 2016

💔 Test failed - travis

@jdm
Copy link
Member

jdm commented Mar 21, 2016

@bors-servo: retry

@bors-servo
Copy link
Contributor

bors-servo commented Mar 21, 2016

Testing commit 9c52f9c with merge 2a71ccb...

bors-servo added a commit that referenced this pull request Mar 21, 2016
Flatten display list transfer into a separately-transferred byte array, and consolidate the multitude of display list messages into one.

Along with the corresponding patches to other Servo repositories, this
improves display list building performance on small display lists by
about 3x.

r? @glennw
@bors-servo
Copy link
Contributor

bors-servo commented Mar 21, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit 9c52f9c into servo:master Mar 21, 2016
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Testing commit 9c52f9c with merge 2a71ccb...
Details
bors-servo added a commit to servo/servo that referenced this pull request Mar 21, 2016
Faster display list transfer

This series of commits improves performance of display list construction in browser.html by about 3x when using WebRender.

It requires servo/webrender_traits#18 and servo/webrender#231.

Anyone should feel free to review if they have time; I'll ask someone in particular once those two upstream commits land.

cc @paulrouget

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9947)
<!-- Reviewable:end -->
bors-servo added a commit to servo/servo that referenced this pull request Mar 22, 2016
Faster display list transfer

This series of commits improves performance of display list construction in browser.html by about 3x when using WebRender.

It requires servo/webrender_traits#18 and servo/webrender#231.

Anyone should feel free to review if they have time; I'll ask someone in particular once those two upstream commits land.

cc @paulrouget

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9947)
<!-- Reviewable:end -->
bors-servo added a commit to servo/servo that referenced this pull request Mar 22, 2016
Faster display list transfer

This series of commits improves performance of display list construction in browser.html by about 3x when using WebRender.

It requires servo/webrender_traits#18 and servo/webrender#231.

Anyone should feel free to review if they have time; I'll ask someone in particular once those two upstream commits land.

cc @paulrouget

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9947)
<!-- Reviewable:end -->
bors-servo added a commit to servo/servo that referenced this pull request Mar 23, 2016
Faster display list transfer

This series of commits improves performance of display list construction in browser.html by about 3x when using WebRender.

It requires servo/webrender_traits#18 and servo/webrender#231.

Anyone should feel free to review if they have time; I'll ask someone in particular once those two upstream commits land.

cc @paulrouget

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/9947)
<!-- Reviewable:end -->
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 1, 2019
…er-display-list-transfer); r=glennw

This series of commits improves performance of display list construction in browser.html by about 3x when using WebRender.

It requires servo/webrender_traits#18 and servo/webrender#231.

Anyone should feel free to review if they have time; I'll ask someone in particular once those two upstream commits land.

cc paulrouget

Source-Repo: https://github.com/servo/servo
Source-Revision: 187ca442337313a76ce115ac92a2da44d91d5b96

UltraBlame original commit: cc64a6b83623b554e40083e93a29be358930016e
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 1, 2019
…er-display-list-transfer); r=glennw

This series of commits improves performance of display list construction in browser.html by about 3x when using WebRender.

It requires servo/webrender_traits#18 and servo/webrender#231.

Anyone should feel free to review if they have time; I'll ask someone in particular once those two upstream commits land.

cc paulrouget

Source-Repo: https://github.com/servo/servo
Source-Revision: 187ca442337313a76ce115ac92a2da44d91d5b96

UltraBlame original commit: cc64a6b83623b554e40083e93a29be358930016e
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 1, 2019
…er-display-list-transfer); r=glennw

This series of commits improves performance of display list construction in browser.html by about 3x when using WebRender.

It requires servo/webrender_traits#18 and servo/webrender#231.

Anyone should feel free to review if they have time; I'll ask someone in particular once those two upstream commits land.

cc paulrouget

Source-Repo: https://github.com/servo/servo
Source-Revision: 187ca442337313a76ce115ac92a2da44d91d5b96

UltraBlame original commit: cc64a6b83623b554e40083e93a29be358930016e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.