- there are additional fields for glob_t struct thus size check is failing.
- to access old mman.h api based on caddr_t, _XOPEN_SOURCE needs not to be defined.
- prxmap_t constified.
- rGc61dcb8f623e: [compiler-rt] fix build on Illumos
Thanks for doing this. It's a first step, but according to a quick test some time ago, much more will be needed. I expect there will be follow-up
patches to address those issues (missing support for ld --version-script, unconditional use of ld --as-needed, probably more)?
One point up front: please be more detailed and specific about which failures you're dealing with here. E.g. the comment about
old mman.h api based on caddr_t
is simply wrong: apart from failing to mention the affected function (madvise in my testing on OpenIndiana 2020.04), the problem is that
madvise isn't declared at all if XOPEN_SOURCE is defined.
Solaris 10 had the same problem, but at some point int the Solaris 11 cycle the prototype was made visible if __EXTENSIONS__ is defined,
just as expected. Independent of the current patch Illumos may want to do this, too.
Please remove the "non posix" part: POSIX always allows for
I guess "There are additional fields that we are not interested in."
If this goes in at all, it needs to be before the various sanitizer_*.h includes above: they can include system headers, too.
In fact, I'm quite unhappy about this: it seems a terribly large hammer to get just a single missing declaration, and I fear there will be undesired side effects now or in the future.
It seems way easier to just declare madvise like
#if SANITIZER_SOLARIS // Illumos' declaration of madvise cannot be made visible if _XOPEN_SOURCE is // defined as g++ does on Solaris. extern "C" int madvise(void *, size_t, int); #endif
just before ReleaseMemoryPagesToOS. This works on both Illumos and Solaris and the risk is far lower.
AFAIK there's currently no way to distinguish between Illumos and Solaris at compile time right now. That would allow introduction of code that's needed by one but harms the other.
Unfortunately I hadn't noticed that you'd silently changed the madvise prototype from the version I'd suggested (and tested on both Solaris 11.4 and OpenIndiana) to one using caddr_t instead of void * for the first arg, this way breaking the Solaris build. Now fixed in D84388.
Please don't do something like this: it just creates unnecessary work.
This is exactly the reason why I asked for a way to distinguish between Illumos and Solaris at compile time (e.g. by predefining __illumos__).
Fixing one by breaking the other just isn't an option.