Bug #529. On Unix, combine pthreed locks with file locking because file

[idzebra-moved-to-github.git] / util / tstflock.c

diff --git a/util/tstflock.c b/util/tstflock.c
index ec26477..cf2039b 100644 (file)
--- a/util/tstflock.c
+++ b/util/tstflock.c
@@ -1,10 +1,11 @@
 /*
- * Copyright (C) 1995-2005, Index Data ApS
+ * Copyright (C) 1995-2006, Index Data ApS
  * See the file LICENSE for details.
  *
- * $Id: tstflock.c,v 1.6 2006-05-10 08:13:46 adam Exp $
+ * $Id: tstflock.c,v 1.7 2006-06-27 11:56:29 adam Exp $
  */
 
+#include <assert.h>
 #include <yaz/test.h>
 #if YAZ_POSIX_THREADS
 #include <pthread.h>
@@ -20,10 +21,10 @@
 #endif
 #include <string.h>
 
-static char seq[20];
+static char seq[40];
 static char *seqp = seq;
 
-#define NUM_THREADS 2
+#define NUM_THREADS 5
 
 static void small_sleep()
 {
@@ -33,21 +34,38 @@ static void small_sleep()
     sleep(1);
 #endif
 }
+
 void *run_func(void *arg)
 {
     int i;
+    int *pdata = (int*) arg;
+    int use_write_lock = *pdata;
     ZebraLockHandle lh = zebra_lock_create(0, "my.LCK");
     for (i = 0; i<2; i++)
     {
-        zebra_lock_w(lh);
-     
-        *seqp++ = 'L';
-        small_sleep();
-        *seqp++ = 'U';
-        
-        zebra_unlock(lh);
+        if (use_write_lock)
+        {
+            zebra_lock_w(lh);
+            
+            *seqp++ = 'L';
+            small_sleep();
+            *seqp++ = 'U';
+            
+            zebra_unlock(lh);
+        }
+        else
+        {
+            zebra_lock_r(lh);
+            
+            *seqp++ = 'l';
+            small_sleep();
+            *seqp++ = 'u';
+            
+            zebra_unlock(lh);
+        }
     }
     zebra_lock_destroy(lh);
+    *pdata = 123;
     return 0;
 }
 
@@ -58,15 +76,17 @@ DWORD WINAPI ThreadProc(void *p)
     return 0;
 }
 
-static void tst_win32()
+static void tst_win32(int num)
 {
     HANDLE handles[NUM_THREADS];
     DWORD dwThreadId[NUM_THREADS];
     int i, id[NUM_THREADS];
     
-    for (i = 0; i<NUM_THREADS; i++)
+    assert (num <= NUM_THREADS);
+    for (i = 0; i<num; i++)
     {
         void *pData = &id[i];
+        id[i] = i >= 2 ? 0 : 1; /* first two are writing.. rest is reading */
         handles[i] = CreateThread(
             NULL,              /* default security attributes */
             0,                 /* use default stack size */
@@ -81,15 +101,23 @@ static void tst_win32()
 #endif
 
 #if YAZ_POSIX_THREADS
-static void tst_pthread()
+static void tst_pthread(int num)
 {
     pthread_t child_thread[NUM_THREADS];
     int i, id[NUM_THREADS];
-    for (i = 0; i<NUM_THREADS; i++)
+
+    assert (num <= NUM_THREADS);
+    for (i = 0; i<num; i++)
+    {
+        id[i] = i >= 2 ? 0 : 1; /* first two are writing.. rest is reading */
         pthread_create(&child_thread[i], 0 /* attr */, run_func, &id[i]);
+    }
 
-    for (i = 0; i<NUM_THREADS; i++)
+    for (i = 0; i<num; i++)
         pthread_join(child_thread[i], 0);
+
+    for (i = 0; i<num; i++)
+        YAZ_CHECK(id[i] == 123);
 }
 #endif
 
@@ -97,18 +125,21 @@ int main(int argc, char **argv)
 {
     YAZ_CHECK_INIT(argc, argv);
 
+    zebra_flock_init();
 #ifdef WIN32
-    tst_win32();
+    tst_win32(2);
 #endif
 #if YAZ_POSIX_THREADS
-    tst_pthread();
+    tst_pthread(2);
 #endif
 
     *seqp++ = '\0';
-    printf("seq=%s\n", seq);
 #if 0
+    printf("seq=%s\n", seq);
+#endif
+#if 1
     /* does not pass.. for bug 529 */
-    YAZ_CHECK(strcmp(seq, "LULULULU") == 0);
+    YAZ_CHECK(strcmp(seq, "LULULULU") == 0); /* for tst_pthread when num=2 */
 #endif
     YAZ_CHECK_TERM;
 }