> 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

Reply via email to