[Gc] Re[2]: gcj 4.6 on OpenBSD/x86

[Ivan Maidanski]

 Hi Kurt,

1. One thing is unclear to me (from the patch): OpenBSD provides (or provided) nice way to suspend/getstate/resume threads, your patch disables it (for recent OpenBSD) in favor of emulation of these operations via signals which has drawbacks (like hard-coding signal numbers, etc.) but inevitable on most pthread-based platforms. What's the reason?

2. There are 2 independent part in your patch: first is for dyn_load.c and for the rest, right? (if yes, I'll break it into to commits)

3. Reason of adding include <sys/param.h> to gc_config_macros.h is to define OpenBSD, right?

4. Some commit message is highly appreciated

5. I don't think we should apply to 7.2d (this seems to be not a fix)

The rest looks good (I haven't tested it on OpenBSD though).

Regards,
Ivan

Fri, 31 May 2013, 15:30 -04:00 from Kurt Miller <kurt at intricatesoftware.com>:
>Hi Ivan,
>
>Sorry about the long delay. Attached please find a diff that update's
>OpenBSD's support to handle our kernel based threads since 5.2 release
>and maintains the prior user-land threads code for pre 5.2.
>
>A similar diff for 7.2d has been tested on the following OpenBSD
>architectures: i386, amd64, sparc64, macppc, hppa, alpha.
>
>Please consider incorporating this diff into your git repository.
>
>Thank you,
>-Kurt
>
>
>On 10/23/12 15:24, Ivan Maidanski wrote:
>> Hi Kurt,
>> 
>> I typically try to care about compatibility with old versions of targets
>> at least at source level.
>> If you know that a certain change, would break support of old OpenBSD
>> releases, please put the code into #ifdef block.
>> But, of course, it is not needed to retest the code on that old OS
>> releases. And, it's acceptable to assume newer OpenBSD by default.
>> 
>> Thank you.
>> 
>> Regards,
>> Ivan
>> 
>> Tue, 23 Oct 2012 13:11:01 -0400 Kurt Miller < kurt at intricatesoftware.com >:
>> 
>>     Thanks. I have the current source tree. Question for you. Do you
>>     care to support OpenBSD < 5.2 release or can we set the minimum
>>     supported version to 5.2?
>> 
>>     In 5.2 we elminiated the userland threads model and replaced it with
>>     a 1-1 kernel supported thread model. We also eliminated some
>>     non-portable functions in 5.2. My preference is to just support 5.2
>>     and up since userland thread model is gone for good and will not be
>>     mantained by any OpenBSD developer now. i.e. pre 5.2 support will
>>     code rot due to lack of testing.
>> 
>>     Thanks,
>>     -Kurt
>> 
>>     On Friday 19 October 2012 04:32:09 pm Ivan Maidanski wrote:
>>     > Hi Kurt,
>>     >
>>     > Please checkout master from  https://github.com/ivmai/bdwgc and
>>     make a pull request.
>>     > Thank you.
>>     >
>>     > Regards,
>>     > Ivan
>>     >
>>     > Tue, 16 Oct 2012 08:44:19 -0400 Kurt Miller
>>     < kurt at intricatesoftware.com
>>     <sentmsg?compose&To=kurt at intricatesoftware.com>>:
>>     > >
>>     > >
>>     > >
>>     >
>>     >
>>     > >
>>     >
>>     >
>>     >
>>     > >Most likely yes. When I originally worked on thread support for
>>     gc on OpenBSD we had user-threads and I used gross hacks to get the
>>     stack pointer. Now that OpenBSD has kernel supported threads, I have
>>     removed the hacks and the patches to support OpenBSD are clean.
>>     > >
>>     > >
>>     > I can port them to upstream now. Is this the correct place for
>>     upstream code?
>>     > >
>>     > > http://www.hpl.hp.com/personal/Hans_Boehm/gc/
>>     > >
>>     > >
>>     > Thanks,
>>     > >
>>     > -Kurt
>>     > >
>>     > >
>>     > On Oct 16, 2012, at 1:44 AM, "Boehm, Hans" < hans.boehm at hp.com
>>     <sentmsg?compose&To=hans.boehm at hp.com>> wrote:
>>     > >
>>     > >
>>     > > Thanks!
>>     > >
>>     > >
>>     > >
>>     > > Have you checked whether the problem also exists in the upstream
>>     version?
>>     > >
>>     > >
>>     > >
>>     > > Hans
>>     > >
>>     > >
>>     > >
>>     > >> -----Original Message-----
>>     > >
>>     > >> From: Kurt Miller [mailto:kurt at intricatesoftware.com]
>>     > >
>>     > >> Sent: Sunday, October 14, 2012 4:27 AM
>>     > >
>>     > >> To: Boehm, Hans
>>     > >
>>     > >> Cc:  java at gcc.gnu.org <sentmsg?compose&To=java at gcc.gnu.org>
>>     > >
>>     > >> Subject: Re: gcj 4.6 on OpenBSD/x86
>>     > >
>>     > >>
>>     > >
>>     > >> I found the root cause was OpenBSD was missing a #define for
>>     > >
>>     > >> HAVE_DL_ITERATE_PHDR. gcj is working now on both x86/x86-64
>>     > >
>>     > >> on OpenBSD. Thanks for the help.
>>     > >
>>     > >>
>>     > >
>>     > >> -Kurt
>>     > >
>>     > >
>>     > >
>>     >
>>     >
>>     >
>>     >
>>     >
>>     >
>>     >
>> 
>> 
>> 
>
>
>_______________________________________________
>Gc mailing list
>Gc at linux.hpl.hp.com
>http://www.hpl.hp.com/hosted/linux/mail-archives/gc/

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://napali.hpl.hp.com/pipermail/gc/attachments/20130601/340755aa/attachment.htm