From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6708 invoked by alias); 6 Apr 2010 15:29:35 -0000 Received: (qmail 6694 invoked by uid 9452); 6 Apr 2010 15:29:31 -0000 Date: Tue, 06 Apr 2010 15:29:00 -0000 Message-ID: <20100406152931.6692.qmail@sourceware.org> From: ccaulfield@sourceware.org To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org Subject: LVM2 ./WHATS_NEW daemons/clvmd/clvmd.c Mailing-List: contact lvm2-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: lvm2-cvs-owner@sourceware.org X-SW-Source: 2010-04/txt/msg00022.txt.bz2 CVSROOT: /cvs/lvm2 Module name: LVM2 Changes by: ccaulfield@sourceware.org 2010-04-06 15:29:31 Modified files: . : WHATS_NEW daemons/clvmd : clvmd.c Log message: Fix a thread race in clvmd that could cause lockups on very busy systems Patches: http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1502&r2=1.1503 http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/daemons/clvmd/clvmd.c.diff?cvsroot=lvm2&r1=1.64&r2=1.65 --- LVM2/WHATS_NEW 2010/04/06 14:07:11 1.1502 +++ LVM2/WHATS_NEW 2010/04/06 15:29:30 1.1503 @@ -1,5 +1,6 @@ Version 2.02.63 - ================================ + Fix another thread race in clvmd. Refactor management of vg->pvs list. Fix lcov rules and generate better coverage report. Improve vg_validate to detect some loops in lists. --- LVM2/daemons/clvmd/clvmd.c 2010/03/18 09:19:31 1.64 +++ LVM2/daemons/clvmd/clvmd.c 2010/04/06 15:29:30 1.65 @@ -1516,7 +1516,8 @@ DEBUGLOG("Waiting to do post command - state = %d\n", client->bits.localsock.state); - if (client->bits.localsock.state != POST_COMMAND) { + if (client->bits.localsock.state != POST_COMMAND && + !client->bits.localsock.finished) { pthread_cond_wait(&client->bits.localsock.cond, &client->bits.localsock.mutex); }