[PATCH] Support 51.011 EFecc format.

Andrzej Zaborowski andrew.zaborowski at intel.com
Thu Dec 17 05:51:28 PST 2009


Hi,

2009/12/16 Denis Kenzior <denkenz at gmail.com>:
> I applied this patch.  However, the way the code works today EFecc will be
> read after IMSI if the SIM is not locked and thus cached normally.  I won't
> mention other possible race conditions here resulting from different driver
> combinations.
>
> This means that any subsequent reads on a device/sim combination of a different
> generation will give potentially different results.  I'd still like us to cache
> SIM files according to phase if possible.  This seems to be the safest / sanest
> approach.

You're right.  Attached patch adds the SIM phase to the index of the
cache.  I used EFphase to figure out the SIM phase becuase this way
seems the least hacky.  This lets us remove some of the autodetection
of things that depend on the phase in other places in ofono, but right
now the autodetection can only be a gain for us so I'm not touching
it.

Do you see any way caching EFecc could have unwanted consequences?

Regards


More information about the ofono mailing list