[jagrat04]

What

Implement sceNetShowIfconfigWithMemory in core/libs/prx/libSceNet/Export.cpp:

• Validates the provided memory pool ID against active pools (g_pools) under g_mutex, returning NET_EBADF if invalid.
• Throws std::runtime_error(__func__) for a valid pool once the net library is initialized, aligning with sibling diagnostics (sceNetShowIfconfig and sceNetShowNetstat) per 04f2d9a60f.
• Adds guest unit tests in core/libs/tests/GuestSceNet.cpp verifying invalid pool failure, destroyed pool failure, and throwing on valid active pool.
• Documents the function in docs/dev/debt/libSceNet.md folded into sceNetShowIfconfig, sceNetShowNetstat.

Tested

• Linux (Ubuntu 24.04 via WSL2, GCC 15): ninja -C build-linux libSceNet guest_sce_net_tests built cleanly; ctest --test-dir build-linux -V -R guest_sce_net passed (1/1 tests passed 100%).
• python tools/check_conventions.py --base origin/main: passed with 0 violations.

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
• Fixes: none
• Unblocks: none
• GPUs tested: n/a
• AI-assisted: yes

[github-actions]

Progress report (b607bfd → ac2591a)

📈 System libraries: 48.47% (+0.01%, +2 implemented functions)

✅ 2 implemented

Library	Name
libkernel	setpgid_nid_postfix
libkernel	setpgrp_nid_postfix

Changed libraries (implemented exports)

Library	Before	After	Change
libkernel	664/704 (94.32%)	666/706 (94.33%)	+0.02 pp

[github-actions]

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 8742		setpgid setpgrp
PR 9378	core/libs/prx/libkernel/System/src/Process.cpp L346 core/libs/tests/GuestParentProcess.cpp L11, L23 docs/dev/debt/libkernel.md L241
PR 9499	docs/dev/debt/libkernel.md L241-242

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 4630	docs/dev/debt/libkernel.md
PR 5021	core/libs/prx/libkernel/System/src/Process.cpp docs/dev/debt/libkernel.md
PR 6149	docs/dev/debt/libkernel.md
PR 6830	docs/dev/debt/libkernel.md
PR 6936	docs/dev/debt/libkernel.md
PR 7088	docs/dev/debt/libkernel.md
PR 7105	docs/dev/debt/libkernel.md
PR 7121	docs/dev/debt/libkernel.md
PR 7149	docs/dev/debt/libkernel.md
PR 7192	docs/dev/debt/libkernel.md
PR 7201	core/libs/prx/libkernel/System/src/Process.cpp
PR 7509	docs/dev/debt/libkernel.md
PR 7511	docs/dev/debt/libkernel.md
PR 8010	docs/dev/debt/libkernel.md
PR 8605	docs/dev/debt/libkernel.md
PR 9133	core/libs/prx/libkernel/System/src/Process.cpp docs/dev/debt/libkernel.md
PR 9160	docs/dev/debt/libkernel.md
PR 9199	docs/dev/debt/libkernel.md
PR 9284	docs/dev/debt/libkernel.md
PR 9293	docs/dev/debt/libkernel.md
PR 9297	docs/dev/debt/libkernel.md
PR 9302	docs/dev/debt/libkernel.md
PR 9509	docs/dev/debt/libkernel.md
PR 9557	docs/dev/debt/libkernel.md
PR 9561	docs/dev/debt/libkernel.md
PR 9566	docs/dev/debt/libkernel.md
PR 9575	docs/dev/debt/libkernel.md
PR 9669	core/libs/prx/libkernel/System/src/Process.cpp docs/dev/debt/libkernel.md
PR 9693	docs/dev/debt/libkernel.md
PR 9929	core/libs/prx/libkernel/System/src/Process.cpp
PR 9947	docs/dev/debt/libkernel.md
PR 10061	docs/dev/debt/libkernel.md

[MarcZX10]

I am reviewing #9506, which adds sceNetSetDnsInfo, sceNetSetDns6Info and sceNetGetDns6Info plus a new registered test and debt entry. The overlap bot could not compare some of these branches because of base conflicts, and this PR author has not yet recorded an agreed merge order. Please confirm whether your current changes overlap these exact exports, CMake/test insertion, or debt section, and whether #9506 can land before your PR or should depend on yours. The #9506 author is being asked to record the agreed order in Depends on and the overlap note.

[jagrat04]

@MarcZX10 Thanks for checking! There is no overlap with #9506:

• Exports: This PR implements sceNetShowIfconfigWithMemory only. It does not touch sceNetSetDnsInfo, sceNetSetDns6Info, or sceNetGetDns6Info.
• Tests & CMake: Tests are in GuestSceNet.cpp and do not touch CMakeLists.txt or GuestNetDnsInfo.cpp.
• Debt documentation: Touches Silent stubs in docs/dev/debt/libSceNet.md without affecting the DNS entries in Unknown function info.
• Merge Order: Either PR can land first without issues (Depends on: none). This branch has also been cleanly rebased onto latest main.

[adriadam10]

Since this was opened, main changed the sibling diagnostics (04f2d9a, "reject unsupported diagnostic output after initialization"): sceNetShowIfconfig and sceNetShowNetstat now throw once the net library is initialized, because the native output is not implemented, and GuestSceNet.cpp pins that (Failed(sceNetShowIfconfig(), 200)). Returning 0 from sceNetShowIfconfigWithMemory for a valid pool is the plausible-success-where-main-threw case we reject.

• core/libs/prx/libSceNet/Export.cpp:2289: keep the pool check (NET_EBADF for an unknown pool), but for a valid pool throw like sceNetShowIfconfig (throw std::runtime_error(__func__)) instead of return 0.
• core/libs/tests/GuestSceNet.cpp:280: drop sceNetShowIfconfigWithMemory(pool) == 0; the invalid-pool rows (-1, destroyed pool) stay and still fail without the change. Pin the valid-pool case as a throw if the harness has a way to (see how line 1690 handles sceNetShowIfconfig).
• docs/dev/debt/libSceNet.md:5: this is no longer a silent stub. Fold it into the existing sceNetShowIfconfig, sceNetShowNetstat line (initialized calls throw).

Rebase onto current main when you push.

[jagrat04]

@adriadam10 Thanks for the review and clear guidance!

Updated as requested:

• Export.cpp: Retained the memory pool validation (NET_EBADF for unknown/invalid pool IDs), and for a valid pool now throw std::runtime_error(__func__) matching sibling diagnostic behavior after initialization.
• GuestSceNet.cpp: Dropped the success return assertion; kept invalid-pool checks (-1 and destroyed pool ID returning NET_EBADF / error 9) and pinned the valid-pool case to assert throwing std::runtime_error.
• libSceNet.md: Removed from silent stubs and folded into the sceNetShowIfconfig, sceNetShowIfconfigWithMemory, sceNetShowNetstat entry stating initialized calls throw.
• Rebased cleanly onto current main (aeeaefad98), verified conventions pass with 0 violations, and confirmed guest_sce_net tests pass 100%.

[adriadam10]

Re-checked on current main: the pool check (NET_EBADF for an unknown or destroyed pool) and the throw for a valid pool match sceNetShowIfconfig/sceNetShowNetstat, and the new test lines fail without the change. Rebased onto main (adjacent conflicts with sceNetShowRouteWithMemory in the test and the Silent stubs debt line, merged by hand keeping both); merged.