[PATCH v4 1/1] bus/cdx: provide driver flag for optional resource mapping

2023-10-16 Thread Abhijit Gangurde
Provide driver flag which gives an option to map the cdx device resource before probing the device driver. External driver can use rte_cdx_map_device() and rte_cdx_unmap_device() APIs to map/ unmap device resource separately. Signed-off-by: Abhijit Gangurde --- v4: - rte_cdx_map_device() and

[PATCH] bus/cdx: Remove ineffective code statement

2023-06-14 Thread Abhijit Gangurde
ret = 0 statement is ineffective since it is overwritten in a loop. Coverity issue: 385379 Fixes: 45ef232af515 ("bus/cdx: introduce AMD CDX bus") Signed-off-by: Abhijit Gangurde --- drivers/bus/cdx/cdx.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/bus/cdx/cdx.c b/drive

[PATCH] bus/cdx: Move debug print before unmapping resource

2023-06-14 Thread Abhijit Gangurde
Debug print is moved before unmapping requested address to fix use after free coverity issue. Coverity issue: 385381 Fixes: 45ef232af515 ("bus/cdx: introduce AMD CDX bus") Signed-off-by: Abhijit Gangurde --- drivers/bus/cdx/cdx.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletio

[PATCH] bus/cdx: Fix resource leak

2023-06-14 Thread Abhijit Gangurde
Freeing the allocated resources on error scenarios. Coverity issue: 385377 Fixes: f29fb5caa135 ("bus/cdx: support MSI") Signed-off-by: Abhijit Gangurde --- drivers/bus/cdx/cdx.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/bus/cdx/cdx.c b/d

[PATCH v2 0/3] bus/cdx: fix coverity issue

2023-06-16 Thread Abhijit Gangurde
This series fixes coverity issues 385379, 385381, 385377 v2: - Merged coverity fix patches into series - Updated commit message Abhijit Gangurde (3): bus/cdx: remove ineffective code statement bus/cdx: move debug print before unmapping resource bus/cdx: fix resource leak drivers/bus/cdx

[PATCH v2 1/3] bus/cdx: remove ineffective code statement

2023-06-16 Thread Abhijit Gangurde
Remove ineffective code statement ret = 0 since it is overwritten in a loop. Coverity issue: 385379 Fixes: 45ef232af515 ("bus/cdx: introduce AMD CDX bus") Signed-off-by: Abhijit Gangurde Acked-by: Nipun Gupta --- v2: - Updated commit message drivers/bus/cdx/cdx.c | 1 - 1 file

[PATCH v2 2/3] bus/cdx: move debug print before unmapping resource

2023-06-16 Thread Abhijit Gangurde
Move debug print before unmapping requested address to fix use after free coverity issue. Coverity issue: 385381 Fixes: 45ef232af515 ("bus/cdx: introduce AMD CDX bus") Signed-off-by: Abhijit Gangurde --- v2: - Updated commit message drivers/bus/cdx/cdx.c | 3 ++- 1 file changed, 2

[PATCH v2 3/3] bus/cdx: fix resource leak

2023-06-16 Thread Abhijit Gangurde
Free the allocated device memory and interrupt handler on error scenarios. Coverity issue: 385377 Fixes: f29fb5caa135 ("bus/cdx: support MSI") Signed-off-by: Abhijit Gangurde --- v2: - Updated commit message drivers/bus/cdx/cdx.c | 10 ++ 1 file changed, 6 insertions(+), 4

[PATCH] bus/cdx: provide driver flag for optional resource mapping

2023-06-26 Thread Abhijit Gangurde
Provide driver flag which gives an option to map the cdx device resource before probing the device driver. Also, make rte_cdx_map_device() API as public to map device resource separately. Signed-off-by: Abhijit Gangurde --- drivers/bus/cdx/bus_cdx_driver.h | 26 ++--- drivers/bus

[PATCH v2] bus/cdx: provide driver flag for optional resource mapping

2023-07-04 Thread Abhijit Gangurde
Provide driver flag which gives an option to map the cdx device resource before probing the device driver. Also, make rte_cdx_map_device() API as public to map device resource separately. Signed-off-by: Abhijit Gangurde --- v2: - Corrected _RTE_BUS_CDX_H_ to RTE_BUS_CDX_H - Improved code

[PATCH v3] bus/cdx: provide driver flag for optional resource mapping

2023-07-10 Thread Abhijit Gangurde
Provide driver flag which gives an option to map the cdx device resource before probing the device driver. Also, make rte_cdx_map_device() API as public to map device resource separately. Signed-off-by: Abhijit Gangurde --- v3: - Changed APIs to __rte_experimental. v2: - Corrected