Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VSH Improvements #13172

Merged
merged 13 commits into from
Jan 9, 2023
Merged

VSH Improvements #13172

merged 13 commits into from
Jan 9, 2023

Conversation

elad335
Copy link
Contributor

@elad335 elad335 commented Jan 6, 2023

  • Implement _sys_prx_register_library.
  • Implement sys_rsx_context_free.
  • sys_net: Implement SO_RCVTIMEO/SO_SENDTIMEO.
  • Implement sys_memory_container_destroy_parent_with_child, fixes out-of-memory crash with VSH.
  • PPU LLVM: Fixes referencing multiple PRX.
  • Fix sys_sm_shutdown reboot.
  • Report size of XMB as 0 on UI so people won't be tempted to remove it and possibly other firmware files inside dev_flash. These files are REQUIRED for rpcs3 to function properly and are not optional.

These fixes allow to proceed further along into game launching within XMB.

Fixes #13154

@elad335 elad335 changed the title sys_prx: Implement PRX LIB register syscall VSH Improvements [WIP] Jan 6, 2023
@elad335
Copy link
Contributor Author

elad335 commented Jan 6, 2023

Last commit adds VSH to game grid, with all it entails such as custom config and shortcut creation.

image

@Megamouse
Copy link
Contributor

Please stop mixing GUI stuff with HLE bugfixes

@elad335 elad335 changed the title VSH Improvements [WIP] VSH Improvements Jan 7, 2023
@elad335 elad335 marked this pull request as ready for review January 7, 2023 06:58
@elad335
Copy link
Contributor Author

elad335 commented Jan 7, 2023

@Darkhost1999 Please retest.

@Darkhost1999
Copy link
Contributor

RPCS3.log
Working well

@ghost
Copy link

ghost commented Jan 7, 2023

Working well for me

rpcs3/Emu/RSX/RSXThread.cpp Outdated Show resolved Hide resolved
@elad335 elad335 requested a review from kd-11 January 8, 2023 09:52
rpcs3/Emu/RSX/RSXThread.cpp Outdated Show resolved Hide resolved
@@ -394,7 +394,7 @@ const std::array<std::pair<ppu_intrp_func_t, std::string_view>, 1024> g_ppu_sysc
BIND_SYSC(sys_memory_container_destroy), //342 (0x156)
BIND_SYSC(sys_memory_container_get_size), //343 (0x157)
NULL_FUNC(sys_memory_budget_set), //344 (0x158)
null_func,//BIND_SYSC(sys_memory_...), //345 (0x159)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing PPUFunction naming I suppose

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean?

Copy link
Contributor Author

@elad335 elad335 Jan 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function in PPUFunction.cpp has been removed a long time ago in favour of inlined naming in this array.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, I'm thinking of syscall list in PPU link table in PPUThread.cpp

Copy link
Contributor Author

@elad335 elad335 Jan 9, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's also fine because all "syscall_0"-"syscall_1023" names are linked by default.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Nekotekina Nekotekina merged commit 0946e59 into RPCS3:master Jan 9, 2023
@elad335 elad335 deleted the patch-11 branch May 6, 2024 17:06
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.

[Regression] (VSH) Building database crash (#13115)
6 participants