RVM
  1. RVM
  2. RVM-50

Remove synchronization in Field/MemberReference resolve

    Details

    • Type: Bug Bug
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 2.9.0.1
    • Fix Version/s: 1000
    • Component/s: Runtime
    • Labels:
      None
    • Number of attachments :
      0

      Description

      Commit 11568 made VM_FieldReference.resolve and VM_MethodReference.resolve
      into synchronized methods. I completely agree with the rationale of getting
      stability. I'm just opening this feature to remind us that we need to
      revisit this at a later point and try to get the synchronization back out
      of the common case.

      http://svn.sourceforge.net/jikesrvm/?rev=11568&view=rev

        Issue Links

          Activity

          Hide
          Peter Donald added a comment -

          This issue just seems to be hiding a symptom of underlying problem. Ideally I would like to see this removed earlier rather than later as I suspect it is related to some of the other issues we are experiencing.

          Show
          Peter Donald added a comment - This issue just seems to be hiding a symptom of underlying problem. Ideally I would like to see this removed earlier rather than later as I suspect it is related to some of the other issues we are experiencing.
          Peter Donald made changes -
          Field Original Value New Value
          Fix Version/s 2.9.1 [ 13529 ]
          Fix Version/s 2.9.2 [ 13599 ]
          David Grove made changes -
          Fix Version/s 2.9.3 [ 13726 ]
          Fix Version/s 2.9.2 [ 13599 ]
          Ian Rogers made changes -
          Link This issue is related to RVM-205 [ RVM-205 ]
          David Grove made changes -
          Fix Version/s 2.9.4 [ 14162 ]
          Fix Version/s 2.9.3 [ 13726 ]
          Hide
          Ian Rogers added a comment -

          The functionality we have is correct, moving this issue to a later release.

          Show
          Ian Rogers added a comment - The functionality we have is correct, moving this issue to a later release.
          Ian Rogers made changes -
          Fix Version/s 1000 [ 14184 ]
          Fix Version/s 2.9.4 [ 14162 ]

            People

            • Assignee:
              Unassigned
              Reporter:
              Peter Donald
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated: