Change in osmo-bsc[master]: hodec 2: favor moving dyn TS

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/.

neels gerrit-no-reply at lists.osmocom.org
Tue Nov 17 23:26:06 UTC 2020


neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/21204 )

Change subject: hodec 2: favor moving dyn TS
......................................................................


Patch Set 4:

(1 comment)

https://gerrit.osmocom.org/c/osmo-bsc/+/21204/4//COMMIT_MSG 
Commit Message:

https://gerrit.osmocom.org/c/osmo-bsc/+/21204/4//COMMIT_MSG@23 
PS4, Line 23: Among lchans on dynamic timeslots that could be moved, this code favors moving
            : later lchans; mainly because it makes for a simpler condition in the code.
> I'm not completely against it, but I'm not sure this is a really good reason... […]
sorry but you clearly are missing the point here :)

The question in this condition is not whether a dyn TS is moved or not, the question is *which one* do you pick. So if there are a choice of N dyn TS with identical ratings, it does not matter which one you pick to be handed over, all of those with the same avg_db are identically qualified candidates.

I have no strong argument to pick the last one, equally you have no strong argument for picking the first one.

The decision to move an lchan of that type is taken elsewhere, and there is a real gain doing so, and that decision is made orthogonally to finding an lchan matching that decision here.



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

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: Ic221b8d2687cdec0bf94410c84a4da43853f0900
Gerrit-Change-Number: 21204
Gerrit-PatchSet: 4
Gerrit-Owner: neels <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-Comment-Date: Tue, 17 Nov 2020 23:26:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: neels <nhofmeyr at sysmocom.de>
Comment-In-Reply-To: pespin <pespin at sysmocom.de>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201117/f95c9781/attachment.htm>


More information about the gerrit-log mailing list