[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <99d1e399-16bd-49fd-9fcf-4db6fc029780@gmail.com>
Date: Fri, 22 Mar 2024 13:32:32 +0100
From: Heiner Kallweit <hkallweit1@...il.com>
Cc: atlas.yu@...onical.com, davem@...emloft.net, edumazet@...gle.com,
hau@...ltek.com, kuba@...nel.org, netdev@...r.kernel.org,
nic_swsd@...ltek.com, pabeni@...hat.com
Subject: Re: r8169 DASH-related issue
On 22.03.2024 11:49, Atlas Yu wrote:
> On Fri, Mar 22, 2024 at 6:16 PM Heiner Kallweit <hkallweit1@...il.com> wrote:
>
>> No, this only checks whether DASH is enabled.
>> I don't think is redundant, because the original change explicitly mentions that
>> DASH fw may impact behavior even if DASH is disabled.
>
> I see, thanks for the clarification.
>
>> I understand that on your test system DASH is disabled. But does your system have
>> a DASH fw or not?
>
> I am not familiar with DASH, my system's DASH type is "RTL_DASH_EP", and I have no
> idea if it has a DASH firmware or not. I am glad to check it if you tell me how.
I don't have access to datasheets and can't tell. Therefore I asked Realtek to comment.
> My patched r8169 driver and r8168 driver both work well on my system.
>
>> My assumption is that the poll loop is relevant on systems with DASH fw, even if
>> DASH is disabled.
>
> I know your concern, but in my case it is wasting 300ms on driver startup. Maybe
> we can find a way to avoid this together.
Before applying the change I'd like to ensure that it doesn't break anything on
systems with a different DASH setup. So let's see whether Realtek provides some
more insight.
Powered by blists - more mailing lists