* [PATCH net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM
@ 2021-06-10 20:56 Heiner Kallweit
2021-06-11 8:04 ` Koba Ko
2021-06-11 20:00 ` patchwork-bot+netdevbpf
0 siblings, 2 replies; 4+ messages in thread
From: Heiner Kallweit @ 2021-06-10 20:56 UTC (permalink / raw)
To: Jakub Kicinski, David Miller, Realtek linux nic maintainers, Koba Ko
Cc: netdev
It has been reported that on RTL8106e the link-up interrupt may be
significantly delayed if the user enables ASPM L1. Per default ASPM
is disabled. The change leaves L1 enabled on the PCIe link (thus still
allowing to reach higher package power saving states), but the
NIC won't actively trigger it.
Reported-by: Koba Ko <koba.ko@canonical.com>
Tested-by: Koba Ko <koba.ko@canonical.com>
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
---
drivers/net/ethernet/realtek/r8169_main.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c
index 64f94a3fe..6a9fe9f7e 100644
--- a/drivers/net/ethernet/realtek/r8169_main.c
+++ b/drivers/net/ethernet/realtek/r8169_main.c
@@ -3508,7 +3508,6 @@ static void rtl_hw_start_8106(struct rtl8169_private *tp)
rtl_eri_write(tp, 0x1b0, ERIAR_MASK_0011, 0x0000);
rtl_pcie_state_l2l3_disable(tp);
- rtl_hw_aspm_clkreq_enable(tp, true);
}
DECLARE_RTL_COND(rtl_mac_ocp_e00e_cond)
--
2.32.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM
2021-06-10 20:56 [PATCH net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM Heiner Kallweit
@ 2021-06-11 8:04 ` Koba Ko
2021-06-11 9:14 ` Heiner Kallweit
2021-06-11 20:00 ` patchwork-bot+netdevbpf
1 sibling, 1 reply; 4+ messages in thread
From: Koba Ko @ 2021-06-11 8:04 UTC (permalink / raw)
To: Heiner Kallweit, Kai Heng Feng
Cc: Jakub Kicinski, David Miller, Realtek linux nic maintainers, netdev
On Fri, Jun 11, 2021 at 4:57 AM Heiner Kallweit <hkallweit1@gmail.com> wrote:
>
> It has been reported that on RTL8106e the link-up interrupt may be
> significantly delayed if the user enables ASPM L1. Per default ASPM
> is disabled. The change leaves L1 enabled on the PCIe link (thus still
> allowing to reach higher package power saving states), but the
> NIC won't actively trigger it.
>
> Reported-by: Koba Ko <koba.ko@canonical.com>
> Tested-by: Koba Ko <koba.ko@canonical.com>
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
> ---
> drivers/net/ethernet/realtek/r8169_main.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c
> index 64f94a3fe..6a9fe9f7e 100644
> --- a/drivers/net/ethernet/realtek/r8169_main.c
> +++ b/drivers/net/ethernet/realtek/r8169_main.c
> @@ -3508,7 +3508,6 @@ static void rtl_hw_start_8106(struct rtl8169_private *tp)
> rtl_eri_write(tp, 0x1b0, ERIAR_MASK_0011, 0x0000);
>
> rtl_pcie_state_l2l3_disable(tp);
> - rtl_hw_aspm_clkreq_enable(tp, true);
> }
As per 0866cd15029b, this also affects the intel soc idle state.
Even the result is positive currently, I think this modification would
have higher risk.
>
> DECLARE_RTL_COND(rtl_mac_ocp_e00e_cond)
> --
> 2.32.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM
2021-06-11 8:04 ` Koba Ko
@ 2021-06-11 9:14 ` Heiner Kallweit
0 siblings, 0 replies; 4+ messages in thread
From: Heiner Kallweit @ 2021-06-11 9:14 UTC (permalink / raw)
To: Koba Ko, Kai Heng Feng
Cc: Jakub Kicinski, David Miller, Realtek linux nic maintainers, netdev
On 11.06.2021 10:04, Koba Ko wrote:
> On Fri, Jun 11, 2021 at 4:57 AM Heiner Kallweit <hkallweit1@gmail.com> wrote:
>>
>> It has been reported that on RTL8106e the link-up interrupt may be
>> significantly delayed if the user enables ASPM L1. Per default ASPM
>> is disabled. The change leaves L1 enabled on the PCIe link (thus still
>> allowing to reach higher package power saving states), but the
>> NIC won't actively trigger it.
>>
>> Reported-by: Koba Ko <koba.ko@canonical.com>
>> Tested-by: Koba Ko <koba.ko@canonical.com>
>> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
>> ---
>> drivers/net/ethernet/realtek/r8169_main.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/drivers/net/ethernet/realtek/r8169_main.c b/drivers/net/ethernet/realtek/r8169_main.c
>> index 64f94a3fe..6a9fe9f7e 100644
>> --- a/drivers/net/ethernet/realtek/r8169_main.c
>> +++ b/drivers/net/ethernet/realtek/r8169_main.c
>> @@ -3508,7 +3508,6 @@ static void rtl_hw_start_8106(struct rtl8169_private *tp)
>> rtl_eri_write(tp, 0x1b0, ERIAR_MASK_0011, 0x0000);
>>
>> rtl_pcie_state_l2l3_disable(tp);
>> - rtl_hw_aspm_clkreq_enable(tp, true);
>> }
>
> As per 0866cd15029b, this also affects the intel soc idle state.
> Even the result is positive currently, I think this modification would
> have higher risk.
>
At the time of 0866cd15029b ASPM was enabled in r8169. Interesting that
after 0866cd15029b nobody reported the link-up issue you're facing.
A few months later (with b75bb8a5b755) we had to disable ASPM again
because of several problem reports. Since then 0866cd15029b has no
practical relevance in mainline.
>>
>> DECLARE_RTL_COND(rtl_mac_ocp_e00e_cond)
>> --
>> 2.32.0
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM
2021-06-10 20:56 [PATCH net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM Heiner Kallweit
2021-06-11 8:04 ` Koba Ko
@ 2021-06-11 20:00 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2021-06-11 20:00 UTC (permalink / raw)
To: Heiner Kallweit; +Cc: kuba, davem, nic_swsd, koba.ko, netdev
Hello:
This patch was applied to netdev/net-next.git (refs/heads/master):
On Thu, 10 Jun 2021 22:56:59 +0200 you wrote:
> It has been reported that on RTL8106e the link-up interrupt may be
> significantly delayed if the user enables ASPM L1. Per default ASPM
> is disabled. The change leaves L1 enabled on the PCIe link (thus still
> allowing to reach higher package power saving states), but the
> NIC won't actively trigger it.
>
> Reported-by: Koba Ko <koba.ko@canonical.com>
> Tested-by: Koba Ko <koba.ko@canonical.com>
> Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
>
> [...]
Here is the summary with links:
- [net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM
https://git.kernel.org/netdev/net-next/c/1ee8856de82f
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-06-11 20:00 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-10 20:56 [PATCH net-next] r8169: avoid link-up interrupt issue on RTL8106e if user enables ASPM Heiner Kallweit
2021-06-11 8:04 ` Koba Ko
2021-06-11 9:14 ` Heiner Kallweit
2021-06-11 20:00 ` patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.