tree 211e84f020cacfe0494e9d126850de1465ea8eba
parent 6990f387b89d87e7acb5141dc516b97c7c709b40
author Vincent Palatin <vpalatin@chromium.org> 1618407146 +0200
committer Commit Bot <commit-bot@chromium.org> 1621235071 +0000

UPSTREAM: sim-mbim: increase the timeout for the MBIM_CID_HOME_PROVIDER query

Increase the timeout from 10 seconds to 30 seconds when reading the
operator information through the MBIM_CID_HOME_PROVIDER query.
We have observed cases where it took more than 12 seconds on a Fibocom
L850-GL modem with specific China Unicom SIMs.

This timeout was actually the cause of a corruption of the operator name
(e.g. returning '@' rather than the real operator name) on the Intel
XMM7360 based L850-GL. The following scenario was happening:
- the base-sim code sends a first MBIM 'home-provider' query to read the
  operator identifier.
- this query times-out after 10 seconds.
- the base-sim immediately sends another MBIM 'home-provider' query to
  read the operator name.
- the modem returns a corrupted operator identifier.
Intel helped for the debugging and indicates that when the first query
happens, the modem starts reading the SIM files (EF SPN/PNN), apparently
it's taking a long time, when it's receives the second query while the
first one is not entirely processed, this triggers a bug (in the modem
firmware) and returns a corrupted name.

(cherry picked from commit f7ec13deb88e9bd292d69ec7442dc892e51f3c52)

BUG=b:176771274
TEST=manual, insert specific SIMs and see it displays the correct
network name.

Change-Id: Ib0a3971389ddc5c2bb6f085aabb7142d79c03c84
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/modemmanager-next/+/2840487
Tested-by: Vincent Palatin <vpalatin@chromium.org>
Reviewed-by: Madhav . <madhavadas@google.com>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Commit-Queue: Vincent Palatin <vpalatin@chromium.org>
(cherry picked from commit 229deec25fef9fd773ee1e60068a480af18ab64f)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/modemmanager-next/+/2847592
Tested-by: rasheed.hsueh <rasheed.hsueh@lcfc.corp-partner.google.com>
Reviewed-by: Henry Sun <henrysun@google.com>
Commit-Queue: Henry Sun <henrysun@google.com>