Change in osmo-bts[master]: oc2g: l1if: delay trx initialization to avoid race condition

dexter 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-Project: osmo-bts
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id3bdc88d28417e422d2c0c33b03be06f1a4706c2
Gerrit-Change-Number: 13208
Gerrit-PatchSet: 2
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
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190312/777d1a0d/attachment.html>


More information about the gerrit-log mailing list