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.

Sign up for GitHub

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

Jump to bottom

Use File::metadata instead of fs::metadata to choose buffer size #47520

Merged
merged 1 commit into from
Jan 18, 2018
Merged

Use File::metadata instead of fs::metadata to choose buffer size #47520

merged 1 commit into from
Jan 18, 2018

Conversation

mbrubeck
Copy link
Contributor

This replaces a stat syscall with fstat or similar, which can be faster. Fixes #47519.

@mbrubeck
Use File::metadata instead of fs::metadata to choose buffer size
e9fdee8 
This replaces a `stat` syscall with `fstat` or similar, which can be
faster.  Fixes rust-lang#47519.
@rust-highfive
Copy link
Collaborator

r? @shepmaster

(rust_highfive has picked a reviewer for you, use r? to override)

@Mark-Simulacrum
Copy link
Member

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Jan 17, 2018

📌 Commit e9fdee8 has been approved by Mark-Simulacrum

@Mark-Simulacrum Mark-Simulacrum added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 17, 2018
@mbrubeck
Copy link
Contributor Author

I confirmed this speeds up these functions by a small constant amount on my Linux system. Example speedup:

test read_32kb_alloc  ... bench:       3,159 ns/iter (+/- 693)
test read_32kb_alloc2 ... bench:       2,875 ns/iter (+/- 395)

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jan 17, 2018
@GuillaumeGomez
Rollup merge of rust-lang#47520 - mbrubeck:fstat, r=Mark-Simulacrum
2606537 
Use File::metadata instead of fs::metadata to choose buffer size

This replaces a `stat` syscall with `fstat` or similar, which can be faster.  Fixes rust-lang#47519.
@GuillaumeGomez GuillaumeGomez mentioned this pull request Jan 17, 2018
Merged
bors added a commit that referenced this pull request Jan 18, 2018
@bors
Auto merge of #47528 - GuillaumeGomez:rollup, r=GuillaumeGomez
3bd4af8 
Rollup of 6 pull requests

- Successful merges: #47250, #47313, #47398, #47468, #47471, #47520
- Failed merges:
@bors bors merged commit e9fdee8 into rust-lang:master Jan 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Mark-Simulacrum Mark-Simulacrum

Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@mbrubeck @rust-highfive @Mark-Simulacrum @bors @shepmaster