x86: prefetch fix #2

Linus noticed a second bug and an uncleanliness:

 - we'd return on any instruction fetch fault

 - we'd use both the value of 16 and the PF_INSTR symbol which are
   the same and make no sense

the cleanup nicely unifies this piece of logic.

Signed-off-by: Ingo Molnar <mingo@elte.hu>
This commit is contained in:
Ingo Molnar 2008-03-27 21:29:09 +01:00
parent fb8c7fb25d
commit 3085354de6
1 changed files with 4 additions and 7 deletions

View File

@ -91,13 +91,10 @@ static int is_prefetch(struct pt_regs *regs, unsigned long addr,
int prefetch = 0; int prefetch = 0;
unsigned char *max_instr; unsigned char *max_instr;
#ifdef CONFIG_X86_32 /*
/* Catch an obscure case of prefetch inside an NX page: */ * If it was a exec (instruction fetch) fault on NX page, then
if ((__supported_pte_mask & _PAGE_NX) && (error_code & 16)) * do not ignore the fault:
return 0; */
#endif
/* If it was a exec fault on NX page, ignore */
if (error_code & PF_INSTR) if (error_code & PF_INSTR)
return 0; return 0;