Change in osmo-bts[master]: oc2g: l1if: delay trx initialization to avoid race condition
gerrit-no-reply at lists.osmocom.org
Tue Mar 12 08:38:34 UTC 2019
dexter has posted comments on this change. ( https://gerrit.osmocom.org/13208 )
Change subject: oc2g: l1if: delay trx initialization to avoid race condition
Patch Set 2:
> I think this commit is good as a proof of concept, but it's a hack
> rather than a fix, so I'd prefer having a better fix for this issue
> (like storing some state and delaying the opstat to a later point
> in time after receiving some event which triggers sending it).
We are aware of that, thats why we first decided not to put it into review at all. We then decided to submit it anyway because for now it is better than nothing and it does not affect other BTS implementations. See also the fixme note int the code.
To view, visit https://gerrit.osmocom.org/13208
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: dexter <pmaier at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: dexter <pmaier at sysmocom.de>
Gerrit-Comment-Date: Tue, 12 Mar 2019 08:38:34 +0000
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the gerrit-log