diff options
author | Ingo Molnar <mingo@elte.hu> | 2009-02-22 07:29:09 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-02-25 12:38:12 -0500 |
commit | c7986acba211e8285e14c9603fb89e6f4ea0b9f8 (patch) | |
tree | fb90edb83df6a2cc0e7ac33896c731016f7f6a5a | |
parent | 478b7aab1682246a3d1e76e27a0aecb2f0013379 (diff) |
time: ntp: micro-optimize ntp_update_offset()
Impact: cleanup, no functionality changed
The time_reftime update in ntp_update_offset() to xtime.tv_sec
is a convoluted way of saying that we want to freeze the frequency
and want the 'secs' delta to be 0. Also make this branch unlikely.
This shaves off 8 bytes from the code size:
text data bss dec hex filename
2504 114 136 2754 ac2 ntp.o.before
2496 114 136 2746 aba ntp.o.after
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/time/ntp.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/kernel/time/ntp.c b/kernel/time/ntp.c index 5202dde2f0af..580a35028693 100644 --- a/kernel/time/ntp.c +++ b/kernel/time/ntp.c | |||
@@ -141,10 +141,10 @@ static void ntp_update_offset(long offset) | |||
141 | * Select how the frequency is to be controlled | 141 | * Select how the frequency is to be controlled |
142 | * and in which mode (PLL or FLL). | 142 | * and in which mode (PLL or FLL). |
143 | */ | 143 | */ |
144 | if (time_status & STA_FREQHOLD || time_reftime == 0) | ||
145 | time_reftime = xtime.tv_sec; | ||
146 | |||
147 | secs = xtime.tv_sec - time_reftime; | 144 | secs = xtime.tv_sec - time_reftime; |
145 | if (unlikely(time_status & STA_FREQHOLD || time_reftime == 0)) | ||
146 | secs = 0; | ||
147 | |||
148 | time_reftime = xtime.tv_sec; | 148 | time_reftime = xtime.tv_sec; |
149 | 149 | ||
150 | offset64 = offset; | 150 | offset64 = offset; |