<p>laforge has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/libosmocore/+/22557">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">ns2: encapsulate calls to nsvc->bind->send_vc()<br><br>Don't call the function pointer of the underlying transport<br>directly, as we many still want to do some common processing,<br>such as statistics counting (see follow-up patches).<br><br>Change-Id: I8d14c7b8d3aacc2bed033072d7934dbd6aab41df<br>---<br>M src/gb/gprs_ns2_message.c<br>1 file changed, 17 insertions(+), 12 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/57/22557/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/gb/gprs_ns2_message.c b/src/gb/gprs_ns2_message.c</span><br><span>index 67ee254..d97c2a5 100644</span><br><span>--- a/src/gb/gprs_ns2_message.c</span><br><span>+++ b/src/gb/gprs_ns2_message.c</span><br><span>@@ -166,6 +166,11 @@</span><br><span> }</span><br><span> </span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+static int ns_vc_tx(struct gprs_ns2_vc *nsvc, struct msgb *msg)</span><br><span style="color: hsl(120, 100%, 40%);">+{</span><br><span style="color: hsl(120, 100%, 40%);">+  return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span> /* transmit functions */</span><br><span> static int ns2_tx_simple(struct gprs_ns2_vc *nsvc, uint8_t pdu_type)</span><br><span> {</span><br><span>@@ -183,7 +188,7 @@</span><br><span> </span><br><span>         nsh->pdu_type = pdu_type;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Transmit a NS-BLOCK on a given NS-VC.</span><br><span>@@ -216,7 +221,7 @@</span><br><span>      msgb_tvlv_put(msg, NS_IE_CAUSE, 1, &cause);</span><br><span>      msgb_tvlv_put(msg, NS_IE_VCI, 2, (uint8_t *) &nsvci);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Transmit a NS-BLOCK-ACK on a given NS-VC.</span><br><span>@@ -245,7 +250,7 @@</span><br><span> </span><br><span>      msgb_tvlv_put(msg, NS_IE_VCI, 2, (uint8_t *) &nsvci);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Transmit a NS-RESET on a given NS-VC.</span><br><span>@@ -278,7 +283,7 @@</span><br><span>      msgb_tvlv_put(msg, NS_IE_VCI, 2, (uint8_t *) &nsvci);</span><br><span>    msgb_tvlv_put(msg, NS_IE_NSEI, 2, (uint8_t *) &nsei);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Transmit a NS-RESET-ACK on a given NS-VC.</span><br><span>@@ -313,7 +318,7 @@</span><br><span>  msgb_tvlv_put(msg, NS_IE_VCI, 2, (uint8_t *)&nsvci);</span><br><span>     msgb_tvlv_put(msg, NS_IE_NSEI, 2, (uint8_t *)&nsei);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-    return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Transmit a NS-UNBLOCK on a given NS-VC.</span><br><span>@@ -399,7 +404,7 @@</span><br><span>    nsh->data[1] = bvci >> 8;</span><br><span>   nsh->data[2] = bvci & 0xff;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-  return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Transmit a NS-STATUS on a given NS-VC.</span><br><span>@@ -454,7 +459,7 @@</span><br><span>     if (cause == NS_CAUSE_BVCI_UNKNOWN)</span><br><span>          msgb_tvlv_put(msg, NS_IE_VCI, 2, (uint8_t *)&bvci);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-     return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> </span><br><span>@@ -514,7 +519,7 @@</span><br><span>                             (const uint8_t *)ip6_elems);</span><br><span>   }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Encode + Transmit a SNS-CONFIG as per Section 9.3.4.</span><br><span>@@ -569,7 +574,7 @@</span><br><span>                             (const uint8_t *)ip6_elems);</span><br><span>   }</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Encode + Transmit a SNS-CONFIG-ACK as per Section 9.3.5.</span><br><span>@@ -608,7 +613,7 @@</span><br><span>   if (cause)</span><br><span>           msgb_tvlv_put(msg, NS_IE_CAUSE, 1, cause);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-  return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> </span><br><span>@@ -657,7 +662,7 @@</span><br><span>       if (ip6_ep_nr >= 0)</span><br><span>               msgb_tv16_put(msg, NS_IE_IPv6_EP_NR, ip6_ep_nr);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-    return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> /*! Encode + Transmit a SNS-SIZE-ACK as per Section 9.3.8.</span><br><span>@@ -692,7 +697,7 @@</span><br><span>     if (cause)</span><br><span>           msgb_tvlv_put(msg, NS_IE_CAUSE, 1, cause);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-  return nsvc->bind->send_vc(nsvc, msg);</span><br><span style="color: hsl(120, 100%, 40%);">+  return ns_vc_tx(nsvc, msg);</span><br><span> }</span><br><span> </span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmocore/+/22557">change 22557</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/+/22557"/><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: I8d14c7b8d3aacc2bed033072d7934dbd6aab41df </div>
<div style="display:none"> Gerrit-Change-Number: 22557 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>