Merge "DO NOT MERGE fix ratratchet when CID change right before RAT change." into pi-dev (cd8dfe2d) · Commits · e / os / android_frameworks_opt_telephony

src/java/com/android/internal/telephony/RatRatcheter.java

+28 −10

Original line number	Diff line number	Diff line
		@@ -27,8 +27,6 @@ import android.telephony.Rlog;
		import android.util.SparseArray;
		import android.util.SparseIntArray;

		import java.util.ArrayList;

		/**
		* This class loads configuration from CarrierConfig and uses it to determine
		* what RATs are within a ratcheting family. For example all the HSPA/HSDPA/HSUPA RATs.
		@@ -49,6 +47,9 @@ public class RatRatcheter {

		private final Phone mPhone;

		private boolean mVoiceRatchetEnabled = true;
		private boolean mDataRatchetEnabled = true;

		/** Constructor */
		public RatRatcheter(Phone phone) {
		mPhone = phone;
		@@ -75,16 +76,33 @@ public class RatRatcheter {
		}
		}

		public void ratchetRat(ServiceState oldSS, ServiceState newSS) {
		public void ratchetRat(ServiceState oldSS, ServiceState newSS, boolean locationChange) {
		// temporarily disable rat ratchet on location change.
		if (locationChange) {
		mVoiceRatchetEnabled = false;
		mDataRatchetEnabled = false;
		return;
		}
		if (mVoiceRatchetEnabled) {
		int newVoiceRat = ratchetRat(oldSS.getRilVoiceRadioTechnology(),
		newSS.getRilVoiceRadioTechnology());
		newSS.setRilVoiceRadioTechnology(newVoiceRat);
		} else if (oldSS.getRilVoiceRadioTechnology() != newSS.getRilVoiceRadioTechnology()) {
		// resume rat ratchet on following rat change within the same location
		mVoiceRatchetEnabled = true;
		}

		if (mDataRatchetEnabled) {
		int newDataRat = ratchetRat(oldSS.getRilDataRadioTechnology(),
		newSS.getRilDataRadioTechnology());
		newSS.setRilDataRadioTechnology(newDataRat);
		} else if (oldSS.getRilVoiceRadioTechnology() != newSS.getRilVoiceRadioTechnology()) {
		// resume rat ratchet on following rat change within the same location
		mVoiceRatchetEnabled = true;
		}

		boolean newUsingCA = oldSS.isUsingCarrierAggregation() \|\|
		newSS.isUsingCarrierAggregation();

		newSS.setRilVoiceRadioTechnology(newVoiceRat);
		newSS.setRilDataRadioTechnology(newDataRat);
		newSS.setIsUsingCarrierAggregation(newUsingCA);
		}

src/java/com/android/internal/telephony/ServiceStateTracker.java

+4 −3

Original line number	Diff line number	Diff line
		@@ -139,7 +139,8 @@ public class ServiceStateTracker extends Handler {
		* and ignore stale responses. The value is a count-down of
		* expected responses in this pollingContext.
		*/
		private int[] mPollingContext;
		@VisibleForTesting
		public int[] mPollingContext;
		private boolean mDesiredPowerState;

		/**
		@@ -2647,8 +2648,8 @@ public class ServiceStateTracker extends Handler {
		// ratchet the new tech up through it's rat family but don't drop back down
		// until cell change or device is OOS
		boolean isDataInService = mNewSS.getDataRegState() == ServiceState.STATE_IN_SERVICE;
		if (!hasLocationChanged && isDataInService) {
		mRatRatcheter.ratchetRat(mSS, mNewSS);
		if (isDataInService) {
		mRatRatcheter.ratchetRat(mSS, mNewSS, hasLocationChanged);
		}

		boolean hasRilVoiceRadioTechnologyChanged =

tests/telephonytests/src/com/android/internal/telephony/ServiceStateTrackerTest.java

+130 −0

Original line number	Diff line number	Diff line
		@@ -158,6 +158,10 @@ public class ServiceStateTrackerTest extends TelephonyTest {
		mBundle.putStringArray(
		CarrierConfigManager.KEY_NON_ROAMING_OPERATOR_STRING_ARRAY, new String[]{"123456"});

		mBundle.putStringArray(CarrierConfigManager.KEY_RATCHET_RAT_FAMILIES,
		// UMTS < GPRS < EDGE
		new String[]{"3,1,2"});

		mSimulatedCommands.setVoiceRegState(NetworkRegistrationState.REG_STATE_HOME);
		mSimulatedCommands.setVoiceRadioTech(ServiceState.RIL_RADIO_TECHNOLOGY_HSPA);
		mSimulatedCommands.setDataRegState(NetworkRegistrationState.REG_STATE_HOME);
		@@ -1330,4 +1334,130 @@ public class ServiceStateTrackerTest extends TelephonyTest {
		assertTrue(actualNitzSignal.mElapsedRealtime <= SystemClock.elapsedRealtime());
		}
		}

		// Edge and GPRS are grouped under the same family and Edge has higher rate than GPRS.
		// Expect no rat update when move from E to G.
		@Test
		public void testRatRatchet() throws Exception {
		CellIdentityGsm cellIdentity = new CellIdentityGsm(-1, -1, -1, -1, -1, -1);
		NetworkRegistrationState dataResult = new NetworkRegistrationState(
		0, 0, 1, 2, 0, false, null, cellIdentity, 1);
		sst.mPollingContext[0] = 2;
		// update data reg state to be in service
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_GPRS,
		new AsyncResult(sst.mPollingContext, dataResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.STATE_IN_SERVICE, mSST.getCurrentDataConnectionState());

		NetworkRegistrationState voiceResult = new NetworkRegistrationState(
		0, 0, 1, 2, 0, false, null, cellIdentity, false, 0, 0, 0);
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_REGISTRATION,
		new AsyncResult(sst.mPollingContext, voiceResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.RIL_RADIO_TECHNOLOGY_EDGE, sst.mSS.getRilVoiceRadioTechnology());

		// EDGE -> GPRS
		voiceResult = new NetworkRegistrationState(
		0, 0, 1, 1, 0, false, null,
		cellIdentity, false, 0, 0, 0);
		sst.mPollingContext[0] = 2;
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_GPRS,
		new AsyncResult(sst.mPollingContext, dataResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.STATE_IN_SERVICE, mSST.getCurrentDataConnectionState());

		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_REGISTRATION,
		new AsyncResult(sst.mPollingContext, voiceResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.RIL_RADIO_TECHNOLOGY_EDGE, sst.mSS.getRilVoiceRadioTechnology());
		}

		// Edge and GPRS are grouped under the same family and Edge has higher rate than GPRS.
		// Bypass rat rachet when cell id changed. Expect rat update from E to G
		@Test
		public void testRatRatchetWithCellChange() throws Exception {
		CellIdentityGsm cellIdentity = new CellIdentityGsm(-1, -1, -1, -1, -1, -1);
		NetworkRegistrationState dataResult = new NetworkRegistrationState(
		0, 0, 1, 2, 0, false, null, cellIdentity, 1);
		sst.mPollingContext[0] = 2;
		// update data reg state to be in service
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_GPRS,
		new AsyncResult(sst.mPollingContext, dataResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.STATE_IN_SERVICE, mSST.getCurrentDataConnectionState());

		NetworkRegistrationState voiceResult = new NetworkRegistrationState(
		0, 0, 1, 2, 0, false, null, cellIdentity, false, 0, 0, 0);
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_REGISTRATION,
		new AsyncResult(sst.mPollingContext, voiceResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.RIL_RADIO_TECHNOLOGY_EDGE, sst.mSS.getRilVoiceRadioTechnology());

		// RAT: EDGE -> GPRS cell ID: -1 -> 5
		cellIdentity = new CellIdentityGsm(-1, -1, -1, 5, -1, -1);
		voiceResult = new NetworkRegistrationState(
		0, 0, 1, 1, 0, false, null, cellIdentity, false, 0, 0, 0);
		sst.mPollingContext[0] = 2;
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_GPRS,
		new AsyncResult(sst.mPollingContext, dataResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.STATE_IN_SERVICE, mSST.getCurrentDataConnectionState());

		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_REGISTRATION,
		new AsyncResult(sst.mPollingContext, voiceResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.RIL_RADIO_TECHNOLOGY_GPRS, sst.mSS.getRilVoiceRadioTechnology());
		}

		// Edge, GPRS and UMTS are grouped under the same family where Edge > GPRS > UMTS .
		// Expect no rat update from E to G immediately following cell id change.
		// Expect ratratchet (from G to UMTS) for the following rat update within the cell location.
		@Test
		public void testRatRatchetWithCellChangeBeforeRatChange() throws Exception {
		// cell ID update
		CellIdentityGsm cellIdentity = new CellIdentityGsm(-1, -1, -1, 5, -1, -1);
		NetworkRegistrationState dataResult = new NetworkRegistrationState(
		0, 0, 1, 2, 0, false, null, cellIdentity, 1);
		sst.mPollingContext[0] = 2;
		// update data reg state to be in service
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_GPRS,
		new AsyncResult(sst.mPollingContext, dataResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.STATE_IN_SERVICE, mSST.getCurrentDataConnectionState());

		NetworkRegistrationState voiceResult = new NetworkRegistrationState(
		0, 0, 1, 2, 0, false, null, cellIdentity, false, 0, 0, 0);
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_REGISTRATION,
		new AsyncResult(sst.mPollingContext, voiceResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.RIL_RADIO_TECHNOLOGY_EDGE, sst.mSS.getRilVoiceRadioTechnology());

		// RAT: EDGE -> GPRS, cell ID unchanged. Expect no rat ratchet following cell Id change.
		voiceResult = new NetworkRegistrationState(
		0, 0, 1, 1, 0, false, null, cellIdentity, false, 0, 0, 0);
		sst.mPollingContext[0] = 2;
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_GPRS,
		new AsyncResult(sst.mPollingContext, dataResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.STATE_IN_SERVICE, mSST.getCurrentDataConnectionState());

		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_REGISTRATION,
		new AsyncResult(sst.mPollingContext, voiceResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.RIL_RADIO_TECHNOLOGY_GPRS, sst.mSS.getRilVoiceRadioTechnology());

		// RAT: GPRS -> UMTS
		voiceResult = new NetworkRegistrationState(
		0, 0, 1, 3, 0, false, null, cellIdentity, false, 0, 0, 0);
		sst.mPollingContext[0] = 2;
		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_GPRS,
		new AsyncResult(sst.mPollingContext, dataResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.STATE_IN_SERVICE, mSST.getCurrentDataConnectionState());

		sst.sendMessage(sst.obtainMessage(ServiceStateTracker.EVENT_POLL_STATE_REGISTRATION,
		new AsyncResult(sst.mPollingContext, voiceResult, null)));
		waitForMs(200);
		assertEquals(ServiceState.RIL_RADIO_TECHNOLOGY_GPRS, sst.mSS.getRilVoiceRadioTechnology());
		}
		}