[PATCH] powerpc/mm/coproc: Handle bad address on coproc slb fault

Balbir Singh bsingharora at gmail.com
Wed Nov 16 11:08:05 AEDT 2016



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.kumar at 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 <bsingharora at gmail.com>

Balbir Singh.


More information about the Linuxppc-dev mailing list