On Tue, Dec 14, 2021 at 6:04 AM Christoph Hellwig <hch@xxxxxx> 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.
