Re: tg3 crashes under high load, when using 100Mbits

[Satish Baddipadige]

On Thu, Feb 15, 2018 at 7:37 PM, Siva Reddy Kallam
 wrote:
> On Mon, Feb 12, 2018 at 10:59 AM, Siva Reddy Kallam
>  wrote:
>> On Fri, Feb 9, 2018 at 10:41 AM, Kai Heng Feng
>>  wrote:
>>> Hi Broadcom folks,
>>>
>>> We are now enabling a new platform with tg3 nic, unfortunately we observed
>>> the bug [1] that dated back to 2015.
>>> I tried commit 4419bb1cedcd ("tg3: Add workaround to restrict 5762 MRRS to
>>> 2048”) but it does’t work.
>>>
>>> Do you have any idea how to solve the issue?
>>>
>>> [1] https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1447664
>>>
>>> Kai-Heng
>>>
>> Thank you for reporting. We will check and update you.
> With link aware mode, the clock speed could be slow and boot code does not
> complete within the expected time with lower link speeds. Need to override
> and the clock in driver. We are checking the feasibility of adding
> this in driver or firmware.

Hi Kai-Heng,

Can you please test the attached patch?

Thanks,
Satish


tg3_5762_clock_override.patch
Description: Binary data

Re: tg3 crashes under high load, when using 100Mbits

[Satish Baddipadige]

On Thu, Feb 15, 2018 at 7:37 PM, Siva Reddy Kallam
 wrote:
> On Mon, Feb 12, 2018 at 10:59 AM, Siva Reddy Kallam
>  wrote:
>> On Fri, Feb 9, 2018 at 10:41 AM, Kai Heng Feng
>>  wrote:
>>> Hi Broadcom folks,
>>>
>>> We are now enabling a new platform with tg3 nic, unfortunately we observed
>>> the bug [1] that dated back to 2015.
>>> I tried commit 4419bb1cedcd ("tg3: Add workaround to restrict 5762 MRRS to
>>> 2048”) but it does’t work.
>>>
>>> Do you have any idea how to solve the issue?
>>>
>>> [1] https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1447664
>>>
>>> Kai-Heng
>>>
>> Thank you for reporting. We will check and update you.
> With link aware mode, the clock speed could be slow and boot code does not
> complete within the expected time with lower link speeds. Need to override
> and the clock in driver. We are checking the feasibility of adding
> this in driver or firmware.

Hi Kai-Heng,

Can you please test the attached patch?

Thanks,
Satish


tg3_5762_clock_override.patch
Description: Binary data

Re: NETDEV WATCHDOG: eth0 (tg3): transmit queue 0 timed out

[Satish Baddipadige]

On Wed, Feb 28, 2018 at 7:40 PM, Siva Reddy Kallam
 wrote:
> On Sat, Feb 24, 2018 at 3:48 PM, Borislav Petkov  wrote:
>> Hi,
>>
>> this didn't happen before but after 4.16-rc1 my tg3 nic stops for
>> whatever reason and the connection to the machine is dead. It didn't show
>> anything in dmesg until today.
>>
>> The IO pagefaults look like it is trying to access something it
>> shouldn't and maybe that's why it times out.
>>
>> It triggers pretty quickly so I'd call it a reliable reproducer and thus
>> I can test patches... :-)
>>
>> Thx.
> Thanks for reporting this. Somehow, this mail moved to my spam folder.
> Hence, delay in response.
> Looks like this is similar to below issue and it was reported some time back.
> https://www.spinics.net/lists/netdev/msg482757.html
> We are actively working on this. We will soon provide you an update on this.

Hi Borislav,

Can you please test the attached patch?

Thanks,
Satish


tg3_5762_clock_override.patch
Description: Binary data

Re: NETDEV WATCHDOG: eth0 (tg3): transmit queue 0 timed out

[Satish Baddipadige]

On Wed, Feb 28, 2018 at 7:40 PM, Siva Reddy Kallam
 wrote:
> On Sat, Feb 24, 2018 at 3:48 PM, Borislav Petkov  wrote:
>> Hi,
>>
>> this didn't happen before but after 4.16-rc1 my tg3 nic stops for
>> whatever reason and the connection to the machine is dead. It didn't show
>> anything in dmesg until today.
>>
>> The IO pagefaults look like it is trying to access something it
>> shouldn't and maybe that's why it times out.
>>
>> It triggers pretty quickly so I'd call it a reliable reproducer and thus
>> I can test patches... :-)
>>
>> Thx.
> Thanks for reporting this. Somehow, this mail moved to my spam folder.
> Hence, delay in response.
> Looks like this is similar to below issue and it was reported some time back.
> https://www.spinics.net/lists/netdev/msg482757.html
> We are actively working on this. We will soon provide you an update on this.

Hi Borislav,

Can you please test the attached patch?

Thanks,
Satish


tg3_5762_clock_override.patch
Description: Binary data