Fix for HDD size calculation, new poweroff processing mechanism, fixed PBUF allocation for Ethernet ARP request, new option for SMSTCPIP to save more RAM. #44
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… align with the official documentation; size is now in 512-byte HDD sectors).
…changed SMAPSendPacket to assume that all PBUFs are aligned. etharp_query requests PBUF_LINK from pbuf_alloc(), but does not reveal the Ethernet header. It is possible that this also writes beyond the end of the PBUF.
…before power off.
…s like the official option from newer LWIP versions.
|
Hi SP193, then you fixed the slight problem you mentioned before? all is ok? after this commit is needed update the PS2SDK? |
|
The "slight" problem that I saw, was in ps2ip_mem_realloc(). With its current design, it does not actually function in the same way as the standard realloc() function. Whereby it does not attempt to allocate new memory, if the specified region of memory cannot be resized. However, it seems like mem_realloc() - the function which ps2ip_mem_realloc() is used to implement, is only used by pbuf_realloc(). pbuf_realloc will only shrink the size of the allocated buffer with mem_realloc(), so I really don't think that ps2ip_mem_realloc() can possibly fail. Hence the only thing that I changed between these two commits, is to add a comment about this. There is actually no problem. :) There are no accompanying updates to the PS2SDK for this pull request. |
|
Thank you very much for the explanation and the work, and have a good weekend :) |
|
Thanks. Have a great weekend too! :D
|
Jay-Jay-OPL
added a commit
to Jay-Jay-OPL/OPL-Daily-Builds
that referenced
this pull request
Mar 5, 2017
Fix for HDD size calculation, new poweroff processing mechanism, fixed PBUF allocation for Ethernet ARP request, new option for SMSTCPIP to save more RAM. (ps2homebrew#44) * Corrected HDD partition size calculation (change in homebrew specs to align with the official documentation; size is now in 512-byte HDD sectors). * (ingame SMSTCPIP) Fixed pbuf allocation for Ethernet ARP request and changed SMAPSendPacket to assume that all PBUFs are aligned. etharp_query requests PBUF_LINK from pbuf_alloc(), but does not reveal the Ethernet header. It is possible that this also writes beyond the end of the PBUF. * Changed GUI power-off to deinitialize the UI and to switch off DEV9, before power off. * Added new MEM_LIBC_MALLOC option to mem.c and enabled it. It functions like the official option from newer LWIP versions.
AKuHAK
pushed a commit
that referenced
this pull request
Sep 30, 2021
…d PBUF allocation for Ethernet ARP request, new option for SMSTCPIP to save more RAM. (#44) * Corrected HDD partition size calculation (change in homebrew specs to align with the official documentation; size is now in 512-byte HDD sectors). * (ingame SMSTCPIP) Fixed pbuf allocation for Ethernet ARP request and changed SMAPSendPacket to assume that all PBUFs are aligned. etharp_query requests PBUF_LINK from pbuf_alloc(), but does not reveal the Ethernet header. It is possible that this also writes beyond the end of the PBUF. * Changed GUI power-off to deinitialize the UI and to switch off DEV9, before power off. * Added new MEM_LIBC_MALLOC option to mem.c and enabled it. It functions like the official option from newer LWIP versions.
citronalco
pushed a commit
to citronalco/OPL-Daily-Builds
that referenced
this pull request
Sep 10, 2023
…d PBUF allocation for Ethernet ARP request, new option for SMSTCPIP to save more RAM. (ps2homebrew#44) * Corrected HDD partition size calculation (change in homebrew specs to align with the official documentation; size is now in 512-byte HDD sectors). * (ingame SMSTCPIP) Fixed pbuf allocation for Ethernet ARP request and changed SMAPSendPacket to assume that all PBUFs are aligned. etharp_query requests PBUF_LINK from pbuf_alloc(), but does not reveal the Ethernet header. It is possible that this also writes beyond the end of the PBUF. * Changed GUI power-off to deinitialize the UI and to switch off DEV9, before power off. * Added new MEM_LIBC_MALLOC option to mem.c and enabled it. It functions like the official option from newer LWIP versions.
This pull request was closed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request checklist
Note: these are not necessarily requirements
Pull Request description
45e99f0 - Fixed HDD partition size calculation. Originally, our HDD.IRX module used to return the partition size in bytes. Eventually, that got changed to fit the official documentation, which returns in 512-byte sectors.
8cadeb4 - Corrected bug in PBUF allocation for Ethernet ARP requests and changed the SMAP output function to always assume that outgoing PBUFs are always aligned (for performance). Previously, the outgoing PBUFs were occasionally unaligned because of this glitch.
4a9a5a7 - As of the recent PS2SDK commit, poweroff no longer automatically switches off DEV9 as a safety measure. OPL will now deinitialize the GUI before switching off DEV9, before finally switching off the PS2.
13d1347 - Our SMSTCPIP module is based on an ancient version of LWIP, which always uses its own memory heap allocator (even though SYSMEM can fill that requirement). This patch adds and enables a new option (MEM_LIBC_MALLOC), which functions like the official version from the newer versions of LWIP.