Skip to content

fix(libkernel): return guest chmod errors and implement sceKernelFutimes - #2787

Merged
adriadam10 merged 2 commits into
boykopovar:mainfrom
tototomate123:fix/hfw-filesystem-split
Oct 11, 2026
Merged

adriadam10 merged 2 commits into
boykopovar:mainfrom
tototomate123:fix/hfw-filesystem-split

Conversation

@tototomate123

@tototomate123 tototomate123 commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

What

Split from #1616, following its review. Return guest errors from kernel chmod and POSIX -1/guest errno from POSIX chmod. Implement sceKernelFutimes while retaining POSIX futimes behavior and timestamp validation. Two commits.

Tested

Linux x86-64, GCC 16.2.1, Release. guest_filesystem passes, including chmod failures, descriptor timestamps and invalid microseconds. Conventions and whitespace checks pass on current main (5edae353).

Depends on

None for build. HFW integration also needs #2785 (relinker), #2786 (loader), #2788 (http), #2789 (apr), #2790 (shader). #1269 (media), #1339 (VRR), #1586 (unwind) and #1436 (imports) are now on main. Local image-table experiments are excluded; open #757 → #758 overlaps them, and #920 is closed without merging. Earlier combined integration reached menu/loading UI; these isolated branches have not been game-tested and gameplay is unverified.

Checklist

  • Based on current main; no duplicate implementation found beyond superseded fix(linux): resolve loader and APR failures exposed by HFW #1616
  • One topic per PR
  • No code comments except technical debt
  • Unimplemented paths throw; offline HTTP limitations documented
  • No investigation notes, logs or images committed
  • No new third-party dependencies
  • Generic behavior; no title-specific dispatch
  • Depends on: none for build; HFW integration requirements listed above
  • AI-assisted: yes, Codex

@github-actions

Copy link
Copy Markdown

Thank you for your interest in the project! We are sorry, but we cannot accept more PRs while we work through the existing backlog. This PR has been automatically closed and labeled "intake paused". This is not a judgment of the quality of your changes.

@github-actions github-actions Bot closed this Oct 10, 2026
@github-actions

Copy link
Copy Markdown

Progress report (5edae35 → 9107be4)

➖ System libraries: 84.52% (+0.0%, +1 implemented functions)

✅ 1 implemented
Library Name
libkernel sceKernelFutimes

@oneandonlydean oneandonlydean removed the intake paused Closed while PR intake is limited to authors with at least 10 PRs merged into main label Oct 11, 2026
@adriadam10

Copy link
Copy Markdown
Collaborator

Checked the chmod/futimes error paths against the guest test (guest_filesystem, incl. null path, missing path, EBADF, invalid usec) and the partial build + ctest (94 tests) on Linux; the new test does not link without the change. Merging once CI is green.

@adriadam10
adriadam10 merged commit effc61e into boykopovar:main Oct 11, 2026
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants