arm64: defconfig: Enable configs for traffic classifier and action module

[PATCH] arm64: defconfig: Enable configs for traffic classifier and action module

Posted by Chintan Vankar 1 month, 1 week ago

Enable CONFIG_NET_CLS_U32 and CONFIG_NET_ACT_SKBEDIT as modules to allow
traffic control operations that can be used during TSN (Time Sensitive
Networking) testing.

Signed-off-by: Chintan Vankar <c-vankar@ti.com>
---

This patch is based on commit '2b7a25df823d' of origin/master
branch of Linux repo.

 arch/arm64/configs/defconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index b67d5b1fc45b..265300c1a62c 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -177,12 +177,14 @@ CONFIG_NET_SCH_TAPRIO=m
 CONFIG_NET_SCH_MQPRIO=m
 CONFIG_NET_SCH_INGRESS=m
 CONFIG_NET_CLS_BASIC=m
+CONFIG_NET_CLS_U32=m
 CONFIG_NET_CLS_FLOWER=m
 CONFIG_NET_CLS_ACT=y
 CONFIG_NET_ACT_GACT=m
 CONFIG_NET_ACT_MIRRED=m
 CONFIG_HSR=m
 CONFIG_NET_ACT_GATE=m
+CONFIG_NET_ACT_SKBEDIT=m
 CONFIG_QRTR_SMD=m
 CONFIG_QRTR_TUN=m
 CONFIG_CAN=m
-- 
2.34.1

Re: [PATCH] arm64: defconfig: Enable configs for traffic classifier and action module

Posted by Krzysztof Kozlowski 1 month, 1 week ago

On 19/02/2026 17:48, Chintan Vankar wrote:
> Enable CONFIG_NET_CLS_U32 and CONFIG_NET_ACT_SKBEDIT as modules to allow
> traffic control operations that can be used during TSN (Time Sensitive
> Networking) testing.

Why do we need it in the defconfig? This is not a full blown distro
config, so any tests of user space or of other non-SoC features by
design require multiple other options.

> 
> Signed-off-by: Chintan Vankar <c-vankar@ti.com>
> ---

Best regards,
Krzysztof

Re: [PATCH] arm64: defconfig: Enable configs for traffic classifier and action module

Posted by Chintan Vankar 1 month, 1 week ago

Hello Krzysztof,

On 20/02/26 12:40, Krzysztof Kozlowski wrote:
> On 19/02/2026 17:48, Chintan Vankar wrote:
>> Enable CONFIG_NET_CLS_U32 and CONFIG_NET_ACT_SKBEDIT as modules to allow
>> traffic control operations that can be used during TSN (Time Sensitive
>> Networking) testing.
> 
> Why do we need it in the defconfig? This is not a full blown distro
> config, so any tests of user space or of other non-SoC features by
> design require multiple other options.
> 

I agree to your point, these options are not strictly required for arm64
platform for an essential features. They are added to simplify TSN
feature validation as I mentioned in the commit message.

Regards,
Chintan.

>>
>> Signed-off-by: Chintan Vankar <c-vankar@ti.com>
>> ---
> 
> Best regards,
> Krzysztof

Re: [PATCH] arm64: defconfig: Enable configs for traffic classifier and action module

Posted by Krzysztof Kozlowski 1 month, 1 week ago

On 20/02/2026 21:53, Chintan Vankar wrote:
> Hello Krzysztof,
> 
> On 20/02/26 12:40, Krzysztof Kozlowski wrote:
>> On 19/02/2026 17:48, Chintan Vankar wrote:
>>> Enable CONFIG_NET_CLS_U32 and CONFIG_NET_ACT_SKBEDIT as modules to allow
>>> traffic control operations that can be used during TSN (Time Sensitive
>>> Networking) testing.
>>
>> Why do we need it in the defconfig? This is not a full blown distro
>> config, so any tests of user space or of other non-SoC features by
>> design require multiple other options.
>>
> 
> I agree to your point, these options are not strictly required for arm64
> platform for an essential features. They are added to simplify TSN
> feature validation as I mentioned in the commit message.

I don't get why do we need defconfig to validate TSN feature, if this is
not a hardware feature. Unless it is and you actually need it for
drivers, but then commit msg is incomplete.

Best regards,
Krzysztof