Re: [389-devel] Please review ticket 47976: deadlock in mep delete post op

2015-11-04 Thread thierry bordaz

Hi Ludwig, Rich,

you were right (as usual !), dblayer_get_pvt_txn/dblayer_push_pvt_txn 
are doing the magic.

Now the fix is a oneline fix !

https://fedorahosted.org/389/attachment/ticket/47976/0001-Ticket-47976-2-deadlock-in-mep-delete-post-op.patch

thanks
thierry
On 11/03/2015 06:01 PM, Ludwig Krispenz wrote:

Hi Thierry,

we already had started to discuss on IRC, but here are my thoughts again.

Is it necessary to explicitely set the txn in the plugin ? The txn 
will be found when ldbm_back_delete() does dblayer_txn_begin(9 and it 
checks the per thread stack of txns.
In my opinion the real problem is not to set  the txn in id2entry, 
which will then try to read a locked page.


Ludwig

On 11/03/2015 05:40 PM, thierry bordaz wrote:

ticket https://fedorahosted.org/389/ticket/47976

fix 
https://fedorahosted.org/389/attachment/ticket/47976/0001-Ticket-47976-deadlock-in-mep-delete-post-op.patch


test case: 
https://fedorahosted.org/389/attachment/ticket/47976/0002-Ticket-47976-test-case.patch




--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel




--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel


--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

Re: [389-devel] Please review ticket 47976: deadlock in mep delete post op

2015-11-03 Thread Ludwig Krispenz

Hi Thierry,

we already had started to discuss on IRC, but here are my thoughts again.

Is it necessary to explicitely set the txn in the plugin ? The txn will 
be found when ldbm_back_delete() does dblayer_txn_begin(9 and it checks 
the per thread stack of txns.
In my opinion the real problem is not to set  the txn in id2entry, which 
will then try to read a locked page.


Ludwig

On 11/03/2015 05:40 PM, thierry bordaz wrote:

ticket https://fedorahosted.org/389/ticket/47976

fix 
https://fedorahosted.org/389/attachment/ticket/47976/0001-Ticket-47976-deadlock-in-mep-delete-post-op.patch


test case: 
https://fedorahosted.org/389/attachment/ticket/47976/0002-Ticket-47976-test-case.patch




--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel


--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel