diff mbox series

[v1,1/1] baseband/acc: fix queue setup failure clean up

Message ID 20250305182716.773937-2-nicolas.chautru@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Maxime Coquelin
Headers
Series baseband/acc: fix queue setup failure clean up |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/loongarch-compilation success Compilation OK
ci/loongarch-unit-testing success Unit Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Functional success Functional PASS
ci/github-robot: build success github build: passed
ci/iol-unit-amd64-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-unit-arm64-testing success Testing PASS
ci/iol-compile-amd64-testing success Testing PASS
ci/iol-compile-arm64-testing success Testing PASS
ci/iol-marvell-Functional success Functional Testing PASS RETEST #1
ci/iol-intel-Performance success Performance Testing PASS RETEST #1
ci/iol-mellanox-Performance success Performance Testing PASS RETEST #1
ci/iol-sample-apps-testing success Testing PASS RETEST #1
ci/iol-broadcom-Performance success Performance Testing PASS RETEST #1
ci/intel-Testing success Testing PASS
ci/iol-intel-Functional success Functional Testing PASS RETEST #1

Commit Message

Nicolas Chautru March 5, 2025, 6:27 p.m. UTC 
Incomplete clean up for vrb2 variant when queue setup fails
to complete successfully.

Fixes: fc65d3dcabe01 ("baseband/acc: refactor queue allocation")
Cc: stable@dpdk.org

Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
---
 drivers/baseband/acc/rte_vrb_pmd.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Patrick Robb March 6, 2025, 2:59 a.m. UTC | #1 
Recheck-request: iol-intel-Performance

There was an infra failure with the Arm Grace server about 12 hours ago
which caused this failure - sending a retest request.
Maxime Coquelin March 12, 2025, 1:48 p.m. UTC | #2 
On 3/5/25 7:27 PM, Nicolas Chautru wrote:
> Incomplete clean up for vrb2 variant when queue setup fails
> to complete successfully.
> 
> Fixes: fc65d3dcabe01 ("baseband/acc: refactor queue allocation")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
> ---
>   drivers/baseband/acc/rte_vrb_pmd.c | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/baseband/acc/rte_vrb_pmd.c b/drivers/baseband/acc/rte_vrb_pmd.c
> index 27620ccc10..2c10ef2c94 100644
> --- a/drivers/baseband/acc/rte_vrb_pmd.c
> +++ b/drivers/baseband/acc/rte_vrb_pmd.c
> @@ -1156,6 +1156,8 @@ vrb_queue_setup(struct rte_bbdev *dev, uint16_t queue_id,
>   free_q:
>   	rte_free(q);
>   	q = NULL;
> +	if (d->device_variant == VRB2_VARIANT)
> +		d->queue_index[conf->op_type]--;
>   
>   	return ret;
>   }

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>

Thanks,
Maxime
Maxime Coquelin March 12, 2025, 3:27 p.m. UTC | #3 
On 3/12/25 2:48 PM, Maxime Coquelin wrote:
> 
> 
> On 3/5/25 7:27 PM, Nicolas Chautru wrote:
>> Incomplete clean up for vrb2 variant when queue setup fails
>> to complete successfully.
>>
>> Fixes: fc65d3dcabe01 ("baseband/acc: refactor queue allocation")
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Nicolas Chautru <nicolas.chautru@intel.com>
>> ---
>>   drivers/baseband/acc/rte_vrb_pmd.c | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/baseband/acc/rte_vrb_pmd.c b/drivers/baseband/ 
>> acc/rte_vrb_pmd.c
>> index 27620ccc10..2c10ef2c94 100644
>> --- a/drivers/baseband/acc/rte_vrb_pmd.c
>> +++ b/drivers/baseband/acc/rte_vrb_pmd.c
>> @@ -1156,6 +1156,8 @@ vrb_queue_setup(struct rte_bbdev *dev, uint16_t 
>> queue_id,
>>   free_q:
>>       rte_free(q);
>>       q = NULL;
>> +    if (d->device_variant == VRB2_VARIANT)
>> +        d->queue_index[conf->op_type]--;
>>       return ret;
>>   }
> 
> Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
> 
> Thanks,
> Maxime
> 

Applied to next-baseband.

Thanks,
Maxime
diff mbox series

Patch

diff --git a/drivers/baseband/acc/rte_vrb_pmd.c b/drivers/baseband/acc/rte_vrb_pmd.c
index 27620ccc10..2c10ef2c94 100644
--- a/drivers/baseband/acc/rte_vrb_pmd.c
+++ b/drivers/baseband/acc/rte_vrb_pmd.c
@@ -1156,6 +1156,8 @@  vrb_queue_setup(struct rte_bbdev *dev, uint16_t queue_id,
 free_q:
 	rte_free(q);
 	q = NULL;
+	if (d->device_variant == VRB2_VARIANT)
+		d->queue_index[conf->op_type]--;
 
 	return ret;
 }