[PATCH v4 09/10] net/gve: enforce DQO Tx packet length limits

Joshua Washington joshwash at google.com
Mon Oct 19 19:35:50 CEST 2026


On Mon, Oct 19, 2026 at 9:55 AM Stephen Hemminger
<stephen at networkplumber.org> wrote:
>
> On Sun, 18 Oct 2026 13:24:03 +0530
> Tathagat Priyadarshi <tathagat.dpdk at gmail.com> wrote:
>
> > > > +static inline bool
> > > > +gve_tx_validate_pkt(const struct rte_mbuf *tx_pkt, bool is_tso)
> > >
> > > Minor nit (not worth blocking over, but maybe can be done as the
> > > patches are being merged): can this be renamed to
> > > gve_tx_validate_pkt_len? This is one step of many in the packet
> > > validation pipeline.
> >
> > Thanks for the review, Joshua. Agreed on renaming to gve_tx_validate_pkt_len.
> > This change impacts patches 9/10 and 10/10. I am happy to submit a v5
> > series if preferred,
> > or the maintainer can rename it upon applying.
>
> Renames should go through patchwork. I can cleanup commit messages and
> release notes, that is where I stop.

In that case, would it be okay to merge as-is? I will be following up
this series directly with some TX datapath fixes of my own which are
kind of related to Tathagat's fixes, so I can probably update the
function name in my patch series, rather than going to v5.


More information about the stable mailing list