1
0
mirror of https://github.com/sqlite/sqlite.git synced 2025-08-07 02:42:48 +03:00

Cleanup of the PAGERTRACE macro. Other comment changes in pager.c. (CVS 6122)

FossilOrigin-Name: ee7b4b60880e80e6fb0b2f93ebc6ee5ad6917f9d
This commit is contained in:
drh
2009-01-06 15:58:57 +00:00
parent be9c0ab276
commit 30d537011a
3 changed files with 40 additions and 65 deletions

View File

@@ -1,5 +1,5 @@
C The\sfix\sin\s(6120)\swasn't\squite\sright.\sThis\sfixes\sit.\s(CVS\s6121) C Cleanup\sof\sthe\sPAGERTRACE\smacro.\s\sOther\scomment\schanges\sin\spager.c.\s(CVS\s6122)
D 2009-01-06T15:28:35 D 2009-01-06T15:58:57
F Makefile.arm-wince-mingw32ce-gcc fcd5e9cd67fe88836360bb4f9ef4cb7f8e2fb5a0 F Makefile.arm-wince-mingw32ce-gcc fcd5e9cd67fe88836360bb4f9ef4cb7f8e2fb5a0
F Makefile.in 05461a9b5803d5ad10c79f989801e9fd2cc3e592 F Makefile.in 05461a9b5803d5ad10c79f989801e9fd2cc3e592
F Makefile.linux-gcc d53183f4aa6a9192d249731c90dbdffbd2c68654 F Makefile.linux-gcc d53183f4aa6a9192d249731c90dbdffbd2c68654
@@ -142,7 +142,7 @@ F src/os_common.h 24525d8b7bce66c374dfc1810a6c9043f3359b60
F src/os_os2.c bed77dc26e3a95ce4a204936b9a1ca6fe612fcc5 F src/os_os2.c bed77dc26e3a95ce4a204936b9a1ca6fe612fcc5
F src/os_unix.c e6eacc7ec735ded605fefcbaf250058baa8feb12 F src/os_unix.c e6eacc7ec735ded605fefcbaf250058baa8feb12
F src/os_win.c 496e3ceb499aedc63622a89ef76f7af2dd902709 F src/os_win.c 496e3ceb499aedc63622a89ef76f7af2dd902709
F src/pager.c 6825837db932f9e4ccb715bcb24d1a24cd22fe43 F src/pager.c 8a64e448f4c6ff72d27e97df28f1c44e28bc2238
F src/pager.h 0793c5e4faed6c278037eb22b2434b318687d615 F src/pager.h 0793c5e4faed6c278037eb22b2434b318687d615
F src/parse.y 4d0e33a702dc3ea7b69d8ae1914b3fbd32e46057 F src/parse.y 4d0e33a702dc3ea7b69d8ae1914b3fbd32e46057
F src/pcache.c 16dc8da6e6ba6250f8dfd9ee46036db1cbceedc6 F src/pcache.c 16dc8da6e6ba6250f8dfd9ee46036db1cbceedc6
@@ -692,7 +692,7 @@ F tool/speedtest16.c c8a9c793df96db7e4933f0852abb7a03d48f2e81
F tool/speedtest2.tcl ee2149167303ba8e95af97873c575c3e0fab58ff F tool/speedtest2.tcl ee2149167303ba8e95af97873c575c3e0fab58ff
F tool/speedtest8.c 2902c46588c40b55661e471d7a86e4dd71a18224 F tool/speedtest8.c 2902c46588c40b55661e471d7a86e4dd71a18224
F tool/speedtest8inst1.c 293327bc76823f473684d589a8160bde1f52c14e F tool/speedtest8inst1.c 293327bc76823f473684d589a8160bde1f52c14e
P 2cdbb468ed81d35a7e1a580683864de60e103083 P ddc2ebfa529b15cdbdd3b6b6d4873cb085cfd3b9
R 990c5e1ec7d21f8528fa930423283fdb R 6b0877b5b9e58d59e7a8a4362c4eedfd
U danielk1977 U drh
Z 046628115eb5f07042a0f536f52e1976 Z f01f333ede5e27c1b8f182af69841c07

View File

@@ -1 +1 @@
ddc2ebfa529b15cdbdd3b6b6d4873cb085cfd3b9 ee7b4b60880e80e6fb0b2f93ebc6ee5ad6917f9d

View File

@@ -18,7 +18,7 @@
** file simultaneously, or one process from reading the database while ** file simultaneously, or one process from reading the database while
** another is writing. ** another is writing.
** **
** @(#) $Id: pager.c,v 1.533 2009/01/06 15:28:35 danielk1977 Exp $ ** @(#) $Id: pager.c,v 1.534 2009/01/06 15:58:57 drh Exp $
*/ */
#ifndef SQLITE_OMIT_DISKIO #ifndef SQLITE_OMIT_DISKIO
#include "sqliteInt.h" #include "sqliteInt.h"
@@ -29,21 +29,13 @@
#if 0 #if 0
int sqlite3PagerTrace=1; /* True to enable tracing */ int sqlite3PagerTrace=1; /* True to enable tracing */
#define sqlite3DebugPrintf printf #define sqlite3DebugPrintf printf
#define PAGERTRACE1(X) if( sqlite3PagerTrace ) sqlite3DebugPrintf(X) #define PAGERTRACE(X) if( sqlite3PagerTrace ){ sqlite3DebugPrintf X; }
#define PAGERTRACE2(X,Y) if( sqlite3PagerTrace ) sqlite3DebugPrintf(X,Y)
#define PAGERTRACE3(X,Y,Z) if( sqlite3PagerTrace ) sqlite3DebugPrintf(X,Y,Z)
#define PAGERTRACE4(X,Y,Z,W) if( sqlite3PagerTrace ) sqlite3DebugPrintf(X,Y,Z,W)
#define PAGERTRACE5(X,Y,Z,W,V) if( sqlite3PagerTrace ) sqlite3DebugPrintf(X,Y,Z,W,V)
#else #else
#define PAGERTRACE1(X) #define PAGERTRACE(X)
#define PAGERTRACE2(X,Y)
#define PAGERTRACE3(X,Y,Z)
#define PAGERTRACE4(X,Y,Z,W)
#define PAGERTRACE5(X,Y,Z,W,V)
#endif #endif
/* /*
** The following two macros are used within the PAGERTRACEX() macros above ** The following two macros are used within the PAGERTRACE() macros above
** to print out file-descriptors. ** to print out file-descriptors.
** **
** PAGERID() takes a pointer to a Pager struct as its argument. The ** PAGERID() takes a pointer to a Pager struct as its argument. The
@@ -107,23 +99,6 @@ int sqlite3PagerTrace=1; /* True to enable tracing */
#define PAGER_EXCLUSIVE 4 /* same as EXCLUSIVE_LOCK */ #define PAGER_EXCLUSIVE 4 /* same as EXCLUSIVE_LOCK */
#define PAGER_SYNCED 5 #define PAGER_SYNCED 5
/*
** If the SQLITE_BUSY_RESERVED_LOCK macro is set to true at compile-time,
** then failed attempts to get a reserved lock will invoke the busy callback.
** This is off by default. To see why, consider the following scenario:
**
** Suppose thread A already has a shared lock and wants a reserved lock.
** Thread B already has a reserved lock and wants an exclusive lock. If
** both threads are using their busy callbacks, it might be a long time
** be for one of the threads give up and allows the other to proceed.
** But if the thread trying to get the reserved lock gives up quickly
** (if it never invokes its busy callback) then the contention will be
** resolved quickly.
*/
#ifndef SQLITE_BUSY_RESERVED_LOCK
# define SQLITE_BUSY_RESERVED_LOCK 0
#endif
/* /*
** This macro rounds values up so that if the value is an address it ** This macro rounds values up so that if the value is an address it
** is guaranteed to be an address that is aligned to an 8-byte boundary. ** is guaranteed to be an address that is aligned to an 8-byte boundary.
@@ -272,7 +247,7 @@ int sqlite3_pager_writej_count = 0; /* Number of pages written to journal */
** was obtained from /dev/random. It is used only as a sanity check. ** was obtained from /dev/random. It is used only as a sanity check.
** **
** Since version 2.8.0, the journal format contains additional sanity ** Since version 2.8.0, the journal format contains additional sanity
** checking information. If the power fails while the journal is begin ** checking information. If the power fails while the journal is being
** written, semi-random garbage data might appear in the journal ** written, semi-random garbage data might appear in the journal
** file after power is restored. If an attempt is then made ** file after power is restored. If an attempt is then made
** to roll the journal back, the database could be corrupted. The additional ** to roll the journal back, the database could be corrupted. The additional
@@ -1238,10 +1213,10 @@ static int pager_playback_one_page(
** Do not attempt to write if database file has never been opened. ** Do not attempt to write if database file has never been opened.
*/ */
pPg = pager_lookup(pPager, pgno); pPg = pager_lookup(pPager, pgno);
PAGERTRACE5("PLAYBACK %d page %d hash(%08x) %s\n", PAGERTRACE(("PLAYBACK %d page %d hash(%08x) %s\n",
PAGERID(pPager), pgno, pager_datahash(pPager->pageSize, aData), PAGERID(pPager), pgno, pager_datahash(pPager->pageSize, aData),
(isMainJrnl?"main-journal":"sub-journal") (isMainJrnl?"main-journal":"sub-journal")
); ));
if( (pPager->state>=PAGER_EXCLUSIVE) if( (pPager->state>=PAGER_EXCLUSIVE)
&& (pPg==0 || 0==(pPg->flags&PGHDR_NEED_SYNC)) && (pPg==0 || 0==(pPg->flags&PGHDR_NEED_SYNC))
&& (pPager->fd->pMethods) && (pPager->fd->pMethods)
@@ -1998,7 +1973,7 @@ int sqlite3PagerOpen(
sqlite3PcacheOpen(szPageDflt, nExtra, !memDb, sqlite3PcacheOpen(szPageDflt, nExtra, !memDb,
!memDb?pagerStress:0, (void *)pPager, pPager->pPCache); !memDb?pagerStress:0, (void *)pPager, pPager->pPCache);
PAGERTRACE3("OPEN %d %s\n", FILEHANDLEID(pPager->fd), pPager->zFilename); PAGERTRACE(("OPEN %d %s\n", FILEHANDLEID(pPager->fd), pPager->zFilename));
IOTRACE(("OPEN %p %s\n", pPager, pPager->zFilename)) IOTRACE(("OPEN %p %s\n", pPager, pPager->zFilename))
/* Fill in Pager.zDirectory[] */ /* Fill in Pager.zDirectory[] */
@@ -2366,7 +2341,7 @@ int sqlite3PagerClose(Pager *pPager){
} }
enable_simulated_io_errors(); enable_simulated_io_errors();
sqlite3EndBenignMalloc(); sqlite3EndBenignMalloc();
PAGERTRACE2("CLOSE %d\n", PAGERID(pPager)); PAGERTRACE(("CLOSE %d\n", PAGERID(pPager)));
IOTRACE(("CLOSE %p\n", pPager)) IOTRACE(("CLOSE %p\n", pPager))
if( pPager->journalOpen ){ if( pPager->journalOpen ){
sqlite3OsClose(pPager->jfd); sqlite3OsClose(pPager->jfd);
@@ -2456,7 +2431,7 @@ static int syncJournal(Pager *pPager){
*/ */
i64 jrnlOff; i64 jrnlOff;
if( pPager->fullSync && 0==(iDc&SQLITE_IOCAP_SEQUENTIAL) ){ if( pPager->fullSync && 0==(iDc&SQLITE_IOCAP_SEQUENTIAL) ){
PAGERTRACE2("SYNC journal of %d\n", PAGERID(pPager)); PAGERTRACE(("SYNC journal of %d\n", PAGERID(pPager)));
IOTRACE(("JSYNC %p\n", pPager)) IOTRACE(("JSYNC %p\n", pPager))
rc = sqlite3OsSync(pPager->jfd, pPager->sync_flags); rc = sqlite3OsSync(pPager->jfd, pPager->sync_flags);
if( rc!=0 ) return rc; if( rc!=0 ) return rc;
@@ -2468,7 +2443,7 @@ static int syncJournal(Pager *pPager){
if( rc ) return rc; if( rc ) return rc;
} }
if( 0==(iDc&SQLITE_IOCAP_SEQUENTIAL) ){ if( 0==(iDc&SQLITE_IOCAP_SEQUENTIAL) ){
PAGERTRACE2("SYNC journal of %d\n", PAGERID(pPager)); PAGERTRACE(("SYNC journal of %d\n", PAGERID(pPager)));
IOTRACE(("JSYNC %p\n", pPager)) IOTRACE(("JSYNC %p\n", pPager))
rc = sqlite3OsSync(pPager->jfd, pPager->sync_flags| rc = sqlite3OsSync(pPager->jfd, pPager->sync_flags|
(pPager->sync_flags==SQLITE_SYNC_FULL?SQLITE_SYNC_DATAONLY:0) (pPager->sync_flags==SQLITE_SYNC_FULL?SQLITE_SYNC_DATAONLY:0)
@@ -2540,8 +2515,8 @@ static int pager_write_pagelist(PgHdr *pList){
i64 offset = (pList->pgno-1)*(i64)pPager->pageSize; i64 offset = (pList->pgno-1)*(i64)pPager->pageSize;
char *pData = CODEC2(pPager, pList->pData, pList->pgno, 6); char *pData = CODEC2(pPager, pList->pData, pList->pgno, 6);
PAGERTRACE4("STORE %d page %d hash(%08x)\n", PAGERTRACE(("STORE %d page %d hash(%08x)\n",
PAGERID(pPager), pList->pgno, pager_pagehash(pList)); PAGERID(pPager), pList->pgno, pager_pagehash(pList)));
IOTRACE(("PGOUT %p %d\n", pPager, pList->pgno)); IOTRACE(("PGOUT %p %d\n", pPager, pList->pgno));
rc = sqlite3OsWrite(pPager->fd, pData, pPager->pageSize, offset); rc = sqlite3OsWrite(pPager->fd, pData, pPager->pageSize, offset);
PAGER_INCR(sqlite3_pager_writedb_count); PAGER_INCR(sqlite3_pager_writedb_count);
@@ -2555,7 +2530,7 @@ static int pager_write_pagelist(PgHdr *pList){
} }
#ifndef NDEBUG #ifndef NDEBUG
else{ else{
PAGERTRACE3("NOSTORE %d page %d\n", PAGERID(pPager), pList->pgno); PAGERTRACE(("NOSTORE %d page %d\n", PAGERID(pPager), pList->pgno));
} }
#endif #endif
if( rc ) return rc; if( rc ) return rc;
@@ -2580,7 +2555,7 @@ static int subjournalPage(PgHdr *pPg){
i64 offset = pPager->stmtNRec*(4+pPager->pageSize); i64 offset = pPager->stmtNRec*(4+pPager->pageSize);
char *pData2 = CODEC2(pPager, pData, pPg->pgno, 7); char *pData2 = CODEC2(pPager, pData, pPg->pgno, 7);
PAGERTRACE3("STMT-JOURNAL %d page %d\n", PAGERID(pPager), pPg->pgno); PAGERTRACE(("STMT-JOURNAL %d page %d\n", PAGERID(pPager), pPg->pgno));
assert( pageInJournal(pPg) || pPg->pgno>pPager->dbOrigSize ); assert( pageInJournal(pPg) || pPg->pgno>pPager->dbOrigSize );
rc = write32bits(pPager->sjfd, offset, pPg->pgno); rc = write32bits(pPager->sjfd, offset, pPg->pgno);
@@ -2638,7 +2613,7 @@ static int pagerStress(void *p, PgHdr *pPg){
} }
if( rc==SQLITE_OK ){ if( rc==SQLITE_OK ){
PAGERTRACE3("STRESS %d page %d\n", PAGERID(pPager), pPg->pgno); PAGERTRACE(("STRESS %d page %d\n", PAGERID(pPager), pPg->pgno));
sqlite3PcacheMakeClean(pPg); sqlite3PcacheMakeClean(pPg);
} }
return rc; return rc;
@@ -2712,8 +2687,8 @@ static int readDbPage(Pager *pPager, PgHdr *pPg, Pgno pgno){
sizeof(pPager->dbFileVers)); sizeof(pPager->dbFileVers));
} }
CODEC1(pPager, pPg->pData, pPg->pgno, 3); CODEC1(pPager, pPg->pData, pPg->pgno, 3);
PAGERTRACE4("FETCH %d page %d hash(%08x)\n", PAGERTRACE(("FETCH %d page %d hash(%08x)\n",
PAGERID(pPager), pPg->pgno, pager_pagehash(pPg)); PAGERID(pPager), pPg->pgno, pager_pagehash(pPg)));
return rc; return rc;
} }
@@ -3249,7 +3224,7 @@ int sqlite3PagerBegin(DbPage *pPg, int exFlag){
return rc; return rc;
} }
pPager->dirtyCache = 0; pPager->dirtyCache = 0;
PAGERTRACE2("TRANSACTION %d\n", PAGERID(pPager)); PAGERTRACE(("TRANSACTION %d\n", PAGERID(pPager)));
if( pPager->useJournal && !pPager->tempFile if( pPager->useJournal && !pPager->tempFile
&& pPager->journalMode!=PAGER_JOURNALMODE_OFF ){ && pPager->journalMode!=PAGER_JOURNALMODE_OFF ){
rc = pager_open_journal(pPager); rc = pager_open_journal(pPager);
@@ -3383,9 +3358,9 @@ static int pager_write(PgHdr *pPg){
IOTRACE(("JOUT %p %d %lld %d\n", pPager, pPg->pgno, IOTRACE(("JOUT %p %d %lld %d\n", pPager, pPg->pgno,
pPager->journalOff, pPager->pageSize)); pPager->journalOff, pPager->pageSize));
PAGER_INCR(sqlite3_pager_writej_count); PAGER_INCR(sqlite3_pager_writej_count);
PAGERTRACE5("JOURNAL %d page %d needSync=%d hash(%08x)\n", PAGERTRACE(("JOURNAL %d page %d needSync=%d hash(%08x)\n",
PAGERID(pPager), pPg->pgno, PAGERID(pPager), pPg->pgno,
((pPg->flags&PGHDR_NEED_SYNC)?1:0), pager_pagehash(pPg)); ((pPg->flags&PGHDR_NEED_SYNC)?1:0), pager_pagehash(pPg)));
/* Even if an IO or diskfull error occurred while journalling the /* Even if an IO or diskfull error occurred while journalling the
** page in the block above, set the need-sync flag for the page. ** page in the block above, set the need-sync flag for the page.
@@ -3421,9 +3396,9 @@ static int pager_write(PgHdr *pPg){
pPg->flags |= PGHDR_NEED_SYNC; pPg->flags |= PGHDR_NEED_SYNC;
pPager->needSync = 1; pPager->needSync = 1;
} }
PAGERTRACE4("APPEND %d page %d needSync=%d\n", PAGERTRACE(("APPEND %d page %d needSync=%d\n",
PAGERID(pPager), pPg->pgno, PAGERID(pPager), pPg->pgno,
((pPg->flags&PGHDR_NEED_SYNC)?1:0)); ((pPg->flags&PGHDR_NEED_SYNC)?1:0)));
} }
} }
@@ -3611,7 +3586,7 @@ int sqlite3PagerDontWrite(DbPage *pDbPage){
** corruption during the next transaction. ** corruption during the next transaction.
*/ */
}else{ }else{
PAGERTRACE3("DONT_WRITE page %d of %d\n", pPg->pgno, PAGERID(pPager)); PAGERTRACE(("DONT_WRITE page %d of %d\n", pPg->pgno, PAGERID(pPager)));
IOTRACE(("CLEAN %p %d\n", pPager, pPg->pgno)) IOTRACE(("CLEAN %p %d\n", pPager, pPg->pgno))
pPg->flags |= PGHDR_DONT_WRITE; pPg->flags |= PGHDR_DONT_WRITE;
#ifdef SQLITE_CHECK_PAGES #ifdef SQLITE_CHECK_PAGES
@@ -3686,7 +3661,7 @@ void sqlite3PagerDontRollback(DbPage *pPg){
sqlite3EndBenignMalloc(); sqlite3EndBenignMalloc();
PAGERTRACE3("DONT_ROLLBACK page %d of %d\n", pPg->pgno, PAGERID(pPager)); PAGERTRACE(("DONT_ROLLBACK page %d of %d\n", pPg->pgno, PAGERID(pPager)));
IOTRACE(("GARBAGE %p %d\n", pPager, pPg->pgno)) IOTRACE(("GARBAGE %p %d\n", pPager, pPg->pgno))
} }
@@ -3788,8 +3763,8 @@ int sqlite3PagerCommitPhaseOne(
return SQLITE_OK; return SQLITE_OK;
} }
PAGERTRACE4("DATABASE SYNC: File=%s zMaster=%s nSize=%d\n", PAGERTRACE(("DATABASE SYNC: File=%s zMaster=%s nSize=%d\n",
pPager->zFilename, zMaster, pPager->dbSize); pPager->zFilename, zMaster, pPager->dbSize));
/* If this is an in-memory db, or no pages have been written to, or this /* If this is an in-memory db, or no pages have been written to, or this
** function has already been called, it is a no-op. ** function has already been called, it is a no-op.
@@ -3861,7 +3836,7 @@ int sqlite3PagerCommitPhaseOne(
Pgno iSkip = PAGER_MJ_PGNO(pPager); Pgno iSkip = PAGER_MJ_PGNO(pPager);
Pgno dbSize = pPager->dbSize; Pgno dbSize = pPager->dbSize;
pPager->dbSize = pPager->dbOrigSize; pPager->dbSize = pPager->dbOrigSize;
for( i=dbSize+1; i<=pPager->dbOrigSize; i++ ){ for( i=pPager->dbSize+1; i<=pPager->dbOrigSize; i++ ){
if( !sqlite3BitvecTest(pPager->pInJournal, i) && i!=iSkip ){ if( !sqlite3BitvecTest(pPager->pInJournal, i) && i!=iSkip ){
rc = sqlite3PagerGet(pPager, i, &pPg); rc = sqlite3PagerGet(pPager, i, &pPg);
if( rc!=SQLITE_OK ) goto sync_exit; if( rc!=SQLITE_OK ) goto sync_exit;
@@ -3949,7 +3924,7 @@ int sqlite3PagerCommitPhaseTwo(Pager *pPager){
assert( pPager->dirtyCache==0 || pPager->journalOpen==0 ); assert( pPager->dirtyCache==0 || pPager->journalOpen==0 );
return SQLITE_OK; return SQLITE_OK;
} }
PAGERTRACE2("COMMIT %d\n", PAGERID(pPager)); PAGERTRACE(("COMMIT %d\n", PAGERID(pPager)));
assert( pPager->state==PAGER_SYNCED || MEMDB || !pPager->dirtyCache ); assert( pPager->state==PAGER_SYNCED || MEMDB || !pPager->dirtyCache );
rc = pager_end_transaction(pPager, pPager->setMaster); rc = pager_end_transaction(pPager, pPager->setMaster);
rc = pager_error(pPager, rc); rc = pager_error(pPager, rc);
@@ -3970,7 +3945,7 @@ int sqlite3PagerCommitPhaseTwo(Pager *pPager){
*/ */
int sqlite3PagerRollback(Pager *pPager){ int sqlite3PagerRollback(Pager *pPager){
int rc = SQLITE_OK; int rc = SQLITE_OK;
PAGERTRACE2("ROLLBACK %d\n", PAGERID(pPager)); PAGERTRACE(("ROLLBACK %d\n", PAGERID(pPager)));
if( !pPager->dirtyCache || !pPager->journalOpen ){ if( !pPager->dirtyCache || !pPager->journalOpen ){
rc = pager_end_transaction(pPager, pPager->setMaster); rc = pager_end_transaction(pPager, pPager->setMaster);
}else if( pPager->errCode && pPager->errCode!=SQLITE_FULL ){ }else if( pPager->errCode && pPager->errCode!=SQLITE_FULL ){
@@ -4232,8 +4207,8 @@ int sqlite3PagerMovepage(Pager *pPager, DbPage *pPg, Pgno pgno, int isCommit){
assert( pPg->nRef>0 ); assert( pPg->nRef>0 );
PAGERTRACE5("MOVE %d page %d (needSync=%d) moves to %d\n", PAGERTRACE(("MOVE %d page %d (needSync=%d) moves to %d\n",
PAGERID(pPager), pPg->pgno, (pPg->flags&PGHDR_NEED_SYNC)?1:0, pgno); PAGERID(pPager), pPg->pgno, (pPg->flags&PGHDR_NEED_SYNC)?1:0, pgno));
IOTRACE(("MOVE %p %d %d\n", pPager, pPg->pgno, pgno)) IOTRACE(("MOVE %p %d %d\n", pPager, pPg->pgno, pgno))
pager_get_content(pPg); pager_get_content(pPg);