open [13177] vfs: misc performance improvements #13177

Open
Accord wants to merge 3 commits from __refs_pull_13177_head into master
Owner

Branch: __refs_pull_13177_head

Labels: early-access-merge, android-merge

liamwhite opened issue at 2024-02-27 02:03:

  1. Not necessary to record the parent directory for offset files as we never care about the parent.
  2. When enumerating a directory to get the file sizes we also happen to know the name of the parent directory and don't have to look it up again.
  3. Check the most common file types first in the loader to increase the chances of early termination.

[Export of Github issue for yuzu-emu/yuzu. Generated on 2024.02.28 at 10:04:32.]

**Branch:** [__refs_pull_13177_head](https://accord.dravee.dev/Accord/yuzu-mirror/src/branch/__refs_pull_13177_head) > **Labels**: `early-access-merge`, `android-merge` #### <div><img width="50" height="50" src="https://avatars.githubusercontent.com/u/9658600?v=4" width="50"></div>[liamwhite](https://github.com/liamwhite) opened issue at [2024-02-27 02:03](https://github.com/yuzu-emu/yuzu/pull/13177): > 1. Not necessary to record the parent directory for offset files as we never care about the parent. > 2. When enumerating a directory to get the file sizes we also happen to know the name of the parent directory and don't have to look it up again. > 3. Check the most common file types first in the loader to increase the chances of early termination. > ------------------------------------------------------------------------------- > [Export of Github issue for [yuzu-emu/yuzu](https://github.com/yuzu-emu/yuzu). Generated on 2024.02.28 at 10:04:32.]
loader: probe common file types first
Some checks failed
codespell / Check for spelling errors (pull_request) Has been cancelled
yuzu verify / verify format (pull_request) Has been cancelled
yuzu-ci / transifex (pull_request) Has been cancelled
yuzu-ci / reuse (pull_request) Has been cancelled
yuzu verify / test build (pull_request) Has been cancelled
yuzu verify / test build-1 (pull_request) Has been cancelled
yuzu verify / test build-2 (pull_request) Has been cancelled
yuzu verify / test build (macos) (pull_request) Has been cancelled
yuzu verify / test build (windows, msvc) (pull_request) Has been cancelled
yuzu verify / android (pull_request) Has been cancelled
f5cc94f054
Some checks failed
codespell / Check for spelling errors (pull_request) Has been cancelled
yuzu verify / verify format (pull_request) Has been cancelled
yuzu-ci / transifex (pull_request) Has been cancelled
yuzu-ci / reuse (pull_request) Has been cancelled
yuzu verify / test build (pull_request) Has been cancelled
yuzu verify / test build-1 (pull_request) Has been cancelled
yuzu verify / test build-2 (pull_request) Has been cancelled
yuzu verify / test build (macos) (pull_request) Has been cancelled
yuzu verify / test build (windows, msvc) (pull_request) Has been cancelled
yuzu verify / android (pull_request) Has been cancelled
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin __refs_pull_13177_head:__refs_pull_13177_head
git switch __refs_pull_13177_head

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff __refs_pull_13177_head
git switch __refs_pull_13177_head
git rebase master
git switch master
git merge --ff-only __refs_pull_13177_head
git switch __refs_pull_13177_head
git rebase master
git switch master
git merge --no-ff __refs_pull_13177_head
git switch master
git merge --squash __refs_pull_13177_head
git switch master
git merge --ff-only __refs_pull_13177_head
git switch master
git merge __refs_pull_13177_head
git push origin master
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
Accord/yuzu-mirror!13177
No description provided.