about summary refs log tree commit diff
diff options
context:
space:
mode:
authorUlrich Drepper <drepper@redhat.com>2006-05-29 07:00:16 +0000
committerUlrich Drepper <drepper@redhat.com>2006-05-29 07:00:16 +0000
commit9636a217e4a0a73113ffdaa9a2f0c8d1749cf1d9 (patch)
treeeb310947247d941a4c71ca880fd1a9a132c8c4a0
parentab50b2ec0193ad37c73d34c0e98ff16486af237e (diff)
downloadglibc-9636a217e4a0a73113ffdaa9a2f0c8d1749cf1d9.tar.gz
glibc-9636a217e4a0a73113ffdaa9a2f0c8d1749cf1d9.tar.xz
glibc-9636a217e4a0a73113ffdaa9a2f0c8d1749cf1d9.zip
* nscd/nscd.h (struct database_dyn): Add prunelock field.
	* nscd/cache.c (prune_cache): Take prunelock before starting the
	work.  Just return in case it is already taken.

	(inet6_option_alloc): Now a simple wrapper around option_alloc.
-rw-r--r--ChangeLog8
-rw-r--r--nscd/cache.c11
-rw-r--r--nscd/nscd.h1
3 files changed, 18 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index d01eb346c9..0f60f1530e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-05-29  Ulrich Drepper  <drepper@redhat.com>
+
+	* nscd/nscd.h (struct database_dyn): Add prunelock field.
+	* nscd/cache.c (prune_cache): Take prunelock before starting the
+	work.  Just return in case it is already taken.
+
 2006-05-25  Ulrich Drepper  <drepper@redhat.com>
 
 	* nis/nis_removemember.c (nis_removemember): Avoid unnecessary
@@ -12,7 +18,7 @@
 	inet6_option_alloc.
 	* inet/inet6_option.c (option_alloc): Renamed from
 	inet6_option_alloc.  Made static.
-	(inet6_option_alloc): New a simple wrapper around option_alloc.
+	(inet6_option_alloc): Now a simple wrapper around option_alloc.
 	(inet6_option_append): Call option_alloc.
 	* include/netinet/in.h: Remove libc_hidden_proto for
 	inet6_option_alloc.
diff --git a/nscd/cache.c b/nscd/cache.c
index 787f8b46f5..899e2caf77 100644
--- a/nscd/cache.c
+++ b/nscd/cache.c
@@ -1,4 +1,4 @@
-/* Copyright (c) 1998, 1999, 2003, 2004, 2005 Free Software Foundation, Inc.
+/* Copyright (c) 1998, 1999, 2003-2005, 2006 Free Software Foundation, Inc.
    This file is part of the GNU C Library.
    Contributed by Ulrich Drepper <drepper@cygnus.com>, 1998.
 
@@ -198,6 +198,13 @@ prune_cache (struct database_dyn *table, time_t now)
   if (cnt == 0)
     return;
 
+  /* This function can be called from the cleanup thread but also in
+     response to an invalidate command.  Make sure only one thread is
+     running.  No need for the second to wait around.  */
+  if (pthread_mutex_trylock (&table->prunelock) != 0)
+    /* Te work is already being done.  */
+    return ;
+
   /* If we check for the modification of the underlying file we invalidate
      the entries also in this case.  */
   if (table->check_file)
@@ -455,4 +462,6 @@ prune_cache (struct database_dyn *table, time_t now)
   /* Run garbage collection if any entry has been removed or replaced.  */
   if (any)
     gc (table);
+
+  pthread_mutex_unlock (&table->prunelock);
 }
diff --git a/nscd/nscd.h b/nscd/nscd.h
index 8b95630807..4a9547221a 100644
--- a/nscd/nscd.h
+++ b/nscd/nscd.h
@@ -58,6 +58,7 @@ typedef enum
 struct database_dyn
 {
   pthread_rwlock_t lock;
+  pthread_mutex_t prunelock;
 
   int enabled;
   int check_file;