Attachment 156089 Details for Bug 241780 – Test patch for the alarm deadlock problem

[patch] Test patch for the alarm deadlock problem

autofs-5.0.1-alarm-deadlock.patch (text/plain), 1.39 KB, created by Ian Kent on 2007-06-04 15:58:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Ian Kent

Created: 2007-06-04 15:58:08 UTC

Size: 1.39 KB

patch

obsolete

>diff --git a/lib/alarm.c b/lib/alarm.c
>index e0e6fd4..4a72605 100755
>--- a/lib/alarm.c
>+++ b/lib/alarm.c
>@@ -168,7 +168,6 @@ void alarm_delete(struct autofs_point *ap)
> static void *alarm_handler(void *arg)
> {
> 	struct list_head *head;
>-	struct autofs_point *ap;
> 	struct timespec expire;
> 	time_t now;
> 	int status;
>@@ -192,10 +191,11 @@ static void *alarm_handler(void *arg)
> 
> 		current = list_entry(head->next, struct alarm, list);
> 
>-		ap = current->ap;
> 		now = time(NULL);
> 
> 		if (current->time <= now) {
>+			struct autofs_point *ap;
>+
> 			list_del(&current->list);
> 
> 			if (current->cancel) {
>@@ -203,11 +203,15 @@ static void *alarm_handler(void *arg)
> 				continue;
> 			}
> 
>+			ap = current->ap;
>+			free(current);
>+			alarm_unlock();
>+
> 			state_mutex_lock(ap);
> 			nextstate(ap->state_pipe[1], ST_EXPIRE);
> 			state_mutex_unlock(ap);
> 
>-			free(current);
>+			alarm_lock();
> 			continue;
> 		}
> 
>@@ -215,6 +219,7 @@ static void *alarm_handler(void *arg)
> 		expire.tv_nsec = 0;
> 
> 		while (1) {
>+			struct autofs_point *ap;
> 			struct alarm *next;
> 
> 			status = pthread_cond_timedwait(&cond, &mutex, &expire);
>@@ -232,12 +237,15 @@ static void *alarm_handler(void *arg)
> 				break;
> 
> 			list_del(&current->list);
>+			ap = current->ap;
> 			free(current);
>+			alarm_unlock();
> 
> 			state_mutex_lock(ap);
> 			nextstate(ap->state_pipe[1], ST_EXPIRE);
> 			state_mutex_unlock(ap);
> 
>+			alarm_lock();
> 			break;
> 		}
> 	}

Actions: View | Diff

Attachments on bug 241780: 155705 | 155783 | 155790 | 155792 | 156053 | 156069 | 156089 | 156232 | 156238 | 156240 | 156433 | 156437