[RFC PATCH v2] net/bonding: restrict secondary control operations

[Weijun Pan]

Sure, please feel free to shorten it.

Thanks,
Weijun

On Sun, Aug 23, 2026 at 10:43 AM Stephen Hemminger
<stephen at networkplumber.org> wrote:
>
> On Sun, 23 Aug 2026 10:16:25 -0500
> Weijun Pan <wpan3636 at gmail.com> wrote:
>
> > diff --git a/doc/guides/prog_guide/link_bonding_poll_mode_drv_lib.rst b/doc/guides/prog_guide/link_bonding_poll_mode_drv_lib.rst
> > index 2fa1ac4028..7ecb05d407 100644
> > --- a/doc/guides/prog_guide/link_bonding_poll_mode_drv_lib.rst
> > +++ b/doc/guides/prog_guide/link_bonding_poll_mode_drv_lib.rst
> > @@ -254,6 +254,19 @@ Like all other PMD, all functions exported by a PMD are lock-free functions
> >  that are assumed not to be invoked in parallel on different logical cores to
> >  work on the same target object.
> >
> > +Bonding device configuration is owned by the primary process. Secondary
> > +processes may attach to an existing bonding device for query and detach
> > +operations, but must not change bonding configuration or device state.
> > +
> > +In a secondary process, bonding control operations such as configuring,
> > +starting or stopping the device, setting up queues, changing members,
> > +changing the bonding mode, updating RSS, changing MAC addresses, changing
> > +MTU, or configuring ``rte_flow`` rules are not supported.
> > +
> > +Secondary process datapath support is limited and bonding mode specific.
> > +Applications should not rely on secondary processes for bonding datapath
> > +operation unless support for the selected mode is explicitly documented.
> > +
> >  It should also be noted that the PMD receive function should not be invoked
> >  directly on a member devices after they have been to a bonding device since
> >  packets read directly from the member device will no longer be available to the
> > diff --git a/doc/guides/rel_notes/release_26_11.rst b/doc/guides/rel_notes/release_26_11.rst
> > index c8cc86295d..8df012c99b 100644
> > --- a/doc/guides/rel_notes/release_26_11.rst
> > +++ b/doc/guides/rel_notes/release_26_11.rst
> > @@ -55,6 +55,12 @@ New Features
> >       Also, make sure to start the actual text at the margin.
> >       =======================================================
> >
> > +* **Updated bonding PMD secondary process handling.**
> > +
> > +  Bonding devices now use a secondary-process device operations table to
> > +  keep query and detach paths available while rejecting unsupported control
> > +  operations before shared ethdev state can be modified. Bonding-specific
> > +  control APIs are also restricted to the primary process.
> >
> >  Removed Items
>
> Mind if I shorten this text, it reads like AI autogenerated verbosity.
> I.e 10x longer than needed. Release notes especially should be succinct.