<p>Vadim Yanitskiy has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/11137">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">trxcon/scheduler: pass talloc ctx directly to sched_prim_init()<br><br>Enforcing pointer to a 'trx_instance' structure is not flexible,<br>because it is used as parent talloc context only.<br><br>Change-Id: I5ab2ef5cea76f955bf72ef54541b3b75cdc2d23f<br>---<br>M src/host/trxcon/sched_prim.c<br>M src/host/trxcon/sched_trx.h<br>2 files changed, 5 insertions(+), 6 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/osmocom-bb refs/changes/37/11137/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/host/trxcon/sched_prim.c b/src/host/trxcon/sched_prim.c</span><br><span>index e71109b..e03f93f 100644</span><br><span>--- a/src/host/trxcon/sched_prim.c</span><br><span>+++ b/src/host/trxcon/sched_prim.c</span><br><span>@@ -42,16 +42,15 @@</span><br><span>  * Initializes a new primitive by allocating memory</span><br><span>  * and filling some meta-information (e.g. lchan type).</span><br><span>  *</span><br><span style="color: hsl(0, 100%, 40%);">- * @param  trx     TRX instance to be used as initial talloc context</span><br><span style="color: hsl(120, 100%, 40%);">+ * @param  ctx     parent talloc context</span><br><span>  * @param  prim    external prim pointer (will point to the allocated prim)</span><br><span>  * @param  pl_len  prim payload length</span><br><span>  * @param  chan_nr RSL channel description (used to set a proper chan)</span><br><span>  * @param  link_id RSL link description (used to set a proper chan)</span><br><span>  * @return         zero in case of success, otherwise a error number</span><br><span>  */</span><br><span style="color: hsl(0, 100%, 40%);">-int sched_prim_init(struct trx_instance *trx,</span><br><span style="color: hsl(0, 100%, 40%);">-     struct trx_ts_prim **prim, size_t pl_len,</span><br><span style="color: hsl(0, 100%, 40%);">-       uint8_t chan_nr, uint8_t link_id)</span><br><span style="color: hsl(120, 100%, 40%);">+int sched_prim_init(void *ctx, struct trx_ts_prim **prim,</span><br><span style="color: hsl(120, 100%, 40%);">+  size_t pl_len, uint8_t chan_nr, uint8_t link_id)</span><br><span> {</span><br><span>        enum trx_lchan_type lchan_type;</span><br><span>      struct trx_ts_prim *new_prim;</span><br><span>@@ -70,7 +69,7 @@</span><br><span>    len += pl_len; /* Requested payload size */</span><br><span> </span><br><span>      /* Allocate a new primitive */</span><br><span style="color: hsl(0, 100%, 40%);">-  new_prim = talloc_zero_size(trx, len);</span><br><span style="color: hsl(120, 100%, 40%);">+        new_prim = talloc_zero_size(ctx, len);</span><br><span>       if (new_prim == NULL) {</span><br><span>              LOGP(DSCH, LOGL_ERROR, "Failed to allocate memory\n");</span><br><span>             return -ENOMEM;</span><br><span>diff --git a/src/host/trxcon/sched_trx.h b/src/host/trxcon/sched_trx.h</span><br><span>index ff288f2..818c95a 100644</span><br><span>--- a/src/host/trxcon/sched_trx.h</span><br><span>+++ b/src/host/trxcon/sched_trx.h</span><br><span>@@ -273,7 +273,7 @@</span><br><span>       enum trx_lchan_type chan);</span><br><span> </span><br><span> /* Primitive management functions */</span><br><span style="color: hsl(0, 100%, 40%);">-int sched_prim_init(struct trx_instance *trx, struct trx_ts_prim **prim,</span><br><span style="color: hsl(120, 100%, 40%);">+int sched_prim_init(void *ctx, struct trx_ts_prim **prim,</span><br><span>    size_t pl_len, uint8_t chan_nr, uint8_t link_id);</span><br><span> int sched_prim_push(struct trx_instance *trx,</span><br><span>   struct trx_ts_prim *prim, uint8_t chan_nr);</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/11137">change 11137</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/11137"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmocom-bb </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I5ab2ef5cea76f955bf72ef54541b3b75cdc2d23f </div>
<div style="display:none"> Gerrit-Change-Number: 11137 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Vadim Yanitskiy <axilirator@gmail.com> </div>