Re: [PATCH v2 4/4] MIPS: memset.S: Add comments to fault fixup handlers

From: James Hogan
Date: Mon May 21 2018 - 11:20:06 EST


On Tue, Apr 17, 2018 at 04:40:03PM +0100, Matt Redfearn wrote:
> diff --git a/arch/mips/lib/memset.S b/arch/mips/lib/memset.S
> index 1cc306520a55..a06dabe99d4b 100644
> --- a/arch/mips/lib/memset.S
> +++ b/arch/mips/lib/memset.S
> @@ -231,16 +231,25 @@
>
> #ifdef CONFIG_CPU_MIPSR6
> .Lbyte_fixup\@:
> + /*
> + * unset_bytes = current_addr + 1
> + * a2 = t0 + 1

The code looks more like a2 = 1 - t0 to me:

> + */
> PTR_SUBU a2, $0, t0
> jr ra
> PTR_ADDIU a2, 1

I.e. t0 counts up to 1 and then stops.

Anyway I've applied patch 3 for 4.18.

Cheers
James

Attachment: signature.asc
Description: PGP signature