Project

General

Profile

Actions

Bug #4399

closed

remsim-client not informed if slotmap is deleted

Added by laforge about 4 years ago. Updated about 4 years ago.

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

100%

Spec Reference:

Description

In the following sequence of events:
  • bankd + client connected to server
  • slotmap is established via RSRES
  • slotmap is communicated to client + bankd
  • communication happens ...
  • slotmap is removed via RSRES
  • slotmap is removed from bankd

There appears to be no communication to the client about the slotmap being removed.


Files

OS4339-2.pcap OS4339-2.pcap 14.3 KB laforge, 02/15/2020 07:38 PM
Actions #1

Updated by laforge about 4 years ago

As we can see in the attached OS4339-2.pcap, there is a ConfigClientBankReq sent to the client when the mapping is created, but there is none when the mapping is removed. On removal time, only a removeMappingReq to the bankd can be seen.

Actions #2

Updated by laforge about 4 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 10

Indeed, rspro_server.c contains a related FIXME:

static void clnt_st_connected_bankd(struct osmo_fsm_inst *fi, uint32_t event, void *data)
{
...
        switch (event) {
...
        case CLNTC_E_REMOVE_MAP_RES: /* Bankd acknowledges mapping was removed */
...
                /* slotmap_del() will remove it from both global and bank list */
                slotmap_del(map->maps, map);
                /* FIXME: update client! */
Actions #4

Updated by laforge about 4 years ago

A corresponding test for the remsim-server behavior is added in https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17187

Actions #5

Updated by laforge about 4 years ago

Test coverage for the remsim-client side is added in https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/17190

Actions #6

Updated by laforge about 4 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 80 to 100

all tests merged, code changes merged, tests pass locally.

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)