init: Fixes for file descriptor accounting #27539

1. 
   Empact commented at 6:33 PM on April 28, 2023: contributor

   This rebases and revises #16003 for clarity of review.

   Aims to fix #18911.

2. 
   DrahtBot commented at 6:33 PM on April 28, 2023: contributor

   <!--e57a25ab6845829454e8d69fc972939a-->

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   <!--006a51241073e994b41acfe9ec718e94-->

   Code Coverage

   For detailed information about the code coverage, see the test coverage report.

   <!--021abf342d371248e50ceaed478a90ca-->

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   Concept ACK	fjahr, TheCharlatan

   If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

   <!--174a7506f384e20aa4161008e828411d-->

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #29415 (Broadcast own transactions only via short-lived Tor or I2P connections by vasild)

   If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

3. Empact force-pushed on Apr 28, 2023
4. DrahtBot added the label CI failed on Apr 28, 2023
5. Empact force-pushed on Apr 28, 2023
6. DrahtBot removed the label CI failed on Apr 28, 2023
7. glozow added the label Docs on May 8, 2023
8. in src/init.cpp:981 in bbad166e42 outdated

    935 | +    }
    936 |  
    937 | -#ifdef USE_POLL
    938 | -    int fd_max = nFD;
    939 | -#else
    940 | -    int fd_max = FD_SETSIZE;
   

   ---

   ---

   

   fjahr commented at 8:14 AM on May 24, 2023:

   Needs an explanation of why it is ok to take FD_SETSIZE completely out of the equation in the commit description.

9. 
   fjahr commented at 8:17 AM on May 24, 2023: contributor

   Concept ACK

   I think it would be very helpful to add comments to explain what is going on. See my accidentally duplicate PR for some suggestions: #27730.

10. achow101 requested review from TheCharlatan on Sep 20, 2023
11. DrahtBot added the label CI failed on Sep 29, 2023
12. DrahtBot removed the label CI failed on Oct 4, 2023
13. 
    TheCharlatan commented at 12:06 PM on October 24, 2023: contributor

    Concept ACK

    First three commits look good, but the last one needs more descriptions and comments as mentioned by fjahr.

14. DrahtBot added the label CI failed on Oct 25, 2023
15. 
    DrahtBot commented at 2:34 PM on February 5, 2024: contributor

    <!--2e250dc3d92b2c9115b66051148d6e47-->

    🤔 There hasn't been much activity lately and the CI seems to be failing.

    If no one reviewed the current pull request by commit hash, a rebase can be considered. While the CI failure may be a false positive, the CI hasn't been running for some time, so there may be a real issue hiding as well. A rebase triggers the latest CI and makes sure that no silent merge conflicts have snuck in.

16. refactor: constantize the unchanging nBind 446818c8c2
17. refactor: Extract RESERVED_FILE_DESCRIPTORS constant 8eefee0984
18. Log the file descriptor stats on init

    Co-authored-by: tryphe <tryphe@noreply.github.com>

    bf3369995e
19. init: reserve file descriptors for bind() and accept()

    And fix the amount of descriptors requested
    
    Co-authored-by: tryphe <tryphe@noreply.github.com>

    44872b32bc
20. Empact force-pushed on Mar 4, 2024
21. DrahtBot removed the label CI failed on Mar 4, 2024
22. 
    TheCharlatan commented at 7:53 PM on May 1, 2024: contributor

    This was rebased fairly recently, but you have not addressed this open comment #27539 (review) . Are you still working on this?

23. 
    sr-gi commented at 5:39 PM on May 8, 2024: member

    I've been reviewing #29415 (which is extending the current brain-hurting approach on master), so I ended up considering fixing this. If @Empact is not working on it anymore I may pick it up

24. 
    fanquake commented at 8:10 AM on May 15, 2024: member

    Closing for now, re #30065.

25. fanquake closed this on May 15, 2024

    ---

26. bitcoin locked this on May 15, 2025

Review Requested
TheCharlatan