tacit/minix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

PFS: unblock the child when unsuspending an accept

Browse Source 
Previously, PFS would incorrectly try to unsuspend the parent (i.e.,
the listening socket), resulting in the child hanging until the other
side performed another action. Test56 started failing on this now.

Change-Id: I231ac5481c83ac45951d33aeecc8149273f48b11
This commit is contained in:
David van Moolenbroek 2013-08-21 01:05:05 +02:00 committed by Gerrit Code Review
parent 842c4eda7f
commit 3b56861db4
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
servers/pfs/uds.c
View File

@ -398,7 +398,7 @@ int do_accept(message *dev_m_in, message *dev_m_out)
int do_connect(message *dev_m_in, message *dev_m_out)
{
int minor;
int minor, child;
struct sockaddr_un addr;
int rc, i, j;
@ -446,26 +446,28 @@ int do_connect(message *dev_m_in, message *dev_m_out)
!strncmp(addr.sun_path, uds_fd_table[i].addr.sun_path,
UNIX_PATH_MAX)) {
if (uds_fd_table[i].child != -1) {
if ((child = uds_fd_table[i].child) != -1) {
/* the server is blocked on accept(2) --
* perform connection to the child
*/
rc = perform_connection(dev_m_in, dev_m_out,
&addr, minor, uds_fd_table[i].child);
&addr, minor, child);
if (rc == OK) {
uds_fd_table[i].child = -1;
#if DEBUG == 1
printf("(uds) [%d] {do_connect} revive %d\n", minor, i);
printf("(uds) [%d] {do_connect} revive %d\n", minor, child);
#endif
/* wake the parent (server) */
uds_fd_table[i].ready_to_revive = 1;
uds_unsuspend(dev_m_in->m_source, i);
uds_fd_table[child].ready_to_revive =
1;
uds_unsuspend(dev_m_in->m_source,
child);
}
return rc;