On Sat, 2023-12-02 at 18:05 -0500, Andrei Matei wrote: > Signed-off-by: Andrei Matei <andreimatei1@xxxxxxxxx> > --- [...] > int allocated_stack; > + /* The state of the stack. Each element of the array describes BPF_REG_SIZE > + * (i.e. 8) bytes worth of stack memory. > + * stack[0] represents bytes [*(r10-8)..*(r10-1)] > + * stack[1] represents bytes [*(r10-16)..*(r10-9)] > + * ... > + * stack[allocated_stack/8 - 1] represents [*(r10-allocated_size)..*(r10-allocated_size+7)] > + */ Nit: this uses variable 'allocated_size' that does not seem defined. Otherwise a nice comment, thanks.