> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-boun...@osuosl.org> On Behalf Of
> Vladimir Oltean
> Sent: Wednesday, February 12, 2025 11:01 PM
> To: Faizal Rahim <faizal.abdul.ra...@linux.intel.com>
> Cc: Nguyen, Anthony L <anthony.l.ngu...@intel.com>; Kitszel, Przemyslaw
> <przemyslaw.kits...@intel.com>; Andrew Lunn <andrew+net...@lunn.ch>;
> David S . Miller <da...@davemloft.net>; Eric Dumazet
> <eduma...@google.com>; Jakub Kicinski <k...@kernel.org>; Paolo Abeni
> <pab...@redhat.com>; Maxime Coquelin <mcoquelin.st...@gmail.com>;
> Alexandre Torgue <alexandre.tor...@foss.st.com>; Simon Horman
> <ho...@kernel.org>; Russell King <li...@armlinux.org.uk>; Alexei
> Starovoitov <a...@kernel.org>; Daniel Borkmann <dan...@iogearbox.net>;
> Jesper Dangaard Brouer <h...@kernel.org>; John Fastabend
> <john.fastab...@gmail.com>; Furong Xu <0x1...@gmail.com>; Russell King
> <rmk+ker...@armlinux.org.uk>; Serge Semin <fancer.lan...@gmail.com>;
> Xiaolei Wang <xiaolei.w...@windriver.com>; Suraj Jaiswal
> <quic_jsu...@quicinc.com>; Kory Maincent <kory.mainc...@bootlin.com>;
> Gal Pressman <g...@nvidia.com>; Jesper Nilsson <jesper.nils...@axis.com>;
> Andrew Halaney <ahala...@redhat.com>; Choong Yong Liang
> <yong.liang.cho...@linux.intel.com>; Kunihiko Hayashi
> <hayashi.kunih...@socionext.com>; Gomes, Vinicius
> <vinicius.go...@intel.com>; intel-wired-...@lists.osuosl.org;
> net...@vger.kernel.org; linux-ker...@vger.kernel.org; linux-stm32@st-md-
> mailman.stormreply.com; linux-arm-ker...@lists.infradead.org;
> b...@vger.kernel.org
> Subject: Re: [Intel-wired-lan] [PATCH iwl-next v4 0/9] igc: Add support for
> Frame Preemption feature in IGC
Please start commit title from slam letters:
Igc: add ...
> On Mon, Feb 10, 2025 at 02:01:58AM -0500, Faizal Rahim wrote:
> > Introduces support for the FPE feature in the IGC driver.
> >
> > The patches aligns with the upstream FPE API:
> >
> https://patchwork.kernel.org/project/netdevbpf/cover/20230220122343.1
> 1
> > 56614-1-vladimir.olt...@nxp.com/
> >
> https://patchwork.kernel.org/project/netdevbpf/cover/20230119122705.7
> 3
> > 054-1-vladimir.olt...@nxp.com/
> >
> > It builds upon earlier work:
> >
> https://patchwork.kernel.org/project/netdevbpf/cover/20220520011538.1
> 0
> > 98888-1-vinicius.go...@intel.com/
> >
> > The patch series adds the following functionalities to the IGC driver:
> > a) Configure FPE using `ethtool --set-mm`.
> > b) Display FPE settings via `ethtool --show-mm`.
> > c) View FPE statistics using `ethtool --include-statistics --show-mm'.
> > e) Enable preemptible/express queue with `fp`:
> > tc qdisc add ... root taprio \
> > fp E E P P
>
> Any reason why you are only enabling the preemptible traffic classes with
> taprio, and not with mqprio as well? I see there will have to be some work
> harmonizing igc's existing understanding of ring priorities with what Kurt
> did in
> 9f3297511dae ("igc: Add MQPRIO offload support"), and I was kind of
> expecting to see a proposal for that as part of this.
