This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA/JVMTI] Fix a couple of thinkos


>>>>> "Keith" == Keith Seitz <keiths@redhat.com> writes:

Keith> First, the JVMTI namespace I
Keith> introduced to jvmti-int.h for event processing must have all the
Keith> members declared extern (and subsequently defined in one place).

This part is ok.

Keith> Second, we cannot simply lock the environment list in
Keith> _Jv_JVMTI_PostEvent. It would prevent another event from posting an
Keith> event because the lock would still be held by the suspended thread.

Hmm, can't we now dispose a jvmti environment while an event is being
posted?  Perhaps leading to something bad?

Keith> +  /* Don't even think about locking the env list here. It will prevent
Keith> +     any events from being posted if some thread suspended as a result
Keith> +     of a previous event. */

We may have to defer manipulations of the env list until it is safe.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]