Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add EIP: Remove BLAKE2 compression precompile #7266

Merged
merged 16 commits into from Jul 20, 2023
Merged

Add EIP: Remove BLAKE2 compression precompile #7266

Changes from 3 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
7148a0c
📝 EIP on `blake2f` removal
pcaversaccio Jul 3, 2023
c85dd16
♻️ Add `CALLCODE` opcode
pcaversaccio Jul 3, 2023
0d0a715
♻️ Correct sum of calls
pcaversaccio Jul 3, 2023
40f46ad
♻️ Add EIP number
pcaversaccio Jul 4, 2023
c55b0b2
♻️ lint
pcaversaccio Jul 4, 2023
6453a9d
🥢 Use Unix timestamp instead of block number
pcaversaccio Jul 7, 2023
ae48786
♻️ Elaborate on the motivation
pcaversaccio Jul 7, 2023
5f34130
Add paragraph to rationale section
pcaversaccio Jul 19, 2023
d904992
Update EIPS/eip-7266.md
pcaversaccio Jul 19, 2023
2b8b4e7
Update EIPS/eip-7266.md
pcaversaccio Jul 19, 2023
926d599
♻ Add suggestions
pcaversaccio Jul 19, 2023
8559189
Merge branch 'blake2f' of https://github.com/pcaversaccio/EIPs into b…
pcaversaccio Jul 19, 2023
56b621c
Merge branch 'master' into blake2f
pcaversaccio Jul 19, 2023
8231113
♻ fix description
pcaversaccio Jul 19, 2023
80306b3
♻ security considerations
pcaversaccio Jul 19, 2023
3268dc3
Merge branch 'master' into blake2f
pcaversaccio Jul 19, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
47 changes: 47 additions & 0 deletions EIPS/eip-remove_blake2f_precompile.md
View file
Open in desktop
@@ -0,0 +1,47 @@
---
eip:

Check failure on line 2 in EIPS/eip-remove_blake2f_precompile.md

View workflow job for this annotation

GitHub Actions / EIP Walidator

preamble header `eip` must be an unsigned integer 

error[preamble-eip]: preamble header `eip` must be an unsigned integer
 --> EIPS/eip-remove_blake2f_precompile.md:2:5
  |
2 | eip:
  |      not a non-negative integer
  |
  = help: see https://ethereum.github.io/eipw/preamble-eip/

Check failure on line 2 in EIPS/eip-remove_blake2f_precompile.md

View workflow job for this annotation

GitHub Actions / EIP Walidator

file name must reflect the preamble header `eip` 

error[preamble-file-name]: file name must reflect the preamble header `eip`
 --> EIPS/eip-remove_blake2f_precompile.md:2:5
  |
2 | eip:
  |      this value
  |
  = help: this file's name should be `eip-.md`
  = help: see https://ethereum.github.io/eipw/preamble-file-name/
pcaversaccio marked this conversation as resolved.
Show resolved Hide resolved
title: Remove blake2f (0x09) Precompile
description: Remove the blake2f (0x09) precompile by changing the precompile behaviour to be a dataless revert
author: Pascal Caversaccio (@pcaversaccio)
discussions-to: https://ethereum-magicians.org/t/discussion-removal-of-ripemd-160-and-blake2f-precompiles/14857
status: Draft
type: Standards Track
category: Core
created: 2023-07-03
---

## Abstract

This EIP removes the [`blake2f`](./eip-152.md) (`0x09`) precompile by changing the precompile behaviour to be a dataless (`0x`) revert.

## Motivation

[EIP-152](./eip-152.md) has never capitalised on a real-world use case. This fact is clearly reflected in the number of times the address `0x09` has been invoked (numbers from the date this EIP was created):
- The most recent call took place on 6 October 2022.
- Since its go-live as part of the Istanbul hard fork on December 7 2019 (block number 9,069,000), `0x09` has been called only 22,131 times.

One of the reasons why [EIP-152](./eip-152.md) has failed is that the envisioned use cases were not validated before inclusion.

The EVM should be optimised for simplicity and future-proofness. In that sense, the precompile `blake2f` (`0x09`) is an obsolete carry-along with no real-world traction and thus should be removed.
pcaversaccio marked this conversation as resolved.
Show resolved Hide resolved

## Specification

The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and "OPTIONAL" in this document are to be interpreted as described in RFC 2119 and RFC 8174.

All `CALL`, `CALLCODE`, `DELEGATECALL`, and `STATICCALL` invocations to the `blake2f` precompile address `0x09` MUST return with a dataless (`0x`) `REVERT`.

## Rationale

The precompile `blake2f` (`0x09`) can be safely used as a test run for the phase-out and removal of EVM functions.

## Backwards Compatibility

This EIP requires a hard fork as it modifies the consensus rules. Note that very few applications are affected by this change and a lead time of 6-12 months can be considered sufficient.
pcaversaccio marked this conversation as resolved.
Show resolved Hide resolved

## Security Considerations

All `CALL`, `CALLCODE`, `DELEGATECALL`, and `STATICCALL` invocations, starting with the hard fork block number that introduces this consensus change, by any smart contract code to the `blake2f` precompile address `0x09` will revert with return data `0x`.
pcaversaccio marked this conversation as resolved.
Show resolved Hide resolved

## Copyright

Copyright and related rights waived via [CC0](../LICENSE.md).