> > diff --git a/lib/ethdev/rte_ethdev.c b/lib/ethdev/rte_ethdev.c > > index d9a03f12f9..ecc6c1fe37 100644 > > --- a/lib/ethdev/rte_ethdev.c > > +++ b/lib/ethdev/rte_ethdev.c > > @@ -6473,6 +6473,36 @@ rte_eth_rx_metadata_negotiate(uint16_t port_id, > uint64_t *features) > > (*dev->dev_ops->rx_metadata_negotiate)(dev, features)); > > } > > > > +int > > +rte_eth_ip_reassembly_conf_set(uint16_t port_id, > > + struct rte_eth_ip_reass_params *conf) > > +{ > > + struct rte_eth_dev *dev; > > + > > + RTE_ETH_VALID_PORTID_OR_ERR_RET(port_id, -ENODEV); > > + dev = &rte_eth_devices[port_id]; > > Should we check here that device is properly configured, but not started yet? Ok will add checks for dev->data->dev_configured and dev->data->dev_started
> Another question - if we have reassembly_conf_set() would it make sense to > have also reassembly_conf_get? > So user can retrieve current ip_reassembly config values? > The set/supported values can be retrieved using rte_eth_dev_info :: reass_capa