On Wed, Jul 4, 2018 at 3:00 PM, Baesken, Matthias <matthias.baes...@sap.com> 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:alan.bate...@oracle.com] >> Sent: Mittwoch, 4. Juli 2018 13:43 >> To: Baesken, Matthias <matthias.baes...@sap.com>; David Holmes >> <david.hol...@oracle.com>; Thomas Stüfe <thomas.stu...@gmail.com> >> Cc: serviceability-dev (serviceability-dev@openjdk.java.net) <serviceability- >> d...@openjdk.java.net> >> 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
