On Wed, Sep 30, 2020 at 10:44:07PM +0300, Ido Schimmel wrote: > On Wed, Sep 30, 2020 at 10:16:45PM +0300, Ioana Ciornei wrote: > > +static int dpaa2_eth_dl_trap_action_set(struct devlink *devlink, > > + const struct devlink_trap *trap, > > + enum devlink_trap_action action, > > + struct netlink_ext_ack *extack) > > +{ > > + /* No support for changing the action of an independent packet trap, > > + * only per trap group - parser error drops > > + */ > > + return -EOPNOTSUPP; > > Please also add an error message via extack > Sure.
- [PATCH net-next v2 0/4] dpaa2-eth: add devlink parser error... Ioana Ciornei
- [PATCH net-next v2 2/4] devlink: add .trap_group_actio... Ioana Ciornei
- Re: [PATCH net-next v2 2/4] devlink: add .trap_gro... Ido Schimmel
- Re: [PATCH net-next v2 2/4] devlink: add .trap... Ioana Ciornei
- [PATCH net-next v2 1/4] devlink: add parser error drop... Ioana Ciornei
- [PATCH net-next v2 3/4] dpaa2-eth: add basic devlink s... Ioana Ciornei
- [PATCH net-next v2 4/4] dpaa2-eth: add support for dev... Ioana Ciornei
- Re: [PATCH net-next v2 4/4] dpaa2-eth: add support... Ido Schimmel
- Re: [PATCH net-next v2 4/4] dpaa2-eth: add sup... Ioana Ciornei
- Re: [PATCH net-next v2 4/4] dpaa2-eth: add support... Jakub Kicinski