Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v8]

2024-09-16 Thread Daniel Jeliński
On Fri, 13 Sep 2024 20:41:27 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request in

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v8]

2024-09-16 Thread Magnus Ihse Bursie
On Fri, 13 Sep 2024 20:41:27 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request in

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v7]

2024-09-13 Thread Brian Burkhalter
On Fri, 13 Sep 2024 19:30:42 GMT, Magnus Ihse Bursie wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8337143: Clean up to address reviewer comments > > make/modules/java.base/lib/CoreLibraries.gmk line 68: > >

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v8]

2024-09-13 Thread Brian Burkhalter
> This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v7]

2024-09-13 Thread Magnus Ihse Bursie
On Thu, 12 Sep 2024 02:10:00 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request in

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Brian Burkhalter
On Fri, 13 Sep 2024 16:45:22 GMT, Daniel Fuchs wrote: >> I would have to check. The failure I observed occurred in both of these tests >> >> test/jdk/sun/net/www/protocol/http/NoNTLM.java >> test/jdk/sun/net/www/protocol/http/TestTransparentNTLM.java >> >> and nowhere else. > > I see. The test

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Daniel Fuchs
On Fri, 13 Sep 2024 16:35:04 GMT, Brian Burkhalter wrote: >> Do we know what code loaded NTLMAuthentication? I'd expect it to be loaded >> by HttpURLConnection, which should have triggered the loading of libnet long >> before it cares about NTLM. > > I would have to check. The failure I observe

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Brian Burkhalter
On Fri, 13 Sep 2024 16:31:36 GMT, Daniel Fuchs wrote: >>> [...] I'd expect that libnet would have been loaded before we reach >>> NTLMAuthentication. >> >> In the (as of now) penultimate webrev 4f47d5a (Merge), >> `WindowsNativeDispatcher` loaded it during boot phase 2. I think that >> withou

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Daniel Fuchs
On Fri, 13 Sep 2024 16:12:28 GMT, Brian Burkhalter wrote: >> Yes, there was an `UnsatisfiedLinkError` in the native method >> `isTrustedSiteAvailable()` in `NTLMAuthentication`. > >> [...] I'd expect that libnet would have been loaded before we reach >> NTLMAuthentication. > > In the (as of no

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Brian Burkhalter
On Fri, 13 Sep 2024 16:08:50 GMT, Brian Burkhalter wrote: >> I wonder - do you see any failure if you don't load libnet from there? > > Yes, there was an `UnsatisfiedLinkError` in the native method > `isTrustedSiteAvailable()` in `NTLMAuthentication`. > [...] I'd expect that libnet would have b

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Brian Burkhalter
On Fri, 13 Sep 2024 16:06:26 GMT, Daniel Fuchs wrote: >> hmm... I don't see any issue in adding the load call to >> `NTLMAuthentication` but I'm surprised that it's even needed: I'd expect >> that libnet would have been loaded before we reach NTLMAuthentication. > > I wonder - do you see any f

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Daniel Fuchs
On Fri, 13 Sep 2024 16:05:37 GMT, Daniel Fuchs wrote: >> @dfuch Would you please check whether the change at line 71 in the updated >> version of `NTLMAuthentication` is the correct place for this load? > > hmm... I don't see any issue in adding the load call to `NTLMAuthentication` > but I'm

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-13 Thread Daniel Fuchs
On Thu, 12 Sep 2024 15:40:27 GMT, Brian Burkhalter wrote: >> Moved to `NTLMAuthentication` static initializer. 853d349 > > @dfuch Would you please check whether the change at line 71 in the updated > version of `NTLMAuthentication` is the correct place for this load? hmm... I don't see any iss

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v7]

2024-09-12 Thread Brian Burkhalter
On Thu, 12 Sep 2024 15:43:32 GMT, Alan Bateman wrote: > Can you hold off integrating, I plan to go over the changes soon. I don't plan to integrate until you have reviewed it. I set the minimum reviewer count to 3 to avoid doing so accidentally. - PR Comment: https://git.openjdk.o

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v7]

2024-09-12 Thread Alan Bateman
On Thu, 12 Sep 2024 15:38:18 GMT, Brian Burkhalter wrote: > The tests passed the JDK tiers 1-3 tests on Linux, macOS, and Windows. In any > case, I will run another round of tests before integrating. Can you hold off integrating, I plan to go over the changes soon. - PR Comment: h

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-12 Thread Brian Burkhalter
On Thu, 12 Sep 2024 02:05:50 GMT, Brian Burkhalter wrote: >> We don't need it in `libjava`, but `NTLMAuthentication` was (perhaps >> unwittingly) relying on it to load `net.dll` during boot phase 2. I'll move >> the load to a more appropriate location in the next commit. > > Moved to `NTLMAuthe

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v7]

2024-09-12 Thread Brian Burkhalter
On Thu, 12 Sep 2024 06:24:50 GMT, Daniel Jeliński wrote: > Thanks for making the changes. LGTM, assuming that tests still pass. The tests passed the JDK tiers 1-3 tests on Linux, macOS, and Windows. In any case, I will run another round of tests before integrating. > src/java.base/unix/native/

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v7]

2024-09-11 Thread Daniel Jeliński
On Thu, 12 Sep 2024 02:10:00 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request in

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Wed, 11 Sep 2024 19:22:00 GMT, Brian Burkhalter wrote: >> src/java.base/windows/classes/sun/nio/fs/WindowsNativeDispatcher.java line >> 1097: >> >>> 1095: >>> 1096: static { >>> 1097: jdk.internal.loader.BootLoader.loadLibrary("net"); >> >> ...do we still need net here? > > We

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Wed, 11 Sep 2024 16:51:43 GMT, Daniel Jeliński wrote: >> Yes, it still needs to be called in a few places, e.g., for classes whose >> native component needs the `fdval()` and `handleval()` functions. > > that's a good point. The comment might need to be updated to reflect that. Comments adde

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Wed, 11 Sep 2024 19:31:02 GMT, Brian Burkhalter wrote: >> Just to be absolutely clear: All my other comments about removing unneeded >> libraries were about libnio, not libjava. I realize I made the comment in >> the PR next to libjava, but my intention was to ask if the list of libraries >

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v7]

2024-09-11 Thread Brian Burkhalter
> This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Tue, 10 Sep 2024 13:26:58 GMT, Magnus Ihse Bursie wrote: >> make/modules/java.base/Lib.gmk line 81: >> >>> 79: libjava/nio/ch \ >>> 80: libnio/ch \ >>> 81: libnio/fs \ >> >> libnio/fs is gone on all platforms other than aix; is this still necessary? > > We can't add e

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Tue, 10 Sep 2024 09:54:56 GMT, Daniel Jeliński wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8337143: Remove loading libnet from Inet6AddressImpl; delete commented out >> #include in Windows IOUtil.c > >

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Wed, 11 Sep 2024 19:28:39 GMT, Magnus Ihse Bursie wrote: >> Apparently I did remove it from libjava and not libnio. It will be fixed in >> the next commit. Thanks for being persistent. > > Just to be absolutely clear: All my other comments about removing unneeded > libraries were about libni

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Magnus Ihse Bursie
On Wed, 11 Sep 2024 19:18:18 GMT, Brian Burkhalter wrote: >> did you remove mswsock from libjava or from libnio? Libnio libraries are >> listed >> [here](https://github.com/openjdk/jdk/blob/f57b6f13e9f375bfd2e8a05afd2b900a4d42285e/make/modules/java.base/Lib.gmk#L89). >> There's no FileDispatche

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Tue, 10 Sep 2024 11:27:05 GMT, Daniel Jeliński wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8337143: Remove loading libnet from Inet6AddressImpl; delete commented out >> #include in Windows IOUtil.c > >

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Wed, 11 Sep 2024 16:48:24 GMT, Daniel Jeliński wrote: >> From a clean build in the CI with `mswsock.lib` removed: >> >> >> [2024-09-11T16:00:17,816Z] FileDispatcherImpl.obj : error LNK2019: >> unresolved external symbol >> TransmitFile referenced in function >> Java_sun_nio_ch_FileDispatch

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Daniel Jeliński
On Tue, 10 Sep 2024 19:58:45 GMT, Brian Burkhalter wrote: >> src/java.base/share/classes/sun/nio/ch/IOUtil.java line 575: >> >>> 573: * Used to trigger loading of native libraries >>> 574: */ >>> 575: public static void load() { } >> >> do we still need this method? > > Yes, it st

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Daniel Jeliński
On Wed, 11 Sep 2024 16:33:28 GMT, Brian Burkhalter wrote: >> Right. This PR moves FileDispatcherImpl.c to libjava, so >> FileDispatcherImpl.obj is no longer there. I'm guessing that our makefiles >> don't detect source files that were removed, and Brian didn't run `make >> clean`. > > From a c

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-11 Thread Brian Burkhalter
On Wed, 11 Sep 2024 06:00:23 GMT, Daniel Jeliński wrote: >> And you did not get `mswsock.lib: FileDispatcherImpl.obj : error LNK2019: >> unresolved external symbol TransmitFile`? > > Right. This PR moves FileDispatcherImpl.c to libjava, so > FileDispatcherImpl.obj is no longer there. I'm guessi

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-10 Thread Daniel Jeliński
On Tue, 10 Sep 2024 21:42:08 GMT, Magnus Ihse Bursie wrote: >> well I'm not satisfied with the answers :) I removed mswsock.lib from libnio >> dependencies >> [here](https://github.com/openjdk/jdk/blob/f57b6f13e9f375bfd2e8a05afd2b900a4d42285e/make/modules/java.base/Lib.gmk#L89) >> and the buil

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-10 Thread Magnus Ihse Bursie
On Tue, 10 Sep 2024 15:15:26 GMT, Daniel Jeliński wrote: >> I believe this question has already been answered >> [here](https://github.com/openjdk/jdk/pull/20317/files#r1707599113). > > well I'm not satisfied with the answers :) I removed mswsock.lib from libnio > dependencies > [here](https:/

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v6]

2024-09-10 Thread Brian Burkhalter
> This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Brian Burkhalter has updated the pull request with a new target base due to a merge or a rebase. The pull request

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-10 Thread Brian Burkhalter
On Tue, 10 Sep 2024 11:06:28 GMT, Daniel Jeliński wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8337143: Remove loading libnet from Inet6AddressImpl; delete commented out >> #include in Windows IOUtil.c > >

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-10 Thread Daniel Jeliński
On Tue, 10 Sep 2024 13:30:05 GMT, Magnus Ihse Bursie wrote: >> make/modules/java.base/lib/CoreLibraries.gmk line 71: >> >>> 69: -framework Foundation \ >>> 70: -framework SystemConfiguration, \ >>> 71: LIBS_windows := advapi32.lib mswsock.lib ole32.lib shell32.lib >>> versio

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-10 Thread Magnus Ihse Bursie
On Tue, 10 Sep 2024 09:56:26 GMT, Daniel Jeliński wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8337143: Remove loading libnet from Inet6AddressImpl; delete commented out >> #include in Windows IOUtil.c > >

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-10 Thread Daniel Jeliński
On Fri, 9 Aug 2024 17:59:12 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request inc

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-09-09 Thread Brian Burkhalter
On Fri, 9 Aug 2024 17:59:12 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request inc

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-08-12 Thread Brian Burkhalter
On Mon, 12 Aug 2024 15:46:13 GMT, Alan Bateman wrote: > please do not integrate this change until I get time Sure, of course I will not. Thanks to @magicus and @dfuch for helping to make it better. - PR Comment: https://git.openjdk.org/jdk/pull/20317#issuecomment-2284529067

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-08-12 Thread Alan Bateman
On Fri, 9 Aug 2024 17:59:12 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request inc

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-08-12 Thread Magnus Ihse Bursie
On Fri, 9 Aug 2024 17:59:12 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request inc

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-08-12 Thread Daniel Fuchs
On Fri, 9 Aug 2024 17:59:12 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request inc

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava

2024-08-09 Thread Brian Burkhalter
On Fri, 26 Jul 2024 21:34:07 GMT, Brian Burkhalter wrote: > > Also think to work through some naming on IOUtil vs. NIOUtil as it won't be > > obvious to maintainers which class to use. > > Maybe `NIOUtil` should be `NetUtil` as its methods appear to be invoked only > by classes involved in net

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Brian Burkhalter
On Thu, 8 Aug 2024 14:33:15 GMT, Brian Burkhalter wrote: >> src/java.base/windows/native/libjava/IOUtil.c line 37: >> >>> 35: #include "nio.h" >>> 36: #include "nio_util.h" >>> 37: /* #include "net_util.h" */ >> >> Is this change intended or is this a left over from some experimentation? > > It

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Brian Burkhalter
On Fri, 9 Aug 2024 15:58:50 GMT, Brian Burkhalter wrote: >> I see. Inet6AddressImpl and Inet4AddressImpl are symetric classes >> implementing InetAddressImpl. Both will make native calls to the "net" >> library - so both require the library to be loaded. >> >> In the JDK, these two classes are

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v5]

2024-08-09 Thread Brian Burkhalter
> This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Brian Burkhalter
On Fri, 9 Aug 2024 15:47:07 GMT, Daniel Fuchs wrote: >> Loading "net" was removed from IOUtil so I am thinking that IOUtil must have >> been initialized somewhere before constructing Inet6AddressImpl, but I've >> not identified where just yet. > > I see. Inet6AddressImpl and Inet4AddressImpl ar

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Daniel Fuchs
On Fri, 9 Aug 2024 15:09:08 GMT, Brian Burkhalter wrote: >> OK, this test uses a private API to create an instance of Inet6AddressImpl; >> This explain why in this test Inet6AddressImpl gets loaded before >> InetAddress. >> >> var impl = new Inet6AddressImpl(); >> >> It should nev

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Brian Burkhalter
On Fri, 9 Aug 2024 09:05:49 GMT, Daniel Fuchs wrote: >> Without loading libnet in Inet6AddressImpl, the test >> java/net/InetAddress/NullCharInHostnameDriver.java fails with >> UnsatisfiedLinkError: >> >> java.lang.UnsatisfiedLinkError: 'java.net.InetAddress[] >> java.net.Inet6AddressImpl.loo

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Brian Burkhalter
On Fri, 9 Aug 2024 09:40:59 GMT, Magnus Ihse Bursie wrote: >> `pthread` is still needed: >> >> open/src/java.base/unix/native/libjava/nio/ch/NativeThread.c:83: error: >> undefined reference to 'pthread_kill' > > Ok then. Thank you for your thorough checking! Thank you for suggesting it! -

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Magnus Ihse Bursie
On Thu, 8 Aug 2024 16:29:18 GMT, Brian Burkhalter wrote: >> I will check. > > `pthread` is still needed: > > open/src/java.base/unix/native/libjava/nio/ch/NativeThread.c:83: error: > undefined reference to 'pthread_kill' Ok then. Thank you for your thorough checking! - PR Review

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-09 Thread Daniel Fuchs
On Thu, 8 Aug 2024 21:23:58 GMT, Brian Burkhalter wrote: >> Thanks for the suggestions. I will look into it. > > Without loading libnet in Inet6AddressImpl, the test > java/net/InetAddress/NullCharInHostnameDriver.java fails with > UnsatisfiedLinkError: > > java.lang.UnsatisfiedLinkError: 'jav

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Brian Burkhalter
On Thu, 8 Aug 2024 16:48:24 GMT, Brian Burkhalter wrote: >> Possibly - if you made isIPv6Supported() in InetAddress.c return false, you >> might be able to see the issue in the same test that you observed failing >> without your change. >> >> InetAddress has a static block that will load the

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Brian Burkhalter
On Thu, 8 Aug 2024 16:39:33 GMT, Daniel Fuchs wrote: >> I'll have to investigate. > > Possibly - if you made isIPv6Supported() in InetAddress.c return false, you > might be able to see the issue in the same test that you observed failing > without your change. > > InetAddress has a static blo

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Daniel Fuchs
On Thu, 8 Aug 2024 16:21:30 GMT, Brian Burkhalter wrote: >> I don't know - you added that code to Inet6AddressImpl - so presumably a >> test was failing without that code? >> Which test was that? It wasn't obvious to me that adding code to load the >> "net" library would be required here. > > I

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Daniel Fuchs
On Thu, 8 Aug 2024 16:09:55 GMT, Brian Burkhalter wrote: >> It may be because we have no IPv4 only machine in the CI? It seems strange >> that IPv6 is treated differently than IPv4 in that respect. > > How would you suggest testing this? I don't know - you added that code to Inet6AddressImpl -

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Brian Burkhalter
On Thu, 8 Aug 2024 16:18:28 GMT, Daniel Fuchs wrote: >> How would you suggest testing this? > > I don't know - you added that code to Inet6AddressImpl - so presumably a test > was failing without that code? > Which test was that? It wasn't obvious to me that adding code to load the > "net" libr

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Brian Burkhalter
On Thu, 8 Aug 2024 15:16:14 GMT, Daniel Fuchs wrote: >> I have not seen any failures in CI testing. Is there a specific test that >> would reveal whether this is a problem? > > It may be because we have no IPv4 only machine in the CI? It seems strange > that IPv6 is treated differently than IPv

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Brian Burkhalter
On Thu, 8 Aug 2024 14:31:09 GMT, Brian Burkhalter wrote: >> And pthread? > > I will check. `pthread` is still needed: open/src/java.base/unix/native/libjava/nio/ch/NativeThread.c:83: error: undefined reference to 'pthread_kill' - PR Review Comment: https://git.openjdk.org/jdk/pul

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Daniel Fuchs
On Thu, 8 Aug 2024 14:32:25 GMT, Brian Burkhalter wrote: >> src/java.base/share/classes/java/net/Inet6AddressImpl.java line 154: >> >>> 152: static { >>> 153: jdk.internal.loader.BootLoader.loadLibrary("net"); >>> 154: } >> >> I am curious about this change - wouldn't it be need

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Brian Burkhalter
On Thu, 8 Aug 2024 12:18:04 GMT, Magnus Ihse Bursie wrote: >> The Windows libs are also still required for `TransmitFile` and >> `WSAGetLastError`: >> >> - ```mswsock.lib: FileDispatcherImpl.obj : error LNK2019: unresolved >> external symbol TransmitFile``` >> - ```ws2_32.lib: FileDispatcherIm

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Brian Burkhalter
On Thu, 8 Aug 2024 09:11:31 GMT, Daniel Fuchs wrote: >> Brian Burkhalter 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 request contains four additional >> comm

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Magnus Ihse Bursie
On Wed, 7 Aug 2024 18:13:25 GMT, Brian Burkhalter wrote: >> `CoreServices` is still necessary ([Uniform Type >> Identifier](https://developer.apple.com/documentation/uniformtypeidentifiers)s). > > The Windows libs are also still required for `TransmitFile` and > `WSAGetLastError`: > > - ```msw

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-08 Thread Daniel Fuchs
On Wed, 7 Aug 2024 15:59:14 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request wit

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-07 Thread Brian Burkhalter
On Wed, 7 Aug 2024 16:03:33 GMT, Brian Burkhalter wrote: >> I will check. Thanks for noticing it. > > `CoreServices` is still necessary ([Uniform Type > Identifier](https://developer.apple.com/documentation/uniformtypeidentifiers)s). The Windows libs are also still required for `TransmitFile` a

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-07 Thread Brian Burkhalter
On Wed, 7 Aug 2024 16:00:26 GMT, Brian Burkhalter wrote: >> make/modules/java.base/lib/CoreLibraries.gmk line 71: >> >>> 69: -framework Foundation \ >>> 70: -framework SystemConfiguration, \ >>> 71: LIBS_windows := advapi32.lib mswsock.lib ole32.lib shell32.lib >>> version.l

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-07 Thread Brian Burkhalter
On Wed, 7 Aug 2024 15:56:06 GMT, Magnus Ihse Bursie wrote: >> Brian Burkhalter 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 request contains four additional >

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v2]

2024-08-07 Thread Brian Burkhalter
On Mon, 5 Aug 2024 15:37:46 GMT, Brian Burkhalter wrote: >> make/modules/java.base/lib/CoreLibraries.gmk line 57: >> >>> 55: OPTIMIZATION := HIGH, \ >>> 56: EXTRA_HEADER_DIRS := \ >>> 57: libnio/ch, \ >> >> This will introduce a source code dependency from libjava to libnio. It

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-07 Thread Magnus Ihse Bursie
On Wed, 7 Aug 2024 15:56:06 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request wit

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v4]

2024-08-07 Thread Brian Burkhalter
> This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Brian Burkhalter has updated the pull request with a new target base due to a merge or a rebase. The incremental

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v3]

2024-08-07 Thread Brian Burkhalter
> This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v2]

2024-08-07 Thread Brian Burkhalter
On Fri, 26 Jul 2024 19:40:24 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request in

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v2]

2024-08-05 Thread Brian Burkhalter
On Mon, 5 Aug 2024 15:19:33 GMT, Magnus Ihse Bursie wrote: > This will introduce a source code dependency from libjava to libnio. It might > be the correct thing to do, but I just want to highlight this. I'll look into improving it. > make/modules/java.base/lib/CoreLibraries.gmk line 71: > >>

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v2]

2024-08-05 Thread Magnus Ihse Bursie
On Fri, 26 Jul 2024 19:40:24 GMT, Brian Burkhalter wrote: >> This proposed change would move the native objects required for NIO file >> interaction from the libnio native library to the libjava native library on >> Linux, macOS, and Windows. > > Brian Burkhalter has updated the pull request in

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava

2024-07-26 Thread Brian Burkhalter
On Thu, 25 Jul 2024 10:05:03 GMT, Alan Bateman wrote: > Also think to work through some naming on IOUtil vs. NIOUtil as it won't be > obvious to maintainers which class to use. Maybe `NIOUtil` should be `NetUtil` as its methods appear to be invoked only by classes involved in networking?

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava [v2]

2024-07-26 Thread Brian Burkhalter
> This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava

2024-07-25 Thread Brian Burkhalter
On Wed, 24 Jul 2024 19:11:42 GMT, Brian Burkhalter wrote: > This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Perhaps `/native/libjava/nio/ch` and `/native/libjava

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava

2024-07-25 Thread Brian Burkhalter
On Wed, 24 Jul 2024 19:11:42 GMT, Brian Burkhalter wrote: > This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. Yes, I was wondering about changing the organization o

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava

2024-07-25 Thread Alan Bateman
On Wed, 24 Jul 2024 19:11:42 GMT, Brian Burkhalter wrote: > This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. I think this will require thinking about how to organi

Re: RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava

2024-07-24 Thread Brian Burkhalter
On Wed, 24 Jul 2024 19:11:42 GMT, Brian Burkhalter wrote: > This proposed change would move the native objects required for NIO file > interaction from the libnio native library to the libjava native library on > Linux, macOS, and Windows. This change passes CI tiers 1-5 jobs on all platforms.

RFR: 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava

2024-07-24 Thread Brian Burkhalter
This proposed change would move the native objects required for NIO file interaction from the libnio native library to the libjava native library on Linux, macOS, and Windows. - Commit messages: - 8337143: (fc, fs) Move filesystem-related native objects from libnio to libjava Cha