diff mbox

Btrfs: use do_div to avoid compile errors on 32bit box

Message ID 1313747324-27477-1-git-send-email-liubo2009@cn.fujitsu.com (mailing list archive)
State New, archived
Headers show

Commit Message

liubo Aug. 19, 2011, 9:48 a.m. UTC
When doing div operation of u64 type, we need to be careful and use do_div
to avoid compile ERROR on 32bit box:

"ERROR: "__udivdi3" [fs/btrfs/btrfs.ko] undefined!
make[1]: *** [__modpost] Error 1

Signed-off-by: Liu Bo <liubo2009@cn.fujitsu.com>
---
 fs/btrfs/extent-tree.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

Comments

Josef Bacik Aug. 19, 2011, 1:22 p.m. UTC | #1
On Fri, Aug 19, 2011 at 05:48:44PM +0800, Liu Bo wrote:
> When doing div operation of u64 type, we need to be careful and use do_div
> to avoid compile ERROR on 32bit box:
> 
> "ERROR: "__udivdi3" [fs/btrfs/btrfs.ko] undefined!
> make[1]: *** [__modpost] Error 1
> 
> Signed-off-by: Liu Bo <liubo2009@cn.fujitsu.com>

Chris just left for vacation, can you send this to Linus/lkml so it gets pulled
in.  Thanks,

Josef
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
liubo Aug. 20, 2011, 1:35 a.m. UTC | #2
On 08/19/2011 09:22 PM, Josef Bacik wrote:
> On Fri, Aug 19, 2011 at 05:48:44PM +0800, Liu Bo wrote:
>> When doing div operation of u64 type, we need to be careful and use do_div
>> to avoid compile ERROR on 32bit box:
>>
>> "ERROR: "__udivdi3" [fs/btrfs/btrfs.ko] undefined!
>> make[1]: *** [__modpost] Error 1
>>
>> Signed-off-by: Liu Bo <liubo2009@cn.fujitsu.com>
> 
> Chris just left for vacation, can you send this to Linus/lkml so it gets pulled
> in.  Thanks,
> 

Already done.

thanks,
liubo

> Josef
> --
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c
index 80d6148..9b495ce 100644
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -6796,14 +6796,14 @@  int btrfs_can_relocate(struct btrfs_root *root, u64 bytenr)
 	index = get_block_group_index(block_group);
 	if (index == 0) {
 		dev_min = 4;
-		min_free /= 2;
+		do_div(min_free, 2);
 	} else if (index == 1) {
 		dev_min = 2;
 	} else if (index == 2) {
 		min_free *= 2;
 	} else if (index == 3) {
 		dev_min = fs_devices->rw_devices;
-		min_free /= dev_min;
+		do_div(min_free, dev_min);
 	}
 
 	mutex_lock(&root->fs_info->chunk_mutex);