Skip to content
This repository was archived by the owner on Oct 4, 2019. It is now read-only.

EIP-150 for Ethereum Classic cpp-ethereum. #10

Open
marcusrbrown wants to merge 23 commits into
base: develop
Choose a base branch
from
Open

EIP-150 for Ethereum Classic cpp-ethereum. #10

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
1c609fd
EIP150 version1 gasPrice change proposal
Oct 10, 2016
41d22dc
update gas price for
Oct 11, 2016
9eaaad0
fixing PR
Oct 11, 2016
975f09a
Implement EIP150 1b for calls
chfast Oct 12, 2016
e78f769
Implement EIP150 1b (63/64 gas) for CREATE
chfast Oct 12, 2016
30a7f20
Refactor EIP150 1b changes for improve readability
chfast Oct 12, 2016
30a38c4
Additional cost of SELFDESTRUCT sending to non-existing account
chfast Oct 12, 2016
3250d78
update EIP150 test filler
Oct 12, 2016
1cbb407
Improve EVMSchedule
chfast Oct 12, 2016
f7c82e5
Rename EIP150 related objects
chfast Oct 12, 2016
f3ed580
test: add gasCheck on creation of a contract
Oct 12, 2016
394cf2c
Rework code for "all but one 64th" rule. NFC
chfast Oct 13, 2016
2adbdc0
EIP150 MemExpandingTests
Oct 13, 2016
ed38acb
more EIP150 state tests
Oct 14, 2016
f013d6b
change frontierCompatibilityModeLimit to homsteadForkBlock
Oct 14, 2016
1a97c6a
Restore stack limit of 1024 for EIP150
chfast Oct 14, 2016
dc81af7
Adapt Homestead tests to EIP150 specs
Oct 14, 2016
375ac97
fix incorrect bnumber in state tests
Oct 14, 2016
ea24960
Move blockchain tests to be run on EIP150 rules
Oct 14, 2016
255c249
update blockchainEIP fillers
Oct 15, 2016
9612186
blockhain tests that crash cpp eth
Oct 15, 2016
2d6fc35
change EIP150 bnumber to 2463000
Oct 17, 2016
7c1d277
Change the EIP-150 block number from 2463000 to 2500000 for Ethereum …
marcusrbrown Oct 17, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion libevm/VM.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,7 @@ void VM::interpretCases()

// After EIP150 hard fork charge additional cost of sending
// ethers to non-existing account.
if (!m_schedule->staticCallDepthLimit && !m_ext->exists(dest))
if (m_schedule->suicideChargesNewAccountGas() && !m_ext->exists(dest))
m_runGas += m_schedule->callNewAccountGas;
Copy link
Member

@elaineo elaineo Oct 21, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

was this change spec'd in the yellow paper? i don't remember seeing it implemented in geth


onOperation();
Expand Down
11 changes: 5 additions & 6 deletions libevm/VMCalls.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,11 +109,11 @@ void VM::caseCreate()
uint64_t initOff = (uint64_t)*m_sp--;
uint64_t initSize = (uint64_t)*m_sp--;

bool depthOk = m_schedule->staticCallDepthLimit ? m_ext->depth < 1024 : true;
bool depthOk = m_schedule->staticCallDepthLimit() ? m_ext->depth < 1024 : true;
if (m_ext->balance(m_ext->myAddress) >= endowment && depthOk)
{
u256 createGas = *m_io_gas;
if (!m_schedule->staticCallDepthLimit)
if (!m_schedule->staticCallDepthLimit())
createGas -= createGas / 64;
u256 gas = createGas;
*++m_sp = (u160)m_ext->create(endowment, gas, bytesConstRef(m_mem.data() + initOff, initSize), *m_onOp);
Expand Down Expand Up @@ -155,9 +155,8 @@ bool VM::caseCallSetup(CallParameters *callParams)
// "Static" costs already applied. Calculate call gas.
u256 requestedCallGas = *m_sp;
u256 maxAllowedCallGas = *m_io_gas - *m_io_gas / 64;
u256 callGas = m_schedule->staticCallDepthLimit
? requestedCallGas
: std::min(requestedCallGas, maxAllowedCallGas);
u256 callGas = m_schedule->staticCallDepthLimit() ? requestedCallGas :
std::min(requestedCallGas, maxAllowedCallGas);
m_runGas = toUint64(callGas);
onOperation();
updateIOGas();
Expand Down Expand Up @@ -186,7 +185,7 @@ bool VM::caseCallSetup(CallParameters *callParams)
uint64_t outOff = (uint64_t)*m_sp--;
uint64_t outSize = (uint64_t)*m_sp--;

bool depthOk = m_schedule->staticCallDepthLimit ? m_ext->depth < 1024 : true;
bool depthOk = m_schedule->staticCallDepthLimit() ? m_ext->depth < 1024 : true;
if (m_ext->balance(m_ext->myAddress) >= callParams->valueTransfer && depthOk)
{
callParams->onOp = *m_onOp;
Expand Down
7 changes: 5 additions & 2 deletions libevmcore/EVMSchedule.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ struct EVMSchedule
EVMSchedule(bool _efcd, bool _hdc, unsigned const& _txCreateGas): exceptionalFailedCodeDeposit(_efcd), haveDelegateCall(_hdc), tierStepGas(std::array<unsigned, 8>{{0, 2, 3, 5, 8, 10, 20, 0}}), txCreateGas(_txCreateGas) {}
bool exceptionalFailedCodeDeposit = true;
bool haveDelegateCall = true;
bool staticCallDepthLimit = true;
bool eip150Mode = false;
unsigned stackLimit = 1024;
std::array<unsigned, 8> tierStepGas;
unsigned expGas = 10;
Expand Down Expand Up @@ -70,6 +70,9 @@ struct EVMSchedule
unsigned extcodecopyGas = 20;
unsigned balanceGas = 20;
unsigned suicideGas = 0;

bool staticCallDepthLimit() const { return !eip150Mode; }
bool suicideChargesNewAccountGas() const { return eip150Mode; }
};

static const EVMSchedule DefaultSchedule = EVMSchedule();
Expand All @@ -80,7 +83,7 @@ static EVMSchedule EVMScheduleEIP150()
{
//EIP150
EVMSchedule schedule = HomesteadSchedule;
schedule.staticCallDepthLimit = false;
schedule.eip150Mode = true;
schedule.extcodesizeGas = 700;
schedule.extcodecopyGas = 700;
schedule.balanceGas = 400;
Expand Down