Integrate native dynamic contracts

Author: jackcmay

Cargo.toml

@@ -85,6 +85,8 @@ jsonrpc-http-server = { git = "https://github.com/paritytech/jsonrpc", rev = "4b
 jsonrpc-macros = { git = "https://github.com/paritytech/jsonrpc", rev = "4b6060b" }
 ipnetwork = "0.12.7"
 itertools = "0.7.8"
+libc = "0.2.43"
+libloading = "0.5.0"
 log = "0.4.2"
 matches = "0.1.6"
 nix = "0.11.0"
@@ -102,7 +104,11 @@ sys-info = "0.5.6"
 tokio = "0.1"
 tokio-codec = "0.1"
 untrusted = "0.6.2"
-libc = "0.2.43"
+
+[dev-dependencies]
+noop = { path = "contracts/noop" }
+print = { path = "contracts/print" }
+move_funds = { path = "contracts/move_funds" }
 
 [[bench]]
 name = "bank"
@@ -118,3 +124,18 @@ name = "signature"
 
 [[bench]]
 name = "sigverify"
+
+[workspace]
+members = [
+    ".",
+    "contracts/noop",
+    "contracts/print",
+    "contracts/move_funds",
+]
+default-members = [
+    ".",
+    "contracts/noop",
+    "contracts/print",
+    "contracts/move_funds",
+]
+

contracts/move_funds/Cargo.toml

@@ -0,0 +1,24 @@
+[package]
+name = "move_funds"
+version = "0.1.0"
+authors = [
+    "Anatoly Yakovenko <[email protected]>",
+    "Greg Fitzgerald <[email protected]>",
+    "Stephen Akridge <[email protected]>",
+    "Michael Vines <[email protected]>",
+    "Rob Walker <[email protected]>",
+    "Pankaj Garg <[email protected]>",
+    "Tyera Eulberg <[email protected]>",
+    "Jack May <[email protected]>",
+]
+
+[dependencies]
+bincode = "1.0.0"
+generic-array = { version = "0.12.0", default-features = false, features = ["serde"] }
+libloading = "0.5.0"
+solana = { path = "../.." }
+
+[lib]
+name = "move_funds"
+crate-type = ["dylib"]
+

contracts/move_funds/src/lib.rs

@@ -0,0 +1,48 @@
+extern crate bincode;
+extern crate solana;
+
+use bincode::deserialize;
+use solana::dynamic_contract::KeyedAccount;
+
+#[no_mangle]
+pub extern "C" fn process(infos: &mut Vec<KeyedAccount>, data: &[u8]) {
+    let tokens: i64 = deserialize(data).unwrap();
+    if infos[0].account.tokens >= tokens {
+        infos[0].account.tokens -= tokens;
+        infos[1].account.tokens += tokens;
+    } else {
+        println!(
+            "Insufficient funds, asked {}, only had {}",
+            tokens, infos[0].account.tokens
+        );
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use bincode::serialize;
+    use solana::bank::Account;
+    use solana::signature::Pubkey;
+
+    #[test]
+    fn test_move_funds() {
+        let tokens: i64 = 100;
+        let data: Vec<u8> = serialize(&tokens).unwrap();
+        let keys = vec![Pubkey::default(); 2];
+        let mut accounts = vec![Account::default(), Account::default()];
+        accounts[0].tokens = 100;
+        accounts[1].tokens = 1;
+
+        {
+            let mut infos: Vec<KeyedAccount> = Vec::new();
+            for (key, account) in keys.iter().zip(&mut accounts).collect::<Vec<_>>() {
+                infos.push(KeyedAccount { key, account });
+            }
+
+            process(&mut infos, &data);
+        }
+        assert_eq!(0, accounts[0].tokens);
+        assert_eq!(101, accounts[1].tokens);
+    }
+}

contracts/noop/Cargo.toml

@@ -0,0 +1,22 @@
+[package]
+name = "noop"
+version = "0.1.0"
+authors = [
+    "Anatoly Yakovenko <[email protected]>",
+    "Greg Fitzgerald <[email protected]>",
+    "Stephen Akridge <[email protected]>",
+    "Michael Vines <[email protected]>",
+    "Rob Walker <[email protected]>",
+    "Pankaj Garg <[email protected]>",
+    "Tyera Eulberg <[email protected]>",
+    "Jack May <[email protected]>",
+]
+
+[dependencies]
+libloading = "0.5.0"
+solana = { path = "../.." }
+
+[lib]
+name = "noop"
+crate-type = ["dylib"]
+

contracts/noop/src/lib.rs

@@ -0,0 +1,6 @@
+extern crate solana;
+
+use solana::dynamic_contract::KeyedAccount;
+
+#[no_mangle]
+pub extern "C" fn process(_infos: &mut Vec<KeyedAccount>, _data: &[u8]) {}

contracts/print/Cargo.toml

@@ -0,0 +1,22 @@
+[package]
+name = "print"
+version = "0.1.0"
+authors = [
+    "Anatoly Yakovenko <[email protected]>",
+    "Greg Fitzgerald <[email protected]>",
+    "Stephen Akridge <[email protected]>",
+    "Michael Vines <[email protected]>",
+    "Rob Walker <[email protected]>",
+    "Pankaj Garg <[email protected]>",
+    "Tyera Eulberg <[email protected]>",
+    "Jack May <[email protected]>",
+]
+
+[dependencies]
+libloading = "0.5.0"
+solana = { path = "../.." }
+
+[lib]
+name = "print"
+crate-type = ["dylib"]
+

contracts/print/src/lib.rs

@@ -0,0 +1,9 @@
+extern crate solana;
+
+use solana::dynamic_contract::KeyedAccount;
+
+#[no_mangle]
+pub extern "C" fn process(infos: &mut Vec<KeyedAccount>, _data: &[u8]) {
+    println!("AccountInfos: {:#?}", infos);
+    //println!("data: {:#?}", data);
+}

src/bank.rs

@@ -7,6 +7,7 @@ use bincode::deserialize;
 use bincode::serialize;
 use budget_contract::BudgetContract;
 use counter::Counter;
+use dynamic_contract::{DynamicContract, KeyedAccount};
 use entry::Entry;
 use hash::{hash, Hash};
 use itertools::Itertools;
@@ -135,6 +136,9 @@ pub struct Bank {
 
     // The latest finality time for the network
     finality_time: AtomicUsize,
+
+    // loaded contracts hashed by contract_id
+    loaded_contracts: RwLock<HashMap<Pubkey, DynamicContract>>,
 }
 
 impl Default for Bank {
@@ -146,6 +150,7 @@ impl Default for Bank {
             transaction_count: AtomicUsize::new(0),
             is_leader: true,
             finality_time: AtomicUsize::new(std::usize::MAX),
+            loaded_contracts: RwLock::new(HashMap::new()),
         }
     }
 }
@@ -306,6 +311,7 @@ impl Bank {
             Ok(called_accounts)
         }
     }
+
     fn load_accounts(
         &self,
         txs: &[Transaction],
@@ -316,6 +322,7 @@ impl Bank {
             .map(|tx| self.load_account(tx, accounts, error_counters))
             .collect()
     }
+
     pub fn verify_transaction(
         tx: &Transaction,
         pre_contract_id: &Pubkey,
@@ -340,11 +347,33 @@ impl Bank {
         }
         Ok(())
     }
+
+    fn loaded_contract(&self, tx: &Transaction, accounts: &mut [Account]) -> bool {
+        let loaded_contracts = self.loaded_contracts.write().unwrap();
+        match loaded_contracts.get(&tx.contract_id) {
+            Some(dc) => {
+                let mut infos: Vec<_> = (&tx.keys)
+                    .into_iter()
+                    .zip(accounts)
+                    .map(|(key, account)| KeyedAccount { key, account })
+                    .collect();
+
+                dc.call(&mut infos, &tx.userdata);
+                true
+            }
+            None => false,
+        }
+    }
+
     /// Execute a transaction.
     /// This method calls the contract's process_transaction method and verifies that the result of
     /// the contract does not violate the bank's accounting rules.
     /// The accounts are committed back to the bank only if this function returns Ok(_).
-    fn execute_transaction(tx: Transaction, accounts: &mut [Account]) -> Result<Transaction> {
+    fn execute_transaction(
+        &self,
+        tx: Transaction,
+        accounts: &mut [Account],
+    ) -> Result<Transaction> {
         let pre_total: i64 = accounts.iter().map(|a| a.tokens).sum();
         let pre_data: Vec<_> = accounts
             .iter_mut()
@@ -354,11 +383,12 @@ impl Bank {
         // Call the contract method
         // It's up to the contract to implement its own rules on moving funds
         if SystemContract::check_id(&tx.contract_id) {
-            SystemContract::process_transaction(&tx, accounts)
+            SystemContract::process_transaction(&tx, accounts, &self.loaded_contracts)
         } else if BudgetContract::check_id(&tx.contract_id) {
             // TODO: the runtime should be checking read/write access to memory
             // we are trusting the hard coded contracts not to clobber or allocate
             BudgetContract::process_transaction(&tx, accounts)
+        } else if self.loaded_contract(&tx, accounts) {
         } else {
             return Err(BankError::UnknownContractId(tx.contract_id));
         }
@@ -415,7 +445,7 @@ impl Bank {
             .zip(txs.into_iter())
             .map(|(acc, tx)| match acc {
                 Err(e) => Err(e.clone()),
-                Ok(ref mut accounts) => Self::execute_transaction(tx, accounts),
+                Ok(ref mut accounts) => self.execute_transaction(tx, accounts),
             }).collect();
         let execution_elapsed = now.elapsed();
         let now = Instant::now();