From: mingming cao <mmc@linux.ibm.com>
To: Jakub Kicinski <kuba@kernel.org>
Cc: netdev@vger.kernel.org, davem@davemloft.net, horms@kernel.org,
edumazet@google.com, pabeni@redhat.com, andrew+netdev@lunn.ch,
nnac123@linux.ibm.com, maddy@linux.ibm.com, mpe@ellerman.id.au,
linuxppc-dev@lists.ozlabs.org, haren@linux.ibm.com,
ricklind@linux.ibm.com, davemarq@linux.ibm.com,
bjking1@linux.ibm.com, shaik.abdulla1@ibm.com
Subject: Re: [PATCH net-next v7 00/15] ibmveth: Add multi-queue RX support
Date: Fri, 2 Oct 2026 19:11:10 -0700 [thread overview]
Message-ID: <7de436b1-82b5-4726-9915-a9a78a20c903@linux.ibm.com> (raw)
In-Reply-To: <20261001155646.5957a373@kernel.org>
On 10/1/26 3:56 PM, Jakub Kicinski wrote:
> On Sat, 26 Sep 2026 10:40:56 -0700 mingming cao wrote:
>> Hi Jacub,
> There's no c in my name, sir. I wonder why people do that.
Sorry about that, Jakub. Noted, and noted on To: as well.
>> I noticed this v7 series is flagged red on patchwork. The apply failure
>> is due to a dependency on my own [PATCH net 0/2] (Message-ID:
>> cover.1790357373.git.mmc@linux.ibm.com) sent the same day — kept
>> separate per your earlier feedback to peel fixes out of feature series.
>> Both touch ibmveth_open() in the same region, and the net pair is not
>> yet in net-next.
>>
>> The v7 MQ series is based on net-next 161ea2d4f2a7 (2026-09-24). The
>> fixes are already subsumed by MQ patches 3 and 6.
>>
>> Shall I wait and rebase MQ once the net pair lands in net-next, or send
>> a v8 now with the net pair folded in? Happy to do either.
> Sorry for the delay, hint: please put just me in the To: if you want me
> to spot the email sooner. We get 400+ messages to the list every day,
> I'm only reading the list as I review :(
>
> To answer your question - please hold off with reposting until the fixes
> are in net-next. We won't merge the patches if they conflict with fixes.
Will do. Thanks for applying the net pair (af0524bf4ce1, 84bec0bf0352).
I will not repost the multi-queue series until they show up in net-next.
A heads-up so the next series is not a surprise: I posted seven more
ibmveth fixes, split out of this series:
[PATCH net-next 0/7] ibmveth: fix hangs, use-after-frees and
netpoll races
https://lore.kernel.org/netdev/cover.1790991039.git.mmc@linux.ibm.com/
<https://lore.kernel.org/netdev/cover.1790991039.git.mmc@linux.ibm.com/>
They fix serious pre-existing bugs in the driver (two hangs, three
use-after-frees and memory corruption), found by AI-assisted review
of this series. They are based on net-next and merge cleanly with
the net pair; they also apply to net if you would rather take them
there.
I am working on v8. My thinking for v8 of the multi-queue series is to
base it on
net-next once both the net pair and these fixes are in. With the
fixes split out it should be smaller than v7, with the MQ patches
mostly extending that handling per queue, and it would address the
review comments on v7. Hope that works for you. Happy to adjust as needed.
Regards,
Mingming
prev parent reply other threads:[~2026-10-03 2:11 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-25 18:38 [PATCH net-next v7 00/15] ibmveth: Add multi-queue RX support Mingming Cao
2026-09-25 18:38 ` [PATCH net-next v7 01/15] ibmveth: Add MQ RX hypercall wrappers and call definitions Mingming Cao
2026-09-25 18:38 ` [PATCH net-next v7 02/15] ibmveth: Prepare MQ RX adapter data structures Mingming Cao
2026-09-25 18:38 ` [PATCH net-next v7 03/15] ibmveth: Refactor RX resource allocation for MQ RX bring-up Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 04/15] ibmveth: Refactor buffer pool management for per-queue MQ RX Mingming Cao
2026-09-25 18:38 ` [PATCH net-next v7 05/15] ibmveth: Refactor RX interrupt control for MQ RX queues Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 06/15] ibmveth: Refactor TX resource allocation in open/close paths Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 07/15] ibmveth: Add RX queue register helpers for MQ Mingming Cao
2026-09-25 18:38 ` [PATCH net-next v7 08/15] ibmveth: Add queue-aware RX buffer submit helper " Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 09/15] ibmveth: Harden RX poll path with helpers Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 10/15] ibmveth: Enable multi-queue RX receive path Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 11/15] ibmveth: Add per-queue RX and TX statistics collection Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 12/15] ibmveth: Report MQ-aware RX counts in ethtool get_channels Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 13/15] ibmveth: Expose per-queue buffer pool details via debugfs Mingming Cao
2026-09-25 18:38 ` [PATCH net-next v7 14/15] ibmveth: Implement incremental MQ RX queue resize Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-25 18:38 ` [PATCH net-next v7 15/15] ibmveth: Complete set_channels down-path and mq_fallback max_rx cap Mingming Cao
2026-09-29 19:33 ` netdev-bot+sashiko
2026-09-26 17:40 ` [PATCH net-next v7 00/15] ibmveth: Add multi-queue RX support mingming cao
2026-10-01 22:56 ` Jakub Kicinski
2026-10-03 2:11 ` mingming cao [this message]
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=7de436b1-82b5-4726-9915-a9a78a20c903@linux.ibm.com \
--to=mmc@linux.ibm.com \
--cc=andrew+netdev@lunn.ch \
--cc=bjking1@linux.ibm.com \
--cc=davem@davemloft.net \
--cc=davemarq@linux.ibm.com \
--cc=edumazet@google.com \
--cc=haren@linux.ibm.com \
--cc=horms@kernel.org \
--cc=kuba@kernel.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=maddy@linux.ibm.com \
--cc=mpe@ellerman.id.au \
--cc=netdev@vger.kernel.org \
--cc=nnac123@linux.ibm.com \
--cc=pabeni@redhat.com \
--cc=ricklind@linux.ibm.com \
--cc=shaik.abdulla1@ibm.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