Skip to main content

Last Call Review of draft-ietf-bpf-isa-02
review-ietf-bpf-isa-02-genart-lc-robles-2024-05-16-00

Request Review of draft-ietf-bpf-isa
Requested revision No specific revision (document currently at 04)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2024-05-16
Requested 2024-05-02
Authors Dave Thaler
I-D last updated 2024-05-16
Completed reviews Genart Last Call review of -02 by Ines Robles (diff)
Opsdir Last Call review of -02 by Ron Bonica (diff)
Assignment Reviewer Ines Robles
State Completed
Request Last Call review on draft-ietf-bpf-isa by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/3Tcqx8AUEEO3JGnPAQfCFtbACyw
Reviewed revision 02 (document currently at 04)
Result Ready w/nits
Completed 2024-05-16
review-ietf-bpf-isa-02-genart-lc-robles-2024-05-16-00
I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://wiki.ietf.org/en/group/gen/GenArtFAQ>.

Document: draft-ietf-bpf-isa-02
Reviewer: Ines Robles
Review Date: 2024-05-16
IETF LC End Date: 2024-05-16
IESG Telechat date: Not scheduled for a telechat

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