diff options
| author | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-02-01 20:42:52 +0100 |
|---|---|---|
| committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-02-05 12:36:41 +0100 |
| commit | f9d19ee85c914d8a8fd53590c2ff6920874ee9d3 (patch) | |
| tree | 4cf03ea2aa253c9c77ff6de3ba141580736eabd0 /src/osmo-bts-virtual | |
| parent | 1948aeb269c2943ecc8a9d9f8a6156fe6cfe77b9 (diff) | |
virtual: l1_if.c: Remove unneeded warning message
Commit 5eb17e28acdd6fba22a1f2e60f4d55aaef18b47a added this warning when
implementing the full function. However, other backends seem to be also
passing the primitive from the stack to l1sap_up without any visible issues.
Furthermore, according to the documentation of l1sap_up, it takes ownership of the
msgb, but doesn't take ownserhip of the prim itself.
Change-Id: I45fe40e3377eac999d1dac5356678195381d94ca
Diffstat (limited to 'src/osmo-bts-virtual')
| -rw-r--r-- | src/osmo-bts-virtual/l1_if.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/osmo-bts-virtual/l1_if.c b/src/osmo-bts-virtual/l1_if.c index cb6d483d..e7478a02 100644 --- a/src/osmo-bts-virtual/l1_if.c +++ b/src/osmo-bts-virtual/l1_if.c @@ -163,8 +163,7 @@ static void virt_um_rcv_cb(struct virt_um_inst *vui, struct msgb *msg) goto nomessage; } - /* forward primitive, forwarded msg will not be freed */ -#warning "we cannot just pass a l1sap primitive on the stack!!!" + /* forward primitive, lsap takes ownership of the msgb. */ l1sap_up(pinst->trx, &l1sap); DEBUGP(DL1P, "Message forwarded to layer 2.\n"); return; |