Opened 3 years ago

Closed 3 years ago

#1589 closed defect (fixed)

[PATCH] Remove connection limit based on FD_SETSIZE

Reported by: sckobras Owned by: Daniel Kobras <d.kobras@…>
Priority: normal Milestone:
Component: sge Version: 8.1.9
Severity: major Keywords:
Cc:

Description

The changeset https://arc.liv.ac.uk/trac/SGE/changeset/20140406123948-0ed77-974278d41c87f2f53e735ad9d271fc2c211d3395/sge (Assume POSIX poll available) got rid of the conditional USE_POLL, but the changes to cl_commlib.c contained two glitches where #ifndef USE_POLL was treated like #ifdef USE_POLL, ie. the preprocessor directives were removed, but the enclosed code block was not. As a result, the releases 8.1.7 up to 8.1.9 impose a connection limit based on FD_SETSIZE, even though poll() is used instead of select(). The attached patch removes both leftover hunks from the original changeset, which means that connections are limited by resource constraints again, as they should be. It fixes connection problems in larger environments with 1000 or more hosts.

Attachments (1)

0001-Remove-connection-limit-based-on-FD_SETSIZE.patch (1.9 KB) - added by sckobras 3 years ago.

Download all attachments as: .zip

Change History (3)

comment:2 Changed 3 years ago by dlove

Oops. Thanks.

comment:3 Changed 3 years ago by Daniel Kobras <d.kobras@…>

  • Owner set to Daniel Kobras <d.kobras@…>
  • Resolution set to fixed
  • Status changed from new to closed

In 4931/sge:

Fix #1589: Remove connection limit based on FD_SETSIZE

Note: See TracTickets for help on using tickets.