Project

General

Profile

Actions

Feature #2949

closed

use RR Assignment for Handover within a cell

Added by neels about 6 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
02/16/2018
Due date:
% Done:

0%

Spec Reference:

Description

Currently, we always use RSL_ACT_INTER_ASYNC and RR Handover Command, both for handover between different cells and for re-assignment within the same cell.

There are patches around to use RSL_ACT_INTRA_IMM_ASS and gsm48_send_rr_ass_cmd() for re-assignment within the same cell, instead.
However, that hasn't worked out in testing so far.

Implement Assignment by actual intra-cell assignment instead of Handover Command.


Related issues

Related to OsmoBSC - Bug #3357: osmo-bsc doesn't perform RSL MODE MODIFY on ASSIGNMENT if channel is compatible.Resolved06/26/2018

Actions
Is duplicate of OsmoBSC - Feature #3277: handover: for intra-cell re-assignment, use Assignment procedure, not HandoverResolvedneels05/17/2018

Actions
Actions #1

Updated by neels about 6 years ago

This patch in essence came from the jolly/new_handover branch, does not seem to be sufficient.

commit adf9dc3065329714d1c67aa04600de23d7a70452
Author: Neels Hofmeyr <neels@hofmeyr.de>
Date:   Mon Feb 12 17:28:56 2018 +0100

    use RR Assignment Command for intra-cell re-assignment

    Change-Id: I26b62731f13cd7d4217b68eb76e298a6216af896

diff --git a/src/libbsc/handover_logic.c b/src/libbsc/handover_logic.c
index cdc21f56f..2d6b6bddb 100644
--- a/src/libbsc/handover_logic.c
+++ b/src/libbsc/handover_logic.c
@@ -158,7 +158,9 @@ int bsc_handover_start(enum hodec_id from_hodec_id, struct gsm_lchan *old_lchan,
     new_lchan->conn->ho_lchan = new_lchan;

     rc = rsl_chan_activate_lchan(new_lchan,
-                     ho->async ? RSL_ACT_INTER_ASYNC : RSL_ACT_INTER_SYNC,
+                     ho->inter_cell
+                       ? (ho->async ? RSL_ACT_INTER_ASYNC : RSL_ACT_INTER_SYNC)
+                       : RSL_ACT_INTRA_IMM_ASS,
                      ho->ho_ref);
     if (rc < 0) {
         LOGPHO(ho, LOGL_INFO, "%s Failure: activate lchan rc = %d\n",
@@ -233,7 +235,10 @@ static int ho_chan_activ_ack(struct gsm_lchan *new_lchan)
     /* we can now send the 04.08 HANDOVER COMMAND to the MS
      * using the old lchan */

-    gsm48_send_ho_cmd(ho->old_lchan, new_lchan, new_lchan->ms_power, ho->ho_ref);
+    if (ho->inter_cell)
+        gsm48_send_ho_cmd(ho->old_lchan, new_lchan, new_lchan->ms_power, ho->ho_ref);
+    else
+        gsm48_send_rr_ass_cmd(ho->old_lchan, new_lchan, new_lchan->ms_power);

     /* start T3103.  We can continue either with T3103 expiration,
      * 04.08 HANDOVER COMPLETE or 04.08 HANDOVER FAIL */
Actions #2

Updated by neels over 5 years ago

  • Is duplicate of Feature #3277: handover: for intra-cell re-assignment, use Assignment procedure, not Handover added
Actions #3

Updated by laforge over 5 years ago

  • Related to Bug #3357: osmo-bsc doesn't perform RSL MODE MODIFY on ASSIGNMENT if channel is compatible. added
Actions #4

Updated by neels over 2 years ago

  • Status changed from New to Resolved

Implemented in
commit 69def1f97e427fa65d242f21110f01cb5d0a2818
hodec 2: do intra-cell congestion resolution by Assignment
Change-Id: Id56a890106b93fcee67ac9401b890e7b63bba421

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)