Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-31 Thread Andrew Lunn
On Fri, May 26, 2017 at 12:13:03PM +0200, Rafa Corvillo wrote: > As modifying sky2 code I have not could get any solution, then I > have modified some parameters on the ethernet interface and using a > MTU = 1503 and disabling TSO (TCP Segmentation Offload) mechanism > all communication errors

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-26 Thread Rafa Corvillo
As modifying sky2 code I have not could get any solution, then I have modified some parameters on the ethernet interface and using a MTU = 1503 and disabling TSO (TCP Segmentation Offload) mechanism all communication errors disappear, both the rx errors due to too long packets and tx timeout

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-16 Thread Rafa Corvillo
Also, check that in sky2_rx_add(), le->length is set to 4K. The value of le->length is set to 1520. Rafa Ah. You probably need to change sky2_get_rx_data_size() as well, to add in the extra 8 bytes. Andrew If I add the extra 8 byte in the function sky2_get_rx_data_size(), the

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-16 Thread Andrew Lunn
> >Also, check that in sky2_rx_add(), le->length is set to 4K. > > > > The value of le->length is set to 1520. > Rafa Ah. You probably need to change sky2_get_rx_data_size() as well, to add in the extra 8 bytes. Andrew

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-16 Thread Rafa Corvillo
On Tue, May 16, 2017 at 03:09:17PM +0200, Rafa Corvillo wrote: Adding 8 bytes (sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN + 8 (EDSA_HLEN)) does not fix the error, because the interface keep having a maximum length of 1518 bytes (sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN). Did you check the value

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-16 Thread Andrew Lunn
On Tue, May 16, 2017 at 03:09:17PM +0200, Rafa Corvillo wrote: > >>Adding 8 bytes (sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN + 8 > >>(EDSA_HLEN)) does not fix the error, because the interface keep > >>having a maximum length of 1518 bytes (sky2->netdev->mtu + ETH_HLEN > >>+ VLAN_HLEN). > > > >Did

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-16 Thread Rafa Corvillo
Adding 8 bytes (sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN + 8 (EDSA_HLEN)) does not fix the error, because the interface keep having a maximum length of 1518 bytes (sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN). Did you check the value being written to here: /* * The receiver

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-16 Thread Andrew Lunn
> Adding 8 bytes (sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN + 8 > (EDSA_HLEN)) does not fix the error, because the interface keep > having a maximum length of 1518 bytes (sky2->netdev->mtu + ETH_HLEN > + VLAN_HLEN). Did you check the value being written to here: /* * The receiver

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-16 Thread Rafa Corvillo
On 08/05/17 14:38, Andrew Lunn wrote: static unsigned sky2_get_rx_threshold(struct sky2_port *sky2) { unsigned size; /* Space needed for frame data + headers rounded up */ size = roundup(sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN, 8); /* Stopping point for

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-08 Thread Andrew Lunn
> >static unsigned sky2_get_rx_threshold(struct sky2_port *sky2) > >{ > > unsigned size; > > > > /* Space needed for frame data + headers rounded up */ > > size = roundup(sky2->netdev->mtu + ETH_HLEN + VLAN_HLEN, 8); > > > > /* Stopping point for hardware truncation

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-05-08 Thread Rafa Corvillo
On 28/04/17 14:22, Andrew Lunn wrote: Since you are using DSA, you will have DSA tags enabled on frames to/from the switch. This adds an extra 8 byte header in the frame. My guess is, it is this header, not the VLAN tag which is causing you MTU issues. But it is strange because, as I have

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-04-28 Thread Andrew Lunn
> >Since you are using DSA, you will have DSA tags enabled on frames > >to/from the switch. This adds an extra 8 byte header in the frame. My > >guess is, it is this header, not the VLAN tag which is causing you MTU > >issues. > > But it is strange because, as I have said above, we have the same

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-04-28 Thread Rafa Corvillo
On 27/04/17 15:04, Andrew Lunn wrote: On Thu, Apr 27, 2017 at 02:05:51PM +0200, Rafa Corvillo wrote: On 25/04/17 17:27, Stephen Hemminger wrote: On Fri, 21 Apr 2017 14:39:00 +0200 Rafa Corvillo wrote: We are working in an ARMv7 embedded system running kernel 4.9

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-04-27 Thread Andrew Lunn
On Thu, Apr 27, 2017 at 02:05:51PM +0200, Rafa Corvillo wrote: > On 25/04/17 17:27, Stephen Hemminger wrote: > >On Fri, 21 Apr 2017 14:39:00 +0200 > >Rafa Corvillo wrote: > > > >>We are working in an ARMv7 embedded system running kernel 4.9 (LEDE build). > >>It is an

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-04-27 Thread Rafa Corvillo
On 25/04/17 17:27, Stephen Hemminger wrote: On Fri, 21 Apr 2017 14:39:00 +0200 Rafa Corvillo wrote: We are working in an ARMv7 embedded system running kernel 4.9 (LEDE build). It is an imx6 board with 2 ethernet interfaces. One of them is connected to a Marvell

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-04-25 Thread Stephen Hemminger
On Fri, 21 Apr 2017 14:39:00 +0200 Rafa Corvillo wrote: > We are working in an ARMv7 embedded system running kernel 4.9 (LEDE build). > It is an imx6 board with 2 ethernet interfaces. One of them is connected to > a Marvell switch. > > The schema of the system is the

Re: [ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-04-24 Thread Rafa Corvillo
I resend the mail with the schema fixed. Sorry for the inconvenience. We are working in an ARMv7 embedded system running kernel 4.9 (LEDE build). It is an imx6 board with 2 ethernet interfaces. One of them is connected to a Marvell switch. The schema of the system is the following:

[ISSUE: sky2 - rx error] Link stops working under heavy traffic load connected to a mv88e6176

2017-04-21 Thread Rafa Corvillo
We are working in an ARMv7 embedded system running kernel 4.9 (LEDE build). It is an imx6 board with 2 ethernet interfaces. One of them is connected to a Marvell switch. The schema of the system is the following: +---+ eth0 | +--+ | | | |