On Tue, Dec 14, 2021 at 6:04 AM Christoph Hellwig [off-list ref] wrote:

 Three LSB bits store instruction class which is one of:

-  ===================     ===============
-  Classic BPF classes     eBPF classes
-  ===================     ===============
-  BPF_LD    0x00          BPF_LD    0x00
-  BPF_LDX   0x01          BPF_LDX   0x01
-  BPF_ST    0x02          BPF_ST    0x02
-  BPF_STX   0x03          BPF_STX   0x03
-  BPF_ALU   0x04          BPF_ALU   0x04
-  BPF_JMP   0x05          BPF_JMP   0x05
-  BPF_RET   0x06          BPF_JMP32 0x06
-  BPF_MISC  0x07          BPF_ALU64 0x07
-  ===================     ===============

I don't want to lose the classic vs extended visual comparison.
These were one the most valuable tables to me.
Maybe instead of intro.rst call it classic_vs_extended.rst ?
or history.rst ?

That would be patch 2 as-is plus extra tables and text
that this patch removes.
There will be a bit of overlap between history.rst
and instruction-set.rst.
I think it's ok.

The rest makes sense to me.

Maybe Daniel has better ideas.