From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Gang Subject: [PATCH] drivers: target: target_core_mod: use div64_u64_rem() instead of operator '%' for u64 Date: Sat, 21 Dec 2013 10:08:55 +0800 Message-ID: <52B4F837.1010403@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Sender: target-devel-owner@vger.kernel.org To: nab@linux-iscsi.org Cc: James Hogan , linux-scsi@vger.kernel.org, target-devel@vger.kernel.org, "linux-kernel@vger.kernel.org" List-Id: linux-scsi@vger.kernel.org In kernel, need use div64_u64_rem() instead of operator '%' for u64, or can not pass compiling (with allmodconfig under metag): MODPOST 2909 modules ERROR: "__umoddi3" [drivers/target/target_core_mod.ko] undefined! Also need u64 type cast for u32 variable multiply u32 variable, or will cause type overflow issue. Signed-off-by: Chen Gang --- drivers/target/target_core_alua.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/target/target_core_alua.c b/drivers/target/target_core_alua.c index dc0d399..ff2aadc 100644 --- a/drivers/target/target_core_alua.c +++ b/drivers/target/target_core_alua.c @@ -489,7 +489,8 @@ static inline int core_alua_state_lba_dependent( u64 first_lba = map->lba_map_first_lba; if (segment_mult) { - start_lba = lba % (segment_size * segment_mult); + u64 tmp = (u64)segment_size * segment_mult; + div64_u64_rem(lba, tmp, &start_lba); last_lba = first_lba + segment_size - 1; if (start_lba >= first_lba && start_lba <= last_lba) { -- 1.7.11.7