Attention is currently required from: fixeria.
8 comments:
File src/osmo-bts-trx/trx_if.c:
Patch Set #2, Line 1298: trx_udp_open
I still think having a helper function is better to avoid code duplication and copy-paste errors.
I prefer it the way it is, to have the setup of both together instead of having to jump from the helepr function to here, which was a pain during working on this patch.Also In the end most of the lines contain different parameters for each iofd.
Patch Set #2, Line 147: msg->data_len
`msgb_length(msg)` here and below
No, this is msg->data_len, no msg->len.
LOGPPHI(l1h->phy_inst, DTRX, LOGL_ERROR,
```suggestion […]
Done
Patch Set #2, Line 1110: trx_if_send_burst
This proposal looks good on paper, but not easy to implement. Here's a simpler solution: […]
^ Yes this commit is what I was planning to implement too.
Patch Set #2, Line 1128: buf = msgb_data(trx_data_last_msg);
setting `buf` here is probably not needed?
It is strictly not needed indeed, but given the complexity with static variables I prefer being on the safe case so that it becomes clear that buf is always set, even on the "goto sendall" path below, to avoid also potential false positives from static analysis.
```suggestion […]
Done
LOGPPHI(l1h->phy_inst, DTRX, LOGL_ERROR,
"send() failed on TRXD with rc=%d (%s)\n",
We're no longer calling `send()` here, right? […]
Done
Copy-paste! […]
Done
To view, visit change 41650. To unsubscribe, or for help writing mail filters, visit settings.