Daniel Petrie wrote:
> --- Guy Hulbert <[EMAIL PROTECTED]> wrote:
>
>   
>> On Wed, 2007-03-21 at 06:50 -0700, Daniel Petrie wrote:
>>     
>>> connection/device using the prior process.  We actually use this
>>> locking pattern in any places in sipX, most recently in the
>>>       
>> s/any/many/ ?
>>     
> Yes. Sorry, we use this locking pattern in many places in sipX:
> SipTransactionList/SipTransaction
> SipProtocolServer/SipClient
> MpInputDeviceManager/MpInputDeviceCOnnection
>
> Cheers,
> Dan
>   
Would it be possible to make it like this?

take lock on the manager
>     take the lock on the connection/device,
>   release the manager lock
>     invoke the frame push to the driver
>     release the lock on the connection/device,
>   take lock on the manager
>   release the manager lock


I haven't looked at the code so I don't know if it would be possible but 
clearly the locking pattern shown previously is deadlock prone and has 
to be fixed.

Jaroslav Libak

_______________________________________________
sipxtapi-dev mailing list
[email protected]
List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/

Reply via email to