[crac] RFR: Merge jdk:jdk-25+18 [v2]

Timofei Pushkin tpushkin at openjdk.org
Mon Jun 30 12:04:00 UTC 2025


On Mon, 30 Jun 2025 08:30:08 GMT, Dmitry Cherepanov <dcherepanov at openjdk.org> wrote:

>> This MR includes merges for tags between jdk-25+15 and jdk-25+18.
>> 
>> Attaching output of --diff-merges=remerge
>> 
>> <details>
>> 
>> <summary>jdk-25+15</summary>
>> 
>> 
>> commit b0b66cb5999e89dd44456682c6bc932fa559e242
>> Merge: eea5480cdda 74df384a987
>> Author: Dmitry Cherepanov <dcherepanov at azul.com>
>> Date:   Mon Jun 9 18:31:26 2025 +0400
>> 
>>     Merge with jdk-25+15
>> 
>> diff --git a/src/hotspot/os/posix/attachListener_posix.cpp b/src/hotspot/os/posix/attachListener_posix.cpp
>> remerge CONFLICT (content): Merge conflict in src/hotspot/os/posix/attachListener_posix.cpp
>> index ca3666be119..4fcb7a24e08 100644
>> --- a/src/hotspot/os/posix/attachListener_posix.cpp
>> +++ b/src/hotspot/os/posix/attachListener_posix.cpp
>> @@ -60,105 +60,6 @@
>>  //    obtain the credentials of client. We check that the effective uid
>>  //    of the client matches this process.
>>  
>> -<<<<<<< eea5480cdda (8358956: [CRaC] Fix NULL usage in hotspot)
>> -=======
>> -// forward reference
>> -class PosixAttachOperation;
>> -
>> -class PosixAttachListener: AllStatic {
>> - private:
>> -  // the path to which we bind the UNIX domain socket
>> -  static char _path[UNIX_PATH_MAX];
>> -  static bool _has_path;
>> -
>> -  // the file descriptor for the listening socket
>> -  static volatile int _listener;
>> -
>> -  static bool _atexit_registered;
>> -
>> - public:
>> -  static void set_path(char* path) {
>> -    if (path == nullptr) {
>> -      _path[0] = '\0';
>> -      _has_path = false;
>> -    } else {
>> -      strncpy(_path, path, UNIX_PATH_MAX);
>> -      _path[UNIX_PATH_MAX-1] = '\0';
>> -      _has_path = true;
>> -    }
>> -  }
>> -
>> -  static void set_listener(int s)               { _listener = s; }
>> -
>> -  // initialize the listener, returns 0 if okay
>> -  static int init();
>> -
>> -  static char* path()                   { return _path; }
>> -  static bool has_path()                { return _has_path; }
>> -  static int listener()                 { return _listener; }
>> -
>> -  static PosixAttachOperation* dequeue();
>> -};
>> -
>> -class SocketChannel : public AttachOperation::RequestReader, public AttachOperation::ReplyWriter {
>> -private:
>> -  int _socket;
>> -public:
>> -  SocketChannel(int socket) : _socket(socket) {}
>> -  ~SocketChannel() {
>> -    close();
>> -  }
>> -
>> -  bool opened() const {
>> -    return _socket != -1;
>> -  }
>> -
>> -  void close() {
>> -    if (opened()) {
>> -      ::shutdown(_socket, SHUT_RDWR);
>> -      ::close(_socket);
>> -      _socket = -1;
>> -    }
>> -  }
>> -
>> -  // RequestRe...
>
> Dmitry Cherepanov has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - fix windows runners
>  - update complete

Marked as reviewed by tpushkin (Committer).

-------------

PR Review: https://git.openjdk.org/crac/pull/238#pullrequestreview-2970966320


More information about the crac-dev mailing list