2023-06-12 10:12:57

by Ravi Gunasekaran

[permalink] [raw]
Subject: [PATCH] arm64: defconfig: Enable HSR by default

Enable HSR feature in kernel build.

Signed-off-by: Ravi Gunasekaran <[email protected]>
---
arch/arm64/configs/defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 8d850be05835..c980268eaea4 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -138,6 +138,7 @@ CONFIG_IP_PNP_BOOTP=y
CONFIG_IPV6=m
CONFIG_NETFILTER=y
CONFIG_BRIDGE_NETFILTER=m
+CONFIG_HSR=m
CONFIG_NF_CONNTRACK=m
CONFIG_NF_CONNTRACK_EVENTS=y
CONFIG_NETFILTER_XT_MARK=m
--
2.17.1



2023-06-12 10:13:04

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable HSR by default

On 12/06/2023 11:43, Ravi Gunasekaran wrote:
> Enable HSR feature in kernel build.

Why? Commit msg should answer to this question.

Best regards,
Krzysztof


2023-06-12 10:18:47

by Ravi Gunasekaran

[permalink] [raw]
Subject: Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR by default



On 6/12/23 3:14 PM, Krzysztof Kozlowski wrote:
> On 12/06/2023 11:43, Ravi Gunasekaran wrote:
>> Enable HSR feature in kernel build.
>
> Why? Commit msg should answer to this question.

I apologize for incomplete description. I will send out a v2.

HSR protocol is implemented by the network stack at /net/hsr/*
In order to create HSR interface, HSR module needs to be loaded.


>
> Best regards,
> Krzysztof
>

--
Regards,
Ravi

2023-06-12 10:32:13

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR by default

On 12/06/2023 11:49, Ravi Gunasekaran wrote:
>
>
> On 6/12/23 3:14 PM, Krzysztof Kozlowski wrote:
>> On 12/06/2023 11:43, Ravi Gunasekaran wrote:
>>> Enable HSR feature in kernel build.
>>
>> Why? Commit msg should answer to this question.
>
> I apologize for incomplete description. I will send out a v2.
>
> HSR protocol is implemented by the network stack at /net/hsr/*
> In order to create HSR interface, HSR module needs to be loaded.

You described what HSR is, but I want to know why do we need it in our
development arm64 kernel. You might want to enable anything and explain
what is this "anything" but this does not solve any of the concerns.

Best regards,
Krzysztof


2023-06-12 12:25:40

by Ravi Gunasekaran

[permalink] [raw]
Subject: Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR by default



On 6/12/23 3:21 PM, Krzysztof Kozlowski wrote:
> On 12/06/2023 11:49, Ravi Gunasekaran wrote:
>>
>>
>> On 6/12/23 3:14 PM, Krzysztof Kozlowski wrote:
>>> On 12/06/2023 11:43, Ravi Gunasekaran wrote:
>>>> Enable HSR feature in kernel build.
>>>
>>> Why? Commit msg should answer to this question.
>>
>> I apologize for incomplete description. I will send out a v2.
>>
>> HSR protocol is implemented by the network stack at /net/hsr/*
>> In order to create HSR interface, HSR module needs to be loaded.
>
> You described what HSR is, but I want to know why do we need it in our
> development arm64 kernel. You might want to enable anything and explain
> what is this "anything" but this does not solve any of the concerns.
>

TI SoCs such as AM64x, AM65x support multiple ethernet ports which can be
used to setup HSR (a redundancy protocol) node in the network. So enabling
the HSR driver to be built as module. The HSR node can be created using a
pair of ethernet ports in the system.

> Best regards,
> Krzysztof
>

--
Regards,
Ravi