Messages in this thread | | | Date | Wed, 20 Aug 2014 11:18:21 -0600 | Subject | amd_mce.c redundant if check? | From | Adam Duskett <> |
| |
I have recently come upon this section of code in arch/x86/kernel/cpu/mcheck/mce_amd.c that seems to be a redundant unnecessary if check.
From line 170 - 176:
if (tr->set_lvt_off) { if (lvt_off_valid(tr->b, tr->lvt_off, lo, hi)) { /* set new lvt offset */ hi &= ~MASK_LVTOFF_HI; hi |= tr->lvt_off << 20; } }
This seems like it's not actually doing anything because it's setting the same value that the bit-field already has to itself.
Any reason for this?
Thanks!
| |