Skip to content

Conversation

@GrayJack
Copy link
Contributor

@GrayJack GrayJack commented Sep 5, 2025

Update the SceModule definition according to RE done on uofw.

According to some talk on discord, this type definition never changed between firmware versions. So this PR also fixes a bug where there should be 160 bits between stub_size and entry_addr fields, but there was 128 bits between those two fields.

@sharkwouter
Copy link
Member

Hey @GrayJack thanks for the PR. It seems there are still some conflicts in the code. I was not able to build it.

@GrayJack
Copy link
Contributor Author

GrayJack commented Sep 9, 2025

Oops, there was a silly type error, I updated it

Avoids type definition conflicts.
@sharkwouter
Copy link
Member

Just to let you know, I have not forgotten about this, but I don't know if I'll be able to test this this week. Next week I'll be able to test and merge this, though.

Copy link
Member

@sharkwouter sharkwouter left a comment

Choose a reason for hiding this comment

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

This looks fine to me, I'll ask in Discord one more time. I was able to build this and build and run OceanPop with this change.

@JoseAaronLopezGarcia
Copy link
Contributor

This looks fine to me, I'll ask in Discord one more time. I was able to build this and build and run OceanPop with this change.

If it's of any help (when it comes to taking a decision): I've personally reviewed this myself and I'm sure it's correct. The SceModule structure is essential for CFW development (and kernel-code in general), and it had inconsistencies, some of which were dangerous (a missing field made other fields not align with reality). The SDK has never updated this structure, forcing CFW (and other kernel-level devs) to use an alternative SceModule2. With this fix we can finally have a unified structure given to us by the kernel itself.

@sharkwouter sharkwouter merged commit c84807b into pspdev:master Sep 15, 2025
1 check passed
@sharkwouter
Copy link
Member

That helps! Thanks for the input @JoseAaronLopezGarcia :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants