Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-14 Thread Andy Shevchenko
On Thu, Dec 14, 2023 at 3:00 PM Christophe Leroy wrote: > Le 14/12/2023 à 13:48, George Stark a écrit : > > [Vous ne recevez pas souvent de courriers de gnst...@salutedevices.com. > > Découvrez pourquoi ceci est important à > > https://aka.ms/LearnAboutSenderIdentification ] > > On 12/14/23 13:06,

Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-14 Thread Christophe Leroy
Le 14/12/2023 à 13:48, George Stark a écrit : > [Vous ne recevez pas souvent de courriers de gnst...@salutedevices.com. > Découvrez pourquoi ceci est important à > https://aka.ms/LearnAboutSenderIdentification ] > > Hello Christophe > > On 12/14/23 13:06, Christophe Leroy wrote: >> >> > ... >

Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-14 Thread George Stark
Hello Christophe On 12/14/23 13:06, Christophe Leroy wrote: ... So you abandonned the idea of using mutex.h ? I'm not the one who make a choice here. The patch [1] you're talking about was seen by everyone but it seems like no one had shown interest. For me personally approach with #def

Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-14 Thread Hans de Goede
Hi, On 12/14/23 11:06, Christophe Leroy wrote: > > > Le 13/12/2023 à 23:30, George Stark a écrit : >> [Vous ne recevez pas souvent de courriers de gnst...@salutedevices.com. >> Découvrez pourquoi ceci est important à >> https://aka.ms/LearnAboutSenderIdentification ] >> >> Using of devm API le

Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-14 Thread Christophe Leroy
Le 13/12/2023 à 23:30, George Stark a écrit : > [Vous ne recevez pas souvent de courriers de gnst...@salutedevices.com. > Découvrez pourquoi ceci est important à > https://aka.ms/LearnAboutSenderIdentification ] > > Using of devm API leads to a certain order of releasing resources. > So all de

Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-14 Thread Hans de Goede
Hi, On 12/13/23 23:38, Andy Shevchenko wrote: > On Thu, Dec 14, 2023 at 12:36 AM Andy Shevchenko > wrote: >> On Thu, Dec 14, 2023 at 12:30 AM George Stark >> wrote: >>> >>> Using of devm API leads to a certain order of releasing resources. >>> So all dependent resources which are not devm-wrapp

Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-13 Thread Andy Shevchenko
On Thu, Dec 14, 2023 at 12:36 AM Andy Shevchenko wrote: > On Thu, Dec 14, 2023 at 12:30 AM George Stark > wrote: > > > > Using of devm API leads to a certain order of releasing resources. > > So all dependent resources which are not devm-wrapped should be deleted > > with respect to devm-release

Re: [PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-13 Thread Andy Shevchenko
On Thu, Dec 14, 2023 at 12:30 AM George Stark wrote: > > Using of devm API leads to a certain order of releasing resources. > So all dependent resources which are not devm-wrapped should be deleted > with respect to devm-release order. Mutex is one of such objects that > often is bound to other re

[PATCH v3 03/11] devm-helpers: introduce devm_mutex_init

2023-12-13 Thread George Stark
Using of devm API leads to a certain order of releasing resources. So all dependent resources which are not devm-wrapped should be deleted with respect to devm-release order. Mutex is one of such objects that often is bound to other resources and has no own devm wrapper. Since mutex_destroy() actua