datarace: simplify raceCheck{Read,Write}() function interfaces

[c11tester.git] / librace.cc

diff --git a/librace.cc b/librace.cc
index 5e775f3f1b471a21e5f473d6e274a481af62d59c..2c36054bad8000fe4231ce822a215f291163d5fb 100644 (file)
--- a/librace.cc
+++ b/librace.cc
@@ -11,8 +11,7 @@ void store_8(void *addr, uint8_t val)
 {
        DEBUG("addr = %p, val = %" PRIu8 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p, val = %" PRIu8 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckWrite(tid, addr, cv);
+       raceCheckWrite(tid, addr);

        (*(uint8_t *)addr) = val;
 }
 
        (*(uint8_t *)addr) = val;
 }
 


@@ -20,9 +19,8 @@ void store_16(void *addr, uint16_t val)
 {
        DEBUG("addr = %p, val = %" PRIu16 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p, val = %" PRIu16 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckWrite(tid, addr, cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 1), cv);
+       raceCheckWrite(tid, addr);
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 1));

        (*(uint16_t *)addr) = val;
 }
 
        (*(uint16_t *)addr) = val;
 }
 


@@ -30,11 +28,10 @@ void store_32(void *addr, uint32_t val)
 {
        DEBUG("addr = %p, val = %" PRIu32 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p, val = %" PRIu32 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckWrite(tid, addr, cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 1), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 2), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 3), cv);
+       raceCheckWrite(tid, addr);
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 1));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 2));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 3));

        (*(uint32_t *)addr) = val;
 }
 
        (*(uint32_t *)addr) = val;
 }
 


@@ -42,15 +39,14 @@ void store_64(void *addr, uint64_t val)
 {
        DEBUG("addr = %p, val = %" PRIu64 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p, val = %" PRIu64 "\n", addr, val);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckWrite(tid, addr, cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 1), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 2), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 3), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 4), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 5), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 6), cv);
-       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 7), cv);
+       raceCheckWrite(tid, addr);
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 1));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 2));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 3));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 4));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 5));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 6));
+       raceCheckWrite(tid, (void *)(((uintptr_t)addr) + 7));

        (*(uint64_t *)addr) = val;
 }
 
        (*(uint64_t *)addr) = val;
 }
 


@@ -58,8 +54,7 @@ uint8_t load_8(const void *addr)
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckRead(tid, addr, cv);
+       raceCheckRead(tid, addr);

        return *((uint8_t *)addr);
 }
 
        return *((uint8_t *)addr);
 }
 


@@ -67,9 +62,8 @@ uint16_t load_16(const void *addr)
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckRead(tid, addr, cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 1), cv);
+       raceCheckRead(tid, addr);
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 1));

        return *((uint16_t *)addr);
 }
 
        return *((uint16_t *)addr);
 }
 


@@ -77,11 +71,10 @@ uint32_t load_32(const void *addr)
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckRead(tid, addr, cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 1), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 2), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 3), cv);
+       raceCheckRead(tid, addr);
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 1));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 2));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 3));

        return *((uint32_t *)addr);
 }
 
        return *((uint32_t *)addr);
 }
 


@@ -89,14 +82,13 @@ uint64_t load_64(const void *addr)
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();
 {
        DEBUG("addr = %p\n", addr);
        thread_id_t tid = thread_current()->get_id();

-       ClockVector *cv = model->get_cv(tid);
-       raceCheckRead(tid, addr, cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 1), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 2), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 3), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 4), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 5), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 6), cv);
-       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 7), cv);
+       raceCheckRead(tid, addr);
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 1));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 2));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 3));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 4));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 5));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 6));
+       raceCheckRead(tid, (const void *)(((uintptr_t)addr) + 7));

        return *((uint64_t *)addr);
 }
        return *((uint64_t *)addr);
 }