On Sat, Dec 4, 2021 at 6:07 AM Jiri Olsa <jolsa@xxxxxxxxxx> wrote: > > As suggested by Andrii, adding variables for registers and ip > address offsets, which makes the code more clear, rather than > abusing single stack_size variable for everything. > > Also describing the stack layout in the comment. > > There is no function change. > > Suggested-by: Andrii Nakryiko <andrii@xxxxxxxxxx> > Signed-off-by: Jiri Olsa <jolsa@xxxxxxxxxx> > --- Thanks for the stack layout diagram! Acked-by: Andrii Nakryiko <andrii@xxxxxxxxxx> > arch/x86/net/bpf_jit_comp.c | 42 ++++++++++++++++++++++++------------- > 1 file changed, 28 insertions(+), 14 deletions(-) > [...]