-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
VSH Improvements #13172
Conversation
Please stop mixing GUI stuff with HLE bugfixes |
@Darkhost1999 Please retest. |
RPCS3.log |
Working well for me |
@@ -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) |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you mean?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These fixes allow to proceed further along into game launching within XMB.
Fixes #13154