Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 2 Sep 2025 21:39:43 -0700
From:      Gleb Smirnoff <glebius@freebsd.org>
To:        Kristof Provost <kp@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: ddd39152743a - main - pf: Introduce M_PF type for pf(4) related memory allocations.
Message-ID:  <aLfGj76MIfsLA9Yu@cell.glebi.us>
In-Reply-To: <202509022110.582LAr4f036441@gitrepo.freebsd.org>
References:  <202509022110.582LAr4f036441@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Sep 02, 2025 at 09:10:53PM +0000, Kristof Provost wrote:
K> The branch main has been updated by kp:
K> 
K> URL: https://cgit.FreeBSD.org/src/commit/?id=ddd39152743abc3cff05e7475c0ce48aa303956a
K> 
K> commit ddd39152743abc3cff05e7475c0ce48aa303956a
K> Author:     Kristof Provost <kp@FreeBSD.org>
K> AuthorDate: 2025-08-18 12:19:40 +0000
K> Commit:     Kristof Provost <kp@FreeBSD.org>
K> CommitDate: 2025-09-02 21:10:20 +0000
K> 
K>     pf: Introduce M_PF type for pf(4) related memory allocations.
K>     
K>     Currently used M_TEMP and M_IFADDR types are unreasonable for that purpose.
K>     This dedicated statistics simplify the future pf(4) unlocking work by decreasing
K>     search area of possible memory leaks.

IMHO, going away from M_TEMP to something else for allocations that last for
duration of a syscall is a regression rather than an improvement.

-- 
Gleb Smirnoff



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?aLfGj76MIfsLA9Yu>