[v1,0/5] dmaengine: idxd: fix memory leak in error handling path

Message

Shuai Xue Jan. 10, 2025, 8:22 a.m. UTC

Shuai Xue (5):
  dmaengine: idxd: fix memory leak in error handling path of
    idxd_setup_wqs
  dmaengine: idxd: fix memory leak in error handling path of
    idxd_setup_engines
  dmaengine: idxd: fix memory leak in error handling path of
    idxd_setup_groups
  dmaengine: idxd: fix memory leak in error handling path of idxd_alloc
  dmaengine: idxd: fix memory leak in error handling path of
    idxd_pci_probe

 drivers/dma/idxd/init.c | 62 ++++++++++++++++++++++++++++++++---------
 1 file changed, 49 insertions(+), 13 deletions(-)

Comments

Shuai Xue Feb. 14, 2025, 6:43 a.m. UTC | #1

在 2025/1/10 16:22, Shuai Xue 写道:
> Shuai Xue (5):
>    dmaengine: idxd: fix memory leak in error handling path of
>      idxd_setup_wqs
>    dmaengine: idxd: fix memory leak in error handling path of
>      idxd_setup_engines
>    dmaengine: idxd: fix memory leak in error handling path of
>      idxd_setup_groups
>    dmaengine: idxd: fix memory leak in error handling path of idxd_alloc
>    dmaengine: idxd: fix memory leak in error handling path of
>      idxd_pci_probe
> 
>   drivers/dma/idxd/init.c | 62 ++++++++++++++++++++++++++++++++---------
>   1 file changed, 49 insertions(+), 13 deletions(-)
> 

Hi, all,

Gentle Ping.

Thanks.
Shuai

Dave Jiang Feb. 14, 2025, 4:17 p.m. UTC | #2

On 2/13/25 11:43 PM, Shuai Xue wrote:
> 
> 
> 在 2025/1/10 16:22, Shuai Xue 写道:
>> Shuai Xue (5):
>>    dmaengine: idxd: fix memory leak in error handling path of
>>      idxd_setup_wqs
>>    dmaengine: idxd: fix memory leak in error handling path of
>>      idxd_setup_engines
>>    dmaengine: idxd: fix memory leak in error handling path of
>>      idxd_setup_groups
>>    dmaengine: idxd: fix memory leak in error handling path of idxd_alloc
>>    dmaengine: idxd: fix memory leak in error handling path of
>>      idxd_pci_probe
>>
>>   drivers/dma/idxd/init.c | 62 ++++++++++++++++++++++++++++++++---------
>>   1 file changed, 49 insertions(+), 13 deletions(-)
>>
> 
> Hi, all,
> 
> Gentle Ping.

- Fenghua, who has left Intel
+ Vinicius, who is the current maintainer


> 
> Thanks.
> Shuai

Markus Elfring Feb. 14, 2025, 5:15 p.m. UTC | #3

…
>  drivers/dma/idxd/init.c | 62 ++++++++++++++++++++++++++++++++---------
…

How do you think about to add any tags (like “Fixes” and “Cc”)
for the presented patches accordingly?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.14-rc2#n145

Regards,
Markus

Dave Jiang Feb. 14, 2025, 5:22 p.m. UTC | #4

On 2/14/25 10:15 AM, Markus Elfring wrote:
> …
>>  drivers/dma/idxd/init.c | 62 ++++++++++++++++++++++++++++++++---------
> …
> 
> How do you think about to add any tags (like “Fixes” and “Cc”)
> for the presented patches accordingly?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.14-rc2#n145

I actually thought about asking for that. But I believe the fixes are against code that just went in for 6.14 so backporting won't be necessary. 

> 
> Regards,
> Markus

Shuai Xue Feb. 15, 2025, 2:01 a.m. UTC | #5

在 2025/2/15 01:22, Dave Jiang 写道:
> 
> 
> On 2/14/25 10:15 AM, Markus Elfring wrote:
>> …
>>>   drivers/dma/idxd/init.c | 62 ++++++++++++++++++++++++++++++++---------
>> …
>>
>> How do you think about to add any tags (like “Fixes” and “Cc”)
>> for the presented patches accordingly?
>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.14-rc2#n145
> 
> I actually thought about asking for that. But I believe the fixes are against code that just went in for 6.14 so backporting won't be necessary.

Will add fixes tags in next version.

> 
>>
>> Regards,
>> Markus

Thanks.
Shuai