Date: Wed, 17 Sep 2025 20:58:46 +0200 From: Kristof Provost <kp@FreeBSD.org> To: Florian Smeets <flo@FreeBSD.org> Cc: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: Re: git: 9d9bc7f462bd - main - pf: set limits before rules Message-ID: <A7B7712B-E27A-4AA1-A823-3D6CB62017FF@FreeBSD.org> In-Reply-To: <4fd761a0-b7ba-45b6-8e9e-e6b696a241a0@FreeBSD.org> References: <202509171415.58HEFh9g010282@gitrepo.freebsd.org> <4fd761a0-b7ba-45b6-8e9e-e6b696a241a0@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --] On 17 Sep 2025, at 20:38, Florian Smeets wrote: > On 17.09.25 16:15, Kristof Provost wrote: >> The branch main has been updated by kp: >> >> URL: >> https://cgit.FreeBSD.org/src/commit/?id=9d9bc7f462bd152d87ab8f1767cad19bab09bf8b >> >> commit 9d9bc7f462bd152d87ab8f1767cad19bab09bf8b >> Author: Kristof Provost <kp@FreeBSD.org> >> AuthorDate: 2025-08-25 13:43:10 +0000 >> Commit: Kristof Provost <kp@FreeBSD.org> >> CommitDate: 2025-09-17 14:15:15 +0000 >> >> pf: set limits before rules >> > > Nice, I ran into this issue recently that the ruleset would not load > on boot because we load large tables. > > I set a higher PFR_KENTRY_HIWAT as a workaround. > > Are you planning to merge this to stable/15? Maybe get this into 15.0? > I have no plans to MFC this, no. There’s a fairly large series of patches that to lesser or greater extent build on each other making MFC-ing more effort and higher risk than it’d otherwise be. I’m certainly not MFC-ing changes other than clear bug fixes to stable/15 at this point in the release cycle. Colin doesn’t need more headaches. — Kristof [-- Attachment #2 --] <!DOCTYPE html> <html> <head> <meta http-equiv="Content-Type" content="text/xhtml; charset=utf-8"> </head> <body><div style="font-family: sans-serif;"><div class="markdown" style="white-space: normal;"> <p dir="auto">On 17 Sep 2025, at 20:38, Florian Smeets wrote:</p> <blockquote style="margin: 0 0 5px; padding-left: 5px; border-left: 2px solid #136BCE; color: #136BCE;"> <p dir="auto">On 17.09.25 16:15, Kristof Provost wrote:</p> <blockquote style="margin: 0 0 5px; padding-left: 5px; border-left: 2px solid #136BCE; border-left-color: #4B89CF; color: #4B89CF;"> <p dir="auto">The branch main has been updated by kp:</p> <p dir="auto">URL: <a href="https://cgit.FreeBSD.org/src/commit/?id=9d9bc7f462bd152d87ab8f1767cad19bab09bf8b">https://cgit.FreeBSD.org/src/commit/?id=9d9bc7f462bd152d87ab8f1767cad19bab09bf8b</a></p>; <p dir="auto">commit 9d9bc7f462bd152d87ab8f1767cad19bab09bf8b<br> Author: Kristof Provost <a href="mailto:kp@FreeBSD.org">kp@FreeBSD.org</a><br> AuthorDate: 2025-08-25 13:43:10 +0000<br> Commit: Kristof Provost <a href="mailto:kp@FreeBSD.org">kp@FreeBSD.org</a><br> CommitDate: 2025-09-17 14:15:15 +0000</p> <pre style="margin-left: 15px; margin-right: 15px; padding: 5px; border: thin solid gray; overflow-x: auto; max-width: 90vw; background-color: #E4E4E4;"><code style="padding: 0 0.25em; background-color: #E4E4E4;"> pf: set limits before rules </code></pre> </blockquote> <p dir="auto">Nice, I ran into this issue recently that the ruleset would not load on boot because we load large tables.</p> <p dir="auto">I set a higher PFR_KENTRY_HIWAT as a workaround.</p> <p dir="auto">Are you planning to merge this to stable/15? Maybe get this into 15.0?</p> </blockquote> <p dir="auto">I have no plans to MFC this, no.</p> <p dir="auto">There’s a fairly large series of patches that to lesser or greater extent build on each other making MFC-ing more effort and higher risk than it’d otherwise be.<br> I’m certainly not MFC-ing changes other than clear bug fixes to stable/15 at this point in the release cycle. Colin doesn’t need more headaches.</p> <p dir="auto">—<br> Kristof</p> </div> </div> </body> </html>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?A7B7712B-E27A-4AA1-A823-3D6CB62017FF>