On 16/11/16 02:06, Aneesh Kumar K.V wrote: > VSID 0 is bad address. Don't create slb entries on coproc fault for > bad address > > Signed-off-by: Aneesh Kumar K.V <aneesh.ku...@linux.vnet.ibm.com> > --- > NOTE: note tested ^^^ ? > > arch/powerpc/mm/copro_fault.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/powerpc/mm/copro_fault.c b/arch/powerpc/mm/copro_fault.c > index 362954f98029..aaa7ec6788b9 100644 > --- a/arch/powerpc/mm/copro_fault.c > +++ b/arch/powerpc/mm/copro_fault.c > @@ -134,6 +134,9 @@ int copro_calculate_slb(struct mm_struct *mm, u64 ea, > struct copro_slb *slb) > pr_debug("%s: invalid region access at %016llx\n", __func__, > ea); > return 1; > } > + /* Bad address */ > + if (!vsid) > + return 1;
Reviewed-by: Balbir Singh <bsinghar...@gmail.com> Balbir Singh.