[LU-8835] More 64 bit divides without do_div.

• Status: Resolved
• Project: Lustre
• Affects Version/s: Lustre 2.10.0
• Fix Version/s: Lustre 2.10.0
• Type: Bug
• Priority: Minor
• Reporter: James A Simmons
• Assignee: Yang Sheng
• Resolution: Fixed
• Severity: 3

Description

While porting more patches to upstream the problem of __umoddi reappeared with new patches that landed. The patches that landed with this defect are:

http://review.whamcloud.com/15316
http://review.whamcloud.com/10458

Issue Links

• duplicates: LU-6163 64-bit divides should use do_div()
• is related to: LU-6215 Sync Lustre external tree with lustre...

Comments

[Joseph Gmitter (Inactive)]

Hi Yang Sheng,

Can you please have a look at this?

Thanks.
Joe

[Gerrit Updater]

James Simmons ([e-mail]) uploaded a new patch: http://review.whamcloud.com/23814
Subject: LU-8835 osc: handle 64 bit time properly in osc_cache_too_much
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3406027771e3a9cc45f085e84a3c072d54974b48

[James A Simmons]

I found the do_div issues in llog.c has already been fixed in master (LU-6134). Its just missing from b2_8_fe. The other do_div bug was introduced with the move to 64 bit time upstream. I submitted a patch here that moves to 64 bit time and resolves the do_div issue. I submitted to master so people can review this work to make sure I did it right,

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/23814/
Subject: LU-8835 osc: handle 64 bit time properly in osc_cache_too_much
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 11dfa318972db8c8d4bdc36848a4aee8072557f7

[Peter Jones]

Fixed in 2.10