RFR: 8329190: (ch) DatagramChannel.receive should throw ClosedChannelException when called on closed channel

[Alan Bateman]

On Tue, 16 Apr 2024 13:35:22 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Do you think this additional `ensureOpen()` that we introduce here would need to be called while holding the `stateLock` monitor? I don't think it would be needed given the implementation of ensureOpen() checks a volatile field but then I see that existing calls to ensureOpen() are done while holding the monitor on `stateLock`.

No, that would lead to contention when doing non-blocking I/O. The cases where you see ensureOpen called while holding the stateLock are blocking I/O ops or non-I/O ops that require more complex coordination with close.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18796#discussion_r1567395321