[PR #3368] [CLOSED] [Draft] Use native file handlers #3417

New issue

Closed

opened 2026-02-27 22:03:37 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 22:03:37 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/shadps4-emu/shadPS4/pull/3368
Author: @marecl
Created: 8/2/2025
Status: ❌ Closed

Base: main ← Head: posixiofile

📝 Commits (10+)

3b66a58 Moved file handling to native calls:
5e7070b Disambiguation, moving parameter resolving to target OS source
96e6dc2 how did these get here
9a47cb2 errno (and Windows equivalent) propagation
d866877 clang
f2d87ce Merge branch 'main' of https://github.com/shadps4-emu/shadPS4 into posixiofile
f6d9870 Moved file handling to native calls:
7a32904 Disambiguation, moving parameter resolving to target OS source
524447b how did these get here
f0e21d2 errno (and Windows equivalent) propagation

📊 Changes

45 files changed (+1555 additions, -706 deletions)

View changed files

📝 CMakeLists.txt (+13 -1)
📝 src/common/config.cpp (+10 -6)
📝 src/common/io_file.cpp (+139 -310)
📝 src/common/io_file.h (+113 -91)
📝 src/common/logging/backend.cpp (+2 -2)
➕ src/common/native_fs.cpp (+87 -0)
➕ src/common/native_fs.h (+80 -0)
➕ src/common/native_fs_posix.cpp (+332 -0)
➕ src/common/native_fs_win32.cpp (+99 -0)
📝 src/common/path_util.cpp (+8 -5)
📝 src/core/devtools/widget/common.h (+5 -1)
📝 src/core/devtools/widget/module_list.h (+7 -3)
📝 src/core/devtools/widget/shader_list.cpp (+2 -1)
📝 src/core/file_format/psf.cpp (+2 -1)
📝 src/core/file_format/trp.cpp (+6 -3)
📝 src/core/file_sys/fs.cpp (+12 -9)
📝 src/core/libraries/app_content/app_content.cpp (+2 -1)
📝 src/core/libraries/kernel/file_system.cpp (+90 -105)
📝 src/core/libraries/kernel/kernel.cpp (+363 -42)
📝 src/core/libraries/np_trophy/trophy_ui.cpp (+11 -7)

...and 25 more files

📄 Description

This PR changes how IOFile is handled on OS-level.
Improves compatibility and granularity with Orbis file-related kernel calls.
Existing (in-app call) behaviour and usage is not affected (I think).

Currently implemented for UNIX (POSIX calls).
Windows implementation pending.
Further tests are needed to confirm correct operation.

Subject to changes.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/shadps4-emu/shadPS4/pull/3368 **Author:** [@marecl](https://github.com/marecl) **Created:** 8/2/2025 **Status:** ❌ Closed **Base:** `main` ← **Head:** `posixiofile` --- ### 📝 Commits (10+) - [`3b66a58`](https://github.com/shadps4-emu/shadPS4/commit/3b66a581bebde5714cc59241a4ee2489ee159f20) Moved file handling to native calls: - [`5e7070b`](https://github.com/shadps4-emu/shadPS4/commit/5e7070bd5bbbb4076e426d31dc1b31c6b7948bb2) Disambiguation, moving parameter resolving to target OS source - [`96e6dc2`](https://github.com/shadps4-emu/shadPS4/commit/96e6dc29fa67e9860394c276cac710cc86d715a0) how did these get here - [`9a47cb2`](https://github.com/shadps4-emu/shadPS4/commit/9a47cb290bb08d9618e9b09a25dd036506d75372) errno (and Windows equivalent) propagation - [`d866877`](https://github.com/shadps4-emu/shadPS4/commit/d86687703b4a50fc801816a35afbd70f45573361) clang - [`f2d87ce`](https://github.com/shadps4-emu/shadPS4/commit/f2d87ce49b51d2eeae873037dd628928f749135c) Merge branch 'main' of https://github.com/shadps4-emu/shadPS4 into posixiofile - [`f6d9870`](https://github.com/shadps4-emu/shadPS4/commit/f6d9870718de94159cbaf63e5bb5382cb97eae22) Moved file handling to native calls: - [`7a32904`](https://github.com/shadps4-emu/shadPS4/commit/7a329047e5c61ccce8dc25df47a24955d0dcc608) Disambiguation, moving parameter resolving to target OS source - [`524447b`](https://github.com/shadps4-emu/shadPS4/commit/524447be46a8b8591163c046fd6a8b59bf8b28f7) how did these get here - [`f0e21d2`](https://github.com/shadps4-emu/shadPS4/commit/f0e21d22396b5ed73b37f940530ca073c5219a65) errno (and Windows equivalent) propagation ### 📊 Changes **45 files changed** (+1555 additions, -706 deletions) <details> <summary>View changed files</summary> 📝 `CMakeLists.txt` (+13 -1) 📝 `src/common/config.cpp` (+10 -6) 📝 `src/common/io_file.cpp` (+139 -310) 📝 `src/common/io_file.h` (+113 -91) 📝 `src/common/logging/backend.cpp` (+2 -2) ➕ `src/common/native_fs.cpp` (+87 -0) ➕ `src/common/native_fs.h` (+80 -0) ➕ `src/common/native_fs_posix.cpp` (+332 -0) ➕ `src/common/native_fs_win32.cpp` (+99 -0) 📝 `src/common/path_util.cpp` (+8 -5) 📝 `src/core/devtools/widget/common.h` (+5 -1) 📝 `src/core/devtools/widget/module_list.h` (+7 -3) 📝 `src/core/devtools/widget/shader_list.cpp` (+2 -1) 📝 `src/core/file_format/psf.cpp` (+2 -1) 📝 `src/core/file_format/trp.cpp` (+6 -3) 📝 `src/core/file_sys/fs.cpp` (+12 -9) 📝 `src/core/libraries/app_content/app_content.cpp` (+2 -1) 📝 `src/core/libraries/kernel/file_system.cpp` (+90 -105) 📝 `src/core/libraries/kernel/kernel.cpp` (+363 -42) 📝 `src/core/libraries/np_trophy/trophy_ui.cpp` (+11 -7) _...and 25 more files_ </details> ### 📄 Description This PR changes how IOFile is handled on OS-level. Improves compatibility and granularity with Orbis file-related kernel calls. Existing (in-app call) behaviour and usage is not affected (I think). Currently implemented for UNIX (POSIX calls). Windows implementation pending. Further tests are needed to confirm correct operation. Subject to changes. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>