[PR #1764] [CLOSED] Get rid (s)elf loader errors #2289

New issue

Closed

opened 2026-02-27 21:15:55 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 21:15:55 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/shadps4-emu/shadPS4/pull/1764
Author: @mailwl
Created: 12/13/2024
Status: ❌ Closed

Base: main ← Head: add_elf_info

📝 Commits (1)

541f21d Get rid (s)elf loader errors

📊 Changes

36 files changed (+458 additions, -96 deletions)

View changed files

📝 .github/workflows/build.yml (+6 -6)
📝 externals/CMakeLists.txt (+3 -0)
📝 externals/sirit (+1 -1)
📝 src/core/address_space.h (+2 -1)
📝 src/core/libraries/audio3d/audio3d.cpp (+1 -1)
📝 src/core/libraries/gnmdriver/gnmdriver.cpp (+1 -1)
📝 src/core/libraries/kernel/process.cpp (+3 -0)
📝 src/core/libraries/libc_internal/libc_internal.cpp (+12 -0)
📝 src/core/loader/elf.h (+17 -0)
📝 src/core/module.cpp (+95 -7)
📝 src/emulator.cpp (+5 -2)
📝 src/shader_recompiler/backend/spirv/emit_spirv_floating_point.cpp (+33 -1)
📝 src/shader_recompiler/backend/spirv/emit_spirv_instructions.h (+8 -1)
📝 src/shader_recompiler/backend/spirv/spirv_emit_context.cpp (+4 -0)
📝 src/shader_recompiler/backend/spirv/spirv_emit_context.h (+2 -0)
📝 src/shader_recompiler/frontend/translate/export.cpp (+5 -0)
📝 src/shader_recompiler/frontend/translate/translate.h (+5 -0)
📝 src/shader_recompiler/frontend/translate/vector_alu.cpp (+36 -1)
📝 src/shader_recompiler/frontend/translate/vector_memory.cpp (+1 -0)
📝 src/shader_recompiler/ir/ir_emitter.cpp (+45 -2)

...and 16 more files

📄 Description

This PR adds no new functionality, mostly remove error logs while loading/parsing ps4 (s)elfs

the only significant change is different alignment for loading modules:
constexpr VAddr SYSTEM_RESERVED_MIN = 0x0800000000ULL;

which help to analize modules with IDA Pro and ps4_module_loader plugin, to look around crashed code

_{🔄 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/1764 **Author:** [@mailwl](https://github.com/mailwl) **Created:** 12/13/2024 **Status:** ❌ Closed **Base:** `main` ← **Head:** `add_elf_info` --- ### 📝 Commits (1) - [`541f21d`](https://github.com/shadps4-emu/shadPS4/commit/541f21d6112438c0cccfb0041460e339a41e8188) Get rid (s)elf loader errors ### 📊 Changes **36 files changed** (+458 additions, -96 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build.yml` (+6 -6) 📝 `externals/CMakeLists.txt` (+3 -0) 📝 `externals/sirit` (+1 -1) 📝 `src/core/address_space.h` (+2 -1) 📝 `src/core/libraries/audio3d/audio3d.cpp` (+1 -1) 📝 `src/core/libraries/gnmdriver/gnmdriver.cpp` (+1 -1) 📝 `src/core/libraries/kernel/process.cpp` (+3 -0) 📝 `src/core/libraries/libc_internal/libc_internal.cpp` (+12 -0) 📝 `src/core/loader/elf.h` (+17 -0) 📝 `src/core/module.cpp` (+95 -7) 📝 `src/emulator.cpp` (+5 -2) 📝 `src/shader_recompiler/backend/spirv/emit_spirv_floating_point.cpp` (+33 -1) 📝 `src/shader_recompiler/backend/spirv/emit_spirv_instructions.h` (+8 -1) 📝 `src/shader_recompiler/backend/spirv/spirv_emit_context.cpp` (+4 -0) 📝 `src/shader_recompiler/backend/spirv/spirv_emit_context.h` (+2 -0) 📝 `src/shader_recompiler/frontend/translate/export.cpp` (+5 -0) 📝 `src/shader_recompiler/frontend/translate/translate.h` (+5 -0) 📝 `src/shader_recompiler/frontend/translate/vector_alu.cpp` (+36 -1) 📝 `src/shader_recompiler/frontend/translate/vector_memory.cpp` (+1 -0) 📝 `src/shader_recompiler/ir/ir_emitter.cpp` (+45 -2) _...and 16 more files_ </details> ### 📄 Description This PR adds no new functionality, mostly remove error logs while loading/parsing ps4 (s)elfs the only significant change is different alignment for loading modules: `constexpr VAddr SYSTEM_RESERVED_MIN = 0x0800000000ULL;` which help to analize modules with IDA Pro and [ps4_module_loader](https://github.com/SocraticBliss/ps4_module_loader) plugin, to look around crashed code --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>