diff options
author | Roel Kluin <roel.kluin@gmail.com> | 2009-12-11 09:43:12 -0500 |
---|---|---|
committer | Jason Wessel <jason.wessel@windriver.com> | 2009-12-11 09:43:12 -0500 |
commit | a5d09d68335bb8422d5e7050c9f03f99ba6cfebd (patch) | |
tree | b9e398daf667b8aca2d968ef822bede5c1fe5b82 /arch/x86/kernel/kgdb.c | |
parent | 3ef884b4c04e857c283cc77ca70ad8f638d94b0e (diff) |
kgdb,x86: remove redundant test
The for loop starts with a breakno of 0, and ends when it's 4. so
this test is always true.
Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Diffstat (limited to 'arch/x86/kernel/kgdb.c')
-rw-r--r-- | arch/x86/kernel/kgdb.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/arch/x86/kernel/kgdb.c b/arch/x86/kernel/kgdb.c index 20a5b368946..f93d015753c 100644 --- a/arch/x86/kernel/kgdb.c +++ b/arch/x86/kernel/kgdb.c | |||
@@ -220,8 +220,7 @@ static void kgdb_correct_hw_break(void) | |||
220 | dr7 |= ((breakinfo[breakno].len << 2) | | 220 | dr7 |= ((breakinfo[breakno].len << 2) | |
221 | breakinfo[breakno].type) << | 221 | breakinfo[breakno].type) << |
222 | ((breakno << 2) + 16); | 222 | ((breakno << 2) + 16); |
223 | if (breakno >= 0 && breakno <= 3) | 223 | set_debugreg(breakinfo[breakno].addr, breakno); |
224 | set_debugreg(breakinfo[breakno].addr, breakno); | ||
225 | 224 | ||
226 | } else { | 225 | } else { |
227 | if ((dr7 & breakbit) && !breakinfo[breakno].enabled) { | 226 | if ((dr7 & breakbit) && !breakinfo[breakno].enabled) { |