Mail Archives: cygwin/2010/02/02/10:16:36
On Tue, Feb 02, 2010 at 09:36:54AM +0000, Andrew West wrote:
>On 01/02/2010 21:24, Dave Korn wrote:
>> On 01/02/2010 20:45, Andrew wrote:
>>
>>
>>> I'm not looking to submit a patch to fix this, I'll leave that up to the
>>> professionals who have a better idea about the whole picture. It's just
>>> I've hit a brick wall with my code with this bug so I'm looking for some
>>> work arounds for myself.
>>>
>> No, really, you've been a ton of help, thanks a million. Try the patch I
>> just posted to the cygwin-patches list, on top of current CVS:
>>
>> http://cygwin.com/ml/cygwin-patches/2010-q1/msg00051.html
>>
>> That should get you going with your current DLLs.
>>
>> Next step is to add the cxx abi functions. I found an old patch lying
>> around, seems I started looking at this back in August and then lost track of
>> it somehow (probably in the rush approaching the end of gcc stage 1, I guess),
>> so I owe you apologies for the inconvenience.
>>
>> cheers,
>> DaveK
>>
>>
>
>O.k. I've changed remove_dll_atexit to;
>
>remove_dll_atexit (const dll *d)
> {
>
> unsigned char *dll_beg = (unsigned char *) d->handle + 0x1000;
> unsigned char *dll_end = (unsigned char *) d->p.data_start;
According to gdb, setting dll_end to this is functionally equivalent to
using AllocationBase + RegionSize. I thought you tried that already.
But, anyway...
>I tested my changes against my simple test case and against my full program and they
>both seems to work fine.
I've gotten rid of remove_dll_atexit and implemented Dave's other idea of
specifically calling cxa_atexit when atexit is called from a DLL. I realize
this morning that this might require some additional tweaking to preserve
the same order of calling but that's easily done.
I'm waiting for approval on a newlib patch before rolling a new snapshot
for this.
cgf
--
Problem reports: http://cygwin.com/problems.html
FAQ: http://cygwin.com/faq/
Documentation: http://cygwin.com/docs.html
Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple
- Raw text -