On Tue, Nov 03, 2020 at 04:31:31PM +0100, KP Singh wrote: > + > +struct storage { > + void *inode; > + unsigned int value; > + /* Lock ensures that spin locked versions of local stoage operations > + * also work, most operations in this tests are still single threaded > + */ > + struct bpf_spin_lock lock; > +}; I think it's a good idea to test spin_lock in local_storage, but it seems the test is not doing it fully. It's only adding it to the storage, but the program is not accessing it.