RE: [PATCH net-next 0/6] igc: Fix corner cases for TSN offload

[Zulkifli, Muhammad Husaini]

Dear Florian,

Thanks for the patch series 😊

> -----Original Message-----
> From: Florian Kauer <florian.kauer@xxxxxxxxxxxxx>
> Sent: Wednesday, 14 June, 2023 10:07 PM
> To: Brandeburg, Jesse <jesse.brandeburg@xxxxxxxxx>; Nguyen, Anthony L
> <anthony.l.nguyen@xxxxxxxxx>; Gomes, Vinicius <vinicius.gomes@xxxxxxxxx>;
> David S . Miller <davem@xxxxxxxxxxxxx>; Eric Dumazet
> <edumazet@xxxxxxxxxx>; Jakub Kicinski <kuba@xxxxxxxxxx>; Paolo Abeni
> <pabeni@xxxxxxxxxx>; Tan Tee Min <tee.min.tan@xxxxxxxxxxxxxxx>; Zulkifli,
> Muhammad Husaini <muhammad.husaini.zulkifli@xxxxxxxxx>; Gunasekaran,
> Aravindhan <aravindhan.gunasekaran@xxxxxxxxx>; Chilakala, Mallikarjuna
> <mallikarjuna.chilakala@xxxxxxxxx>
> Cc: intel-wired-lan@xxxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; kurt@xxxxxxxxxxxxx; florian.kauer@xxxxxxxxxxxxx
> Subject: [PATCH net-next 0/6] igc: Fix corner cases for TSN offload
> 
> The igc driver supports several different offloading capabilities relevant in the
> TSN context. Recent patches in this area introduced regressions for certain
> corner cases that are fixed in this series.
> 
> Each of the patches (except the first one) addresses a different regression that
> can be separately reproduced. Still, they have overlapping code changes so they
> should not be separately applied.
> 
> Especially #4 and #6 address the same observation, but both need to be applied
> to avoid TX hang occurrences in the scenario described in the patches.
> 
> Signed-off-by: Florian Kauer <florian.kauer@xxxxxxxxxxxxx>
> Reviewed-by: Kurt Kanzenbach <kurt@xxxxxxxxxxxxx>
> 
> Florian Kauer (6):
>   igc: Rename qbv_enable to taprio_offload_enable
>   igc: Do not enable taprio offload for invalid arguments
>   igc: Handle already enabled taprio offload for basetime 0
>   igc: No strict mode in pure launchtime/CBS offload
>   igc: Fix launchtime before start of cycle
>   igc: Fix inserting of empty frame for launchtime

All six patches, as far as I can see here, have the Fixes tag. Should they go to Net instead of Net-Next?

> 
>  drivers/net/ethernet/intel/igc/igc.h      |  2 +-
>  drivers/net/ethernet/intel/igc/igc_main.c | 24 ++++++++-------------
> drivers/net/ethernet/intel/igc/igc_tsn.c  | 26 ++++++++++++++++++++---
>  3 files changed, 33 insertions(+), 19 deletions(-)
> 
> --
> 2.39.2