> -----Original Message-----
> From: Brandes, Shai <shaib...@amazon.com>
> Sent: Friday, March 8, 2024 10:27 PM
> To: Ferruh Yigit <ferruh.yi...@amd.com>
> Cc: dev@dpdk.org
> Subject: RE: [PATCH v3 00/33] net/ena: v2.9.0 driver release
>
> Sure, will upload a new seried, thanks!
>
> בתאריך 8 במרץ 2024 19:36, Ferruh Yigit <ferruh.yi...@amd.com> כתב:
>
> 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.
>
>
>
> On 3/6/2024 12:24 PM, shaib...@amazon.com wrote:
> > From: Shai Brandes <shaib...@amazon.com>
> >
> > Hi all, the ena v2.9.0 release introduces:
> > 1. HAL upgrade:
> > - renamed the 'base' folder to be 'hal'
> > - separated the HAL patches instead of a bulk update.
> > 2. Restructured ena stats and metrics.
> > 3. Restructured the LLQ configuration:
> > - configurable via devarg.
> > - support device recommendation.
> > - restructure the logic in driver.
> > 4. Added support for the admin queue to work only in poll-mode
> > - configurable via devarg.
> > - allows to bind ports to uio_pci_generic kernel driver.
> > 5. Reworked the device close to exhaust interrupt callbacks and alarms.
> > 6. Fixed a bug in fast mbuf free.
> > Best regards.
> >
> > ---
> > v3:
> > * Fixed missing admin queue missing intialization in patch 0032
> >
> > v2:
> > * Fixed minor spelling issues from checkpatch
> >
> >
> > Shai Brandes (33):
> > net/ena: rework the metrics multi-process functions
> > net/ena: report new supported link speed capabilities
> > net/ena: update imissed stat with Rx overruns
> > net/ena: sub-optimal configuration notifications support
> > net/ena: fix fast mbuf free
> > net/ena: rename base folder to hal
> > net/ena: restructure the llq policy setting process
> > net/ena/hal: exponential backoff exp limit
> > net/ena/hal: add a new csum offload bit
> > net/ena/hal: added a bus parameter to ena memcpy macro
> > net/ena/hal: optimize Rx ring submission queue
> > net/ena/hal: rename fields in completion descriptors
> > net/ena/hal: use correct read once on u8 field
> > net/ena/hal: add completion descriptor corruption check
> > net/ena/hal: malformed Tx descriptor error reason
> > net/ena/hal: phc feature modifications
> > net/ena/hal: restructure interrupt handling
> > net/ena/hal: add unlikely to error checks
> > net/ena/hal: missing admin interrupt reset reason
> > net/ena/hal: check for existing keep alive notification
> > net/ena/hal: modify memory barrier comment
> > net/ena/hal: rework Rx ring submission queue
> > net/ena/hal: remove operating system type enum
> > net/ena/hal: handle command abort
> > net/ena/hal: add support for device reset request
> > net/ena: cosmetic changes
> > net/ena/hal: modify customer metrics memory management
> > net/ena/hal: cosmetic changes
> > net/ena: update device-preferred size of rings
> > net/ena: exhaust interrupt callbacks in device close
> > net/ena: support max large llq depth from the device
> > net/ena: control path pure polling mode
> > net/ena: upgrade driver version to 2.9.0
> >
>
> Hi Shai,
>
> I did review only first 10 patches, there are some common patterns to
> address in the perspective and commit logs.
>
> Can you please update whole series according to comments? I will review
> remaining patches in the new version.
>
> Thanks,
> Ferruh
>
[Brandes, Shai] sure, thanks for the comments, we will align accordingly.