Change in osmo-bts[master]: scheduler: treat subsequent lchan (de)activation as error

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

fixeria gerrit-no-reply at lists.osmocom.org
Mon Oct 19 08:53:34 UTC 2020


fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bts/+/20698 )

Change subject: scheduler: treat subsequent lchan (de)activation as error
......................................................................


Patch Set 2:

> Patch Set 2:
> 
> Is there any a behavioral change in this commit? looks more like a cosmetic one..

If the callers of this function check returned value, then yes. Before this change we used to skip matching channels that are already (de)activated and return 0, but now we return -EINVAL. Just like in case when no channel matches.


-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bts/+/20698
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bts
Gerrit-Branch: master
Gerrit-Change-Id: I6256280cae35b2b4d7a8ba4b3913ca69cde22611
Gerrit-Change-Number: 20698
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-CC: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Mon, 19 Oct 2020 08:53:34 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201019/3526cd27/attachment.htm>


More information about the gerrit-log mailing list