Re: [PATCH net v7 1/7] net: mana: reference-count CQs looked up from the EQ handler

From: Jakub Kicinski

Date: Tue Aug 18 2026 - 12:22:06 EST


On Tue, 18 Aug 2026 09:20:05 -0700 Jakub Kicinski wrote:
> On Thu, 13 Aug 2026 10:42:33 -0700 Long Li wrote:
> > }
> >
> > +static struct gdma_queue *mana_gd_get_cq(struct gdma_context *gc, u32 cq_id);
> > +static void mana_gd_put_cq(struct gdma_queue *cq);
>
> Forward declarations are considered a bad smell in the kernel.
> Please add these functions here, I don't see a reason for this
> they are purely calling kernel APIs.
>
> > static void mana_gd_process_eqe(struct gdma_queue *eq)

Well, prolly not gonna have time to review the rest this week
due to the merge window, so pls TAL at the AI review as well
before you respin:
https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260813174243.3044348-2-longli@xxxxxxxxxxxxx