summaryrefslogtreecommitdiffstats
path: root/fs/ocfs2
diff options
context:
space:
mode:
authorMilton Miller <miltonm@bga.com>2010-11-17 22:20:11 -0600
committerJoel Becker <joel.becker@oracle.com>2010-11-18 14:10:56 -0800
commita2a2f55291918f6cf9287d7beaecc7bc007a9f1c (patch)
tree8fd82a27d2a90e302edcb0ce443b12913615cab2 /fs/ocfs2
parent1989a80a60d2f620bad99196d6c1801c2afd7c71 (diff)
downloadtalos-op-linux-a2a2f55291918f6cf9287d7beaecc7bc007a9f1c.tar.gz
talos-op-linux-a2a2f55291918f6cf9287d7beaecc7bc007a9f1c.zip
ocfs2: char is not always signed
Commit 1c66b360fe262 (Change some lock status member in ocfs2_lock_res to char.) states that these fields need to be signed due to comparision to -1, but only changed the type from unsigned char to char. However, it is a compiler option if char is a signed or unsigned type. Change these fields to signed char so the code will work with all compilers. Signed-off-by: Milton Miller <miltonm@bga.com> Signed-off-by: Joel Becker <joel.becker@oracle.com>
Diffstat (limited to 'fs/ocfs2')
-rw-r--r--fs/ocfs2/ocfs2.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/ocfs2/ocfs2.h b/fs/ocfs2/ocfs2.h
index 1efea3615589..70dd3b1798f1 100644
--- a/fs/ocfs2/ocfs2.h
+++ b/fs/ocfs2/ocfs2.h
@@ -159,9 +159,9 @@ struct ocfs2_lock_res {
char l_name[OCFS2_LOCK_ID_MAX_LEN];
unsigned int l_ro_holders;
unsigned int l_ex_holders;
- char l_level;
- char l_requested;
- char l_blocking;
+ signed char l_level;
+ signed char l_requested;
+ signed char l_blocking;
/* Data packed - type enum ocfs2_lock_type */
unsigned char l_type;
OpenPOWER on IntegriCloud