Remove unused imports and disable persist performance metrics on bare metal (#3880)

* Remove unused imports.

* Disable persist performance metrics when compiling on bare metal.

Author: bhartnett

execution_chain/db/aristo/aristo_init/rocks_db/rdb_desc.nim

@@ -14,7 +14,6 @@
 {.push raises: [].}
 
 import
-  std/os,
   std/concurrency/atomics,
   stew/endians2,
   ../../../core_db/backend/rocksdb_desc,

execution_chain/db/core_db/base.nim

@@ -13,13 +13,16 @@
 import
   std/typetraits,
   chronicles,
-  chronos/timer,
   eth/common/[accounts, base, hashes],
   ../../constants,
   ../[kvt, aristo],
   ../kvt/kvt_init/init_common,
   ./base/[base_desc, base_helpers]
 
+# Persist performance metrics are disabled on bare metal
+when not defined(`any`) and not defined(standalone):
+  import chronos/timer
+
 export
   CoreDbAccount,
   CoreDbErrorCode,
@@ -96,22 +99,30 @@ proc persist*(db: CoreDbRef, txFrame: CoreDbTxRef) =
     #      kvt changes written to memory but not to disk because of an aristo
     #      error), we have to panic instead.
 
-    let kvtTick = Moment.now()
-    db.kvt.persist(kvtBatch[], txFrame.kTx)
-    let mptTick = Moment.now()
-    db.mpt.persist(mptBatch[], txFrame.aTx)
-
-    let endTick = Moment.now()
-    db.kvt.putEndFn(kvtBatch[]).isOkOr:
-      raiseAssert $error
-
-    db.mpt.putEndFn(mptBatch[]).isOkOr:
-      raiseAssert $error
+    when not defined(`any`) and not defined(standalone):
+      let kvtTick = Moment.now()
+      db.kvt.persist(kvtBatch[], txFrame.kTx)
+      let mptTick = Moment.now()
+      db.mpt.persist(mptBatch[], txFrame.aTx)
+
+      let endTick = Moment.now()
+      db.kvt.putEndFn(kvtBatch[]).isOkOr:
+        raiseAssert $error
+      db.mpt.putEndFn(mptBatch[]).isOkOr:
+        raiseAssert $error
+
+      debug "Core DB persisted",
+        kvtDur = mptTick - kvtTick,
+        mptDur = endTick - mptTick,
+        endDur = Moment.now() - endTick
+    else:
+      db.kvt.persist(kvtBatch[], txFrame.kTx)
+      db.mpt.persist(mptBatch[], txFrame.aTx)
+      db.kvt.putEndFn(kvtBatch[]).isOkOr:
+        raiseAssert $error
+      db.mpt.putEndFn(mptBatch[]).isOkOr:
+        raiseAssert $error
 
-    debug "Core DB persisted",
-      kvtDur = mptTick - kvtTick,
-      mptDur = endTick - mptTick,
-      endDur = Moment.now() - endTick
   else:
     discard kvtBatch.expect("should always be able to create batch")
     discard mptBatch.expect("should always be able to create batch")

execution_chain/evm/code_stream.nim

@@ -5,7 +5,7 @@
 #  * MIT license ([LICENSE-MIT](LICENSE-MIT) or http://opensource.org/licenses/MIT)
 # at your option. This file may not be copied, modified, or distributed except according to those terms.
 
-import chronicles, stint, stew/byteutils, ./interpreter/op_codes, ./code_bytes
+import stint, stew/byteutils, ./interpreter/op_codes, ./code_bytes
 
 export code_bytes
 

execution_chain/evm/interpreter_dispatch.nim

@@ -12,7 +12,8 @@
 
 import
   std/[macros, strformat],
-  pkg/[chronicles, chronos, stew/byteutils],
+  chronicles,
+  stew/byteutils,
   ".."/[constants, db/ledger],
   "."/[code_stream, computation, evm_errors],
   "."/[message, precompiles, state, types],

execution_chain/transaction/call_common.nim

@@ -10,7 +10,6 @@
 
 import
   eth/common/eth_types, stint,
-  chronos,
   results,
   ../evm/[types, state],
   ../evm/[message, precompiles, internals, interpreter_dispatch],