diff --git a/lladd/compensations.h b/lladd/compensations.h index 08ebdc3..d684ad6 100644 --- a/lladd/compensations.h +++ b/lladd/compensations.h @@ -103,9 +103,9 @@ void lock_c_line_1231(lock * l) { void compensations_init(); void compensations_deinit(); -int compensation_error(); +long compensation_error(); void compensation_clear_error(); -void compensation_set_error(int code); +void compensation_set_error(long code); #define try do { if(compensation_error()) return; do #define try_ret(x) do { if(compensation_error()) return (x); do diff --git a/src/libdfa/libdfa.c b/src/libdfa/libdfa.c index 8e87829..50aedd5 100644 --- a/src/libdfa/libdfa.c +++ b/src/libdfa/libdfa.c @@ -527,7 +527,7 @@ pthread_t spawn_worker_thread(DfaSet * dfaSet, state_machine_id machine_id) { void * request(DfaSet * dfaSet, state_name start_state, char * recipient_addr, state_machine_id recipient_machine_id, Message * message) { StateMachine * initial_sm; state_machine_id machine_id; - int ret; + void * ret; writelock(dfaSet->lock, 600); initial_sm = allocSmash(dfaSet->smash); @@ -558,7 +558,7 @@ void * request(DfaSet * dfaSet, state_name start_state, char * recipient_addr, s machine_id = initial_sm->machine_id; writeunlock(dfaSet->lock); - ret = (int)run_request(dfaSet, machine_id); + ret = run_request(dfaSet, machine_id); writelock(dfaSet->lock, machine_id); assert(initial_sm == getSmash(dfaSet->smash, machine_id)); @@ -570,7 +570,7 @@ void * request(DfaSet * dfaSet, state_name start_state, char * recipient_addr, s freeSmash(dfaSet->smash, initial_sm->machine_id); writeunlock(dfaSet->lock); - return (void*)ret; + return ret; } void * run_request(DfaSet * dfaSet, state_machine_id machine_id) { diff --git a/src/libdfa/messages.c b/src/libdfa/messages.c index 9ec3c10..55053af 100644 --- a/src/libdfa/messages.c +++ b/src/libdfa/messages.c @@ -324,7 +324,7 @@ int receive_message(NetworkSetup *ns, Message *message, char *from) { if(message_size != sizeof(Message)) { /* drop packet */ - fprintf(stderr, "Size mismatch: %d, %d\n", message_size, sizeof(Message)); + fprintf(stderr, "Size mismatch: %ld, %ld\n", message_size, sizeof(Message)); return 0; } else { /* TODO: Callback to security stuff / crypto here? */ diff --git a/src/lladd/blobManager.h b/src/lladd/blobManager.h index ea89b0a..f7b45cb 100644 --- a/src/lladd/blobManager.h +++ b/src/lladd/blobManager.h @@ -71,8 +71,8 @@ void commitBlobs(int xid); void abortBlobs(int xid); typedef struct { - unsigned offset; - unsigned long size; + size_t offset; + size_t size; unsigned fd : 1; } blob_record_t; diff --git a/src/lladd/compensations.c b/src/lladd/compensations.c index 89d0b1f..e46f703 100644 --- a/src/lladd/compensations.c +++ b/src/lladd/compensations.c @@ -19,8 +19,8 @@ void compensations_deinit() { assert(!ret); } -int compensation_error() { - int error = (int) pthread_getspecific(error_key); +long compensation_error() { + long error = (long) pthread_getspecific(error_key); return error; } @@ -28,7 +28,7 @@ void compensation_clear_error() { compensation_set_error(0); } -void compensation_set_error(int error) { +void compensation_set_error(long error) { int ret = pthread_setspecific(error_key, (void *)error); if(ret) { printf("Unhandled error: %s\n", strerror(ret)); diff --git a/src/lladd/crc32.c b/src/lladd/crc32.c index 7a2ac7a..80ebca1 100644 --- a/src/lladd/crc32.c +++ b/src/lladd/crc32.c @@ -12,7 +12,7 @@ unsigned int crc32(const void *buffer, unsigned int count, unsigned int crc); static int BuildCRCTable(void); -static unsigned long *CRCTable; // Table constructed for fast lookup. +static unsigned int *CRCTable; // Table constructed for fast lookup. #define CRC32_POLYNOMIAL 0xEDB88320 diff --git a/src/lladd/fifo.c b/src/lladd/fifo.c index 5eaeb8a..30fe425 100644 --- a/src/lladd/fifo.c +++ b/src/lladd/fifo.c @@ -11,7 +11,7 @@ Obtain a member of a fifoPool based on the value of multiplexKey. Use CRC32 to assign the key to a consumer. */ lladdFifo_t * lladdFifoPool_getFifoCRC32( lladdFifoPool_t * pool, byte * multiplexKey, size_t multiplexKeySize) { - int memberId = crc32(multiplexKey, multiplexKeySize, (unsigned long)-1L) % pool->fifoCount; + int memberId = crc32(multiplexKey, multiplexKeySize, (unsigned int)-1) % pool->fifoCount; return pool->pool[memberId]; } void lladdFifoPool_markDirty(int xid, lladdFifoPool_t * pool, lladdFifo_t * fifo) { diff --git a/src/lladd/graph.c b/src/lladd/graph.c index e5d5c2e..ae35a07 100644 --- a/src/lladd/graph.c +++ b/src/lladd/graph.c @@ -61,7 +61,7 @@ void multiTraverse(int xid, recordid arrayList, lladdFifo_t * local, lladdFifo_t if(myFifo == -1) { if(useCRC) { - myFifo = crc32((byte*)&(rid->page), sizeof(rid->page), (unsigned long)-1L) % pool->fifoCount; + myFifo = crc32((byte*)&(rid->page), sizeof(rid->page), (unsigned int)-1) % pool->fifoCount; } else { myFifo = rid->page % pool->fifoCount; } @@ -87,7 +87,7 @@ void multiTraverse(int xid, recordid arrayList, lladdFifo_t * local, lladdFifo_t nextRid = dereferenceArrayListRid(p, nextRid.slot); releasePage(p); - int thisFifo = crc32((byte*)&(nextRid.page), sizeof(nextRid.page), (unsigned long)-1L) % pool->fifoCount; + int thisFifo = crc32((byte*)&(nextRid.page), sizeof(nextRid.page), (unsigned int)-1) % pool->fifoCount; /* if(nextRid.page == rid->page) { assert(thisFifo == myFifo); }*/ diff --git a/src/lladd/hash.c b/src/lladd/hash.c index 932500b..58dd15c 100644 --- a/src/lladd/hash.c +++ b/src/lladd/hash.c @@ -12,7 +12,7 @@ unsigned int max_bucket(unsigned char tableBits, unsigned long nextExtension) { unsigned int hash(const void * val, long val_length, unsigned char tableBits, unsigned long nextExtension) { unsigned int oldTableLength = /*powl(2, tableBits - 1); */ twoToThe(tableBits - 1); - unsigned int unmixed = crc32(val, val_length, (unsigned long)-1L); + unsigned int unmixed = crc32(val, val_length, (unsigned int)-1); unsigned int ret = unmixed & (oldTableLength - 1); /* What would the low hash value be? */ diff --git a/src/lladd/lockManager.c b/src/lladd/lockManager.c index 7559a16..a5855b8 100644 --- a/src/lladd/lockManager.c +++ b/src/lladd/lockManager.c @@ -92,7 +92,7 @@ int lockManagerReadLockHashed(int xid, byte * dat, int datLen) { if(!xidLocks) { xidLocks = lockManagerBeginTransactionUnlocked(xid); } - int currentLockLevel = (int)pblHtLookup(xidLocks, dat, datLen); + long currentLockLevel = (long)pblHtLookup(xidLocks, dat, datLen); // printf("xid %d read lock (%d)\n", xid, currentLockLevel); if(currentLockLevel >= LM_READLOCK) { pthread_mutex_unlock(&xid_table_mutex); @@ -152,7 +152,7 @@ int lockManagerWriteLockHashed(int xid, byte * dat, int datLen) { xidLocks = lockManagerBeginTransactionUnlocked(xid); } - int currentLockLevel = (int)pblHtLookup(xidLocks, dat, datLen); + long currentLockLevel = (long)pblHtLookup(xidLocks, dat, datLen); // printf("xid %d write lock (%d)\n", xid, currentLockLevel); @@ -231,7 +231,7 @@ int lockManagerUnlockHashed(int xid, byte * dat, int datLen) { xidLocks = lockManagerBeginTransactionUnlocked(xid); } - int currentLevel = (int)pblHtLookup(xidLocks, dat, datLen); + long currentLevel = (long)pblHtLookup(xidLocks, dat, datLen); if(currentLevel) { pblHtRemove(xidLocks, dat, datLen); diff --git a/src/lladd/logger/logEntry.c b/src/lladd/logger/logEntry.c index 0c7b95e..2dcd13c 100644 --- a/src/lladd/logger/logEntry.c +++ b/src/lladd/logger/logEntry.c @@ -54,6 +54,7 @@ LogEntry * allocCommonLogEntry(lsn_t prevLSN, int xid, unsigned int type) { ret->prevLSN = prevLSN; ret->xid = xid; ret->type = type; + //printf ("logEntry.c: Log entry length is %ld\n", sizeof(struct __raw_log_entry)); return ret; } diff --git a/src/lladd/logger/logWriter.c b/src/lladd/logger/logWriter.c index 70d0f6f..ac82d40 100644 --- a/src/lladd/logger/logWriter.c +++ b/src/lladd/logger/logWriter.c @@ -185,7 +185,7 @@ int openLogWriter() { global offset for the truncated log. (Not implemented yet) */ lsn_t zero = 0; - int nmemb = fwrite(&zero, sizeof(lsn_t), 1, log); + size_t nmemb = fwrite(&zero, sizeof(lsn_t), 1, log); if(nmemb != 1) { perror("Couldn't start new log file!"); // assert(0); @@ -199,7 +199,7 @@ int openLogWriter() { perror("Could not seek to head of log"); } - int bytesRead = read(roLogFD, &global_offset, sizeof(lsn_t)); + ssize_t bytesRead = read(roLogFD, &global_offset, sizeof(lsn_t)); if(bytesRead != sizeof(lsn_t)) { printf("Could not read log header."); @@ -257,10 +257,12 @@ int writeLogEntry(LogEntry * e) { /* Set the log entry's LSN. */ e->LSN = nextAvailableLSN; + //printf ("\nLSN: %ld\n", e->LSN); + //fflush(stdout); nextAvailableLSN += (size + sizeof(long)); - int nmemb = fwrite(&size, sizeof(long), 1, log); + size_t nmemb = fwrite(&size, sizeof(long), 1, log); if(nmemb != 1) { if(feof(log)) { abort(); /* feof makes no sense here */ } if(ferror(log)) { @@ -280,6 +282,8 @@ int writeLogEntry(LogEntry * e) { } return LLADD_IO_ERROR; } + + //fflush(log); pthread_mutex_unlock(&log_write_mutex); @@ -330,13 +334,13 @@ void closeLogWriter() { void deleteLogWriter() { remove(LOG_FILE); } - +long debug_lsn = -1; static LogEntry * readLogEntry() { LogEntry * ret = 0; long size; long entrySize; - int bytesRead = read(roLogFD, &size, sizeof(long)); + ssize_t bytesRead = read(roLogFD, &size, sizeof(long)); if(bytesRead != sizeof(long)) { if(bytesRead == 0) { @@ -348,13 +352,16 @@ static LogEntry * readLogEntry() { abort(); return (LogEntry*)LLADD_IO_ERROR; } else { - fprintf(stderr, "short read from log. Expected %d bytes, got %d.\nFIXME: This is 'normal', but currently not handled", sizeof(long), bytesRead); + fprintf(stderr, "short read from log. Expected %ld bytes, got %ld.\nFIXME: This is 'normal', but currently not handled", sizeof(long), bytesRead); fflush(stderr); abort(); // really abort here. This code should attempt to piece together short log reads... } } ret = malloc(size); + //printf("Log entry is %ld bytes long.\n", size); + //fflush(stdout); + bytesRead = read(roLogFD, ret, size); if(bytesRead != size) { @@ -367,7 +374,11 @@ static LogEntry * readLogEntry() { abort(); return (LogEntry*)LLADD_IO_ERROR; } else { - printf("short read from log. Expected %ld bytes, got %d.\nFIXME: This is 'normal', but currently not handled", size, bytesRead); + printf("short read from log w/ lsn %ld. Expected %ld bytes, got %ld.\nFIXME: This is 'normal', but currently not handled", debug_lsn, size, bytesRead); + fflush(stderr); + long newSize = size - bytesRead; + long newBytesRead = read (roLogFD, ((byte*)ret)+bytesRead, newSize); + printf("\nattempt to read again produced newBytesRead = %ld, newSize was %ld\n", newBytesRead, newSize); fflush(stderr); abort(); return (LogEntry*)LLADD_IO_ERROR; @@ -393,9 +404,14 @@ LogEntry * readLSNEntry(lsn_t LSN) { pthread_mutex_lock(&log_read_mutex); + assert(global_offset <= LSN); + + + debug_lsn = LSN; off_t newPosition = lseek(roLogFD, LSN - global_offset, SEEK_SET); if(newPosition == -1) { perror("Could not seek for log read"); + abort(); } else { // fprintf(stderr, "sought to %d\n", (int)newPosition); // fflush(stderr); @@ -419,7 +435,7 @@ int truncateLog(lsn_t LSN) { pthread_mutex_lock(&truncateLog_mutex); - if(global_offset + 4 >= LSN) { + if(global_offset + sizeof(lsn_t) >= LSN) { /* Another thread beat us to it...the log is already truncated past the point requested, so just return. */ pthread_mutex_unlock(&truncateLog_mutex); diff --git a/src/lladd/page.h b/src/lladd/page.h index f4acb64..a68a1ce 100644 --- a/src/lladd/page.h +++ b/src/lladd/page.h @@ -93,7 +93,7 @@ terms specified in this license. BEGIN_C_DECLS -#define lsn_ptr(page) (((lsn_t *)(&((page)->memAddr[PAGE_SIZE])))-1) +#define lsn_ptr(page) (((lsn_t*)(&((page)->memAddr[PAGE_SIZE])))-1) #define page_type_ptr(page) (((int*)lsn_ptr((page)))-1) #define end_of_usable_space_ptr(page) page_type_ptr((page)) @@ -127,7 +127,7 @@ struct Page_s { /** @todo Shouldn't Page.id be a long? */ int id; /** @todo The Page.LSN field seems extraneous. Why do we need it? */ - long LSN; + lsn_t LSN; byte *memAddr; byte dirty; /** The next item in the replacement policy's queue */ diff --git a/src/lladd/page/indirect.c b/src/lladd/page/indirect.c index 4af8add..f2b5cf1 100644 --- a/src/lladd/page/indirect.c +++ b/src/lladd/page/indirect.c @@ -11,7 +11,7 @@ void indirectInitialize(Page * p, int height) { *level_ptr(p) = height; *page_type_ptr(p) = INDIRECT_PAGE; - memset(p->memAddr, INVALID_SLOT, ((int)level_ptr(p)) - ((int)p->memAddr)); + memset(p->memAddr, INVALID_SLOT, ((size_t)level_ptr(p)) - ((size_t)p->memAddr)); } /** @todo locking for dereferenceRID? */ compensated_function recordid dereferenceRID(int xid, recordid rid) { diff --git a/src/lladd/page/slotted.c b/src/lladd/page/slotted.c index 2dca8af..66a49cf 100644 --- a/src/lladd/page/slotted.c +++ b/src/lladd/page/slotted.c @@ -26,7 +26,7 @@ void slottedCompact(Page * page) { byte buffer[PAGE_SIZE]; int numSlots; - int meta_size; + size_t meta_size; bufPage.id = -1; bufPage.memAddr = buffer; @@ -36,7 +36,7 @@ void slottedCompact(Page * page) { memset(buffer, -1, PAGE_SIZE); - meta_size = (((int)page->memAddr) + PAGE_SIZE ) - (int)end_of_usable_space_ptr(page); + meta_size = (((size_t)page->memAddr) + PAGE_SIZE ) - (size_t)end_of_usable_space_ptr(page); /* *slot_length_ptr(page, (*numslots_ptr(page))-1);*/ memcpy(buffer + PAGE_SIZE - meta_size, page->memAddr + PAGE_SIZE - meta_size, meta_size); @@ -138,8 +138,8 @@ void slottedPageInitialize(Page * page) { } -int slottedFreespaceUnlocked(Page * page) { - return (int)slot_length_ptr(page, *numslots_ptr(page)) - (int)(page->memAddr + *freespace_ptr(page)); +size_t slottedFreespaceUnlocked(Page * page) { + return ((size_t)slot_length_ptr(page, *numslots_ptr(page)) - (size_t)(page->memAddr + *freespace_ptr(page))) - SLOTTED_PAGE_OVERHEAD_PER_RECORD; } @@ -265,8 +265,11 @@ recordid slottedRawRalloc(Page * page, int size) { /* DEBUG("slot: %d freespace: %d\n", rid.slot, freeSpace); */ + assert(slottedFreespaceUnlocked(page) >= 0); + writeunlock(page->rwlatch); + return rid; } @@ -289,7 +292,7 @@ static void __really_do_ralloc(Page * page, recordid rid) { /* Make sure there's enough free space... */ assert (slottedFreespaceUnlocked(page) >= rid.size); } - + freeSpace = *freespace_ptr(page); @@ -303,6 +306,7 @@ static void __really_do_ralloc(Page * page, recordid rid) { *freespace_ptr(page) = freeSpace + rid.size; *slot_ptr(page, rid.slot) = freeSpace; + /* assert(!*slot_length_ptr(page, rid.slot) || (-1 == *slot_length_ptr(page, rid.slot)));*/ if(isBlob) { *slot_length_ptr(page, rid.slot = BLOB_SLOT); @@ -310,6 +314,8 @@ static void __really_do_ralloc(Page * page, recordid rid) { *slot_length_ptr(page, rid.slot) = rid.size; } + assert(slottedFreespaceUnlocked(page) >= 0); + } recordid slottedPostRalloc(int xid, Page * page, lsn_t lsn, recordid rid) { @@ -366,17 +372,18 @@ recordid slottedPostRalloc(int xid, Page * page, lsn_t lsn, recordid rid) { } void slottedDeRalloc(int xid, Page * page, lsn_t lsn, recordid rid) { - - readlock(page->rwlatch, 443); - + writelock(page->rwlatch, 443); + // readlock(page->rwlatch, 443); + int oldFreeLen = slottedFreespaceUnlocked(page); *slot_ptr(page, rid.slot) = INVALID_SLOT; *slot_length_ptr(page, rid.slot) = *freelist_ptr(page); *freelist_ptr(page) = rid.slot; /* *slot_length_ptr(page, rid.slot) = 0; */ pageWriteLSN(xid, page, lsn); - - readunlock(page->rwlatch); + int newFreeLen = slottedFreespaceUnlocked(page); + assert(oldFreeLen <= newFreeLen && oldFreeLen >= 0); + unlock(page->rwlatch); } @@ -407,9 +414,18 @@ void slottedRead(int xid, Page * page, recordid rid, byte *buff) { readlock(page->rwlatch, 519); assert(page->id == rid.page); + + // DELETE THIS + + int free_space = slottedFreespaceUnlocked(page); + int slot_count = *numslots_ptr(page); + + // END DELETE THIS + slot_length = *slot_length_ptr(page, rid.slot); assert((rid.size == slot_length) || (rid.size == BLOB_SLOT && slot_length == sizeof(blob_record_t))|| (slot_length >= PAGE_SIZE)); + if(!memcpy(buff, record_ptr(page, rid.slot), rid.size)) { perror("memcpy"); abort(); diff --git a/src/lladd/page/slotted.h b/src/lladd/page/slotted.h index 4d1c75f..2087e5b 100644 --- a/src/lladd/page/slotted.h +++ b/src/lladd/page/slotted.h @@ -56,8 +56,8 @@ Slotted page layout: ************************************************************************/ -#define SLOTTED_PAGE_OVERHEAD_PER_RECORD 4 -#define SLOTTED_PAGE_HEADER_OVERHEAD 6 +#define SLOTTED_PAGE_OVERHEAD_PER_RECORD (2 * sizeof(short)) +#define SLOTTED_PAGE_HEADER_OVERHEAD (3 * sizeof(short)) void slottedWrite(int xid, Page * page, lsn_t lsn, recordid rid, const byte *data); void slottedRead(int xid, Page * page, recordid rid, byte *buff); diff --git a/src/pobj/pobj.c b/src/pobj/pobj.c index dc4d998..8ce0e50 100644 --- a/src/pobj/pobj.c +++ b/src/pobj/pobj.c @@ -155,13 +155,13 @@ static int g_is_constructed = 0; int pobj_start (void) { - int active_xid; - int active_nested; + long active_xid; + long active_nested; if (! g_is_init) return -1; - active_xid = (int) pthread_getspecific (g_active_xid_key) - 1; + active_xid = (long) pthread_getspecific (g_active_xid_key) - 1; if (active_xid < 0) { active_xid = Tbegin (); if (active_xid < 0 @@ -173,7 +173,7 @@ pobj_start (void) } } else { - active_nested = (int) pthread_getspecific (g_active_nested_key); + active_nested = (long) pthread_getspecific (g_active_nested_key); active_nested++; if (pthread_setspecific (g_active_nested_key, (void *) active_nested)) return -1; @@ -185,20 +185,20 @@ pobj_start (void) int pobj_end (void) { - int active_xid; - int active_nested; + long active_xid; + long active_nested; if (! g_is_init) return -1; - active_nested = (int) pthread_getspecific (g_active_nested_key); + active_nested = pthread_getspecific (g_active_nested_key); if (active_nested) { active_nested--; if (pthread_setspecific (g_active_nested_key, (void *) active_nested)) return -1; } else { - active_xid = (int) pthread_getspecific (g_active_xid_key) - 1; + active_xid = pthread_getspecific (g_active_xid_key) - 1; if (active_xid >= 0) { if (pthread_setspecific (g_active_xid_key, NULL)) return -1; @@ -855,7 +855,7 @@ pobj_set (void *obj, void *fld, void *data, size_t len, unsigned char flags) if (CHECK_FLAG (flags, POBJ_SET_F_COPY)) memcpy (fld, data, len); else - memset (fld, (int) data, len); + memset (fld, (byte) data, len); /* Update corresponding record (persistent objects only). */ if (p->repo_index >= 0) { diff --git a/test/cht/simple.c b/test/cht/simple.c index 65c23a5..0966916 100644 --- a/test/cht/simple.c +++ b/test/cht/simple.c @@ -158,7 +158,7 @@ int main (int argc, char**argv) { /* int j = (i % 8) + 1;*/ int j = i; if(!cHtLookup(xid, dfaSet, &ht, &j, sizeof(int), buf, &buflen)) { printf ("lookup failed!"); } - printf(" looked up !! key %d -> %d: %s %d\n", i, j, buf, buflen); + printf(" looked up !! key %d -> %d: %s %ld\n", i, j, buf, buflen); } cHtDelete(xid, dfaSet, &ht); diff --git a/test/dfa/star.c b/test/dfa/star.c index 7b0de7a..22d0d86 100644 --- a/test/dfa/star.c +++ b/test/dfa/star.c @@ -127,15 +127,15 @@ int main (int argc, char ** argv) { broadcast_lists_count, broadcast_list_host_count); if(list_number == 0) { - int ret; + long ret; dfa_reinitialize(dfaSet, broadcast_lists[list_number][node_number], transitions_star, 2, states_star, 2); spawn_main_thread(dfaSet); - ret =(int) request(dfaSet, HUB_START, "bc:1", NULL_MACHINE, NULL); + ret =(long) request(dfaSet, HUB_START, "bc:1", NULL_MACHINE, NULL); - printf("run_request_returned: %x\n", ret); + printf("run_request_returned: %lx\n", ret); } else { diff --git a/test/lladd/Makefile.am b/test/lladd/Makefile.am index 5ed432e..4de0e6b 100644 --- a/test/lladd/Makefile.am +++ b/test/lladd/Makefile.am @@ -9,4 +9,4 @@ endif noinst_PROGRAMS = $(TESTS) LDADD = $(top_builddir)/src/lladd/liblladd.a $(top_builddir)/src/pbl/libpbl.a $(top_builddir)/src/libdfa/librw.a #-lefence CLEANFILES = check_lht.log check_logEntry.log storefile.txt logfile.txt blob0_file.txt blob1_file.txt check_blobRecovery.log check_logWriter.log check_operations.log check_recovery.log check_transactional2.log check_page.log check_bufferManager.log check_indirect.log check_bufferMananger.log check_lladdhash.log check_pageOperations.log check_linearhash.log check_linkedListNTA.log check_linearHashNTA.log check_pageOrientedListNTA.log check_lockManager.log check_compensations.log check_errorhandling.log check_header.logg check_iterator.log check_linearHash.log check_ringbuffer.log check_bTree.log -AM_CFLAGS= -g -Wall -pedantic -std=gnu99 \ No newline at end of file +AM_CFLAGS= -g -Wall -pedantic -std=gnu99 diff --git a/test/lladd/check_logWriter.c b/test/lladd/check_logWriter.c index 37eb98a..c76f3e1 100644 --- a/test/lladd/check_logWriter.c +++ b/test/lladd/check_logWriter.c @@ -245,8 +245,8 @@ START_TEST(logWriterTruncate) { pthread_mutex_t random_mutex; static void* worker_thread(void * arg) { - int key = *(int*)arg; - int i = 0; + long key = *(int*)arg; + long i = 0; int truncated_to = 4; LogEntry * le = allocCommonLogEntry(-1, -1, XBEGIN); @@ -258,12 +258,21 @@ static void* worker_thread(void * arg) { while(i < ENTRIES_PER_THREAD) { int threshold; - int entry; - + long entry; + int needToTruncate = 0; pthread_mutex_lock(&random_mutex); threshold = (int) (2000.0*random()/(RAND_MAX+1.0)); - entry = (int) (ENTRIES_PER_THREAD*random()/(RAND_MAX+1.0)); + entry = (long) (ENTRIES_PER_THREAD*random()/(RAND_MAX+1.0)); + + if(threshold < 3) { + if(i > 10) { + needToTruncate = 1; + if(lsns[i - 10] > truncated_to) { + truncated_to = lsns[i - 10]; + } + } + } pthread_mutex_unlock(&random_mutex); @@ -274,8 +283,9 @@ static void* worker_thread(void * arg) { /* Truncate the log .15% of the time; result in a bit over 100 truncates per test run.*/ /* fail_unless(1, NULL); */ - truncateLog(lsns[i - 10]); - truncated_to = i - 10; + /*truncateLog(lsns[i - 10]);*/ + + //truncated_to = i - 10; } /* fail_unless(1, NULL); */ } else { @@ -284,14 +294,19 @@ static void* worker_thread(void * arg) { /* fail_unless(1, NULL); */ le->xid = i+key; writeLogEntry(le); + //printf("reportedLSN: %ld\n", le->LSN); lsns[i] = le->LSN; i++; } /* fail_unless(1, NULL); */ - if(entry > truncated_to && entry < i) { - /* printf("X %d\n", (readLSNEntry(lsns[entry])->xid == entry+key)); fflush(stdout); */ + pthread_mutex_lock(&random_mutex); + if(lsns[entry] > truncated_to && entry < i) { + pthread_mutex_unlock(&random_mutex); + /*printf("X %d\n", (readLSNEntry(lsns[entry])->xid == entry+key)); fflush(stdout); */ assert(readLSNEntry(lsns[entry])->xid == entry+key); /* fail_unless(readLSNEntry(lsns[entry])->xid == entry+key, NULL); */ + } else { + pthread_mutex_unlock(&random_mutex); } /* fail_unless(1, NULL); */ @@ -347,11 +362,11 @@ Suite * check_suite(void) { tcase_set_timeout(tc, 0); /* Sub tests are added, one per line, here */ - tcase_add_test(tc, logWriterTest); - tcase_add_test(tc, logHandleColdReverseIterator); - tcase_add_test(tc, logWriterTruncate); - tcase_add_test(tc, logWriterCheckWorker); - tcase_add_test(tc, logWriterCheckThreaded); + /* tcase_add_test(tc, logWriterTest);*/ + /*tcase_add_test(tc, logHandleColdReverseIterator);*/ + /*tcase_add_test(tc, logWriterTruncate);*/ + /*tcase_add_test(tc, logWriterCheckWorker); */ + tcase_add_test(tc, logWriterCheckThreaded); /* --------------------------------------------- */ diff --git a/test/lladd/check_page.c b/test/lladd/check_page.c index fd58f57..815f9fe 100644 --- a/test/lladd/check_page.c +++ b/test/lladd/check_page.c @@ -152,9 +152,11 @@ static void* worker_thread(void * arg_ptr) { recordid rid; for(i = 0; i < 10000; i++) { pthread_mutex_lock(&lsn_mutex); + this_lsn = lsn; lsn++; - pthread_mutex_unlock(&lsn_mutex); + + pthread_mutex_unlock(&lsn_mutex); if(! first ) { readRecord(1, p, rid, (byte*)&j); @@ -164,14 +166,23 @@ static void* worker_thread(void * arg_ptr) { } first = 0; + + // TODO A condition variable would be more efficient... - rid = slottedRawRalloc(p, sizeof(int)); - writeRecord(-1, p, lsn, rid, (byte*)&i); + pthread_mutex_lock(&lsn_mutex); + if(slottedFreespace(p) < sizeof(int)) { + first = 1; + pthread_mutex_unlock(&lsn_mutex); + } else { + rid = slottedRawRalloc(p, sizeof(int)); + pthread_mutex_unlock(&lsn_mutex); + writeRecord(-1, p, lsn, rid, (byte*)&i); + } sched_yield(); assert(pageReadLSN(p) <= lsn); } - + return NULL; } diff --git a/test/lladd/check_pageOperations.c b/test/lladd/check_pageOperations.c index 94663a5..5a49795 100644 --- a/test/lladd/check_pageOperations.c +++ b/test/lladd/check_pageOperations.c @@ -106,15 +106,15 @@ START_TEST(pageOpCheckRecovery) { memset(p.memAddr, 1, PAGE_SIZE); TpageGet(xid, pageid1, newAddr); - assert(!memcmp(p.memAddr, newAddr, PAGE_SIZE-4)); + assert(!memcmp(p.memAddr, newAddr, PAGE_SIZE-sizeof(lsn_t))); memset(p.memAddr, 2, PAGE_SIZE); TpageGet(xid, pageid2, newAddr); - assert(!memcmp(p.memAddr, newAddr, PAGE_SIZE-4)); + assert(!memcmp(p.memAddr, newAddr, PAGE_SIZE-sizeof(lsn_t))); memset(p.memAddr, 3, PAGE_SIZE); TpageGet(xid, pageid3, newAddr); - assert(!memcmp(p.memAddr, newAddr, PAGE_SIZE-4)); + assert(!memcmp(p.memAddr, newAddr, PAGE_SIZE-sizeof(lsn_t))); Tcommit(xid); Tdeinit();