Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
In-Reply-To: <20171004165605.17123-1-prasannatsmkumar@gmail.com>
References: <20171004165605.17123-1-prasannatsmkumar@gmail.com>
From:   PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com>
Date:   Sat, 21 Oct 2017 12:42:49 +0530
Message-ID: <CANc+2y4HfpxA5=Jj6uKD3Z3O_1RFUFgVO1rxDvH-jQ-jUP8=eA@mail.gmail.com>
Subject: Re: [PATCH] lib: memmove: Use optimised memcpy if possible
To:     linux-kernel@vger.kernel.org, dan.carpenter@oracle.com,
        mchehab@kernel.org, Paul Burton <paul.burton@imgtec.com>
Cc:     PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 4 October 2017 at 22:26, PrasannaKumar Muralidharan
<prasannatsmkumar@gmail.com> wrote:
> When there is no overlap between src and dst use optimised memcpy if it
> is available.
>
> Signed-off-by: Paul Burton <paul.burton@imgtec.com>
> Signed-off-by: PrasannaKumar Muralidharan <prasannatsmkumar@gmail.com>
> ---
> This change is a small part of a patch [1] from Paul Burton. I have
> added his Signed-off by. I do not know whether it is correct. Please let
> me know if it has to be changed, I will send a v2.
>
> This patch is boot tested with qemu for MIPS architecture by removing
> mips's memmove routine. This patch does not contain MIPS changes. I
> will try to find out why [1] was not taken already and figure out what
> to do.
>
> 1. https://patchwork.linux-mips.org/patch/14517/
>
>  lib/string.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/lib/string.c b/lib/string.c
> index 9921dc2..462ab7b 100644
> --- a/lib/string.c
> +++ b/lib/string.c
> @@ -825,6 +825,17 @@ void *memmove(void *dest, const void *src, size_t count)
>         char *tmp;
>         const char *s;
>
> +#ifdef __HAVE_ARCH_MEMCPY
> +       /* Use optimised memcpy when there is no overlap */
> +       const char *s_end = src + count;
> +       const char *d = dest;
> +       char *d_end = dest + count;
> +
> +       s = src;
> +       if ((d_end <= s) || (s_end <= d))
> +               return memcpy(dest, src, count);
> +#endif /* __HAVE_ARCH_MEMCPY */
> +
>         if (dest <= src) {
>                 tmp = dest;
>                 s = src;
> --
> 2.10.0
>

Is there anything more that I have to do for this patch? As there is
no review comment I am wondering what I missed.

Thanks,
PrasannaKumar

From 1580346948986428938@xxx Wed Oct 04 16:58:16 +0000 2017
X-GM-THRID: 1580346948986428938
X-Gmail-Labels: Inbox,Category Forums