[PATCH 3/3] staging: vchiq_arm: Make vchiq_shutdown never fail
Stefan Wahren
wahrenst at gmx.net
Sun Jul 13 13:20:48 PDT 2025
Am 13.07.25 um 20:46 schrieb Maíra Canal:
> Hi Stefan,
>
> On 11/07/25 12:40, Stefan Wahren wrote:
>> Most of the users of vchiq_shutdown ignore the return value,
>> which is bad because this could lead to resource leaks.
>> So instead of changing all calls to vchiq_shutdown, it's easier
>> to make vchiq_shutdown never fail.
>>
>> Fixes: 71bad7f08641 ("staging: add bcm2708 vchiq driver")
>> Signed-off-by: Stefan Wahren <wahrenst at gmx.net>
>> ---
>> drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c | 3 +--
>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git
>> a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
>> b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
>> index 6434cbdc1a6e..721b15b7e13b 100644
>> --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
>> +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c
>> @@ -393,8 +393,7 @@ int vchiq_shutdown(struct vchiq_instance *instance)
>
> If vchiq_shutdown() will always return 0 and most users ignore the
> return value, how about turning it into a void function?
I will send a separate patch for this later, because this fix is easier
to backport.
Regards
>
> Best Regards,
> - Maíra
>
>> struct vchiq_state *state = instance->state;
>> int ret = 0;
>> - if (mutex_lock_killable(&state->mutex))
>> - return -EAGAIN;
>> + mutex_lock(&state->mutex);
>> /* Remove all services */
>> vchiq_shutdown_internal(state, instance);
>
More information about the linux-arm-kernel
mailing list