From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3 0/3]: bug fixes in the ixgbe PF PMD Rx flow Date: Tue, 17 Mar 2015 00:16:17 +0100 Message-ID: <16211802.UMTJk1lyst@xps13> References: <1426195052-4640-1-git-send-email-vladz@cloudius-systems.com> <2601191342CEEE43887BDE71AB977258213F5E9B@irsmsx105.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: Vlad Zolotarov Return-path: In-Reply-To: <2601191342CEEE43887BDE71AB977258213F5E9B-pww93C2UFcwu0RiL9chJVbfspsVTdybXVpNB7YpNyf8@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2015-03-13 11:07, Ananyev, Konstantin: > From: dev [mailto:dev-bounces-VfR2kkLFssw@public.gmane.org] On Behalf Of Vlad Zolotarov > > This series contains some bug fixes that were found during my work on the ixgbe LRO > > patches. Sending this series separately on Thomas request so that it may be integrated > > into the 2.0 release. > > > > New in v3: > > - Adjusted to the new structs naming in the master. > > - Fixed rx_bulk_alloc_allowed and rx_vec_allowed initialization: > > - Don't set them to FALSE in rte_eth_dev_stop() flow - the following > > rte_eth_dev_start() will need them. > > - Reset them to TRUE in rte_eth_dev_configure() and not in a probe() flow. > > This will ensure the proper behaviour if port is re-configured. > > - Rename: > > - ixgbe_rx_vec_condition_check() -> ixgbe_rx_vec_dev_conf_condition_check() > > - set_rx_function() -> ixgbe_set_rx_function() > > - Clean up the logic in ixgbe_set_rx_function(). > > - Define stubs with __attribute__((weak)) instead of using #ifdef's. > > - Styling: beautify ixgbe_rxtx.h a bit. > > > > New in v2: > > - Fixed a compilation failure. > > > > > > Vlad Zolotarov (3): > > ixgbe: Use the rte_le_to_cpu_xx()/rte_cpu_to_le_xx() when > > reading/setting HW ring descriptor fields > > ixgbe: Bug fix: Properly configure Rx CRC stripping for x540 devices > > ixgbe: Unify the rx_pkt_bulk callback initialization > > Acked-by: Konstantin Ananyev Applied, thanks