All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vinicius Costa Gomes <vinicius.gomes@intel.com>
To: Ferenc Fejes <ferenc.fejes@ericsson.com>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Cc: "jesse.brandeburg@intel.com" <jesse.brandeburg@intel.com>,
	"anthony.l.nguyen@intel.com" <anthony.l.nguyen@intel.com>
Subject: Re: igc: missing HW timestamps at TX
Date: Mon, 18 Jul 2022 11:46:34 -0300	[thread overview]
Message-ID: <87tu7emqb9.fsf@intel.com> (raw)
In-Reply-To: <VI1PR07MB4080DC45051E112EEC6D7734E18D9@VI1PR07MB4080.eurprd07.prod.outlook.com>

Hi Ferenc,

Ferenc Fejes <ferenc.fejes@ericsson.com> writes:

> (Ctrl+Enter'd by mistake)
>
> My question here: is there anything I can quickly try to avoid that
> behavior? Even when I send only a few (like 10) packets but on fast
> rate (5us between packets) I get missing TX HW timestamps. The receive
> side looks much more roboust, I cannot noticed missing HW timestamps
> there.

There's a limitation in the i225/i226 in the number of "in flight" TX
timestamps they are able to handle. The hardware has 4 sets of registers
to handle timestamps.

There's an aditional issue that the driver as it is right now, only uses
one set of those registers.

I have one only briefly tested series that enables the driver to use the
full set of TX timestamp registers. Another reason that it was not
proposed yet is that I still have to benchmark it and see what is the
performance impact.

If you are feeling adventurous and feel like helping test it, here is
the link:

https://github.com/vcgomes/net-next/tree/igc-multiple-tstamp-timers-lock-new


Cheers,
-- 
Vinicius

  reply	other threads:[~2022-07-18 14:46 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-17 14:42 igc: missing HW timestamps at TX Ferenc Fejes
2022-07-17 14:46 ` Ferenc Fejes
2022-07-18 14:46   ` Vinicius Costa Gomes [this message]
2022-07-19  7:40     ` Ferenc Fejes
2022-08-11  8:54       ` Ferenc Fejes
2022-08-11 13:33         ` Vinicius Costa Gomes
2022-08-12 14:13           ` Ferenc Fejes
2022-08-12 20:16             ` Vladimir Oltean
2022-08-15  6:47               ` Ferenc Fejes
2022-08-15 22:04                 ` Vinicius Costa Gomes
2022-08-16  9:33                 ` Vladimir Oltean
2022-08-17  7:44                   ` Ferenc Fejes
2022-08-15 21:39               ` Vinicius Costa Gomes
2022-08-15 22:26                 ` Vladimir Oltean
2022-08-15 23:07                   ` Vinicius Costa Gomes
2022-08-16  8:51                     ` Kurt Kanzenbach
2022-08-16 20:45                       ` Vinicius Costa Gomes
2022-08-17  6:10                         ` Kurt Kanzenbach
2022-08-17 19:24                           ` Vinicius Costa Gomes
2022-08-16  9:10                     ` Vladimir Oltean
2022-08-16 18:11                       ` Vinicius Costa Gomes

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=87tu7emqb9.fsf@intel.com \
    --to=vinicius.gomes@intel.com \
    --cc=anthony.l.nguyen@intel.com \
    --cc=ferenc.fejes@ericsson.com \
    --cc=jesse.brandeburg@intel.com \
    --cc=netdev@vger.kernel.org \
    /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.