<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/20732">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/20732/2/src/select.c">File src/select.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/20732/2/src/select.c@158">Patch Set #2, Line 158:</a> <code style="font-family:monospace,monospace">                                 g_poll.poll_size + POLL_ALLOC_GRANULARITY);</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">what would be the advantage? I don't really care much either way.</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">A process starting to alocate big quantities of fds is expected to keep allocating lots more, so you adapt the number of reallocations based on that (meaning for processes with lots of fds you don't end up calling realloc lots of times, incrementing 1024 at a time).</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/libosmocore/+/20732/2/src/select.c@363">Patch Set #2, Line 363:</a> <code style="font-family:monospace,monospace"> rc = poll(g_poll.poll, n_poll, polling ? 0 : osmo_timers_nearest_ms());</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">keeping multiple implementations around has the risk that over time only one of them will be tested  […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Agree with moving to epoll at some point in time, but still I'd keep the select() support around which may come handy at any point, and it's not really a lot of specific code to keep anyway.</p><p style="white-space: pre-wrap; word-wrap: break-word;">At least let's have that for a while and drop it in a different patch to it's clear what's needed to readd it if somebody needs it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This way we ensure we are prepared to support several backends, which will require once we add an epoll implementation since that's linux-specific afaik and we'll need to keep something else for older linux versions too.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/20732">change 20732</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmocore/+/20732"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I9e80da68a144b36926066610d0d3df06abe09bca </div>
<div style="display:none"> Gerrit-Change-Number: 20732 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 19 Oct 2020 10:17:46 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Comment-In-Reply-To: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>