Re: [2.6.22.y] {02/17} - invalid-semicolon - series for stable kernel #2

From: Oliver Pinter
Date: Wed Feb 06 2008 - 18:00:33 EST


---8<---
From: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxx>
Subject: [POWERPC] Fix invalid semicolon after if statement
Patch-mainline: 2.6.23-rc4 2b02d13996fe28478e45605de9bd8bdca25718de
Acked-by: schwab@xxxxxxx

A similar fix to netfilter from Eric Dumazet inspired me to
look around a bit by using some grep/sed stuff as looking for
this kind of bugs seemed easy to automate. This is one of them
I found where it looks like this semicolon is not valid.

Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxx>
Acked-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
Signed-off-by: Paul Mackerras <paulus@xxxxxxxxx>

---
arch/powerpc/mm/hash_utils_64.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

--- a/arch/powerpc/mm/hash_utils_64.c 2007-07-08 19:32:17.000000000 -0400
+++ b/arch/powerpc/mm/hash_utils_64.c 2007-08-27 14:01:20.000000000 -0400
@@ -795,7 +795,7 @@ void hash_preload(struct mm_struct *mm,

#ifdef CONFIG_PPC_MM_SLICES
/* We only prefault standard pages for now */
- if (unlikely(get_slice_psize(mm, ea) != mm->context.user_psize));
+ if (unlikely(get_slice_psize(mm, ea) != mm->context.user_psize))
return;
#endif
---8<---

On 2/2/08, Oliver Pinter (Pintér Olivér) <oliver.pntr@xxxxxxxxx> wrote:
> mainline: 2b02d13996fe28478e45605de9bd8bdca25718de
>
> --
> Thanks,
> Oliver
>


--
Thanks,
Oliver
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/