On Mon, 2 Oct 2023 23:12:24 GMT, Brian Burkhalter wrote:
> remove stripLongOrUNCPrefix() except in normalize()
I haven't had time to look at the test changes yet but the I think the code
changes look okay so I think go ahead and create the CSR as this behavior
change will need to be tracked.
On Tue, 3 Oct 2023 02:19:00 GMT, Jiangli Zhou wrote:
> Please help review this trivial change that adds missing
> DEF_STATIC_JNI_OnLoad for librmi.
>
> Thanks
Surprised this one didn't have DEF_STATIC_JNI_OnLoad already. Change looks
okay, can you update the copyright date before
On Thu, 21 Sep 2023 00:46:45 GMT, Brian Burkhalter wrote:
>> Brian Burkhalter has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> 8287843: Move prefix stripping to separate method; add to isAbsolute
>
> Support for `isAbsolute` was added.
On Fri, 29 Sep 2023 05:41:11 GMT, Jaikiran Pai wrote:
> Can I please get a review of this change which removes unused (internal)
> method from the `private` `URLClassPath$JarLoader`?
>
> The `validIndex` method which is being removed here was being used when JAR
> index was supported. We
On 28/09/2023 17:21, Егор Зиборов wrote:
Hello, everyone
I'm new there and writing this letter during advice of Dalibor Topic
I faced an issue with SOE in ThreadLocal and want to add this java9 annotation
on ThreadLocal#set. Does anyone have any concerns about it?
This annotation is for very
On Wed, 27 Sep 2023 16:50:33 GMT, Jorn Vernee wrote:
>> This patch contains the implementation of the foreign linker & memory API
>> JEP for Java 22. The initial patch is composed of commits brought over
>> directly from the [panama-foreign
>> repo](https://github.com/openjdk/panama-foreign).
On Wed, 27 Sep 2023 17:29:29 GMT, Maurizio Cimadamore
wrote:
>> This patch adds a new internal annotation that is used to mark all
>> restricted me
>> thods in the FFM API. The new annotation is similar to the one we used for
>> preview API methods.
>>
>> We plan to use the new annotation
On Wed, 27 Sep 2023 16:19:04 GMT, Brian Burkhalter wrote:
>> On Windows, do not return `true` from the `java.io.File` methods
>> `setReadable(boolean, boolean)` and `setExecutable(boolean, boolean)` if the
>> file does not exist.
>
> Brian Burkhalter has updated the pull request incrementally
On Wed, 27 Sep 2023 16:12:46 GMT, Jorn Vernee wrote:
> Side note: I don't believe I have to add all the different error message
> translations right? Only the English version?
That's right, the translations will be updated towards the end of the release.
-
PR Review Comment:
On Wed, 27 Sep 2023 00:53:25 GMT, Jorn Vernee wrote:
>> This patch contains the implementation of the foreign linker & memory API
>> JEP for Java 22. The initial patch is composed of commits brought over
>> directly from the [panama-foreign
>> repo](https://github.com/openjdk/panama-foreign).
On Wed, 27 Sep 2023 00:53:25 GMT, Jorn Vernee wrote:
>> This patch contains the implementation of the foreign linker & memory API
>> JEP for Java 22. The initial patch is composed of commits brought over
>> directly from the [panama-foreign
>> repo](https://github.com/openjdk/panama-foreign).
On Wed, 27 Sep 2023 07:47:19 GMT, Alan Bateman wrote:
> A test only change to a stress test for virtual thread parking/unparking to
> limit execution time on a larger systems. Right run, the test bashes
> parking/unparking for 1, 2, 3, ... up to the number of half the hardware
On Tue, 26 Sep 2023 18:03:54 GMT, Brian Burkhalter wrote:
>> On Windows, do not return `true` from the `java.io.File` methods
>> `setReadable(boolean, boolean)` and `setExecutable(boolean, boolean)` if the
>> file does not exist.
>
> Brian Burkhalter has updated the pull request incrementally
A test only change to a stress test for virtual thread parking/unparking to
limit execution time on a larger systems. Right run, the test bashes
parking/unparking for 1, 2, 3, ... up to the number of half the hardware
threads. It is changed to limit it to 4 iterations. It is also dialed down
On Tue, 26 Sep 2023 14:19:55 GMT, Technici4n wrote:
> Fixes the issue (hopefully) by resolving automatic modules and automatic
> module dependencies after propagation of non-automatic transitive
> dependencies. The module tests run.
>
> I also added a few asserts to validate that the
On Mon, 25 Sep 2023 20:25:41 GMT, Brian Burkhalter wrote:
>> On Windows, do not return `true` from the `java.io.File` methods
>> `setReadable(boolean, boolean)` and `setExecutable(boolean, boolean)` if the
>> file does not exist.
>
> Brian Burkhalter has updated the pull request incrementally
On Mon, 25 Sep 2023 15:52:12 GMT, Aleksei Voitylov
wrote:
> test java.lang.String.RegionMatches1Tests fails on all platforms with
> -XX:-CompactStrings option and on ARM32 where Compact Strings is disabled by
> default. The fix is to return true immediately if len is negative, since for
>
On Mon, 25 Sep 2023 12:28:40 GMT, Chen Liang wrote:
> In the constructor of String, many locations the user-supplied byte or char
> arrays are read multiple times with a plain memory access; if a user
> previously wrote to one of such locations out of happens-before order,
> distinct plain
On Fri, 22 Sep 2023 22:46:43 GMT, Brian Burkhalter wrote:
>> On Windows, do not return `true` from the `java.io.File` methods
>> `setReadable(boolean, boolean)` and `setExecutable(boolean, boolean)` if the
>> file does not exist.
>
> Brian Burkhalter has updated the pull request incrementally
On Mon, 25 Sep 2023 10:04:13 GMT, Chen Liang wrote:
>>> UTF8 decoder does not perform any internal state mutation during decoding;
>>
>> Are you sure? I think `CharsetDecoder::decode` will modify the `status`
>> field.
>
> right, it does have a `state` field.
CharsetDecoder is specified to
On Tue, 5 Sep 2023 07:53:36 GMT, Afshin Zafari wrote:
> A new benchmark for measuring the NMT overhead in `summary` and `detail`
> modes.
> The tests are run using:
>
> make CONF=debug test TEST="micro:java.util.NMTBenchmark"
> MICRO="RESULTS_FORMAT=json"
>
> The results are written to a
On Tue, 19 Sep 2023 20:03:51 GMT, Naoto Sato wrote:
>> Fixed the failing (well, false-positive) test case. Made the following
>> changes to the test:
>>
>> - Corrected the path to the mapping files directory
>> - Made sure to fail if the directory path is incorrect
>> - Took care of `GB18030`
On Thu, 21 Sep 2023 18:51:50 GMT, Naoto Sato wrote:
>> Fixing a test case that fails in some time zones. Making sure the test is
>> run in `UTC` zone will fix the issue. Confirmed the fix by manually setting
>> machine's time zone to Europe/Dublin.
>
> Naoto Sato has updated the pull request
On Thu, 21 Sep 2023 18:35:57 GMT, Naoto Sato wrote:
> But yes, it is appropriate to replace the invocation. Replaced them all.
Thanks
-
PR Review Comment: https://git.openjdk.org/jdk/pull/15829#discussion_r1333926851
On Wed, 20 Sep 2023 23:11:27 GMT, Brian Burkhalter wrote:
>> On Windows, do not return `true` from the `java.io.File` methods
>> `setReadable(boolean, boolean)` and `setExecutable(boolean, boolean)` if the
>> file does not exist.
>
> Brian Burkhalter has updated the pull request incrementally
On Thu, 21 Sep 2023 00:46:45 GMT, Brian Burkhalter wrote:
> Support for `isAbsolute` was added.
Okay, this looks better but I'm still concerned about other potential
inconsistencies. Can you try out getParent with different input to see if any
issues come up related to the prefix length.
On Wed, 20 Sep 2023 16:12:36 GMT, Naoto Sato wrote:
>> Fixing a test case that fails in some time zones. Making sure the test is
>> run in `UTC` zone will fix the issue. Confirmed the fix by manually setting
>> machine's time zone to Europe/Dublin.
>
> Naoto Sato has updated the pull request
On Wed, 20 Sep 2023 18:47:33 GMT, Brian Burkhalter wrote:
> > File::isAbsolute, it looks like it will return true for input like
> > `\\\?\\foo` but it will be treated by toAbsolutePath as a relative path.
>
> Right on:
Ideally the prefix would be removed at the front door, meaning when
On Wed, 20 Sep 2023 18:00:40 GMT, Coleen Phillimore wrote:
>> Please review this trivial change to remove the UseHeavyMonitors develop
>> option, in favor of the now non-experimental LockingMode=LM_MONITOR (0)
>> option. Tested with tier1 locally.
>
> Coleen Phillimore has updated the pull
On Wed, 20 Sep 2023 18:10:17 GMT, Brian Burkhalter wrote:
>> In the Windows implementation of java.io.File.getCanonicalPath, strip any
>> long path or UNC prefix before canonicalizing the remainder of the pathname.
>
> Brian Burkhalter has updated the pull request incrementally with one
>
On Wed, 20 Sep 2023 11:21:51 GMT, Daniel Fuchs wrote:
> Thanks Tim. Should 8308995 be listed in the `@bug` clause of these two tests?
I don't think so as these tests are just used to check that changes haven't
broken anything.
-
PR Comment:
On Tue, 19 Sep 2023 22:48:20 GMT, Brian Burkhalter wrote:
>> On Windows, do not return `true` from the `java.io.File` methods
>> `setReadable(boolean, boolean)` and `setExecutable(boolean, boolean)` if the
>> file does not exist.
>
> Brian Burkhalter has updated the pull request incrementally
On Wed, 20 Sep 2023 02:14:48 GMT, Brian Burkhalter wrote:
> It could be that if after stripping the `\\\?\` prefix the result is relative
> or drive-relative, then it should first be resolved in the same way before
> proceeding.
Right, which is why the one-arg WinNTFileSystem.resolve may be
On Thu, 10 Aug 2023 17:21:31 GMT, Severin Gehwolf wrote:
>> Please review this patch which adds a "jmodless" jlink mode to the JDK.
>> Fundamentally this patch adds an option to use `jlink` even though your JDK
>> install might not come with the packaged modules (directory `jmods`). This
>>
On 18/09/2023 14:51, Andrew Leonard wrote:
Thanks for the clarification Alan.
To ensure the reproducibility of the whole JDK image regardless of the
specific bootjdk used, would it make sense once the "Build JDK" has
been built, we re-build jrt-fs.jar again using the "Build JDK" ? Thus
On Fri, 21 Jul 2023 18:01:45 GMT, Alan Bateman wrote:
> Thread::getState is an API for monitoring and management purposes to report
> the thread state. If a virtual thread is parked with LockSupport.parkNanos,
> its state is reported as WAITING when it should be TIMED_WAITIN
On Wed, 6 Sep 2023 15:55:21 GMT, Tim Prinzing wrote:
>>> https://bugs.openjdk.org/browse/JDK-8310979 - better exception handling
>>> https://bugs.openjdk.org/browse/JDK-8310978 - missing code paths for event
>>> generation https://bugs.openjdk.org/browse/JDK-8310994 - non-blocking,
>>> event
ange is straight-forward with additional state for
> timed-parking/parked/pinned. The existing virtual/ThreadAPI.java test is
> expanded to this scenario. A new test is added for JVM TI GetThreadState to
> test waiting/timed-waited cases (including pinned) as test coverage seems
> pat
On Mon, 18 Sep 2023 17:30:25 GMT, Daniel Jeliński wrote:
> WinHTTP functions are only used when an application:
> - uses DefaultProxySelector to resolve proxies, and
> - is run with -Djava.net.useSystemProxies=true
>
> In all other cases, loading winhttp.dll is a waste of resources.
>
>
On Mon, 18 Sep 2023 01:44:10 GMT, David Holmes wrote:
> Surely that is not a specified exported thread state though ?? Why would we
> care about PINNED (timed or otherwise) in the current context?
There are internal states and then mappings to the thread states defined by the
APIs
On Thu, 14 Sep 2023 18:03:48 GMT, Brian Burkhalter wrote:
> If this code does not interact with ACL-based security, then it's not clear
> to me that there's anything to be done here.
Right, these methods were created with POSIX file permissions in mind. On the
surface, the only result it can
On Wed, 13 Sep 2023 09:50:35 GMT, Alan Bateman wrote:
> Unsafe.{shouldBeInitialized,ensureClassInitialized} are deprecated for
> removal since JDK 15. It's time to remove them.
> Lookup.ensureInitialized(Class) was added in Java 15 as a standard API to
> ensure that an acce
On 15/09/2023 09:09, Andrew Leonard wrote:
Hi,
jrt-fs.jar is the only jar that is not built (or re-built) with the
"Build JDK", and I was wondering why?
The JAR file contains the "jrt" file system provider for the runtime.
IDE or other tools running on JDK 8/11/etc. need to be able to
On Fri, 15 Sep 2023 00:36:10 GMT, David Holmes wrote:
> What is TIMED PINNING?
LockSupport.parkNanos when the carrier can't be released due to monitor or
native frame.
-
PR Review Comment: https://git.openjdk.org/jdk/pull/14978#discussion_r1326860516
On Thu, 27 Jul 2023 01:54:58 GMT, Serguei Spitsyn wrote:
> Looks good.
> Thanks,
> Serguei
Thanks Serguei. I had to pause this to work on other things. When I sync'ed up
the branch I found I had to update recently introduced test
jvmti/vthread/VThreadEventTest/VThreadEventTest.java as it was
On Wed, 26 Jul 2023 12:18:02 GMT, David Holmes wrote:
> Yes but TIMED and SUSPENDED are not alike. I assume SUSPENDED is a
> stand-alone bit because you can be in various states and suspended at the
> same time. But TIMED isn't like that.
The SUSPENDED bit can only be set when the virtual
ange is straight-forward with additional state for
> timed-parking/parked/pinned. The existing virtual/ThreadAPI.java test is
> expanded to this scenario. A new test is added for JVM TI GetThreadState to
> test waiting/timed-waited cases (including pinned) as test coverage seems
> pat
Unsafe.{shouldBeInitialized,ensureClassInitialized} are deprecated for removal
since JDK 15. It's time to remove them. Lookup.ensureInitialized(Class) was
added in Java 15 as a standard API to ensure that an accessible class is
initialized.
A search of 175973022 classes in 484751 artifacts
raight-forward, it's just additional bit to indicate that the
> parking/parked/pinned states are timed. The existing virtual/ThreadAPI.java
> test is expanded to this scenario. A new test is added for JVM TI
> GetThreadState to test waiting/timed-waited cases (including pinned) as test
> c
On Fri, 8 Sep 2023 15:54:05 GMT, Alan Bateman wrote:
> There are several methods defined by sun.misc.Unsafe that have standard API
> equivalents for many years and releases. The change proposed here is to
> deprecate, for removal, the park, unpark, getLoadAverage, loadFence,
>
On Tue, 12 Sep 2023 01:36:34 GMT, Brian Burkhalter wrote:
>> On Windows, do not return `true` from the `java.io.File` methods
>> `setReadable(boolean, boolean)` and `setExecutable(boolean, boolean)` if the
>> file does not exist.
>
> src/java.base/windows/native/libjava/WinNTFileSystem_md.c
On Mon, 11 Sep 2023 06:31:33 GMT, Per Minborg wrote:
> Currently, all deprecated methods of `Unsafe` are at the end of the file.
Are you looking at the internal Unsafe, rather than sun.misc.Unsafe? This PR is
sun.misc.Unsafe only. Previous deprecation didn't re-order the methods. With
On Wed, 30 Aug 2023 13:56:42 GMT, Alan Bateman wrote:
> In the virtual thread implementation, thread identity switches to the carrier
> before freezing and switches back to the virtual thread after thawing. This
> was a forced move due to issues getting JVMTI to work with virtua
On Wed, 6 Sep 2023 05:01:38 GMT, Serguei Spitsyn wrote:
> The fix looks okay to me. From JVMTI point of view this transition is hidden,
> so no inconsistency can be observed. However, I wonder if the
> AsyncGetStackTraces may have similar issue as the JFR had. Unfortunately we
> do not have
On Fri, 8 Sep 2023 01:07:17 GMT, Brian Burkhalter wrote:
>> In the Windows implementation of java.io.File.getCanonicalPath, strip any
>> long path or UNC prefix before canonicalizing the remainder of the pathname.
>
> Brian Burkhalter has updated the pull request incrementally with one
>
On Fri, 8 Sep 2023 18:45:02 GMT, Brent Christian wrote:
> I would also like to see what a FileChannel implementation looks like. Am I
> right that this would allow the `prefs` native library to be removed entirely?
Yes. FileChannel.open can be called with the file permissions to atomically set
There are several methods defined by sun.misc.Unsafe that have standard API
equivalents for many years and releases. The change proposed here is to
deprecate, for removal, the park, unpark, getLoadAverage, loadFence,
storeFence, and fullFence methods. Code using these methods should move to
JFR thread sampler to skip sampling when
> it samples during a transition.
>
> Testing: tier1-5
Alan Bateman has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull reque
On Fri, 8 Sep 2023 01:07:17 GMT, Brian Burkhalter wrote:
>> In the Windows implementation of java.io.File.getCanonicalPath, strip any
>> long path or UNC prefix before canonicalizing the remainder of the pathname.
>
> Brian Burkhalter has updated the pull request incrementally with one
>
On Fri, 8 Sep 2023 07:44:21 GMT, Matthias Baesken wrote:
> We could do this mid-term in some follow up action.
It shouldn't be hard to try it. If it works out then it would mean this old
crufty code goes away and the JDK is in a better place. If it doesn't work out
then the a plan B would be
On Mon, 4 Sep 2023 11:57:47 GMT, Matthias Baesken wrote:
>>> Hi Alan , Your assumption 'I assume the use of System.getProperty is
>>> problematic when running with a SM.' is most likely correct.
>>
>> You'll need to test with a SM that denies reading the system property to be
>> sure. There
On Thu, 31 Aug 2023 21:31:40 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal is to develop faster sort routines for x86_64 CPUs by taking
>> advantage of AVX512 instructions. This enhancement provides an order of
>> magnitude speedup for Arrays.sort() using int, long, float and double arrays.
On Wed, 6 Sep 2023 21:38:39 GMT, Brian Burkhalter wrote:
> In the Windows implementation of java.io.File.getCanonicalPath, strip any
> long path or UNC prefix before canonicalizing the remainder of the pathname.
Would it be possible to summarise how this behaves when File just presents this
On Wed, 6 Sep 2023 15:55:21 GMT, Tim Prinzing wrote:
> I think it's useful if trying to trace the calls (i.e. set to 0ms).
> Apparently the security manager was being used for that by some.
The SM isn't called once connected so I don't think anyone could have every
done that. Yes, you could
On 06/09/2023 16:17, Volker Simonis wrote:
:
I'm familiar with JEP 260. But wouldn't you agree that an
"encapsulated" monitoring API is an oxymoron? A monitoring API is by
design intended for external usage and completely useless to the
platform itself. There's no single usage of the
On Wed, 6 Sep 2023 16:49:39 GMT, Chris Plummer wrote:
> > I wonder if this is the right thing to do for the hprof files. I believe
> > they originated from some hprof tools that we no longer ship. 3rd parties
> > might choose to integrate them into their own tools.
>
> Do you think I should
JFR thread sampler to skip sampling when
> it samples during a transition.
>
> Testing: tier1-5
Alan Bateman has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull requ
On 06/09/2023 14:02, Volker Simonis wrote:
:
I wonder why "sun.management" was encapsulated in the first place? I
understand that it is not an "officially supported" API, but I find it
still quite useful.
sun.management.* is JDK internal so not something for code outside the
JDK to use
On Wed, 6 Sep 2023 08:18:45 GMT, JoKern65 wrote:
> After push of [JDK-8307478](https://bugs.openjdk.org/browse/JDK-8307478) ,
> the following test started to fail on AIX :
> com/sun/tools/attach/warnings/DynamicLoadWarningTest.java;
> The problem was described in
>
On Tue, 5 Sep 2023 23:15:53 GMT, Jonathan Gibbons wrote:
> One has to wonder about the `**/*_OLD.java` files, but that would be a
> different cleanup
The IBM double byte charsets were re-implemented in JDK 7. I think the old
implementations moved to the test tree so it could be used to test
On Fri, 1 Sep 2023 14:59:19 GMT, Jaikiran Pai wrote:
> I asked for Mark's and Joe's inputs on this in the linked JBS issue
> https://bugs.openjdk.org/browse/JDK-8233160. Mark has suggested that we keep
> this `java.vendor.url.bug` system property non-optional and also not to add
> any
On Fri, 1 Sep 2023 11:32:45 GMT, Markus Grönlund wrote:
>> Thanks. One other thing that I see when more testing with generational ZGC
>> is that ZPageAllocator::alloc_page will record an event and this can happen
>> during a mount transition. So I think JfrStackTrace::record also needs to
>>
JFR thread sampler to skip sampling when
> it samples during a transition.
>
> Testing: tier1-5
Alan Bateman has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull reque
On Fri, 28 Jul 2023 18:41:48 GMT, Joe Wang wrote:
> Add a JDK Impl specific property 'jdk.xml.dtd.support' for applications to
> specify how DTDs are handled. This property is uniformly supported across the
> JDK XML libraries. It complements, rather than replaces, the existing
> properties
On Fri, 1 Sep 2023 23:12:28 GMT, Justin Lu wrote:
> Please review this PR and [CSR](https://bugs.openjdk.org/browse/JDK-8315558)
> which is a conformance change to document some exceptions in the
> specification of java.text.StringCharacterIterator.
@justin-curtis-lu You are correct that it
On Fri, 1 Sep 2023 13:10:19 GMT, Matthias Baesken wrote:
> Hi Alan , Your assumption 'I assume the use of System.getProperty is
> problematic when running with a SM.' is most likely correct.
You'll need to test with a SM that denies reading the system property to be
sure. There are classes in
On Fri, 1 Sep 2023 12:36:28 GMT, Matthias Baesken wrote:
>> We run into some BackingStoreException: Couldn't get file lock. e.g. here :
>>
>> [JShell] Exception in thread "main" java.lang.IllegalStateException:
>> java.util.prefs.BackingStoreException: Couldn't get file lock.
>> [JShell] at
On Fri, 1 Sep 2023 10:47:25 GMT, Markus Grönlund wrote:
>> Just to add that Patricio suggested today to run the stress tests with
>> -Xint and that does lead to triggering the assert quickly when the thread is
>> sampled in native. There are several native methods that are
>>
On Fri, 25 Aug 2023 09:49:20 GMT, Alan Bateman wrote:
>> I want to add a log output similar to JDK-8301627 to Runtime.halt().
>> To avoid double logging of Runtime.exit(), add a flag to indicate whether
>> logging was done, and fix it so that logging is done only once.
>&
On Thu, 31 Aug 2023 17:09:40 GMT, Mandy Chung wrote:
>> 8268829: Provide an optimized way to walk the stack with Class object only
>>
>> `StackWalker::walk` creates one `StackFrame` per frame and the current
>> implementation
>> allocates one `StackFrameInfo` and one `MemberName` objects per
On Thu, 31 Aug 2023 14:29:41 GMT, iaroslavski wrote:
> Alan, you mentioned that DualPivotQuicksort will need detailed review. Can we
> go ahead and start reviewing? Laurent checked performance, JMH results look
> fine.
As before, I think the main question with this change is whether adding
On Thu, 31 Aug 2023 17:31:34 GMT, Patricio Chilano Mateo
wrote:
>> src/hotspot/share/jfr/periodic/sampling/jfrThreadSampler.cpp line 415:
>>
>>> 413: } else {
>>> 414: assert(NATIVE_SAMPLE == type, "invariant");
>>> 415: if (thread_state_in_native(thread) &&
>>>
On Thu, 31 Aug 2023 12:13:40 GMT, Adam Sotona wrote:
> ModuleTarget and ModuleResolution attributes were flagged as 'allow multiple'
> in the Classfile API.
> This patch removed the flags and allows at most one instance of each
> attribute.
>
> Please review.
>
> Thanks,
> Adam
Thanks for
On Thu, 31 Aug 2023 10:38:32 GMT, Markus Grönlund wrote:
>> In the virtual thread implementation, thread identity switches to the
>> carrier before freezing and switches back to the virtual thread after
>> thawing. This was a forced move due to issues getting JVMTI to work with
>> virtual
On Thu, 31 Aug 2023 06:53:45 GMT, Jaikiran Pai wrote:
> Can I please get a review of this change which proposes to address
> https://bugs.openjdk.org/browse/JDK-8233160?
>
> It has been noted in https://bugs.openjdk.org/browse/JDK-8232753 that:
>
>> The java.vendor.url.bug property has been
On Wed, 30 Aug 2023 16:29:31 GMT, Mandy Chung wrote:
> Indeed, Set.of factory methods are easy to use. I'm okay with taking it out.
It would be easy to add in the future if needed, say if more options were
introduced and the common case requires 2 or more options.
-
PR Comment:
In the virtual thread implementation, thread identity switches to the carrier
before freezing and switches back to the virtual thread after thawing. This was
a forced move due to issues getting JVMTI to work with virtual threads. JVMTI
can now hide events during transitions so we can invert the
On Wed, 30 Aug 2023 17:38:01 GMT, Stefan Karlsson wrote:
> I wouldn't be opposed to a change that:
>
> * Keeps the `createJavaProcessBuilder` name
> * Renames `createTestJvm` to `createJavaProcessBuilderPrependTestOpts`
> * Renames `executeTestJvm` to `executeJavaPrependTestOpts`
> * Removes
On Wed, 30 Aug 2023 20:57:43 GMT, Mandy Chung wrote:
> Loom added a special filtering of Continuation.yield0 in stack walker. After
> the initial implementation, JDK-8304919 marks the yielding and entering
> methods with `@Hidden` and hidden frames are filtered by stack walker by
> default.
On Wed, 30 Aug 2023 15:03:27 GMT, Daniel Fuchs wrote:
>> Mandy Chung has updated the pull request incrementally with three additional
>> commits since the last revision:
>>
>> - update mode to be int rather than long
>> - update tests
>> - Review feedback on javadoc
>
>
On Wed, 30 Aug 2023 12:02:12 GMT, chenggwang wrote:
> Hi Can anyone help me to review this PR @sormuras @asotona or any other
> reviewer?
I think you first need to make a case for changing the CyclicBarrier API as
opposed to dealing with the phases in your BarrierAction or using the Phaser
On Wed, 30 Aug 2023 00:37:26 GMT, Srinivas Vamsi Parasa
wrote:
> Hi Vladimir, Just verified that the test/jdk/java/util/Arrays/Sorting.java is
> triggering the intrinsic without additional flags
Just to add that Sorting.java has short and long run modes. The default when
running with jtreg
On Tue, 29 Aug 2023 20:51:56 GMT, Mandy Chung wrote:
>> 8268829: Provide an optimized way to walk the stack with Class object only
>>
>> `StackWalker::walk` creates one `StackFrame` per frame and the current
>> implementation
>> allocates one `StackFrameInfo` and one `MemberName` objects per
On Mon, 28 Aug 2023 21:27:25 GMT, Srinivas Vamsi Parasa
wrote:
>> The goal is to develop faster sort routines for x86_64 CPUs by taking
>> advantage of AVX512 instructions. This enhancement provides an order of
>> magnitude speedup for Arrays.sort() using int, long, float and double arrays.
On Tue, 29 Aug 2023 12:35:19 GMT, Severin Gehwolf wrote:
> @AlanBateman Gentle ping.
On my list, it's a lot to get through and a number of aspects to this that I
think will require refinement and discussion.
-
PR Comment:
On Tue, 29 Aug 2023 16:59:22 GMT, Daniel Fuchs wrote:
>> how about "s/filtering/excluding"?
>
> Yes - that's better.
The example use filter so it might be better to have the text use the same
word, up to you, I think "filtering out known implementation classes" works too.
-
PR
On Tue, 29 Aug 2023 00:16:40 GMT, Mandy Chung wrote:
>> 8268829: Provide an optimized way to walk the stack with Class object only
>>
>> `StackWalker::walk` creates one `StackFrame` per frame and the current
>> implementation
>> allocates one `StackFrameInfo` and one `MemberName` objects per
On Mon, 28 Aug 2023 16:31:06 GMT, Lance Andersen wrote:
> Hi all,
>
> Please review this PR which updates zlib from 1.2.13 to 1.3 in openJDK
>
> The [Zlib Data Compression Library](https://github.com/madler/zlib ) has
> released Zlib 1.3 on August 18, 2023.
>
> There are a [small number of
On Mon, 28 Aug 2023 07:28:48 GMT, Matthias Baesken wrote:
> Hi Alan , when adding the test group sun/tools/jhsdb in TEST.root to the
> exclusiveAccess.dirs , I cannot see the error any more. So it seems your
> suggestions is correct .
My command was actually wondering why we didn't have a
On Mon, 28 Aug 2023 17:15:36 GMT, Lance Andersen wrote:
> It was a clean copy(which I have been able to do as of zlib 1.2.13). Sorry I
> should have mentioned this in the PR request
That's okay, I just wanted to check to be sure that the ChangeLog_java (renamed
adler32.c -> zadler32.c, crc32c
601 - 700 of 1993 matches
Mail list logo
