aboutsummaryrefslogtreecommitdiffstats
path: root/fs/hfsplus
diff options
context:
space:
mode:
authorGeert Uytterhoeven <geert@linux-m68k.org>2013-11-14 17:32:18 -0500
committerLinus Torvalds <torvalds@linux-foundation.org>2013-11-14 19:32:23 -0500
commita99b7069aab8fc3fb4f26d15795dc280b52e38b1 (patch)
treeaa9d2dde6d72e7c5dc1e8ba58cc7da9561f83b55 /fs/hfsplus
parent498d319bb512992ef0784c278fa03679f2f5649d (diff)
hfsplus: Fix undefined __divdi3 in hfsplus_init_header_node()
ERROR: "__divdi3" [fs/hfsplus/hfsplus.ko] undefined! Introduced by commit 099e9245e04d ("hfsplus: implement attributes file's header node initialization code"). i_size_read() returns loff_t, which is long long, i.e. 64-bit. node_size is size_t, which is either 32-bit or 64-bit. Hence "i_size_read(attr_file) / node_size" is a 64-by-32 or 64-by-64 division, causing (some versions of) gcc to emit a call to __divdi3(). Fortunately node_size is actually 16-bit, as the sole caller of hfsplus_init_header_node() passes a u16. Hence change its type from size_t to u16, and use do_div() to perform a 64-by-32 division. Not seen in m68k/allmodconfig in -next, so it really depends on the verion of gcc. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Cc: Vyacheslav Dubeyko <slava@dubeyko.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/hfsplus')
-rw-r--r--fs/hfsplus/xattr.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/fs/hfsplus/xattr.c b/fs/hfsplus/xattr.c
index efc85b1377cc..3c6136f98c73 100644
--- a/fs/hfsplus/xattr.c
+++ b/fs/hfsplus/xattr.c
@@ -129,7 +129,7 @@ static int can_set_xattr(struct inode *inode, const char *name,
129 129
130static void hfsplus_init_header_node(struct inode *attr_file, 130static void hfsplus_init_header_node(struct inode *attr_file,
131 u32 clump_size, 131 u32 clump_size,
132 char *buf, size_t node_size) 132 char *buf, u16 node_size)
133{ 133{
134 struct hfs_bnode_desc *desc; 134 struct hfs_bnode_desc *desc;
135 struct hfs_btree_header_rec *head; 135 struct hfs_btree_header_rec *head;
@@ -139,8 +139,9 @@ static void hfsplus_init_header_node(struct inode *attr_file,
139 char *bmp; 139 char *bmp;
140 u32 used_nodes; 140 u32 used_nodes;
141 u32 used_bmp_bytes; 141 u32 used_bmp_bytes;
142 loff_t tmp;
142 143
143 hfs_dbg(ATTR_MOD, "init_hdr_attr_file: clump %u, node_size %zu\n", 144 hfs_dbg(ATTR_MOD, "init_hdr_attr_file: clump %u, node_size %u\n",
144 clump_size, node_size); 145 clump_size, node_size);
145 146
146 /* The end of the node contains list of record offsets */ 147 /* The end of the node contains list of record offsets */
@@ -154,7 +155,9 @@ static void hfsplus_init_header_node(struct inode *attr_file,
154 155
155 head = (struct hfs_btree_header_rec *)(buf + offset); 156 head = (struct hfs_btree_header_rec *)(buf + offset);
156 head->node_size = cpu_to_be16(node_size); 157 head->node_size = cpu_to_be16(node_size);
157 head->node_count = cpu_to_be32(i_size_read(attr_file) / node_size); 158 tmp = i_size_read(attr_file);
159 do_div(tmp, node_size);
160 head->node_count = cpu_to_be32(tmp);
158 head->free_nodes = cpu_to_be32(be32_to_cpu(head->node_count) - 1); 161 head->free_nodes = cpu_to_be32(be32_to_cpu(head->node_count) - 1);
159 head->clump_size = cpu_to_be32(clump_size); 162 head->clump_size = cpu_to_be32(clump_size);
160 head->attributes |= cpu_to_be32(HFS_TREE_BIGKEYS | HFS_TREE_VARIDXKEYS); 163 head->attributes |= cpu_to_be32(HFS_TREE_BIGKEYS | HFS_TREE_VARIDXKEYS);