> On 3 Jan 2023, at 17:47, Alexandr Nedvedicky <sas...@fastmail.net> wrote:
>
> Hello,
>
> On Tue, Jan 03, 2023 at 03:31:42PM +1000, David Gwynne wrote:
>> the pf_state_tree type represents the rb tree that pf_state_key structs
>> go into. currently pf_state_key is declared in pfvar_priv.h (because
>> it's a kernel private data structure) but pf_state_tree was left in
>> pfvar.h. this moves it to pfvar_priv.h, because it is also kernel
>> private.
>>
>> while here, this moves it from the RB tree macros to RBT which shrinks
>> pf.o by about 13k on amd64.
>
> diff reads OK to me. Do I assume right 'tree_id' tree will
> be also moved from RB_*() to RBT_() in another commit?
yes, sorry, i was going to say that.
> thanks and OK sashan
i'll put it in when im awake.
cheers,
dlg