On Wed, Dec 21, 2022 at 4:11 PM Eduard Zingerman <eddyz87@xxxxxxxxx> wrote: > > On Tue, 2022-12-20 at 13:03 -0800, Andrii Nakryiko wrote: > > On Fri, Dec 16, 2022 at 6:17 PM Eduard Zingerman <eddyz87@xxxxxxxxx> wrote: > > > > > > Adds a macro __test_state_freq, the macro expands as a btf_decl_tag of a > > > special form that instructs test_loader that the flag BPF_F_TEST_STATE_FREQ > > > has to be passed to BPF verifier when program is loaded. > > > > > > > I needed similar capabilities locally, but I went a slightly different > > direction. Instead of defining custom macros and logic, I define just > > __flags(X) macro and then parse flags either by their symbolic name > > (or just integer value, which might be useful sometimes for > > development purposes). I've also added support for matching multiple > > messages sequentially which locally is in the same commit. Feel free > > to ignore that part, but I think it's useful as well. So WDYT about > > the below? > > Makes total sense. I can replace my patch with your patch in the > patchset, or just wait until your changes land. Mine will take a bit more time and will be stuck in discussions anyways, so the more prerequisites land before it the better. Please go ahead and add it to your patch set. > > > > >
