diff options
| author | Aleksander Morgado <aleksandermj@chromium.org> | 2024-07-30 22:56:06 +0000 |
|---|---|---|
| committer | Aleksander Morgado <aleksandermj@chromium.org> | 2024-09-30 07:55:05 +0000 |
| commit | 8e0cbd9c94ffd7f9122ade0f4cea198c7c4dfd0a (patch) | |
| tree | 3ea34d4425eaf0538345a02361f9a14c897a671e /include/ModemManager.h | |
| parent | a24a6ee4c5af47859e433fe9559946515fb1869c (diff) | |
base-modem: remove support for parallel enable/disable operations
Follow up changes will introduce support to run exclusive enable and
disable operations, effectively ensuring that only one enabling or
disabling task is processed at any given time, so the support for
completing parallel operations with the same result is no longer
needed.
That logic was anyway flawed, as it did not consider complex cases
like a disable request received between two enable requests, the order
of incoming requests was not maintained.
The main reason for adding the parallel enable/disable operation logic
was for the case where 2 separate users tried to enable the modem at
the same time. In that case, we did not want to run 2 separate
enabling operations from scratch, and completing one would complete
the second one as well. With the new operation lock logic, the same
result is achieved, as the 2nd enabling operation would find the modem
already enabled, if the 1st one succeeded.
This removal is done before introducing the new exclusive operation
support in the enable/disable path in order to make it easier and more
consistent with other operations that will also be included in the
synchronization.
Diffstat (limited to 'include/ModemManager.h')
0 files changed, 0 insertions, 0 deletions