Read corruption possible during ldb_search
garming at catalyst.net.nz
Tue Apr 25 22:03:18 UTC 2017
As far as I know, all the known deadlocks and data corruption should be
fixed in this patchset.
What I think Andrew was trying to say was that there was no direct test
of my patch ([PATCH 07/17] ldb_tdb: Ensure we correctly decrement
ltdb->read_lock_count) which fixes the original read consistency issue
we were attempting to resolve. Instead he wrote a cmocka test to show
that separately (prior to that we only detected it through spurious
deadlocks during make test).
On 26/04/17 00:20, Stefan Metzmacher via samba-technical wrote:
> Am 25.04.2017 um 13:08 schrieb Andrew Bartlett:
>> I would like to make progress on the ldb and tdb locking issue.
>> I've now included a patch that demonstrates that with ldb_tdb 1.1.29 we
>> allow writes to interfere with indexed reads, meaning that if a write
>> happens during a search, the search output is not consistent.
>> To demonstrate it, run ./bin/ldb_tdb_mod_op_test in a lib/ldb
>> standalone build with this commit reverted:
>> "ldb_tdb: Ensure we correctly decrement ltdb->read_lock_count"
>> Therefore the locking changes are more than just a performance issue,
>> and I would request that we move forward with it, given the inability
>> to isolate the possible decade-old locking issue on Solaris.
>> In the alternate, could you please propose a route forward on this?
> I used this branch
> (with this top commit)
> and combined that with the patches I posted here:
> The result is attached and available here:
> So my most important question:
> Do you think the patches in my branch fix all of our known deadlock/
> data corruption bugs?
> All mails in this thread indicated to me that there're more bugs to be
> But this could also be a misunderstanding.
> If there're no more bugs to be fixed and I start with a more deep
> review, otherwise please base your additions on top of my branch.
> Regarding the Solaris deadlock problem, I guess we can ignore it
> as the test we wrote also works on Solaris 10, but I'll run the
> full standalone testsuites for ldb and tdb again before it might
> be pushed to master.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 490 bytes
Desc: OpenPGP digital signature
More information about the samba-technical