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?

thanks and OK sashan

Reply via email to