ISA: Comments from Ines Robles

[dthaler]

https://datatracker.ietf.org/doc/review-ietf-bpf-isa-02-genart-lc-robles-2024-05-16/

Summary:

This document specifies the BPF instruction set architecture (ISA). The document is clear and well-written. No major issues were found, just some minor suggestions.

Major issues: None Minor issues: None Nits/editorial comments:

• In the introduction, maybe?: "eBPF (which is no longer an acronym for anything), also commonly referred to as BPF" --> eBPF (which originally stood for "extended Berkeley Packet Filter" but is no longer an acronym), also commonly referred to as BPF...

• It would be nice to add caption to the tables (from Table 3 to Table 18).

Thanks for this document,

Ines

[dthaler]

Proposed patch adding captions posted at https://mailarchive.ietf.org/arch/msg/bpf/wu_vtrec-wZdQAqpSi_1P66iHlU/

[dthaler]

Patch was merged but indentation was off for two tables causing the caption to not render correctly. Proposed fix posted to https://mailarchive.ietf.org/arch/msg/bpf/P_AeRgHrfQ1RG7UpapDn6kKTaRg/

[dthaler]

Fixed in draft-03