Message ID | 1401458186-29985-1-git-send-email-ben.dooks@codethink.co.uk |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
Hello. On 05/30/2014 05:56 PM, Ben Dooks wrote: The subject should have "R8A779x". > The current behaviour of the sh_eth driver is not to use the RNC bit > for the receive ring. This means that every packet recieved is not only > generating an IRQ but it also stops the receive ring DMA as well until > the driver re-enables it after unloading the packet. > > This means that a number of the following errors are generated due to > the receive packet FIFO overflowing due to nowhere to put packets: > > net eth0: Receive FIFO Overflow > > Setting the RMCR_RNC configuration has so far been tested with an NFS > root filesystem and the driver has not failed yet. It is not yet known > why this is not set for R8A779x operation (Feedback on this issue or > other testing is welcome) I have provided such feedback (and many people provided testing), so that the last sentence doesn't make sense anymore. > There are further test reports from Sergei Shtylov and others for > both the R8A7790 and R8A7791. > Tested-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> > Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> Other than the remarks above: Acked-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 30/05/14 15:00, Sergei Shtylyov wrote: > Hello. > > On 05/30/2014 05:56 PM, Ben Dooks wrote: > > The subject should have "R8A779x". > >> The current behaviour of the sh_eth driver is not to use the RNC bit >> for the receive ring. This means that every packet recieved is not only >> generating an IRQ but it also stops the receive ring DMA as well until >> the driver re-enables it after unloading the packet. >> >> This means that a number of the following errors are generated due to >> the receive packet FIFO overflowing due to nowhere to put packets: >> >> net eth0: Receive FIFO Overflow >> >> Setting the RMCR_RNC configuration has so far been tested with an NFS >> root filesystem and the driver has not failed yet. It is not yet known >> why this is not set for R8A779x operation (Feedback on this issue or >> other testing is welcome) > > I have provided such feedback (and many people provided testing), so > that the last sentence doesn't make sense anymore. I suppose it could be removed, it depends on how pedantic people want to be with the changelogs.
From: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Date: Fri, 30 May 2014 18:00:10 +0400 > Hello. > > On 05/30/2014 05:56 PM, Ben Dooks wrote: > > The subject should have "R8A779x". > >> The current behaviour of the sh_eth driver is not to use the RNC bit >> for the receive ring. This means that every packet recieved is not >> only >> generating an IRQ but it also stops the receive ring DMA as well until >> the driver re-enables it after unloading the packet. >> >> This means that a number of the following errors are generated due to >> the receive packet FIFO overflowing due to nowhere to put packets: >> >> net eth0: Receive FIFO Overflow >> >> Setting the RMCR_RNC configuration has so far been tested with an NFS >> root filesystem and the driver has not failed yet. It is not yet known >> why this is not set for R8A779x operation (Feedback on this issue or >> other testing is welcome) > > I have provided such feedback (and many people provided testing), so > that the last sentence doesn't make sense anymore. > >> There are further test reports from Sergei Shtylov and others for >> both the R8A7790 and R8A7791. > >> Tested-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> >> Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk> > > Other than the remarks above: > > Acked-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Ben, please fix the commit message and resubmit, -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c index 6a9509c..1e3e5d2 100644 --- a/drivers/net/ethernet/renesas/sh_eth.c +++ b/drivers/net/ethernet/renesas/sh_eth.c @@ -468,6 +468,7 @@ static struct sh_eth_cpu_data r8a779x_data = { .ecsr_value = ECSR_PSRTO | ECSR_LCHNG | ECSR_ICD, .ecsipr_value = ECSIPR_PSRTOIP | ECSIPR_LCHNGIP | ECSIPR_ICDIP, .eesipr_value = 0x01ff009f, + .rmcr_value = RMCR_RNC, .tx_check = EESR_FTC | EESR_CND | EESR_DLC | EESR_CD | EESR_RTO, .eesr_err_check = EESR_TWB | EESR_TABT | EESR_RABT | EESR_RFE |