Problems with: Improvements to fork handling (2/5)
Sat May 28 20:50:00 GMT 2011
On Wed, May 11, 2011 at 02:31:37PM -0400, Ryan Johnson wrote:
>This patch has the parent sort its dll list topologically by
>dependencies. Previously, attempts to load a DLL_LOAD dll risked pulling
>in dependencies automatically, and the latter would then not benefit
>from the code which "encourages" them to land in the right places. The
>dependency tracking is achieved using a simple class which allows to
>introspect a mapped dll image and pull out the dependencies it lists.
>The code currently rebuilds the dependency list at every fork rather
>than attempt to update it properly as modules are loaded and unloaded.
>Note that the topsort optimization affects only cygwin dlls, so any
>windows dlls which are pulled in dynamically (directly or indirectly)
>will still impose the usual risk of address space clobbers.
I applied this patch and the one after it but then noticed that zsh started
producing: "bad address: " errors.
path:4: bad address: /share/bin/dopath
term:1: bad address: /bin/tee
The errors disappear when I back this patch out.
FWIW, I was running "zsh -l". I have somewhat complicated
.zshrc/.zlogin/.zshenv files. I'll post them if needed.
Until this is fixed, this patch and the subsequent ones which rely on
it, can't go in. I did commit this fix but it has been backed out now.
More information about the Cygwin-patches