Skip to content

8276819: javax/print/PrintServiceLookup/FlushCustomClassLoader.java fails to free #1704

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
mrserb wants to merge 69 commits into from
Closed

8276819: javax/print/PrintServiceLookup/FlushCustomClassLoader.java fails to free #1704

mrserb wants to merge 69 commits into from

Conversation

@mrserb
Copy link
Member

@mrserb mrserb commented Aug 29, 2023 •
edited by openjdk bot
Loading

Hi all,

This pull request contains a backport of commit 7c2c5858 from the openjdk/jdk repository.

The commit being backported was authored by Sergey Bylokhov on 12 Dec 2021 and was reviewed by Prasanta Sadhukhan and Alexey Ivanov.

Thanks!

Progress

  • Change must be properly reviewed (1 review required, with at least 1 Reviewer)
  • Change must not contain extraneous whitespace
  • Commit message must refer to an issue

Integration blocker

 ⚠️ Dependency #1625 must be integrated first

Issue

  • JDK-8276819: javax/print/PrintServiceLookup/FlushCustomClassLoader.java fails to free (Bug - P4)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk17u-dev.git pull/1704/head:pull/1704
$ git checkout pull/1704

Update a local copy of the PR:
$ git checkout pull/1704
$ git pull https://git.openjdk.org/jdk17u-dev.git pull/1704/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 1704

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

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk17u-dev/pull/1704.diff

luchenlin and others added 30 commits July 26, 2023 06:17
@luchenlin @GoeLin
8307165: java/awt/dnd/NoFormatsDropTest/NoFormatsDropTest.java timed out
90202ef 
Backport-of: 14df5c130e5bf5139e3e2ef6d7b9d64883d94d18
@luchenlin @GoeLin
8270331: [TESTBUG] Error: Not a test or directory containing tests: j…
2a70aaa 
…ava/awt/print/PrinterJob/InitToBlack.java

Backport-of: 2ef9767aae8e1798661fea615b096833bdb9d985
@GoeLin
8289510: Improve test coverage for XPath Axes: namespace
9ed5671 
Backport-of: 3d254d3c8ea7f409e8f709413f80d88913eb0ce8
@GoeLin
8293177: Verify version numbers in legal files
89b4429 
Backport-of: 69ede5baeda6645aa3e961a02cbd40db965fc6a1
@GoeLin
8296384: [TESTBUG] sun/security/provider/SecureRandom/AbstractDrbg/Sp…
61442ce 
…ecTest.java intermittently timeout

Backport-of: 82561de722b9ca580c0c1a53050c711b64611352
@GoeLin
8299617: CurrencySymbols.properties is missing the copyright notice
7d29233 
Backport-of: d5b80abcbfff57c7728d3e42a696a762f08bc7ad
@GoeLin
8301367: Add exception handler method to the BaseLdapServer
2940973 
Backport-of: 03b23a1e1bd724019ef4fdbee3463d0646329164
@GoeLin
8307526: [JFR] Better handling of tampered JFR repository
aaf47b3 
Reviewed-by: lucy
Backport-of: 66d2736521611fbe7652356894d046c17d7cf743
@schmelter-sap
8289917: Metadata for regionsRefilled of G1EvacuationStatistics event…
ba17497 
… is wrong

Backport-of: 728157fa03913991088f6bb257a8bc16706792a9
@shipilev
8309746: Reconfigure check should include make/conf/version-numbers.conf
d44c446 
Reviewed-by: mbaesken
Backport-of: 955fc2fabafc4c4d79d137bf86194228d5cf8ad8
8307603: [AIX] Broken build after JDK-8307301
b4c8cc8 
Backport-of: bb3e44d8b604b4e3facc4f49f625cc0b7ff2abd7
@zifeihan @RealFYang
8311923: TestIRMatching.java fails on RISC-V
c486b9b 
Reviewed-by: phh, fyang
Backport-of: e7adbdb1f1506b82392907f7e4a5c8882d3198eb
@shipilev
8307766: Linux: Provide the option to override the timer slack
735e1da 
Backport-of: 7173c3009e0999f13eaa4bee5eedee7326f9d124
@shipilev
8273803: Zero: Handle "zero" variant in CommandLineOptionTest.java
482f224 
Backport-of: 2d13fb21af783ebbe3dd5c2632aea11347317026
@mrserb
8313576: GCC 7 reports compiler warning in bundled freetype 2.13.0
a5cdc22 
Backport-of: 8248e351d0bed263fb68d8468004a4286e6391af
@TheRealMDoerr
8289797: tools/launcher/I18NArgTest.java fails on Japanese Windows en…
6349de6 
…vironment

Backport-of: a11477ce259d6d2a053117a2300ebb171145cfde
@TheRealMDoerr
8195589: T6587786.java failed after JDK-8189997
2f5a952 
Backport-of: 21b62fef335aa981a239415532fa8b804fb5676d
@TheRealMDoerr
8267860: Off-by-one bug when searching arrays in AlpnGreaseTest
4786a00 
Backport-of: 2adef6a1f84d478bb38b179795f08ffa43680e36
@TheRealMDoerr
8309095: Remove UTF-8 character from TaskbarPositionTest.java
9c8731b 
Backport-of: 07f2070411b3bff5608166ceae0e0bdbf155e7c2
@RealCLanger
8302182: Update Public Suffix List to 88467c9
ec2abc2 
Backport-of: 01b85129116dd2cc762e518ac631305bd8511764
@mrserb
8283276: java/io/ObjectStreamClass/ObjectStreamClassCaching.java fail…
fe3920e 
…s with various GCs

Backport-of: d4a795d75aef8d787934f5c05e146c61138a408a
@TheRealMDoerr
8301167: Update VerifySignedJar to actually exercise and test verific…
8aad696 
…ation

Backport-of: 05ea083b0563ddacf3e38dc329ba00dc4bac9b29
@TheRealMDoerr
8307079: Update test java/awt/Choice/DragOffNoSelect.java
3d86810 
Backport-of: 89711f376751d4cfa05758705867afedfafeb602
@TheRealMDoerr
6381945: (cal) Japanese calendar unit test system should avoid multip…
242513f 
…le static imports

Backport-of: f79b3d42f07b703f0e3b9fc67c92dee260b0e602
@TheRealMDoerr
8255548: Missing coverage for javax.xml.crypto.dom.DOMCryptoContext
fe9b7c6 
Backport-of: 76cda9f44a80b1979e6e1b7a21431ef631f80782
@tstuefe
8273092: Sort classlist in JDK image
762c351 
Reviewed-by: aph
Backport-of: 1996f649a3a30b7ac4b547a762417f807f5fa414
@TheRealMDoerr
8289646: configure script failed on WSL
38204b0 
Backport-of: 3f1174aa4709aabcfde8b40deec88b8ed466cc06
@TheRealMDoerr
8308232: nsk/jdb tests don't pass -verbose flag to the debuggee
8683db8 
Backport-of: c6f20db945c6217aea84cebd6c97dbf8b93c48a4
@TheRealMDoerr
8156889: ListKeychainStore.sh fails in some virtualized environments
10942c1 
Backport-of: 82c8b28f4005c3c335d4fd60c2e3b55adfd200c6
@TheRealMDoerr
8308090: Add container tests for on-the-fly resource quota updates
6ab5177 
Backport-of: dc8bc6c98ca1f9b441cf71c641675fe29dda9162
shipilev and others added 21 commits August 21, 2023 09:14
@shipilev
8284273: Early crashes in os::print_context on AArch64
d725961 
Reviewed-by: stuefe
Backport-of: 83cec4b6cc25b9b0db751bbe873444259cdcb21c
@shipilev
8309591: Socket.setOption(TCP_QUICKACK) uses wrong level
78bbd57 
Reviewed-by: phh
Backport-of: 56a73a6f0f3d38379cecea1de5eacb751febca95
@feilongjiang @RealFYang
8314117: RISC-V: Incorrect VMReg encoding in RISCV64Frame.java
74a12fc 
Backport-of: 5c9162288570a140138a0055cd9c4e88fe40e69d
@RealFYang
8310873: Re-enable locked_create_entry symbol check in runtime/NMT/Ch…
994f16f 
…eckForProperDetailStackTrace.java for RISC-V

Backport-of: 87e6fab2c498558d4593f6c0537c998a61779aaf
@shipilev
8314658: [17u] GHA: Sync up debian-version for cross-builds
3f8e995 
Reviewed-by: phh
@shipilev
8293851: hs_err should print more stack in hex dump
d35a896 
Backport-of: cbd0688b321ff88a405be4f7929d6862e543ab50
@shipilev
8314501: Shenandoah: sun/tools/jhsdb/heapconfig/JMapHeapConfigTest.ja…
49ffbcf 
…va fails

Backport-of: 812f475bc4ea84225e8bbb0b5a677eed0af864dd
@lutkerd
8314552: Fix javadoc tests to work with jtreg 7
e830464 
Reviewed-by: phh, shade, andrew
@TheRealMDoerr
8299827: Add resolved IP address in connection exception for sockets
40add10 
Backport-of: 92d8326e4037605897d7c4eb4b3edb63a2fc11b0
@shipilev
8283724: Incorrect description for jtreg-failure-handler option
e070c31 
Backport-of: 0901548833a0125f15fede64bc2e7dbe84fed42d
@shipilev
8313701: GHA: RISC-V should use the official repository for bootstrap
e84d9dd 
Reviewed-by: phh
Backport-of: 538f9557b87f750264231f04bfbc91d15f8af8c0
@shipilev
8314730: GHA: Drop libfreetype6-dev transitional package in favor of …
d9985d0 
…libfreetype-dev

Reviewed-by: stuefe
Backport-of: 69d900d2ce97e5479020cff9a63c471d07e39989
@MBaesken
8276651: java/lang/ProcessHandle tests fail with "RuntimeException: I…
1421540 
…nput/output error" in java.lang.ProcessHandleImpl$Info.info0

Backport-of: d24b7b7026cf85f1aecf44f60819762872cfd5c1
@shipilev
8293114: JVM should trim the native heap
f667b35 
Reviewed-by: stuefe
Backport-of: 9e4fc568a6f1a93c84a84d6cc5220c6eb4e546a5
@shipilev
8312525: New test runtime/os/TestTrimNative.java#trimNative is failin…
3c426c7 
…g: did not see the expected RSS reduction

Reviewed-by: mdoerr
Backport-of: ad34be1f329edc8e7155983835cc70d733c014b8
@shipilev
8314426: runtime/os/TestTrimNative.java is failing on slow machines
fdc4130 
Reviewed-by: stuefe
Backport-of: 20e94784c9f7c30e95550c72aedb5e986a153114
@yukikimmura
8313815: The exception messages printed by jcmd ManagementAgent.start…
7286bb8 
… are corrupted on Japanese Windows

Reviewed-by: kevinw, cjplummer, phh
8295737: macOS: Print content cut off when width > height with portra…
4c123ac 
…it orientation

Backport-of: d00a767047ec41e233e711dbc5fe7b8818e72f28
8311033: [macos] PrinterJob does not take into account Sides attribute
47ed2e5 
Backport-of: a3d67231a71fbe37c509fcedd54c679b4644c0d9
@gnu-andrew
8284772: GHA: Use GCC Major Version Dependencies Only
1d65a31 
Reviewed-by: shade
Backport-of: 62defc3dfc4b9ba5adfe3189f34fe8b3f59b94a0
Backport 7c2c58587d4eda5523331eae45e7d897252dc097
b01bf8e
@bridgekeeper
Copy link

bridgekeeper bot commented Aug 29, 2023

👋 Welcome back serb! A progress list of the required criteria for merging this PR into pr/1625 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 changed the title Backport 7c2c58587d4eda5523331eae45e7d897252dc097 8276819: javax/print/PrintServiceLookup/FlushCustomClassLoader.java fails to free Aug 29, 2023
@openjdk
Copy link

openjdk bot commented Aug 29, 2023

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

@openjdk openjdk bot added backport Port of a pull request already in a different code base clean Identical backport; no merge resolution required labels Aug 29, 2023
@openjdk openjdk bot removed the clean Identical backport; no merge resolution required label Aug 29, 2023
@openjdk
Copy link

openjdk bot commented Aug 29, 2023

⚠️ @mrserb This pull request contains merges that bring in commits not present in the target repository. Since this is not a "merge style" pull request, these changes will be squashed when this pull request in integrated. If this is your intention, then please ignore this message. If you want to preserve the commit structure, you must change the title of this pull request to Merge <project>:<branch> where <project> is the name of another project in the OpenJDK organization (for example Merge jdk:master).

@mrserb mrserb closed this Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

backport Port of a pull request already in a different code base

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.