Re: [PATCH v1 00/24] net/ena: new features and fixes

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 5/9/2018 1:44 PM, Michal Krawczyk wrote:
> The ENA driver was updated with the new features and few fixes and minor
> changes are introduced.
> First of all, the communication layer which is delivered by vendor was
> updated - the version in the HEAD is a bit outdated now. ENA is able to
> communicate with the driver through Admin queue by using admin interrupts
> instead of polling.
> Admin interrupts are also used for handling AENQ events, which are used for
> the following new features:
>   - LSC handlers
>   - watchdog and device rest
>   - monitoring the admin queue
>   - handling ENA notifications (getting hints from device)
> For the watchdog and admin queue monitoring, the timers had to be used, so
> the makefile was modified to do not cut out the librte_timer.
> 
> From other fixes and changes:
>   - legacy LLQ was removed which is now deprecated API
>   - Rx out of order completion was added to enable cleaning up packets out
>     of order
>   - Tx mbufs are now linearized if they exceed supported number of segments
>   - pass information about maximum number of Tx and Rx descriptors
>   - the IO queue number is now taking into consideration maximum number of
>     sq and cq
>   - Tx id requested for sending is now being validated and the reset is
>     being triggered if it is invalid
>   - branch predictioning was added for better performance
>   - error checking and returned values were fixed
>   - macros for allocating memory in communication layer were fixed
>   - information about numa mode is now being passed to the NIC
> 
> Michal Krawczyk (10):
>   net/ena: update ena_com to the newer version
>   net/ena: add interrupt handler for admin queue
>   net/ena: add stop and uninit routines
>   net/ena: add LSC intr support and AENQ handling
>   net/ena: restart only initialized queues instead of all
>   net/ena: add reset routine
>   mk: link librte_timer with --whole-archive
>   net/ena: add watchdog and keep alive AENQ handler
>   net/ena: add checking for admin queue state
>   net/ena: add RX out of order completion
> 
> Rafal Kozik (14):
>   net/ena: remove support of legacy LLQ
>   net/ena: handle ENA notification
>   net/ena: make watchdog configurable
>   net/ena: linearize Tx mbuf
>   net/ena: add info about max number of Tx/Rx descriptors
>   net/ena: unimplemented handler error
>   net/ena: rework configuration of IO queue numbers
>   net/ena: validate Tx req id
>   net/ena: add (un)likely statements
>   net/ena: adjust error checking and cleaning
>   net/ena: update numa node
>   net/ena: check pointer before memset
>   net/ena: change memory type
>   net/ena: fix GENMASK_ULL macro

Hi Michał,

This arrived too late for this release cycle, let us close this release first
and consider set for next release.

Thanks,
ferruh