8000 ISA: add definition and/or reference for big/little endian · Issue #156 · ietf-wg-bpf/ebpf-docs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

ISA: add definition and/or reference for big/little endian #156

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.

8000 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
Open
dthaler opened this issue Jun 20, 2024 · 4 comments
Open

ISA: add definition and/or reference for big/little endian #156

dthaler opened this issue Jun 20, 2024 · 4 comments
Labels
fixed in editors copy Fixed in github repo ISA Affects ISA draft

Comments

@dthaler
Copy link
Collaborator
dthaler commented Jun 20, 2024

John Scudder comments:

Thanks for this.

Although one would hope that “big-endian” and “little-endian” are widely
understood, it might not be a bad idea to supply definitions and/or a reference.
@dthaler
Copy link
Collaborator Author
dthaler commented Jun 20, 2024

Paul Wouters comments:

Can the headers in the tables such as in Section 3 be extended so it is
more clear how many bits are in the various fields ? eg the IETF style is:

                     1                   2                   3
 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
|    opcode     |     regs      |            offset             |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
|                              imm                              |
+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+

Why "host vs big" endian instead of "host order vs network order" or
"little vs big endian" ? I understand not all hosts are "host endian",
I am just a little confused why to fixup two common sets into a new set.

NITS:

boilerplate got rendered badly

@dthaler dthaler added the ISA Affects ISA draft label Jun 20, 2024
@dthaler
Copy link
Collaborator Author
dthaler commented Jun 23, 2024

The appropriate reference for endianness is IEN 137

@dthaler
Copy link
Collaborator Author
dthaler commented Jun 23, 2024 •
edited
Loading

Proposed patch posted at https://mailarchive.ietf.org/arch/msg/bpf/G0yjlzZ4GazyQC2gN0jW8YjqGkk/
(Boilerplate fix will be done as part of #160)

@dthaler dthaler added have proposed text Proposed patch posted fixed in bpf-next Awaiting propagation to ebpf-docs and removed have proposed text Proposed patch posted labels Jun 23, 2024
dthaler added a commit to dthaler/ebpf-docs-1 that referenced this issue Jun 24, 2024
@dthaler
Update instruction-set.rst to latest from bpf-next
516d227 
Addresses ietf-wg-bpf#156 ietf-wg-bpf#159 ietf-wg-bpf#160

Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
@dthaler dthaler mentioned this issue Jun 24, 2024
Merged
dthaler added a commit that referenced this issue Jun 24, 2024
@dthaler
Update instruction-set.rst to latest from bpf-next (#165)
ba14743 
Addresses #156 #159 #160

Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
@dthaler dthaler added fixed in editors copy Fixed in github repo and removed fixed in bpf-next Awaiting propagation to ebpf-docs labels Jun 24, 2024
@dthaler
Copy link
Collaborator Author
dthaler commented Jun 25, 2024

Fixed in draft -04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fixed in editors copy Fixed in github repo ISA Affects ISA draft
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dthaler
0