public inbox for lvm2-cvs@sourceware.org
help / color / mirror / Atom feed
* LVM2 daemons/clvmd/clvmd.c ./WHATS_NEW
@ 2010-12-22 12:10 zkabelac
0 siblings, 0 replies; only message in thread
From: zkabelac @ 2010-12-22 12:10 UTC (permalink / raw)
To: lvm-devel, lvm2-cvs
CVSROOT: /cvs/lvm2
Module name: LVM2
Changes by: zkabelac@sourceware.org 2010-12-22 12:10:57
Modified files:
daemons/clvmd : clvmd.c
. : WHATS_NEW
Log message:
Log error state from pthread_join operation
Value jstat is unused - so replace it with logging via log_sys_error().
Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/daemons/clvmd/clvmd.c.diff?cvsroot=lvm2&r1=1.89&r2=1.90
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/WHATS_NEW.diff?cvsroot=lvm2&r1=1.1850&r2=1.1851
--- LVM2/daemons/clvmd/clvmd.c 2010/12/20 13:48:28 1.89
+++ LVM2/daemons/clvmd/clvmd.c 2010/12/22 12:10:56 1.90
@@ -666,7 +666,6 @@
/* EOF on pipe or an error, close it */
if (len <= 0) {
- int jstat;
void *ret = &status;
close(thisfd->fd);
@@ -677,7 +676,9 @@
/* Reap child thread */
if (thisfd->bits.pipe.threadid) {
- jstat = pthread_join(thisfd->bits.pipe.threadid, &ret);
+ if (pthread_join(thisfd->bits.pipe.threadid, &ret))
+ log_sys_error("pthread_join", "");
+
thisfd->bits.pipe.threadid = 0;
if (thisfd->bits.pipe.client != NULL)
thisfd->bits.pipe.client->bits.localsock.
@@ -1041,7 +1042,6 @@
/* EOF or error on socket */
if (len <= 0) {
int *status;
- int jstat;
DEBUGLOG("EOF on local socket: inprogress=%d\n",
thisfd->bits.localsock.in_progress);
@@ -1068,9 +1068,10 @@
pthread_cond_signal(&thisfd->bits.localsock.cond);
pthread_mutex_unlock(&thisfd->bits.localsock.mutex);
- jstat =
- pthread_join(thisfd->bits.localsock.threadid,
- (void **) &status);
+ if (pthread_join(thisfd->bits.localsock.threadid,
+ (void **) &status))
+ log_sys_error("pthread_join", "");
+
DEBUGLOG("Joined child thread\n");
thisfd->bits.localsock.threadid = 0;
--- LVM2/WHATS_NEW 2010/12/21 21:08:51 1.1850
+++ LVM2/WHATS_NEW 2010/12/22 12:10:56 1.1851
@@ -1,5 +1,6 @@
Version 2.02.80 -
====================================
+ Log error message for pthread_join() failure in clvmd.
Version 2.02.79 - 20th December 2010
====================================
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2010-12-22 12:10 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-12-22 12:10 LVM2 daemons/clvmd/clvmd.c ./WHATS_NEW zkabelac
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).