Skip to content

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

Merged
bors merged 1 commit intorust-lang:masterfrom
mbrubeck:fstat
Jan 18, 2018
Merged

Use File::metadata instead of fs::metadata to choose buffer size#47520
bors merged 1 commit intorust-lang:masterfrom
mbrubeck:fstat

Conversation

@mbrubeck
Copy link
Contributor

@mbrubeck mbrubeck commented Jan 17, 2018

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
Contributor

rust-highfive commented Jan 17, 2018

r? @shepmaster

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

@Mark-Simulacrum
Copy link
Member

Mark-Simulacrum commented Jan 17, 2018

@bors r+ rollup

@bors
Copy link
Collaborator

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

mbrubeck commented Jan 17, 2018

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