Re: [PATCH v2 1/2] soc: fsl: dpio: Use scope-based resource management in dpaa2_io_store_create()

[Markus Elfring]

…
>> +++ b/drivers/soc/fsl/dpio/dpio-service.c
…
>> -	size = max_frames * sizeof(struct dpaa2_dq) + 64;
>> -	ret->alloced_addr = kzalloc(size, GFP_KERNEL);
>> -	if (!ret->alloced_addr) {
>> -		kfree(ret);
>> +	ret->alloced_addr = kzalloc(max_frames * sizeof(struct dpaa2_dq) + 64, GFP_KERNEL);
>> +	if (!ret->alloced_addr)
>>  		return NULL;
> 
> Why convert the ret allocation but not the ret->alloced_addr allocation?
I suggest to reconsider the suggested influence of the attribute “__free(kfree)”
once more also according to data structure members.

Regards,
Markus