Permit builtin programs to be swapped out at epoch boundaries

solana-labs · Jun 3, 2020 · 17d365b · 17d365b
1 parent 1303c49
commit 17d365b
Show file tree

Hide file tree

Showing 2 changed files with 53 additions and 8 deletions.
diff --git a/runtime/src/bank.rs b/runtime/src/bank.rs
@@ -10,7 +10,7 @@ use crate::{
     accounts_db::{ErrorCounters, SnapshotStorages},
     accounts_index::Ancestors,
     blockhash_queue::BlockhashQueue,
-    builtin_programs::get_builtin_programs,
+    builtin_programs::{get_builtin_programs, get_epoch_activated_builtin_programs},
     epoch_stakes::{EpochStakes, NodeVoteAccounts},
     message_processor::{MessageProcessor, ProcessInstruction},
     nonce_utils,
@@ -456,6 +456,18 @@ impl Bank {
             {
                 entered_epoch_callback(&mut new)
             }
+
+            if let Some(builtin_programs) =
+                get_epoch_activated_builtin_programs(self.operating_mode(), self.epoch)
+            {
+                for program in builtin_programs.iter() {
+                    self.add_builtin_program(
+                        &program.name,
+                        program.id,
+                        program.process_instruction,
+                    );
+                }
+            }
         }
 
         new.update_epoch_stakes(leader_schedule_epoch);
@@ -2048,7 +2060,7 @@ impl Bank {
     }
 
     pub fn finish_init(&mut self) {
-        let builtin_programs = get_builtin_programs();
+        let builtin_programs = get_builtin_programs(self.operating_mode(), self.epoch);
         for program in builtin_programs.iter() {
             self.add_builtin_program(&program.name, program.id, program.process_instruction);
         }

diff --git a/runtime/src/builtin_programs.rs b/runtime/src/builtin_programs.rs
@@ -16,13 +16,30 @@ impl BuiltinProgram {
     }
 }
 
-pub fn get_builtin_programs() -> Vec<BuiltinProgram> {
+fn new_system_program_activation_epoch(operating_mode: OperatingMode) -> Epoch {
+    match operating_mode {
+        OperatingMode::Development => 0,
+        OperatingMode::Preview => std::u64::MAX / 2,
+        OperatingMode::Stable => std::u64::MAX / 2,
+    }
+}
+
+/// All builtin programs that should be active at the given (operating_mode, epoch)
+pub fn get_builtin_programs(operating_mode: OperatingMode, epoch: Epoch) -> Vec<BuiltinProgram> {
     vec![
-        BuiltinProgram::new(
-            "system_program",
-            system_program::id(),
-            system_instruction_processor::process_instruction,
-        ),
+        if epoch < new_system_program_activation_epoch(operating_mode) {
+            BuiltinProgram::new(
+                "system_program",
+                system_program::id(),
+                system_instruction_processor::process_instruction, // TODO: This is the OLD system instruction processor
+            )
+        } else {
+            BuiltinProgram::new(
+                "system_program",
+                system_program::id(),
+                system_instruction_processor::process_instruction, // TODO: This should be the NEW system instruction processor
+            )
+        },
         BuiltinProgram::new(
             "config_program",
             solana_config_program::id(),
@@ -40,3 +57,19 @@ pub fn get_builtin_programs() -> Vec<BuiltinProgram> {
         ),
     ]
 }
+
+/// Builtin programs that activate at the given (operating_mode, epoch)
+pub fn get_epoch_activated_builtin_programs(
+    operating_mode: OperatingMode,
+    epoch: Epoch,
+) -> Option<Vec<BuiltinProgram>> {
+    if epoch == new_system_program_activation_epoch(operating_mode) {
+        Some(vec![BuiltinProgram::new(
+            "system_program",
+            system_program::id(),
+            system_instruction_processor::process_instruction,  // TODO: This should be the NEW system instruction processor
+        )])
+    } else {
+        None
+    }
+}