Project

General

Profile

Bug #4414

remsim-client-st2 doesn't perform modem reset after bankd reconnect

Added by laforge about 1 month ago. Updated 26 days ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
remsim-client
Target version:
-
Start date:
02/21/2020
Due date:
% Done:

100%


Description

If I execute the following sequence of events:
  • start remsim-client/server/bankd
  • create a slotmap
    • modem is automatically reset at that point
  • modem starts to talk to remote sim
  • at some point, delete slotmap
  • create a new slotmap (could be the same as the old, or to a different card
    • modem is not automatically reset at that point

This lack of a modem reset means that the communication fails for a long time until some timeout is hit: As the modem may hve issued a not-yet-completed APDU to the first SIM card, it never receives any response.

Associated revisions

Revision 0e968cce (diff)
Added by laforge 26 days ago

client: major restructuring around new main_fsm

The remsim_client code already used FSMs for the connections
to both remsim-server and remsim-bankd. However the 'main' part of the
program was not yet implemented as a FSM, making it somewhat difficult
to perform the right actions in every possible situation.

This commit re-structures the code around a central main_fsm, which
gets notified from the per-connection FSMs and which handles the common
processing. It also handles the execution of external script commands,
and hence further unifies the code base between the different backends
(simtrace2, ifd_handler, shell)

Closes: #4414

Change-Id: I44a430bc5674dea00ed72a0b28729ac8bcb4e022

History

#1 Updated by laforge about 1 month ago

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

This requires the introduction of a proper state machine for the 'main' program of remsim-client. I'm working on it.

#2 Updated by laforge about 1 month ago

  • % Done changed from 10 to 40

work-in-progress code in laforge/client-fsm branch. Currently undergoing manual testing.

#3 Updated by laforge 29 days ago

  • Priority changed from High to Urgent

#4 Updated by laforge 26 days ago

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

patches for main_fsm merged

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)