On Fri, 2023-11-17 at 12:21 +0100, Jiri Pirko wrote: > Fri, Nov 17, 2023 at 06:52:49AM CET, xuejun.zh...@intel.com wrote: > > Hello Jiri & Jakub, > > > > Thanks for looking into our last patch with devlink API. Really appreciate > > your candid review. > > > > Following your suggestion, we have looked into 3 tc offload options to > > support queue rate limiting > > > > #1 mq + matchall + police > > This looks most suitable. Why it would not work?
AFAICS, it should work, but it does not look the most suitable to me: beyond splitting a "simple" task in separate entities, it poses a constraint on the classification performed on the egress device. Suppose the admin wants to limit the egress bandwidth on the given tx queue _and_ do some application specific packet classification and actions. That would not be possible right? Thanks! Paolo _______________________________________________ Intel-wired-lan mailing list Intel-wired-lan@osuosl.org https://lists.osuosl.org/mailman/listinfo/intel-wired-lan
