public inbox for java-prs@sourceware.org
help / color / mirror / Atom feed
From: "nwourms at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: java-prs@gcc.gnu.org
Subject: [Bug libgcj/50895] Build failure in jni.cc
Date: Thu, 03 Jan 2013 03:03:00 -0000	[thread overview]
Message-ID: <bug-50895-8172-GCKy8jsLq7@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-50895-8172@http.gcc.gnu.org/bugzilla/>


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50895

Nicholas Wourms <nwourms at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nwourms at gmail dot com

--- Comment #6 from Nicholas Wourms <nwourms at gmail dot com> 2013-01-03 03:03:13 UTC ---
(In reply to comment #4)
> A question about this build-failure.  Were you using posix-threading model for
> 4.6 ?

I can't speak for Ruben, but now that thread models have been inextricably
entwined into the core compiler, libjava really should be updated to handle
mingw64's posix thread support. It shouldn't be too difficult, but does require
some unraveling of the various Posix/Win32 native functions.

I think the main problem is the lack of any decent signal handling in win32. As
such, files such as:

libjava/java/lang/natWin32Process.cc
libjava/gnu/java/nio/natVMSelectorWin32.cc

seem to involve event handling that is linked to Win32 threading model,
specifically using the _Jv_Win32GetInterruptEvent() function. It appears that
libjava/win32-threads.cc also has some similarities to the underlying
pthread_cond* implementation in mingw-w64's pthreads library, so in theory it
could just be a matter of porting the interrupt event handlers to
posix-threads.cc to provide ifdef'd alternates to the signal handlers.

Alternatively, Kai and friends at mingw-w64 could implement improved signal
handling in librt, but that may be defeating the purpose of mingw.


  parent reply	other threads:[~2013-01-03  3:03 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-28  8:34 [Bug libgcj/50895] New: " vanboxem.ruben at gmail dot com
2012-01-23 23:45 ` [Bug libgcj/50895] " ktietz at gcc dot gnu.org
2012-01-24 15:29 ` vanboxem.ruben at gmail dot com
2012-02-14 20:23 ` ktietz at gcc dot gnu.org
2012-12-10 11:21 ` ktietz at gcc dot gnu.org
2012-12-10 12:32 ` vanboxem.ruben at gmail dot com
2013-01-03  3:03 ` nwourms at gmail dot com [this message]
2013-12-17 19:44 ` ktietz at gcc dot gnu.org
2014-02-16 13:15 ` jackie.rosen at hushmail dot com

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-50895-8172-GCKy8jsLq7@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=java-prs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).