FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Redhat > Cluster Development

 
 
LinkBack Thread Tools
 
Old 11-21-2007, 04:49 PM
 
Default [Cluster-devel] cluster/group/gfs_controld lock_dlm.h

CVSROOT: /cvs/cluster
Module name: cluster
Changes by: teigland@sourceware.org 2007-11-21 17:49:16

Modified files:
group/gfs_controld: lock_dlm.h

Log message:
ASSERT was doing fprintf(stderr) which goes somewhere we don't want
when running as a daemon.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/group/gfs_controld/lock_dlm.h.diff?cvsroot=cluster&r1=1.29&r2=1.30

--- cluster/group/gfs_controld/lock_dlm.h 2007/09/04 19:22:52 1.29
+++ cluster/group/gfs_controld/lock_dlm.h 2007/11/21 17:49:16 1.30
@@ -105,13 +105,12 @@
} while (0)

#define ASSERT(x)
-{
+do {
if (!(x)) {
- fprintf(stderr, "
Assertion failed on line %d of file %s

"
- "Assertion: "%s"
",
- __LINE__, __FILE__, #x);
- }
-}
+ log_error("Assertion failed on line %d of file %s
"
+ "Assertion: "%s"
", __LINE__, __FILE__, #x);
+ }
+} while (0)

struct mountpoint {
struct list_head list;
 
Old 11-21-2007, 04:50 PM
 
Default [Cluster-devel] cluster/group/gfs_controld lock_dlm.h

CVSROOT: /cvs/cluster
Module name: cluster
Branch: RHEL5
Changes by: teigland@sourceware.org 2007-11-21 17:50:16

Modified files:
group/gfs_controld: lock_dlm.h

Log message:
ASSERT was doing fprintf(stderr) which goes somewhere we don't want
when running as a daemon.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/group/gfs_controld/lock_dlm.h.diff?cvsroot=cluster&only_with_tag=RHEL 5&r1=1.21.2.8&r2=1.21.2.9

--- cluster/group/gfs_controld/lock_dlm.h 2007/09/04 19:27:34 1.21.2.8
+++ cluster/group/gfs_controld/lock_dlm.h 2007/11/21 17:50:15 1.21.2.9
@@ -105,13 +105,12 @@
} while (0)

#define ASSERT(x)
-{
+do {
if (!(x)) {
- fprintf(stderr, "
Assertion failed on line %d of file %s

"
- "Assertion: "%s"
",
- __LINE__, __FILE__, #x);
- }
-}
+ log_error("Assertion failed on line %d of file %s
"
+ "Assertion: "%s"
", __LINE__, __FILE__, #x);
+ }
+} while (0)

struct mountpoint {
struct list_head list;
 
Old 11-21-2007, 05:19 PM
David Teigland
 
Default [Cluster-devel] cluster/group/gfs_controld lock_dlm.h

On Wed, Nov 21, 2007 at 05:50:16PM -0000, teigland@sourceware.org wrote:
> CVSROOT: /cvs/cluster
> Module name: cluster
> Branch: RHEL5
> Changes by: teigland@sourceware.org 2007-11-21 17:50:16
>
> Modified files:
> group/gfs_controld: lock_dlm.h
>
> Log message:
> ASSERT was doing fprintf(stderr) which goes somewhere we don't want
> when running as a daemon.

created bz 394721 for this
 

Thread Tools




All times are GMT. The time now is 09:40 AM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org