[PATCH v3 1/2] app/testpmd: assign share group dynamically

[Dariusz Sosnowski]

> -----Original Message-----
> From: Stephen Hemminger <stephen at networkplumber.org>
> Sent: Wednesday, March 25, 2026 7:51 PM
> To: Dariusz Sosnowski <dsosnowski at nvidia.com>
> Cc: Aman Singh <aman.deep.singh at intel.com>; dev at dpdk.org; NBU-
> Contact-Thomas Monjalon (EXTERNAL) <thomas at monjalon.net>; Raslan
> Darawsheh <rasland at nvidia.com>; Adrian Schollmeyer
> <a.schollmeyer at syseleven.de>
> Subject: Re: [PATCH v3 1/2] app/testpmd: assign share group dynamically
> 
> On Wed, 25 Mar 2026 19:02:53 +0100
> Dariusz Sosnowski <dsosnowski at nvidia.com> wrote:
> 
> > +static void
> > +try_release_share_group(struct share_group_slot *slot) {
> > +     uint16_t pi;
> > +     bool group_not_used = true;
> > +
> > +     /* Check if any port still uses this share group. */
> > +     RTE_ETH_FOREACH_DEV(pi) {
> > +             if (ports[pi].dev_info.switch_info.domain_id == slot->domain_id
> &&
> > +                 ports[pi].dev_info.switch_info.rx_domain == slot->rx_domain) {
> > +                     group_not_used = false;
> > +                     break;
> > +             }
> > +     }
> > +     if (group_not_used) {
> > +             slot->share_group = 0;
> > +             slot->domain_id = 0;
> > +             slot->rx_domain = 0;
> > +     }
> > +}
> 
> Just add a return and skip the group_not_used boolean.

Fixed in v4

Best regards,
Dariusz Sosnowski