#47320 put conn on work_q not poll list if conn has buffered more_data
Closed: Fixed None Opened 6 years ago by rmeggins.

in connection_threadmain(), if the connection is not in turbo mode, if we are using buffered reads and the connection has more data in the buffer after reading a PDU, we should not put the connection back into the poll() list, we should put the connection immediately back on the work_q to allow another thread to read the next PDU.


commit ed88c40
Author: Rich Megginson rmeggins@redhat.com
Date: Fri Apr 5 14:29:16 2013 -0600

Thanks to Rich for the advice.

Description: Additional change to
commit ed88c40
to fix the self-deadlock.
Replacing bind_credentials_set with bind_credentials_set_nolock
in handle_handshake_done. The handle_handshake_done is only
called from connection_read_operation, where lock on c_mutex
is already obtained. Thus, the replacement is safe.

Pushed to master: commit a784486

Metadata Update from @nhosoi:
- Issue assigned to rmeggins
- Issue set to the milestone: 1.3.2 - 04/13 (April)

3 years ago

Login to comment on this ticket.