All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Brandes, Shai" <shaibran@amazon.com>
To: "Morten Brørup" <mb@smartsharesystems.com>,
	"ferruh.yigit@amd.com" <ferruh.yigit@amd.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: RE: [PATCH 0/1] net/ena: adding a histogram framework to 2.11.0
Date: Thu, 31 Oct 2024 12:45:39 +0000	[thread overview]
Message-ID: <8ab650f4bddd4bbc8245690bde407d7b@amazon.com> (raw)
In-Reply-To: <98CBD80474FA8B44BF855DF32C47DC35E9F843@smartserver.smartshare.dk>


> -----Original Message-----
> From: Morten Brørup <mb@smartsharesystems.com>
> Sent: Tuesday, October 29, 2024 10:55 AM
> To: Brandes, Shai <shaibran@amazon.com>; ferruh.yigit@amd.com
> Cc: dev@dpdk.org
> Subject: RE: [EXTERNAL] [PATCH 0/1] net/ena: adding a histogram framework
> to 2.11.0
> 
> CAUTION: This email originated from outside of the organization. Do not click
> links or open attachments unless you can confirm the sender and know the
> content is safe.
> 
> 
> 
> > From: Shai Brandes <shaibran@amazon.com>
> >
> > This update is an enhancement to the ENA driver version 2.11.0 release.
> > It is a "nice to have" feature for the 24.11 release, should it be
> > included before the code freeze deadline.
> >
> > This addition leverages an internal framework used in our testing
> > processes, which may offer significant benefits for customers.
> > It calculates the differences between specified start and end points
> > in the code, generating a histogram of time intervals to provide
> > detailed insights into performance.
> 
> Yes, histograms can be extremely useful.
> The SmartShare applications use histograms for a variety of purposes, e.g.
> function profiling (like yours), number of packets per RX burst, number of
> packets per flow, and much more.
> The distribution index can be linear (like yours), log2, or - for higher
> resolution - floating point with 3 or 4 bits mantissa and 5 or more bits
> exponent.
> 
> Please provide generic histograms as a proper library, not tied to a specific
> driver, and not specifically using "time" as parameter.
Acknowledged, thanks for your insights!

      reply	other threads:[~2024-10-31 12:45 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-29  8:02 [PATCH 0/1] net/ena: adding a histogram framework to 2.11.0 shaibran
2024-10-29  8:02 ` [PATCH 1/1] net/ena: add histogram support shaibran
2024-10-29 16:22   ` Stephen Hemminger
2024-10-31 12:46     ` Brandes, Shai
2024-10-29  8:54 ` [PATCH 0/1] net/ena: adding a histogram framework to 2.11.0 Morten Brørup
2024-10-31 12:45   ` Brandes, Shai [this message]

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=8ab650f4bddd4bbc8245690bde407d7b@amazon.com \
    --to=shaibran@amazon.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@amd.com \
    --cc=mb@smartsharesystems.com \
    /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.