Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR [v2]

2021-01-21 Thread Brian Burkhalter
On Thu, 21 Jan 2021 18:58:53 GMT, Alan Bateman wrote: >> Brian Burkhalter has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8259943: Suppress RESTARTABLE for non-AIX. > > src/java.base/unix/native/libjava/io_util_md.c line 173: > >> 171:

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR [v3]

2021-01-21 Thread Alan Bateman
On Thu, 21 Jan 2021 19:12:16 GMT, Brian Burkhalter wrote: >> Please review this small change to handle `EINTR` from `close()` in >> `fileDescriptorClose()`. The function `handleGetLength()` is also changed to >> handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the >>

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR [v3]

2021-01-21 Thread Brian Burkhalter
> Please review this small change to handle `EINTR` from `close()` in > `fileDescriptorClose()`. The function `handleGetLength()` is also changed to > handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the > file. Brian Burkhalter has updated the pull request incrementally

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR [v2]

2021-01-21 Thread Alan Bateman
On Thu, 21 Jan 2021 18:56:59 GMT, Brian Burkhalter wrote: >> Please review this small change to handle `EINTR` from `close()` in >> `fileDescriptorClose()`. The function `handleGetLength()` is also changed to >> handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the >>

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR [v2]

2021-01-21 Thread Brian Burkhalter
> On Jan 21, 2021, at 11:06 AM, Alan Bateman wrote: > > On Thu, 21 Jan 2021 18:56:59 GMT, Brian Burkhalter wrote: > >>> Please review this small change to handle `EINTR` from `close()` in >>> `fileDescriptorClose()`. The function `handleGetLength()` is also changed >>> to handle `EINTR`

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR [v2]

2021-01-21 Thread Brian Burkhalter
> Please review this small change to handle `EINTR` from `close()` in > `fileDescriptorClose()`. The function `handleGetLength()` is also changed to > handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the > file. Brian Burkhalter has updated the pull request incrementally

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR

2021-01-21 Thread Brian Burkhalter
On Thu, 21 Jan 2021 07:32:39 GMT, Alan Bateman wrote: >> Please review this small change to handle `EINTR` from `close()` in >> `fileDescriptorClose()`. The function `handleGetLength()` is also changed to >> handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the >> file. >

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR

2021-01-20 Thread Alan Bateman
On Wed, 20 Jan 2021 23:01:19 GMT, Brian Burkhalter wrote: > Please review this small change to handle `EINTR` from `close()` in > `fileDescriptorClose()`. The function `handleGetLength()` is also changed to > handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the > file.

Re: RFR: 8259943: FileDescriptor.close0 does not handle EINTR

2021-01-20 Thread Naoto Sato
On Wed, 20 Jan 2021 23:01:19 GMT, Brian Burkhalter wrote: > Please review this small change to handle `EINTR` from `close()` in > `fileDescriptorClose()`. The function `handleGetLength()` is also changed to > handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the > file.

RFR: 8259943: FileDescriptor.close0 does not handle EINTR

2021-01-20 Thread Brian Burkhalter
Please review this small change to handle `EINTR` from `close()` in `fileDescriptorClose()`. The function `handleGetLength()` is also changed to handle `EINTR` from `fstat64()` to match other uses of `fstat64()` in the file. - Commit messages: - 8259943: FileDescriptor.close0 does