diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2010-02-24 13:23:58 +1030 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2010-02-24 13:23:58 +1030 |
commit | ec96ea690edbe3398d690b4a953d487ca1773f1c (patch) | |
tree | 24346da6a7faddd3b04478e58b08d8b809994ce1 /lib/tdb/common/check.c | |
parent | 1bf482b9ef9ec73dd7ee4387d7087aa3955503dd (diff) | |
download | samba-ec96ea690edbe3398d690b4a953d487ca1773f1c.tar.gz samba-ec96ea690edbe3398d690b4a953d487ca1773f1c.tar.bz2 samba-ec96ea690edbe3398d690b4a953d487ca1773f1c.zip |
tdb: handle processes dying during transaction commit.
tdb transactions were designed to be robust against the machine
powering off, but interestingly were never designed to handle the case
where an administrator kill -9's a process during commit. Because
recovery is only done on tdb_open, processes with the tdb already
mapped will simply use it despite it being corrupt and needing
recovery.
The solution to this is to check for recovery every time we grab a
data lock: we could have gained the lock because a process just died.
This has no measurable cost: here is the time for tdbtorture -s 0 -n 1
-l 10000:
Before:
2.75 2.50 2.81 3.19 2.91 2.53 2.72 2.50 2.78 2.77 = Avg 2.75
After:
2.81 2.57 3.42 2.49 3.02 2.49 2.84 2.48 2.80 2.43 = Avg 2.74
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'lib/tdb/common/check.c')
0 files changed, 0 insertions, 0 deletions