Warning: Can't synchronize with repository "(default)" (No changeset 96d22ec3fa3ef6de3ea8dc0d7d398adc9aa071cf in the repository). Look in the Trac log for more information.

Ticket #343 (closed defect: fixed)

Opened 5 years ago

Last modified 5 years ago

Bug in state transitions

Reported by: spaetz Owned by: mickey
Priority: major Milestone: milestone5.5
Component: framework/odeviced Version:
Keywords: Cc:


Please assign to mickey, I discussed this with in on IRC.

The attach (yet untested) patch solves this problem:

1) if we currently prohibitTransition to a state we will check if we are in that exact state and go back by one, currently. As e.g. requesting the Resource "Display" prohibits state "idle_dim" this means that we will be thrown back to "busy" if we are in "idle_dim" but nothing will happen if we are already in "idle_prelock". The attached patch will solve that by throwing us into the last allowed state if we are *in or beyond* the prohibited state.

(A theoretical issue is that "previousState" currently wraps around when we are in the beginning. So if *something* prohibits state "awake" it will go into "suspend" state. But it behaved the same way before this patch).

Generally, I think it would be more efficient to use indices as state parameters rather than throwing strings around and looking up their index all the time. But that is a minor (and probably future issue).


framework_subsystems_odeviced_idlenotifier.py.diff (491 bytes) - added by spaetz 5 years ago.
patch to framework/subsystems/odeviced/idlenotifier.py

Change History

Changed 5 years ago by spaetz

patch to framework/subsystems/odeviced/idlenotifier.py

comment:1 Changed 5 years ago by spaetz

  • Owner changed from jluebbe to mickey
  • Component changed from framework/oeventsd to framework/odeviced

comment:2 Changed 5 years ago by stefan

  • Milestone set to milestone5.5

comment:3 Changed 5 years ago by mickey

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.