> Dear Joshua, > > > Thank you for the patch. > > Am 14.10.25 um 01:13 schrieb Joshua Hay: > > From: Pavan Kumar Linga <[email protected]> > > > > Move both TX and RX queue resources to the newly introduced > > idpf_q_vec_rsrc structure. > > What is the motivation for doing this?
The main motivation is to decouple the queue resources from the vport structure. The previous implementations of the config path functions assumes we're operating on all the queues attached to a specific vport. By moving the queue resources into this queue resource specific struct, we can configure arbitrary queue groups, and more importantly ones that are not associated to a specific vport struct, e.g. control queues. This is critical for use cases where additional control queues are necessary. The PTP secondary mailbox is one such use case, which is needed to improve PTP latency. With this change, any additional control queues can be allocated, configured, and enabled, without having to allocate a dummy vport struct associated with those queues. > > > > While at it, declare the loop iterator in loop and use the correct type. > > Please make this a separate commit, as the diff is already big enough. Sure, will do. Thanks! Josh > > > Kind regards, > > Paul > > > > Reviewed-by: Anton Nadezhdin <[email protected]> > > Signed-off-by: Pavan Kumar Linga <[email protected]> > > Signed-off-by: Joshua Hay <[email protected]> > > --- > > v8: > > - rebase on AF_XDP series > > - remove dev param from rx_desc_rel and access through q_vector > > - introduce per queue RSC flag to avoid vport check > > --- > > drivers/net/ethernet/intel/idpf/idpf.h | 69 +-- > > .../net/ethernet/intel/idpf/idpf_ethtool.c | 91 ++-- > > drivers/net/ethernet/intel/idpf/idpf_lib.c | 71 +-- > > drivers/net/ethernet/intel/idpf/idpf_ptp.c | 17 +- > > drivers/net/ethernet/intel/idpf/idpf_txrx.c | 451 +++++++++--------- > > drivers/net/ethernet/intel/idpf/idpf_txrx.h | 21 +- > > .../net/ethernet/intel/idpf/idpf_virtchnl.c | 258 +++++----- > > .../net/ethernet/intel/idpf/idpf_virtchnl.h | 12 +- > > drivers/net/ethernet/intel/idpf/xdp.c | 37 +- > > drivers/net/ethernet/intel/idpf/xdp.h | 6 +- > > drivers/net/ethernet/intel/idpf/xsk.c | 7 +- > > 11 files changed, 561 insertions(+), 479 deletions(-) > > […]
