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.

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

Add EIP: System logs #9002

Merged
merged 8 commits into from
Oct 30, 2024
Merged
Changes from all commits
Commits
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
134 changes: 134 additions & 0 deletions EIPS/eip-7799.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,134 @@
---
eip: 7799
title: System logs
description: Per-block logs without associated transactions
author: Etan Kissling (@etan-status), Gajinder Singh (@g11tech)
discussions-to: https://ethereum-magicians.org/t/eip-7799-system-logs/21497
status: Draft
type: Standards Track
category: Core
created: 2024-10-29
requires: 1559, 4895, 6466, 7708
---

## Abstract

This EIP defines an extension for eth_getLogs to provide logs for events that are not associated with a given transaction, such as block rewards and withdrawals.

## Motivation

With [EIP-7708](./eip-7708.md) wallets gain the ability to use eth_getLogs to track changes to their ETH balance. However, the ETH balance may change without an explicit transaction, through block production and withdrawals. By having such operations emit block-level system logs, eth_Logs provides a complete picture of ETH balance changes.

## 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.

### System logs list

A new list is introduced to track all block level logs emitted from system interactions. The definition uses the `Log` SSZ type from [EIP-6466](./eip-6466.md).

```python
system_logs = List[Log, MAX_LOGS_PER_RECEIPT](
log_0, log_1, log_2, ...)
```

### Priority fee processing

[EIP-1559](./eip-1559.md) priority fees SHALL no longer be credited after each individual transaction. Instead, they SHALL be summed up and credited after all transactions of a block are processed but before [EIP-4895](./eip-4895.md) withdrawals are processed.

A log SHALL be appended to the system logs list to reflect this credit of priority fees.

| Field | Value |
| - | - |
| `address` | `0xfffffffffffffffffffffffffffffffffffffffe` ([`SYSTEM_ADDRESS`](./eip-4788.md)) |
| `topics[0]` | `0x5dfe9c0fd3043bb299f97cfece428f0396cf8b7890c525756e4ea5c0ff7d61b2` (`keccak256('PriorityRewards(address,uint256)')`) |
| `topics[1]` | `to` address (zero prefixed to fill uint256) |
| `data` | `amount` in Wei (big endian uint256) |

### Withdrawal processing

A log SHALL be appended to the system logs list on every [EIP-4895](./eip-4895.md) withdrawal.

| Field | Value |
| - | - |
| `address` | `0xfffffffffffffffffffffffffffffffffffffffe` ([`SYSTEM_ADDRESS`](./eip-4788.md)) |
| `topics[0]` | `0x7fcf532c15f0a6db0bd6d0e038bea71d30d808c7d98cb3bf7268a95bf5081b65` (`keccak256('Withdrawal(address,uint256)')`) |
| `topics[1]` | `to` address (zero prefixed to fill uint256) |
| `data` | `amount` in Wei (big endian uint256) |

### Genesis processing

A log SHALL be appended to the system logs list when generating genesis blocks for networks that adopt this EIP from the beginning.

| Field | Value |
| - | - |
| `address` | `0xfffffffffffffffffffffffffffffffffffffffe` ([`SYSTEM_ADDRESS`](./eip-4788.md)) |
| `topics[0]` | `0xba2f6409ffd24dd4df8e06be958ed8c1706b128913be6e417989c74969b0b55a` (`keccak256('Genesis(address,uint256)')`) |
| `topics[1]` | `to` address (zero prefixed to fill uint256) |
| `data` | `amount` in Wei (big endian uint256) |

### Execution block header changes

The [execution block header](https://github.com/ethereum/devp2p/blob/6b259a7003b4bfb18365ba690f4b00ba8a26393b/caps/eth.md#block-encoding-and-validity) is extended with a new field, `system-logs-root`.

```python
block_header.system_logs_root = system_logs.hash_tree_root()
```

### JSON-RPC API

Block header objects in the context of the JSON-RPC API are extended to include:

- `systemLogsRoot`: `DATA`, 32 Bytes

Log objects in the context of the JSON-RPC API are updated as follows:

- `logIndex`: `QUANTITY` - The additional system logs are indexed consecutively after the existing logs from transaction receipts
- `transactionIndex`: `QUANTITY` - `null` is also used for system logs; pending logs can still be identified by checking the `blockHash` for `null`
- `transactionHash`: DATA, 32 Bytes - `null` is also used for system logs

### Engine API

In the engine API, the `ExecutionPayload` for versions corresponding to forks adopting this EIP is extended to include:

- `systemLogsRoot`: `DATA`, 32 Bytes

As part of `engine_forkchoiceUpdated`, Execution Layer implementations SHALL verify that `systemLogsRoot` for each block matches the actual value computed from local processing. This extends on top of existing `receiptsRoot` validation.

### Consensus `ExecutionPayload` changes

The consensus `ExecutionPayload` type is extended with a new field to store the system logs root.

```python
class ExecutionPayload(Container):
...
system_logs_root: Root
```

## Rationale

Together with [EIP-7708](./eip-7708.md) this EIP provides the ability for wallets to compute the exact ETH balance from logs without requiring download of every single block header and all withdrawals.

The block reward from priority fees no longer has to be summed up by processing all receipts and can be obtained from the system logs root, making it efficiently provable.

Batched crediting of priority fees improves parallel execution of transactions, as a transaction can no longer start with insufficient fees and only become eligible for execution after incremental priority fees have been credited.

### Alternatives / Future

- Instead of combining the priority fees into a single payment, each transaction could emit the priority fee payment as a separate log. A per-transaction fee log is needed anyway for the fee burn. One could extend the payload of that as 2 sets of uint256, one for the burn, and one for the priority fee, instead of a combined fee; and also add the fee recipient as an additional topic to these logs so that it is indexed for eth_getLogs search. However, that would prevent parallel execution benefits as every transaction would continue to require an exclusive lock on the fee recipient's account to credit the fee, and the total priority fees earned by a block would not be efficiently provable as it would require processing all of the receipts within the block.

- The information from withdrawals is now duplicated across both the `withdrawals` list and the system logs. Maybe the information could be massaged into the emitted `Withdrawal` log data, allowing the existing `withdrawals` list to be retired. The current list design requires the wallet to obtain all block headers and all withdrawals to ensure they obtained all data pertaining to the observed account. Allowing them to be queried using `eth_getLogs` is more in line with how deposits are being tracked.

- The log definitions themselves are subject to change. Aligning them with [ERC-20](./eip-20.md) for plain credits provides consistency. For withdrawals, the log data could match the form of deposit logs and [EIP-7685](./eip-7685.md) request logs.

## Backwards Compatibility

The fee recipient now receives priority fees at the end of the block rather than incrementally after each transaction, making it only possible to spend them in the next block. This may require updates to block builder infrastructure and change liquidity requirements for MEV use cases.

## Security Considerations

The emitted logs use `SYSTEM_ADDRESS` as their `address` which cannot conflict with user controlled smart contracts.

## Copyright

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