On Friday, October 4th, 2024 at 2:15 AM, Alan Maguire <alan.maguire@xxxxxxxxxx> wrote: > > On 04/10/2024 06:13, Ihor Solodrai wrote: > > > This patchset changes how BTF encoders collect and store function > > information in a table searcheable by name, where their > > btf_encoder_function_state is maintained. > > > hi Ihor > > I'll take a more detailed look shortly, but I'd suggest respinning the > series against the updated "next" branch shortly as there are changes in > flight that Stephen is working on to encode variables; they > significantly change the processing of variables, see [1] for details. > They have partly landed and the rest will likely land soon. It might > make things easier to wait until those changes land and respin, and I > don't think they invalidate the approach you suggest, just will require > some merging around variable handling I think. > > [1] https://lore.kernel.org/dwarves/8734ldm72u.fsf@xxxxxxxxxx/ Hi Alan. Yes, I am aware of the pending changes. This is a reason I was a bit in a hurry, because I already had experience re-doing this from scratch :) This patchset was done on top of next (2b35957). I don't expect this version to be merged, but I wanted to get feedback from you and Arnaldo, so decided to submit. After Stephen's changes are applied to next, I'll submit v2 of the patch. Thanks! [...]
