Linux Netfilter development
 help / color / mirror / Atom feed
From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Julian Anastasov <ja@ssi.bg>
Cc: Jakub Kicinski <kuba@kernel.org>,
	netfilter-devel@vger.kernel.org, davem@davemloft.net,
	netdev@vger.kernel.org, pabeni@redhat.com, edumazet@google.com,
	horms@kernel.org, fw@strlen.de
Subject: Re: [PATCH net 06/12] ipvs: bound LBLCR and LBLC cache growth
Date: Thu, 10 Sep 2026 01:47:28 +0200	[thread overview]
Message-ID: <aqHwEDfPTQBcLH2m@chamomile> (raw)
In-Reply-To: <2051dadb-fa1a-02bd-42fb-57719e6acb7c@ssi.bg>

Hi Julian,

On Wed, Sep 09, 2026 at 03:24:48PM +0300, Julian Anastasov wrote:
> 
> 	Hi Pablo,
> 
> On Fri, 4 Sep 2026, Julian Anastasov wrote:
> 
> > On Thu, 3 Sep 2026, Jakub Kicinski wrote:
> > 
> > > This is an AI-generated review of your patch. The human sending this
> > > email has considered the AI review valid, or at least plausible.
> > > ---
> > > ipvs: bound LBLCR and LBLC cache growth
[...]
> > 	Yes, we remember this place from the previous Sashiko
> > review but the new change depends on the current patch to
> > apply gracefully.
> 
> 	I expected this patch to be applied before posting
> a separate fix for the missing atomic_dec(&tbl->entries).
> Another option is to post the atomic_dec fix on top of the
> current patch or better before it, so that both can be applied together.
> If needed, I can repost them together in a new patchset, so that
> they can be reviewed together properly.

Apologies for the inconvenience, I decided to keep back the three
patches that LLMs are barfing when I saw the "Changed Requested" label
in patchwork. Too bad.

Please, go ahead post them together, including a cover letter
explaining the logic, when bugs are multidimensional, LLMs become a
bit strict because they want you to fix all at once, that's hard.

Thanks.

  reply	other threads:[~2026-09-09 23:47 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-03  0:41 [PATCH net 00/12] Netfilter/IPVS fixes for net Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 01/12] ipvs: reject invalid states in connection template sync records Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 02/12] ipvs: fix reversed sequence option serialization Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 03/12] netfilter: nf_conntrack_sip: fix OOB read in sip_skip_whitespace() Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 04/12] netfilter: cttimeout: prevent UAF during module unload Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 05/12] netfilter: nf_log: unregister loggers before per-net teardown Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 06/12] ipvs: bound LBLCR and LBLC cache growth Pablo Neira Ayuso
2026-09-04  2:01   ` Jakub Kicinski
2026-09-04  4:20     ` Julian Anastasov
2026-09-09 12:24       ` Julian Anastasov
2026-09-09 23:47         ` Pablo Neira Ayuso [this message]
2026-09-10 10:26           ` Julian Anastasov
2026-09-03  0:41 ` [PATCH net 07/12] netfilter: nft_payload: restrict checksum offsets to known values Pablo Neira Ayuso
2026-09-04  2:01   ` Jakub Kicinski
2026-09-04  5:56     ` Florian Westphal
2026-09-03  0:41 ` [PATCH net 08/12] netfilter: nfnetlink_log: cope with concurrent instance destruction Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 09/12] netfilter: nfnetlink_queue: hold nfnl mutex in event notifier Pablo Neira Ayuso
2026-09-04  2:01   ` Jakub Kicinski
2026-09-04  5:56     ` Florian Westphal
2026-09-03  0:41 ` [PATCH net 10/12] netfilter: arp_tables: remove the 32bit compat interface Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 11/12] netfilter: ip6_tables: set F_PROTO when proto value is nonzero Pablo Neira Ayuso
2026-09-03  0:41 ` [PATCH net 12/12] netfilter: report NLM_F_DUMP_FILTERED when all is filtered out Pablo Neira Ayuso
2026-09-04  2:04 ` [PATCH net 00/12] Netfilter/IPVS fixes for net Jakub Kicinski
2026-09-04  5:57   ` Florian Westphal
2026-09-04 10:55     ` Pablo Neira Ayuso
2026-09-04 10:59       ` Florian Westphal

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aqHwEDfPTQBcLH2m@chamomile \
    --to=pablo@netfilter.org \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=fw@strlen.de \
    --cc=horms@kernel.org \
    --cc=ja@ssi.bg \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=netfilter-devel@vger.kernel.org \
    --cc=pabeni@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox