lkml.org 
[lkml]   [2021]   [Jan]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH 0/2] D_CAN RX buffer size improvements
Hi everyone,

This series was recently brought to my attention, in connection to a
long-standing
packet drop issue that we had on a Sitara-based product.

I haven't tested this personally, but I've been notified that this was
backported
to the v4.19 kernel, and the packet drop was fixed.

It seems the series never managed to get upstreamed,
but I think this should be resurrected and merged (probably with after
some cleanup/review).

Thanks,
Ezequiel

On Fri, 8 Feb 2019 at 10:31, Andrejs Cainikovs
<Andrejs.Cainikovs@netmodule.com> wrote:
>
> Re-sending entire patchset due to missed cover letter, sorry.
>
> This patchset introduces support for 64 D_CAN message objects with an option of
> unequal split between RX/TX.
>
> The rationale behind this is that there are lots of frame loss on higher bus
> speeds. Below are test results from my custom Sitara AM3352 based board:
>
> Sender: timeout 15m cangen can0 -g 0 -i x
> Target: candump can0,0~0,#FFFFFFFF -td -c -d -e
>
> * Without patches:
> - 15 minute RX test, 500kbps
> - 16 RX / 16 TX message objects
> - 77 received frames lost out of 4649415
>
> * With patches applied:
> - 15 hours RX test, 500kbps
> - 56 RX / 8 TX message objects
> - 41 received frames lost out of 279303376
>
> Please note, I do not have ability to test pure C_CAN, so it is left untested.
>
> ---
>
> Andrejs Cainikovs (2):
> can: c_can: support 64 message objects for D_CAN
> can: c_can: configurable amount of D_CAN RX objects
>
> drivers/net/can/c_can/Kconfig | 20 ++++++++++
> drivers/net/can/c_can/c_can.c | 93 +++++++++++++++++++++++++++----------------
> drivers/net/can/c_can/c_can.h | 20 +++++++---
> 3 files changed, 94 insertions(+), 39 deletions(-)
>
> ---
> 2.11.0
>

\
 
 \ /
  Last update: 2021-01-13 19:45    [W:0.108 / U:0.556 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site