All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@web.de>
To: Hongzhan Chen <hongzhan.chen@intel.com>, xenomai@lists.linux.dev
Subject: Re: [PATCH 0/1] net/drivers: igc: introduce rt_igc driver
Date: Fri, 26 Aug 2022 18:26:46 +0200	[thread overview]
Message-ID: <ad5ac2e9-e662-ee94-94fa-c7d42b6e4b68@web.de> (raw)
In-Reply-To: <20220822011821.21283-1-hongzhan.chen@intel.com>

On 22.08.22 03:18, Hongzhan Chen wrote:
> ported basic network functions not including TSN.
>
> 1. passed three network related smokey test: UDP, raw, dgram, But
>    I do not know if these three test is enough to validate the
>    driver, please let me know if there is other tests need to
>    cover.

If those tests exchanged more than a hand-full of packets (to make sure
we are not leaking buffers, thus will run out of resources after a
while) and if you checked that the reported latency is comparable to
rt_igb e.g., then we have a good indication that the driver works. The
rest is field-testing.

> 2. In addtion, another thing I want to discuss here is TSN functions of
>    i225 has the very similiar effect with TDMA not only from clock sync,
>    master and slave arch, timing mangement but the difference for TSN is
>    that most of fucntions is implemented by hardware. We are considering
>    if it is feasible to implement TSN-enabled TDMA-TSN driver to make
>    use of i225 hardware feature because TDMA itself is really heavy.

Yeah, time-triggered send is basically what RTmac/TDMA introduced via
software almost two decades ago. I'm not sure, though, if we should map
the configuration of TSN capabilities of modern hardware on interfaces
(RTmac & Co.) that were designed that long ago. If it happens to work
our easily, it's a nice experiment, but we would likely miss other
things (Qbv transmission windows e.g.). And there is also the question
how to set up an operate PTP aside RTnet.

Jan

  parent reply	other threads:[~2022-08-26 16:33 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-22  1:18 [PATCH 0/1] net/drivers: igc: introduce rt_igc driver Hongzhan Chen
2022-08-22  1:18 ` [PATCH 1/1] " Hongzhan Chen
2022-09-05 12:00   ` Jan Kiszka
2022-09-05 12:03     ` Jan Kiszka
2022-09-05 13:55       ` Jan Kiszka
2022-09-06  2:26         ` Chen, Hongzhan
2022-08-26 16:26 ` Jan Kiszka [this message]
2022-08-27  6:35   ` [PATCH 0/1] " Florian Bezdeka
2022-09-06  3:02     ` Chen, Hongzhan
2022-08-31  8:24   ` Chen, Hongzhan
2022-09-05 11:59     ` Jan Kiszka
2022-09-06  7:56       ` Chen, Hongzhan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ad5ac2e9-e662-ee94-94fa-c7d42b6e4b68@web.de \
    --to=jan.kiszka@web.de \
    --cc=hongzhan.chen@intel.com \
    --cc=xenomai@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.