On Wed, Jul 4, 2018 at 3:00 PM, Baesken, Matthias <[email protected]> wrote: > Ok, I change the comment to "AIX recommends to repeat the close call on > EINTR" and push , is that fine with you ? >
Sure. I do not need another webrev. ..Thomas > Best regards, Matthias > >> -----Original Message----- >> From: Alan Bateman [mailto:[email protected]] >> Sent: Mittwoch, 4. Juli 2018 13:43 >> To: Baesken, Matthias <[email protected]>; David Holmes >> <[email protected]>; Thomas Stüfe <[email protected]> >> Cc: serviceability-dev ([email protected]) <serviceability- >> [email protected]> >> Subject: Re: 8206145 : dbgsysSocketClose - do not restart close if errno is >> EINTR [linux] - was : RE: RFR : 8205959 : Do not restart close if errno is >> EINTR >> >> On 04/07/2018 12:37, Baesken, Matthias wrote: >> > Hi all, here is another webrev : >> > >> > http://cr.openjdk.java.net/~mbaesken/webrevs/8206145.2/ >> > >> > - switched to the coding proposed by Thomas >> > - added a small comment >> > >> The code looks okay but the comment is a bit strange. A simple "AIX >> recommends to repeat the close call on EINTR" should be ignore and drop >> the bug reference. >> >> -Alan
