Skip to content

fix(libSceNet): silent stub for sceNetShowIfconfig - #6285

Closed
mazen320 wants to merge 2 commits into
boykopovar:mainfrom
mazen320:fix/net-show-ifconfig-silent-stub
Closed

mazen320 wants to merge 2 commits into
boykopovar:mainfrom
mazen320:fix/net-show-ifconfig-silent-stub

Conversation

@mazen320

@mazen320 mazen320 commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

What

sceNetShowIfconfig no longer calls NotImplemented_nid_no_patch. It returns 0 with no ifconfig dump.

PPSA01530 and PPSA13356 import it with an unknown signature and fail to load without it. Debt moved into Silent stubs. New guest_net_show_ifconfig covers the return.

Tested

Windows 11. python tools/check_conventions.py --base upstream/main passed. Built the original libSceNet and test targets at this PR head in a separate CMake harness, using freshly built libc/kernel from source trees verified identical to those in #6286. ctest --output-on-failure -R ^guest_net_show_ifconfig$ passed on 2026-10-15. This is local targeted validation; upstream full builds remain skipped.

Checklist

  • Based on current main; no other open PR implements the same functions
  • One topic per PR; follow-ups go in a new PR
  • No comments in code except technical debt
  • Unimplemented paths throw (NotImplemented_nid_no_patch); silent stubs are listed under Silent stubs in their library's technical debt file
  • No notes, investigation .md files or images added to the repository (attach them to this PR)
  • New third-party code is a submodule built from source
  • Generic behaviour, not specific to one title
  • Depends on: none
  • Unblocks: PPSA01530 / PPSA13356 load (ShowIfconfig stop; pad unknown NID is fix(libScePad): silent stub for scePadUnknown_n3kSX62fgNo #6283)
  • GPUs tested: n/a
  • AI-assisted: yes

PPSA01530 and PPSA13356 fail to load without the unknown-signature import;
return 0 with no ifconfig dump and list it under Silent stubs.
@mazen320

Copy link
Copy Markdown
Contributor Author

Overlap (#6283)

Bot may flag docs/dev/debt/libSceNet.md (PPSA01530/13356 unknown-imports bullet). Production change here is sceNetShowIfconfig returns 0. #6283 silent-stubs the co-listed pad NID — rebase debt wording if that PR lands first. Depends on: none.

@github-actions

Copy link
Copy Markdown

Progress report (0afa6a6 → 37a4dc2)

📈 System libraries: 88.74% (+0.03%, +1 implemented functions)

✅ 1 implemented
Library Name
libSceNet sceNetShowIfconfig
⚪ 1 behaviourally empty (constant return or throw)
Library Name
libSceNet sceNetShowIfconfig

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Other open pull requests touch the same code. Merging one will break the other, or both implement the same thing. If this one builds on them, list them in Depends on.

Pull request Conflicting files (lines on main) Same exports Same new files
PR 5676 docs/dev/debt/libSceNet.md L5
PR 5928 docs/dev/debt/libSceNet.md L2, L5

Some comparisons could not be completed because a pull request conflicts with the base branch. Review these shared files manually before concluding that there is no overlap.

Pull request Shared files not compared
PR 6044 core/libs/prx/libSceNet/CMakeLists.txt

@mazen320

Copy link
Copy Markdown
Contributor Author

Overlap

Bot flagged docs/dev/debt/libSceNet.md with #5676. This PR is Net sceNetShowIfconfig silent stub only. Depends on: none.

@mazen320

Copy link
Copy Markdown
Contributor Author

Agree with the silent-stub bar from #6274 / CONTRIBUTING: need UI-only (or safe-to-ignore / never-called) evidence, not only a load stop.

This PR only established that the title fails to load without the export. Restored throwing and closing until that evidence exists.

@mazen320

Copy link
Copy Markdown
Contributor Author

Closing: silent stub withdrawn; export(s) throw again pending UI-only / never-called evidence.

@mazen320 mazen320 closed this Oct 15, 2026
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.

1 participant