net/i40e: fix packet type parser error

Message ID 1530754589-62675-1-git-send-email-beilei.xing@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Qi Zhang
Headers
Series net/i40e: fix packet type parser error |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Xing, Beilei July 5, 2018, 1:36 a.m. UTC
  To parse packet type correctly, profile needs to be
loaded again to update tables in SW even if profile
has exited. But previously packet type parser will
be false after loading profile when profile already
exists.
This patch fixes the issue.

Fixes: 0585f5c3d264 ("net/i40e: fix DDP profile DEL operation")
Cc: stable@dpdk.org

Signed-off-by: Beilei Xing <beilei.xing@intel.com>
---
 drivers/net/i40e/rte_pmd_i40e.c | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Lin, Xueqin July 5, 2018, 6:35 a.m. UTC | #1
> -----Original Message-----
 > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Beilei Xing
 > Sent: Thursday, July 5, 2018 9:36 AM
 > To: Zhang, Qi Z <qi.z.zhang@intel.com>
 > Cc: dev@dpdk.org; stable@dpdk.org
 > Subject: [dpdk-dev] [PATCH] net/i40e: fix packet type parser error
 >
 > To parse packet type correctly, profile needs to be loaded again to
 > update tables in SW even if profile has exited. But previously packet
 > type parser will be false after loading profile when profile already exists.
 > This patch fixes the issue.
 >
 > Fixes: 0585f5c3d264 ("net/i40e: fix DDP profile DEL operation")
 > Cc: stable@dpdk.org
 >
 > Signed-off-by: Beilei Xing <beilei.xing@intel.com>
Tested-by: Xueqin Lin <xueqin.lin@intel.com>
  

Patch

diff --git a/drivers/net/i40e/rte_pmd_i40e.c b/drivers/net/i40e/rte_pmd_i40e.c
index 921dfc0..bba62b1 100644
--- a/drivers/net/i40e/rte_pmd_i40e.c
+++ b/drivers/net/i40e/rte_pmd_i40e.c
@@ -1709,6 +1709,7 @@  rte_pmd_i40e_process_ddp_package(uint16_t port, uint8_t *buff,
 				PMD_DRV_LOG(ERR, "Profile of group 0 already exists.");
 			else if (is_exist == 3)
 				PMD_DRV_LOG(ERR, "Profile of different group already exists");
+			i40e_update_customized_info(dev, buff, size, op);
 			rte_free(profile_info_sec);
 			return -EEXIST;
 		}