<p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25415">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/25415/2/src/vty/cpu_sched_vty.c">File src/vty/cpu_sched_vty.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/+/25415/2/src/vty/cpu_sched_vty.c@425">Patch Set #2, Line 425:</a> <code style="font-family:monospace,monospace">      if (applynow) {</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">All we allow ourselves to do is to apply some changes immediately, and others only at whatever other natural point in time (bts reconnect, etc.), and we document that via attributes.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">That's precisely my point here. User configures it and it is expected to be applied at runtime by the app code, for which each thread calls the related function to apply its config. So it's not generic VTY code or the user deciding when to do the mangling with threads at runtime, but rather the app code asking the infra to apply desired thread option at a specific point of time. If one wants to change the usual thread config, it is expected to be applied next time the process is started. If someone really wants to mangle with thread cpu affinity while the process is running, better go do it with linux tools allowing so (they exist) since you anyway should know what you are doing.<br>So in general when you use this VTY commands, you are configuring where to pin the threads, but when that actually happens is defined by the app itself. You are simply configuring where to allow each thread.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">"enable" style commands don't change persistent configuration.  That's what the configure node and its sub-nodes are for.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I think I wrote there " store it temporarily but not written back to cfg file (have a ->temporary=true)." so I think we agree here? In any case, that's given the event that we really want to use the VTY to apply instantaneous cpu affinity interactively rather than simply configuring it to do so upon startup.</p><p style="white-space: pre-wrap; word-wrap: break-word;">That's my five cents, up to you whether you want to change the state of things here. I'm not for it, but I'm not going to block it. If you want to merge this, then at least provide proper update to the documentation of the cpu affinity section explaining the new behavior and ideas.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/25415">change 25415</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/+/25415"/><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: Id8405099e6b316c2e14fb0c9b3c5e80a68a91277 </div>
<div style="display:none"> Gerrit-Change-Number: 25415 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Hoernchen <ewild@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 14 Sep 2021 09:32:26 +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>