commit 98638ed0c030ca943ea5ed8c0f0b03883e6e62e0
author Ken Mixter <kmixter@chromium.org> Mon May 23 21:07:09 2011
committer Ken Mixter <kmixter@chromium.org> Tue May 24 00:26:08 2011
tree b7b679ff138fbb4be4d3500f710eff653b6993be
parent 8b32cb882a1916c34ed42461fbe4af68dcec46ca

entd: if pkcs11 system is down back off on retries until trying each hour

BUG=none
TEST=login in state where token is not initialized, verify backoff based on logs

Change-Id: Ied5659279d22b305845ead262735bd6a6f47c7e7
Reviewed-on: http://gerrit.chromium.org/gerrit/1386
Tested-by: Ken Mixter <kmixter@chromium.org>
Reviewed-by: Robert Ginda <rginda@chromium.org>

base_policy/policy-utils.js

diff --git a/base_policy/policy-utils.js b/base_policy/policy-utils.js
index 11b7415..806fa00 100644
--- a/base_policy/policy-utils.js
+++ b/base_policy/policy-utils.js
@@ -346,13 +346,18 @@
   this.info('Initializing PKCS#11 library, attempt: ' +
             this.pkcs11.initCount);
 
+  // Compute retry interval, starting at 1 second, doubling every failure,
+  // until reaching an hour between checks.
+  var doubling = Math.min(this.pkcs11.initCount - 1, 12);
+  var next_retry = 1000 * (1 << doubling);
+
   if ('isTokenReady' in entd.tpm) {
     if (!entd.tpm.isTokenReady) {
       // The TPM token is not yet initialized by cryptohomed and we
       // must not load PKCS11 until it has finished.  Re-check again
       // in a second.
-      this.info('TPM token not yet initialized, delaying library load.');
-      entd.setTimeout(util.bindp(this, "initPkcs11"), 1000);
+      this.info('Cryptohome has not yet initialized TPM token, delaying.');
+      entd.setTimeout(util.bindp(this, "initPkcs11"), next_retry);
       return false;
     }
   }
@@ -366,8 +371,8 @@
   }
 
   if (!this.pkcs11.api) {
-    // Initialization failed, try again in a second.
-    entd.setTimeout(util.bindp(this, "initPkcs11"), 1000);
+    // Initialization failed, try again.
+    entd.setTimeout(util.bindp(this, "initPkcs11"), next_retry);
     return false;
   }