lkml.org 
[lkml]   [2019]   [Apr]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 1/2] stmmac: introduce flag to dynamically disable TX offload for rockchip devices
Date
From: Philipp Tomsich <philipp.tomsich@theobroma-systems.com>
Date: Mon, Apr 01, 2019 at 20:12:21

> + Christoph.
>
> > On 01.04.2019, at 21:06, Heiko Stübner <heiko@sntech.de> wrote:
> >
> > Am Montag, 1. April 2019, 20:54:45 CEST schrieb Robin Murphy:
> >> On 01/04/2019 19:18, Leonidas P. Papadakos wrote:
> >>> From: =?UTF-8?q?Kamil=20Trzci=C5=84ski?= <ayufan@ayufan.eu>
> >>>
> >>> Some rockchip boards exhibit an issue where tx checksumming does not work
> with
> >>> packets larger than 1498.
> >>
> >> Is it really a board-level problem? I'm no networking expert, but the
> >> nature of the workaround suggests this is more likely to be some
> >> inherent limitation of the IP block in the SoC, rather than something to
> >> do with how the external pins get wired up. Does anyone have an RK3328
> >> or RK3399 board that provably *does* checksum large packets correctly?
> >
> > I don't have that many rk3399-boards with actual ethernet and even only
> > the rock64 from rk3328-land, but at least my rk3399-firefly also seems
> > affected by this.
> >
> > But so far the rk3399-puma board from Theobroma did not show that ethernet
> > issue for me, so I've added two Theobroma people who may or may not tell
> > if they've also seen that issue.
> >
> >>
> >>> This is bad for network stability.
> >>>
> >>> The previous approach was using force_thresh_dma_mode in the board dts,
> which
> >>> does more than we need.
> >>
> >> If indeed it is a SoC-level thing (or at least we want to treat it as
> >> such), then couldn't we just hang it off the existing SoC-specific
> >> compatibles in dwmac-rk.c and avoid the need for a new DT property at
> >> all? After all, that's precisely why SoC-specific compatibles are a
> >> thing in the first place.
> >>

This can happen when FIFO size + PBL settings are not big enough for COE.

Can you please share the above settings ?

Thanks,
Jose Miguel Abreu
\
 
 \ /
  Last update: 2019-04-02 10:00    [W:0.066 / U:0.328 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site