Please review ticket 47976: deadlock in mep delete post op

[thierry bordaz]

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

fix 
https://fedorahosted.org/389/attachment/ticket/47976/0001-Ticket-47976-de...

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

[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-de...
>
>  test case: 
>  https://fedorahosted.org/389/attachment/ticket/47976/0002-Ticket-47976-te...
>
>
>
>  --
>  389-devel mailing list
>  389-devel(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/389-devel

[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-...

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-de...
> >
> > test case: 
> >
> https://fedorahosted.org/389/attachment/ticket/47976/0002-Ticket-47976-te...
> >
> >
> >
> > --
> > 389-devel mailing list
> > 389-devel(a)lists.fedoraproject.org
> > https://admin.fedoraproject.org/mailman/listinfo/389-devel
>
>
>
>  --
>  389-devel mailing list
>  389-devel(a)lists.fedoraproject.org
>  https://admin.fedoraproject.org/mailman/listinfo/389-devel