Re: [PATCHv3 net-next 04/15] bpf: don't (ab)use instructions to store state

2016-09-14 Thread Alexei Starovoitov
On Wed, Sep 14, 2016 at 08:00:12PM +0100, Jakub Kicinski wrote: > Storing state in reserved fields of instructions makes > it impossible to run validator on programs already > marked as read-only. Allocate and use an array of > per-instruction state instead. > > While touching the error path renam

[PATCHv3 net-next 04/15] bpf: don't (ab)use instructions to store state

2016-09-14 Thread Jakub Kicinski
Storing state in reserved fields of instructions makes it impossible to run validator on programs already marked as read-only. Allocate and use an array of per-instruction state instead. While touching the error path rename and move existing jump target. Suggested-by: Alexei Starovoitov Signed-o