From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761608AbcIZGC0 (ORCPT ); Mon, 26 Sep 2016 02:02:26 -0400 Received: from mail-db5eur01on0046.outbound.protection.outlook.com ([104.47.2.46]:41597 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1761482AbcIZGCY (ORCPT ); Mon, 26 Sep 2016 02:02:24 -0400 Authentication-Results: spf=pass (sender IP is 193.47.165.134) smtp.mailfrom=mellanox.com; linaro.org; dkim=none (message not signed) header.d=none;linaro.org; dmarc=pass action=none header.from=mellanox.com; From: Liav Rehana To: CC: , , , , Liav Rehana Subject: [PATCH] timekeeping: Change type of nsec variable to unsigned in its calculation. Date: Mon, 26 Sep 2016 08:45:36 +0300 Message-ID: <1474868736-22724-1-git-send-email-liavr@mellanox.com> X-Mailer: git-send-email 1.7.1 MIME-Version: 1.0 Content-Type: text/plain X-TM-AS-Product-Ver: SMEX-11.0.0.1191-8.000.1202-22520.000 X-TM-AS-Result: No--5.515200-8.000000-31 X-TM-AS-User-Approved-Sender: No X-TM-AS-User-Blocked-Sender: No X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-HT: Tenant X-Forefront-Antispam-Report: CIP:193.47.165.134;IPV:NLI;CTRY:IL;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(7916002)(2980300002)(438002)(189002)(199003)(8676002)(81166006)(586003)(11100500001)(8936002)(81156014)(305945005)(356003)(7846002)(50986999)(33646002)(6916009)(19580395003)(19580405001)(36756003)(50226002)(86362001)(189998001)(4001430100002)(2351001)(229853001)(106466001)(4326007)(47776003)(49486002)(626004)(107886002)(2906002)(110136003)(50466002)(5660300001)(48376002)(92566002)(5003940100001)(87936001)(77096005);DIR:OUT;SFP:1101;SCL:1;SRVR:DB6PR0501MB2694;H:mtlcas13.mtl.com;FPR:;SPF:Pass;PTR:mail13.mellanox.com;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;AM1FFO11FD022;1:6pYbmolZFt+28gnUAEbS/NaQMzVwWM04L3N6uzfeTl5dV9DFQXhchvWNJKAO4Tcob9Cs2cpgaf6Vnc1HHCWrsFsl1si0ewb8Bc07qLpo9NTgEtVmYfUqxZBX4lOdA/JptZy3xb6wAGR2ovJERL8cFgRS2/BeClIWsDTvXMVatVO1jxF/WqdVOEdXSmRNgS1M9Ds1bgu1CzmSR1iluUBRBAuH6c8w9gMXvcIUF9PzJWc2bzWtta41uWHQxTaO8JH9A8GR7lKRIqidmo9OYuVYxMUeCGzfiMrOwpqmqT1XSHdS2tkDfi/umTzaZiRhomXjPLM7x3tE1Z0oRb70vWTvOAH50Zx7ZWverkj+nnT4srNqGi2grG23bniK4P9D5qrJuEshSNpKxZdarpy50zSobEu+HNOR3FwfXu9tswhg+18PPOsR/clPVjkdR6AgBeCqTy+mV6yDxSAfacWi5EasmYz/JGU9ntyKrF0C7sA+Dfev9S7o5n/EssSjeR5SrJ09 X-MS-Office365-Filtering-Correlation-Id: 23d06f7f-b04e-48e8-a18e-08d3e5d06ee0 X-Microsoft-Exchange-Diagnostics: 1;DB6PR0501MB2694;2:0eZZLfKuDB1AUTUgd5tH2ifFF5HEu6UjUYiqL/6rC5uFwMuqU8b+z3WCGL+HREeBFHZGQnUfWQyjYBBd6DnET+5tp60hEFlROthNhZpEOhkNm6X7lVk7O2sHOvfyj5J1UUA12XBpcZodFI/gV8bwtGkX3eBBxJ/KrXloYSbqlyGzT7LkoTRWaLpmxHiL90mW;3:nIrLAeJhzR9k7XqfeFUpbvcmULhWUPbTjzfKP9rQqnw9vllD03RSeApOIV6CYzapcCRxdg9Ak2+QQftM+bRt3/Y/zpMwfmgdnWgHMvfkdEauATzGqZNcnfV6S7YoFOXz+5JoM7/mTlwwUtN3j6rRSQPCs4G9t7c7aqau0SJTqA+NiQDQvREEHsiHCk0DcjmcmQVJTLG02n7651Kat0qudYx+LBvUyJBXWlsjhs9g8WhwR/EUnXnl4RAGSyzwdBLWHwBDJwP/zYeLFHfpQAQw4g==;25:ERkj1ZvTPBHWYKhSklsUaD4XTNSFH7q3x1II0ckteoKymQGIKlf2zL0E/jZCoJGOPfsMtHNrlviJkRJVPH7dX5e7BFLeo4dSxOOfATOQZa5cmbRw9TSqDKwjTCbNyt36aFxIRNUml7U44FvhNhnfHHtpUq7/HlEVXVPWdZBBl4yIKqCt5fd4h0T30EqDUIcQoNY8u3Ue5cNaia9KIkDkx9Mp8hhbWnrEsmxoO6xnC9yEluhbblmH0F4JFkaWSzRdayuP7dhI7vNDtZXoTGRiXMX8eW1mMgV+3URPAFahfoVrV47hV/t4QhPrWKgPDiHW7Dgb+agN44BMBWg1GfHOciIBtDShHG35QcNwlrrYAtbTgc5ltGobvjygPBmpBTvmkwaFNQB4bohjylDpPE7ul7DZG4pQTe0JzqrteHuXyFE= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(8251501002);SRVR:DB6PR0501MB2694; X-Microsoft-Exchange-Diagnostics: 1;DB6PR0501MB2694;31:FnhAdrfFfBlXDJ/jDe4FE3qOVkqRyDhKSd+Ak1pAzAlVDt1EQcfuZaZu5livkM1MIniBqVZg11cRzRHlFQ6htrlm+VDZtd//IkjCIENrE2i84PsMcQsmVeGwhBdz/pC9tp4WuLVfRouUPvfjuYz9DEZfb5MHnqbnD3pI+BWfuv/VchQ1kFMw7IFB0q2VsLNzjjbD5tA2fWka45gYe6K2p4Fvyz/Wo5mue7fAAZDPcWg=;20:ZlVRhYiqaBbKF+/564iY5xh1kT3HPp2OFeNfO23rWaXYGdLtVaxynJXF2eQATWhkcW96OyA3SySfc3mAZwlJvygDj/ccAG6JF7XqNpxQQpZkl71PsYM5K/0qRIjCdRKvbbMrg0feZdRYBMN6tXK45d5U8+SmE6HKlhxOlhik2/zpUOhCejqVkqRANaVFb9wPq5y6zlPyKgSuMeyf7drJ4RPorVSTdl8sidtwu6A53/PdYKsYEeZ8OFvEr2rZPQug/mhUIZbSdAUwcyR/ZLNgwL+5mlnra8sNnAZaVF/UXpRbGgr3fZezjAUYaP/SYYkiTUzonsdd9krJ5JmWe9Ovh2jlLo2mf37lG4JPkwp8Xm9K/FM361Pn4+e0dbZiGOyeAkE7TI+02MB6I4KQyGQZwIE5L0rsLBIcesBeuENgO2QkdA0ppi4b8MuL9bxyZ3MGf+JmeYw4LkbONqjsDrovHgrsKADdXeIQv7IPcGxCDKVWPuQOHVSSAMRWIOR1/73a X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(13016025)(13018025)(8121501046)(5005006)(3002001)(10201501046)(6055026);SRVR:DB6PR0501MB2694;BCL:0;PCL:0;RULEID:;SRVR:DB6PR0501MB2694; X-Microsoft-Exchange-Diagnostics: 1;DB6PR0501MB2694;4:xzVIc6QtNI3nIkkYtjMTvDZRZ06bWYfeqWSchn2qtqe3GXDPteLWuUtWjbapA/BzxmqG/cWhluEYHflTTiJDE14gUFvHVWdh0kor8aSXowWJek8zrBpt1l8LFz8+MlDipvrip2bfHPc0Pp9nC3a8fI6acjIZWy96NPLfLTLhC7oneYAZW+y4XyLz/Rh7/GLUuWOvR+RUKN/GVGiQ7eoATfSdbo9E1dddH/3vL9EKtKNHm90z+tTE8nXAx85/5KmN7Ij57GCBYAhi5Dqci1rWWuUfyQyLTGRvccH6i8UPYwaRNOvDEpuq/g3J5gXSeCoc7UzNeJ7AAQJAp0eritk46wG2kRck7sjeBBxe7/oIv3MIBbogWyEHg7woqIiL6oz399nsMa1GfygcYsvSEs0tFWIsMIRw7f3q45+bQ9IjnylZA+NnB8oC0ooyod4XHvSQ X-Forefront-PRVS: 00770C4423 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;DB6PR0501MB2694;23:Db8DUE3l5JdcrLS8ymYx2RIk42w24UErc2BRG0W?= =?us-ascii?Q?hcgNFrKEk0Zkou2ocbEXYxsh+g3GVQGTATiwc6mw4NNyd2UEgunEn2m6nB3G?= =?us-ascii?Q?dc+vUFkkS7uLp8/eBKk2g8QrvHDvNeUmng5H45FXKVSffFQGVRPt+7xWRoX4?= =?us-ascii?Q?IYur6ED4W4cVf743PtUfVaKAiQskT6b1TkPF1ElwIgouu7ForiUpdWYBXMsf?= =?us-ascii?Q?E3NBI3YwBuz6POVrOod51WvxFJ7LUcnQ/gBZ14hd3OPdHl1NWlMBQMsyOIll?= =?us-ascii?Q?M3/Ghcz5rw2Zvgw/pnuGRl0/QYPCVxMOmelYukgZL3vyJhVBzBGl3HR6C34p?= =?us-ascii?Q?BBOogZE5gGMr5UJquI+NqwAnLfVYXJ2kQKv+zmWAMPoM+aiKgDiE714t8F0s?= =?us-ascii?Q?Hd/AgiW7LaIPEwISkaVtu3jiO8KmtqdwbmQIZZGtSZmtgKSIfTE85CWoGEpM?= =?us-ascii?Q?dF14TvL9sT82l+yuC6F0gSIUGjl8hXgBF7RWzERbojwu3F5mh3RTmJlf375q?= =?us-ascii?Q?MLKsW2a3WAr9gEG3SKU1JffDRwh0IhNUUE8w9CMOrIiiE/ki+/wwx7+rA/eo?= =?us-ascii?Q?qodPNmVzeahROPgrSW87vY7mGlJ2953foDB+y7IOQY1f8davQUG7fIoA9PbX?= =?us-ascii?Q?fMQYWqtfAF91NBpUrD/F/GqshSPBEusneCk8BxKo94Lj1la1R1QohmFbuTz3?= =?us-ascii?Q?GbhizrBkewWZOSq0aXjIW4VXFBhy7q9PNU6iQE1s/adN41Z4WowHkXhKg9wh?= =?us-ascii?Q?q/Iuf72oYD9pfhDPOSqelSki+JHXpfzx0Nz6WmUeYqbXDVg3UJ8MPAWeL2Nn?= =?us-ascii?Q?51o1jMX5cy6ErhSXdikApUSuIVey/m1UCsS3JgFcxnxN8Bu+L4Fu62Xo21yk?= =?us-ascii?Q?Ysb9QuNX2jPK83zoHcwS3DgCp7lMPmJMccUunNu17YPcWZKNowHO9qpqyaWk?= =?us-ascii?Q?a6/kXx5fv3oHorl4AA4xBKUUpAsIlbM6TMSNRjevG7lTjp6igaLQxKovVzW4?= =?us-ascii?Q?yVfVncyxoZv0arnTWjqyOaq87MJjjcfwDY+DUnYsZXGR7wcKl4uZdTClcFqK?= =?us-ascii?Q?5LL9drPhfeGbG3pvRq+oOhrb0QHaa?= X-Microsoft-Exchange-Diagnostics: 1;DB6PR0501MB2694;6:VSKrd3oIWmdtKkD5EsfEIpIrUIxAKNJYsDK02ce7oLsXUpP0FPlLcAVm6UGvby5A33+b8MsxB6NQch4syMVhJksH269opCBtFWcEJhN8eMItkIFMBAbztCqDQETw54lQz0tEeSKaYWlTi/8uCIG89YB0Gajk9GNqGr4IzfOJvxbex27IaVh5t7gH5bJoVsvOIA/31AEXwIEWSs7yywDyu4dR2oAzQaGaZVhNhZ8y3gRO80o0gE83rBg6Eq78T8mhe0edsFrKV+2gB/SK6ERj4DOO/BU2EQKkbfxDDL74+KR+kqWUNwmqdl+2W9BQ1Gwf5MDa3sFWI249PFFn24CM8g==;5:d9vStboDzFJNVKUiWLMnNjLIziGT7rcFm8qeUBl2z7g4Eycf5TTCVQSkleQdC9uRFAf6RK3enFEwxYSDu3JUXVEzYtfbHTXpN7kHIMswwZjMS25a2nzENPXornflsSj/bwU5vcDgMxMvPDayXQUUnw==;24:sjS83zG/vm3W1pr/Rl1NCni+mEtVDRWooyzOT8/W0B92wu5z46qFc8YlE0DV/IuzOkdAaU+s1Ukj9iy09+G8gLZau7O/olgJAqrW60Pz9c0=;7:VQ3yZmgX71FhfTxKNVkSptHk5I9dml4VvQBMOVQkwU6jWU4gwwb4n007XO/GzCbKWegmpXQF+QcFHutn9hSqTH6JjcvUj8g2IvCE/OjCby1qTwaA0JSSs7u3j3ubzy8EeZWpmZZO+rTU4f1j5lX0EXxvZQbsSKTXiB9cUVGzpbxVCp5ZizUGJSDP4c1lmv9iaxx/bvgXopHNfkJrp53NKyNkzDHVSDLVniX4p/qK40FjmSCjJXGbzLZA16YRATQsj+V3J+ISER+/Te6Ytf5XjzEOjxL9s6eQxA8OgFIQCoz5InhRAj6KTR8tAusUGoCk SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Sep 2016 05:46:16.3673 (UTC) X-MS-Exchange-CrossTenant-Id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=a652971c-7d2e-4d9b-a6a4-d149256f461b;Ip=[193.47.165.134];Helo=[mtlcas13.mtl.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0501MB2694 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Liav Rehana During the calculation of the nsec variable in the inline function timekeeping_delta_to_ns, it may undergo a sign extension if its msb is set just before the shift. The sign extension may, in some cases, gain it a value near the maximum value of the 64-bit range. This is bad when it is later used in a division function, such as __iter_div_u64_rem, where the amount of loops it will go through to calculate the division will be too large. The following commit fixes that chance of sign extension, while maintaining the type of the nsec variable as signed for other functions that use this variable, for possible legit negative time intervals. Signed-off-by: Liav Rehana --- kernel/time/timekeeping.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/time/timekeeping.c b/kernel/time/timekeeping.c index 479d25c..f2c0067 100644 --- a/kernel/time/timekeeping.c +++ b/kernel/time/timekeeping.c @@ -299,10 +299,10 @@ u32 (*arch_gettimeoffset)(void) = default_arch_gettimeoffset; static inline u32 arch_gettimeoffset(void) { return 0; } #endif -static inline s64 timekeeping_delta_to_ns(struct tk_read_base *tkr, +static inline u64 timekeeping_delta_to_ns(struct tk_read_base *tkr, cycle_t delta) { - s64 nsec; + u64 nsec; nsec = delta * tkr->mult + tkr->xtime_nsec; nsec >>= tkr->shift; -- 1.7.1