Uploaded image for project: 'JikesRVM'
  1. JikesRVM
  2. RVM-566

JNI: Implement AttachCurrentThread/DetachCurrentThread

    Details

    • Type: New Feature
    • Status: Open
    • Priority: Medium
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 3.2
    • Component/s: Runtime: JNI
    • Labels:
      None

      Description

      We need to implement the following JNI Invocation API Functions:
      AttachCurrentThread, DetachCurrentThread, and AttachCurrentThreadAsDaemon.

        Attachments

          Issue Links

            Activity

            Hide
            ebrangs Erik Brangs added a comment -

            Da Feng's x64 fixes include changes to the JNI Functions FindClass and DefineClass that ensure that a proper classloader is returned to JNI even for threads that were newly attached via JNI. We cannot test this without an implementation of the functions for attaching and detaching threads.

            This makes this issue a transitive dependency of Da Feng's fixes for x64 if we don't want to separate the FindClass/DefineClass bugfix from the x64 patch set.

            There is an implementation of thread attaching and detaching in MRP-110 but it was not part of Da Feng's x64 patch set for the RVM. Da Feng has a contributor statement for MRP (see MRP-43, readmeAndCLa.tar.gz) so I suppose that it would be legally ok for us to use that code.

            Show
            ebrangs Erik Brangs added a comment - Da Feng's x64 fixes include changes to the JNI Functions FindClass and DefineClass that ensure that a proper classloader is returned to JNI even for threads that were newly attached via JNI. We cannot test this without an implementation of the functions for attaching and detaching threads. This makes this issue a transitive dependency of Da Feng's fixes for x64 if we don't want to separate the FindClass/DefineClass bugfix from the x64 patch set. There is an implementation of thread attaching and detaching in MRP-110 but it was not part of Da Feng's x64 patch set for the RVM. Da Feng has a contributor statement for MRP (see MRP-43, readmeAndCLa.tar.gz) so I suppose that it would be legally ok for us to use that code.

              People

              • Assignee:
                Unassigned
                Reporter:
                dgrove David Grove
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated: