-C Add\ssome\stests\sfor\sthe\satomic-write\soptimization.\s(CVS\s4275)
-D 2007-08-23T08:06:45
+C Add\ssome\sfurther\stests\sand\sa\sbugfix\sfor\sthe\satomic-write\soptimization.\s(CVS\s4276)
+D 2007-08-23T11:07:10
F Makefile.in 0c0e53720f658c7a551046442dd7afba0b72bfbe
F Makefile.linux-gcc 65241babba6faf1152bf86574477baab19190499
F README 9c4e2d6706bdcc3efdd773ce752a8cdab4f90028
F src/os_unix.h 5768d56d28240d3fe4537fac08cc85e4fb52279e
F src/os_win.c 29c0e19c1072679a4c7818c49fab2f35d2ad7747
F src/os_win.h 41a946bea10f61c158ce8645e7646b29d44f122b
-F src/pager.c 54087a81a44e38e30cf8bfed41a73acc9ece84a8
+F src/pager.c 7b0eee5f01f26449837f690baca7d929f683609c
F src/pager.h 53087c6fb9db01aed17c7fd044662a27507e89b8
F src/parse.y 2d2ce439dc6184621fb0b86f4fc5aca7f391a590
F src/pragma.c 9b989506a1b7c8aecd6befb8235e2f57a4aba7e5
F src/test3.c a7d011c51d6b2e2a73c43983d5c2b731d69c74d7
F src/test4.c c2c0f5dc907f1346f5d4b65eb5799f11eb9e4071
F src/test5.c 3a6a5717a149d7ca2e6d14f5be72cf7555d54dc4
-F src/test6.c 2c141f367ba483eef99a7f4d00f07431caff791e
+F src/test6.c 5d6286568b12ec6c813cb30d2a14de8229f8e388
F src/test7.c a9d509d0e9ad214b4772696f49f6e61be26213d1
F src/test8.c e6a543c8b248efe120ae33a6859fcd55dcf46a96
F src/test9.c b46c8fe02ac7cca1a7316436d8d38d50c66f4b2f
F test/corrupt3.test 263e8bb04e2728df832fddf6973cf54c91db0c32
F test/crash.test 24020168cc42977a4dd83ff78d2b5eb6577715db
F test/crash2.test 423c6ec404d15b7d7d0e40aef0a26740cce6075f
+F test/crash3.test caa79052f29ee5cb17ecede2c386251eb932c05f
F test/crashtest1.c 09c1c7d728ccf4feb9e481671e29dda5669bbcc2
F test/createtab.test b562aba1a65be49935fc43a04e90766e39231804
F test/date.test 4834d022b2fa5982cafed20938f7523a7475e4cd
F test/insert4.test 1e27f0a3e5670d5f03c1636f699aa44270945bca
F test/interrupt.test 81555fb0f8179bb2d0dc7151fd75428223f93cf2
F test/intpkey.test af4fd826c4784ec5c93b444de07adea0254d0d30
-F test/io.test ca9db7cd57a1b02cd983863c1be1153e1900e68b
+F test/io.test 4368da68ae82a8496cd3904a886345b75303979f
F test/ioerr.test 491d42c49bbec598966d26b01ed7901f55e5ee2d
F test/ioerr2.test f938eadb12108048813869b86beee4a2f98e34b8
F test/join.test af0443185378b64878750aa1cf4b83c216f246b4
F test/tableapi.test 036575a98dcce7c92e9f39056839bbad8a715412
F test/tclsqlite.test 593f3b30221e85786965d9e5670ae4f96b4e4159
F test/temptable.test c36f3e5a94507abb64f7ba23deeb4e1a8a8c3821
-F test/tester.tcl c50fd942fe3c8fc36c45be161fcd9b60603f033c
+F test/tester.tcl d6fd4e9eac69a7bc7791ca2f80c91945d1f6d1cc
F test/thread1.test 776c9e459b75ba905193b351926ac4019b049f35
F test/thread2.test 6d7b30102d600f51b4055ee3a5a19228799049fb
F test/threadtest1.c 6029d9c5567db28e6dc908a0c63099c3ba6c383b
F www/vdbe.tcl 87a31ace769f20d3627a64fa1fade7fed47b90d0
F www/version3.tcl 890248cf7b70e60c383b0e84d77d5132b3ead42b
F www/whentouse.tcl fc46eae081251c3c181bd79c5faef8195d7991a5
-P e4e74cd0f9343448ea38e57f08bb4f0616825f31
-R 630334db112909a8743c928d15787556
+P e2cc7b4a3476a733b2701546f6b4ec9abc18152b
+R 98112005a7fda39581e1ca266aa94e0a
U danielk1977
-Z 01e41e90624e3bb8ea0ae5e4ad13b9f8
+Z 5d888b8b25790997a283d2df12c0d28a
-e2cc7b4a3476a733b2701546f6b4ec9abc18152b
\ No newline at end of file
+5f0fb894f44069c4aa9b8dba62b4d8a262c991de
\ No newline at end of file
** file simultaneously, or one process from reading the database while
** another is writing.
**
-** @(#) $Id: pager.c,v 1.368 2007/08/22 22:04:37 drh Exp $
+** @(#) $Id: pager.c,v 1.369 2007/08/23 11:07:10 danielk1977 Exp $
*/
#ifndef SQLITE_OMIT_DISKIO
#include "sqliteInt.h"
assert(SQLITE_IOCAP_ATOMIC512==(512>>8));
assert(SQLITE_IOCAP_ATOMIC64K==(65536>>8));
- if( !fd->pMethods || (dc&(SQLITE_IOCAP_ATOMIC|(nPage<<8))&&nSector<=nPage) ){
+ if( !fd->pMethods || (dc&(SQLITE_IOCAP_ATOMIC|(nPage>>8))&&nSector<=nPage) ){
return JOURNAL_HDR_SZ(pPager) + JOURNAL_PG_SZ(pPager);
}
return 0;
#ifndef SQLITE_OMIT_DISKIO /* This file is a no-op if disk I/O is disabled */
+/* #define TRACE_CRASHTEST */
+
typedef struct CrashFile CrashFile;
typedef struct CrashGlobal CrashGlobal;
typedef struct WriteBuffer WriteBuffer;
}
}
+#ifdef TRACE_CRASHTEST
+ printf("Sync %s (is %s crash)\n", pFile->zName, (isCrash?"a":"not a"));
+#endif
+
sqlite3OsFileSize((sqlite3_file *)pFile, &iSize);
ppPtr = &g.pWriteList;
rc = sqlite3OsTruncate(pRealFile, pWrite->iOffset);
}
*ppPtr = pWrite->pNext;
+#ifdef TRACE_CRASHTEST
+ if( isCrash ){
+ printf("Writing %d bytes @ %d\n", pWrite->nBuf, (int)pWrite->iOffset);
+ }
+#endif
sqlite3_free(pWrite);
break;
}
case 2: { /* Do nothing */
ppPtr = &pWrite->pNext;
+#ifdef TRACE_CRASHTEST
+ if( isCrash ){
+ printf("Omiting %d bytes @ %d\n", pWrite->nBuf, (int)pWrite->iOffset);
+ }
+#endif
break;
}
case 3: { /* Trash sectors */
assert(pWrite->zBuf);
+#ifdef TRACE_CRASHTEST
+ printf("Trashing %d sectors @ sector %d\n", 1+iLast-iFirst, iFirst);
+#endif
+
zGarbage = sqlite3_malloc(g.iSectorSize);
if( zGarbage ){
sqlite3_int64 i;
--- /dev/null
+# 2007 August 23
+#
+# The author disclaims copyright to this source code. In place of
+# a legal notice, here is a blessing:
+#
+# May you do good and not evil.
+# May you find forgiveness for yourself and forgive others.
+# May you share freely, never taking more than you give.
+#
+#***********************************************************************
+#
+# $Id: crash3.test,v 1.1 2007/08/23 11:07:10 danielk1977 Exp $
+
+set testdir [file dirname $argv0]
+source $testdir/tester.tcl
+
+ifcapable !crashtest {
+ finish_test
+ return
+}
+
+proc do_test2 {name tcl res1 res2} {
+ set script [subst -nocommands {
+ do_test $name {
+ set res1 {$res1}
+ set res2 {$res2}
+ set res [eval {$tcl}]
+ if {[set res] eq [set res1] || [set res] eq [set res2]} {
+ set res "{[set res1]} or {[set res2]}"
+ }
+ set res
+ } {{$res1} or {$res2}}
+ }]
+ uplevel $script
+}
+
+# Each iteration of the following loop sets up the database to contain
+# the following schema and data:
+#
+# CREATE TABLE abc(a, b, c);
+# INSERT INTO abc VALUES(1, 2, 3);
+#
+# Then execute the SQL statement, scheduling a crash for part-way through
+# the first sync() of either the database file or the journal file (often
+# the journal file is not required - meaning no crash occurs).
+#
+# After the crash (or absence of a crash), open the database and
+# verify that:
+#
+# * The integrity check passes, and
+# * The contents of table abc is either {1 2 3} or the value specified
+# to the right of the SQL statement below.
+#
+# The procedure is repeated 10 times for each SQL statement. Five times
+# with the crash scheduled for midway through the first journal sync (if
+# any), and five times with the crash midway through the database sync.
+#
+set tn 1
+foreach {sql res2} [list \
+ {INSERT INTO abc VALUES(4, 5, 6)} {1 2 3 4 5 6} \
+ {DELETE FROM abc} {} \
+ {INSERT INTO abc SELECT * FROM abc} {1 2 3 1 2 3} \
+ {UPDATE abc SET a = 2} {2 2 3} \
+ {INSERT INTO abc VALUES(4, 5, randstr(1000,1000))} {n/a} \
+ {CREATE TABLE def(d, e, f)} {n/a} \
+] {
+ for {set ii 0} {$ii < 10} {incr ii} {
+
+ db close
+ file delete -force test.db test.db-journal
+ sqlite3 db test.db
+ do_test crash3-1.$tn.1 {
+ execsql {
+ BEGIN;
+ CREATE TABLE abc(a, b, c);
+ INSERT INTO abc VALUES(1, 2, 3);
+ COMMIT;
+ }
+ } {}
+ db close
+
+ set crashfile test.db
+ if {($ii%2)==0} { append crashfile -journal }
+ set rand "SELECT randstr($tn,$tn);"
+ do_test crash3-1.$tn.2 [subst {
+ crashsql -file $crashfile -char atomic {$rand $sql}
+ sqlite3 db test.db
+ execsql { PRAGMA integrity_check; }
+ }] {ok}
+
+ do_test2 crash3-1.$tn.3 {
+ execsql { SELECT * FROM abc }
+ } {1 2 3} $res2
+
+ incr tn
+ }
+}
+
+finish_test
+
# IO traffic generated by SQLite (making sure SQLite is not writing out
# more database pages than it has to, stuff like that).
#
-# $Id: io.test,v 1.3 2007/08/23 08:06:45 danielk1977 Exp $
+# $Id: io.test,v 1.4 2007/08/23 11:07:10 danielk1977 Exp $
set testdir [file dirname $argv0]
source $testdir/tester.tcl
}
} {1 2 3 4 5 6 7 8}
+# Test that the atomic write optimisation is not enabled if the sector
+# size is larger than the page-size.
+#
+do_test io-2.9.1 {
+ sqlite3_simulate_device -char atomic -sectorsize 2048
+ execsql {
+ BEGIN;
+ INSERT INTO abc VALUES(9, 10);
+ }
+ file exists test.db-journal
+} {1}
+do_test io-2.9.2 {
+ execsql { ROLLBACK; }
+ db close
+ file delete -force test.db test.db-journal
+ sqlite3 db test.db
+ execsql {
+ PRAGMA page_size = 2048;
+ CREATE TABLE abc(a, b);
+ }
+ execsql {
+ BEGIN;
+ INSERT INTO abc VALUES(9, 10);
+ }
+ file exists test.db-journal
+} {0}
+do_test io-2.9.3 {
+ execsql { COMMIT }
+} {}
+
+# Test a couple of the more specific IOCAP_ATOMIC flags
+# (i.e IOCAP_ATOMIC2K etc.).
+#
+do_test io-2.10.1 {
+ sqlite3_simulate_device -char atomic1k
+ execsql {
+ BEGIN;
+ INSERT INTO abc VALUES(11, 12);
+ }
+ file exists test.db-journal
+} {1}
+do_test io-2.10.2 {
+ execsql { ROLLBACK }
+ sqlite3_simulate_device -char atomic2k
+ execsql {
+ BEGIN;
+ INSERT INTO abc VALUES(11, 12);
+ }
+ file exists test.db-journal
+} {0}
+
sqlite3_simulate_device -char {} -sectorsize 0
finish_test
# This file implements some common TCL routines used for regression
# testing the SQLite library
#
-# $Id: tester.tcl,v 1.87 2007/08/23 02:47:54 drh Exp $
+# $Id: tester.tcl,v 1.88 2007/08/23 11:07:10 danielk1977 Exp $
# Make sure tclsqlite3 was compiled correctly. Abort now with an
# error message if not.
# error message. This is "child process exited abnormally" if the crash
# occured.
#
-# crashsql -delay CRASHDELAY -file CRASHFILE ?-blocksize BLOCKSIZE $sql
+# crashsql -delay CRASHDELAY -file CRASHFILE ?-blocksize BLOCKSIZE? $sql
#
proc crashsql {args} {
if {$::tcl_platform(platform)!="unix"} {
set blocksize ""
set crashdelay 1
set crashfile ""
+ set dc ""
set sql [lindex $args end]
for {set ii 0} {$ii < [llength $args]-1} {incr ii 2} {
if {$n>1 && [string first $z -delay]==0} {set crashdelay $z2} \
elseif {$n>1 && [string first $z -file]==0} {set crashfile $z2} \
elseif {$n>1 && [string first $z -blocksize]==0} {set blocksize "-s $z2" } \
+ elseif {$n>1 && [string first $z -characteristics]==0} {set dc "-c {$z2}" } \
else { error "Unrecognized option: $z" }
}
set cfile [file join [pwd] $crashfile]
set f [open crash.tcl w]
- puts $f "sqlite3_crashparams $blocksize $crashdelay $cfile"
+ puts $f "sqlite3_crashparams $blocksize $dc $crashdelay $cfile"
puts $f "set sqlite_pending_byte $::sqlite_pending_byte"
puts $f "sqlite3 db test.db"