Skip to content
This repository has been archived by the owner on Feb 2, 2023. It is now read-only.

8216557: Aarch64: Add support for Concurrent Class Unloading #282

Closed
wants to merge 2 commits into from
Closed

8216557: Aarch64: Add support for Concurrent Class Unloading #282

wants to merge 2 commits into from

Conversation

snazarkin
Copy link
Contributor

@snazarkin snazarkin commented Oct 23, 2021

I'd like to backport feature implemented for x86 platform but missed for aarch64. The patch doesn't not apply cleanly for following files:

  1. 8234562 was not backported yet, so release_store/load_acquire are part of OrderAccess class
  2. 8234737 is missed, so Atomic::add use old function signature
  3. macroAssembler_aarch64.hpp copyright year
  4. sharedRutime.cpp contains 8253375 that confuses patcher not expected it

Tested with hotspot tier1/2


Progress

  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue
  • Change must be properly reviewed

Issue

  • JDK-8216557: Aarch64: Add support for Concurrent Class Unloading

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/jdk13u-dev pull/282/head:pull/282
$ git checkout pull/282

Update a local copy of the PR:
$ git checkout pull/282
$ git pull https://git.openjdk.java.net/jdk13u-dev pull/282/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 282

View PR using the GUI difftool:
$ git pr show -t 282

Using diff file

Download this PR as a diff file:
https://git.openjdk.java.net/jdk13u-dev/pull/282.diff

@bridgekeeper
Copy link

bridgekeeper bot commented Oct 23, 2021

👋 Welcome back snazarki! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@openjdk openjdk bot added the rfr Pull request is ready for review label Oct 23, 2021
@snazarkin snazarkin changed the title 8216557 backport Backport 739e8e322d568ab051a905c6369f4b6280f2ee22 Oct 23, 2021
@openjdk openjdk bot changed the title Backport 739e8e322d568ab051a905c6369f4b6280f2ee22 8216557: Aarch64: Add support for Concurrent Class Unloading Oct 23, 2021
@openjdk
Copy link

openjdk bot commented Oct 23, 2021

This backport pull request has now been updated with issue from the original commit.

@openjdk openjdk bot added the backport label Oct 23, 2021
@mlbridge
Copy link

mlbridge bot commented Oct 23, 2021

Webrevs

@snazarkin snazarkin closed this Mar 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport rfr Pull request is ready for review
1 participant