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.

Reply via email to