-
-
Notifications
You must be signed in to change notification settings - Fork 459
Overhaul Address #2226
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
Overhaul Address #2226
Conversation
|
Can you also add this: } else if (vc->retinfo->vtype == Valve_VirtualAddress) {
void *addr = (void *)vc->retbuf;
return g_pSM->ToPseudoAddress(addr);
} else {to here sourcemod/extensions/sdktools/vcaller.cpp Lines 532 to 545 in b71d3c0
This would allow things like calling |
151ec78 to
7f8dfd6
Compare
4d69f79 to
ce846d4
Compare
|
I think this is great, and indeed much better than the define. |
ab34ad7 to
dffd568
Compare
dffd568 to
021ed61
Compare
|
A big thanks to @Malifox for taking the first step into trying this PR with a well established plugin, which helped me hunt down a ton of bugs. |
In the current ongoing effort for sourcemod to fully support 64 bits, we are introducing "virtual address". # Explanation Because SourcePawn does not yet support a 64 bits-wide type it's been impossible for any plugins to hold addresses in regular 32-bits wide variable. A first attempt at solving this issue was made in commit ce1a4dc therein dubbed "PseudoAddress", however this turned out to be an unsatisfactory solution, as any 'high' address if offsetted could turn invalid (or outright be impossible to map). This leaves us with three alternatives : - New type - Convert Address into a handle - Virtual Address A new type is the most destructive solution, as it entails breaking every single Address related method. While that solution is still not off the table, we're reserving it as the last attempt should this commit fail. Converting into a handle type is a good compromise between a brand new type whilst also preserving the Address methods. However, this comes with two issues: the first being that you can no longer offset Address, the second is that we would require authors to free the handle type which will be very confusing. This will likely not be implemented. # Virtual address Under a reasonable assumption, we've noted that the average plugin is unlikely to play with more than 4 GB of memory; this shouldn't be too surprising as all valve games were once 32bits and therefore limited to 4GB. Assuming this stays mostly true and a plugin isn't interested with the mapped memory of lesser known modules (like soundlib or matlib), it is fair to assume plugins are unlikely to access more than 4GB of mapped memory. Working with this in mind, we map the memory the plugins are likely to access to our custom virtual address ranges (from 0 to 4Gb, the values of which can fit on 32bits variable). If any memory was missed and plugins were to try an access it later those ranges will be late-mapped to our virtual address ranges until we run out of them. In order to use virtual addressing, whether on 32 bits or 64 bits. Plugins must now "#include <virtual_address>", as well as use the new SDKCall_VirtualAddress, SDKType_VirtualAddress, LoadAddressFromAddress & StoreAddressToAddress where it's appropriate to.
Another Address PR!
The idea and implementation is primarily @dvander's, this PR just makes the bridge to SM.
In order to use the new virtual address :
If enabled, plugins need to ensure their SDKCalls are using the new
SDKType_VirtualAddress, whether they're on 32bits or 64bits. Otherwise the plugin will most definitively crash the server.Marking as draft for now until I can get a few server operators to try this out. Extra natives idea are welcomed !