Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix broken data retry" into honeycomb-LTE

Browse Source
This commit is contained in:
Wink Saville
2011-05-20 16:47:54 -07:00
committed by Android (Google) Code Review
parent f483ed9040 5ce6268faa
commit 2b534e770d
1 changed files with 36 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

65
telephony/java/com/android/internal/telephony/gsm/GsmDataConnectionTracker.java
View File

@@ -927,37 +927,43 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
return false;
}
dc = findReadyDataConnection(apn);
// First, check to see if ApnContext already has DC.
// This could happen if the retries are currently engaged.
dc = (GsmDataConnection)apnContext.getDataConnection();
if (dc == null) {
if (DBG) log("setupData: No ready GsmDataConnection found!");
// TODO: When allocating you are mapping type to id. If more than 1 free,
// then could findFreeDataConnection get the wrong one??
dc = findFreeDataConnection();
}
dc = findReadyDataConnection(apn);
if (dc == null) {
dc = createDataConnection();
}
if (dc == null) {
if (DBG) log("setupData: No ready GsmDataConnection found!");
// TODO: When allocating you are mapping type to id. If more than 1 free,
// then could findFreeDataConnection get the wrong one??
dc = findFreeDataConnection();
}
if (dc == null) {
if (DBG) log("setupData: No free GsmDataConnection found!");
return false;
}
if (dc == null) {
dc = createDataConnection();
}
dc.setProfileId( profileId );
dc.setActiveApnType(apnContext.getApnType());
int refCount = dc.incAndGetRefCount();
if (DBG) log("setupData: init dc and apnContext refCount=" + refCount);
if (dc == null) {
if (DBG) log("setupData: No free GsmDataConnection found!");
return false;
}
// configure retry count if no other Apn is using the same connection.
if (refCount == 1) {
configureRetry(dc, apnContext.getApnType());
dc.setProfileId( profileId );
dc.setActiveApnType(apnContext.getApnType());
int refCount = dc.incAndGetRefCount();
if (DBG) log("setupData: init dc and apnContext refCount=" + refCount);
// configure retry count if no other Apn is using the same connection.
if (refCount == 1) {
configureRetry(dc, apnContext.getApnType());
}
DataConnectionAc dcac = mDataConnectionAsyncChannels.get(dc.getDataConnectionId());
apnContext.setDataConnectionAc(mDataConnectionAsyncChannels.get(dc.getDataConnectionId()));
apnContext.setApnSetting(apn);
apnContext.setDataConnection(dc);
}
DataConnectionAc dcac = mDataConnectionAsyncChannels.get(dc.getDataConnectionId());
apnContext.setDataConnectionAc(mDataConnectionAsyncChannels.get(dc.getDataConnectionId()));
apnContext.setApnSetting(apn);
apnContext.setDataConnection(dc);
Message msg = obtainMessage();
msg.what = EVENT_DATA_SETUP_COMPLETE;
@@ -1655,6 +1661,12 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
}
apnContext.setState(State.FAILED);
mPhone.notifyDataConnection(Phone.REASON_APN_FAILED, apnContext.getApnType());
int refCount = releaseApnContext(apnContext, false);
if (DBG) {
log("onDataSetupComplete: permanent error apn=%s" + apnString +
" refCount=" + refCount);
}
} else {
if (DBG) log("onDataSetupComplete: Not all permanent failures, retry");
startDelayedRetry(cause, apnContext);
@@ -1667,11 +1679,6 @@ public final class GsmDataConnectionTracker extends DataConnectionTracker {
sendMessageDelayed(obtainMessage(EVENT_TRY_SETUP_DATA, apnContext),
APN_DELAY_MILLIS);
}
int refCount = releaseApnContext(apnContext, false);
if (DBG) {
log("onDataSetupComplete: error apn=%s" + apnString + " refCount=" + refCount);
}
}
}