Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-27 Thread Nick Hu
Hi Paul, On Tue, Aug 27, 2019 at 05:33:11PM +0800, Andrey Ryabinin wrote: > > > On 8/27/19 12:07 PM, Nick Hu wrote: > > Hi Andrey > > > > On Thu, Aug 22, 2019 at 11:59:02PM +0800, Andrey Ryabinin wrote: > >> On 8/7/19 10:19 AM, Nick Hu wrote: > >>> There are some features which need this

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-27 Thread Andrey Ryabinin
On 8/27/19 12:07 PM, Nick Hu wrote: > Hi Andrey > > On Thu, Aug 22, 2019 at 11:59:02PM +0800, Andrey Ryabinin wrote: >> On 8/7/19 10:19 AM, Nick Hu wrote: >>> There are some features which need this string operation for compilation, >>> like KASAN. So the purpose of this porting is for the

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-27 Thread Nick Hu
Hi Andrey On Thu, Aug 22, 2019 at 11:59:02PM +0800, Andrey Ryabinin wrote: > On 8/7/19 10:19 AM, Nick Hu wrote: > > There are some features which need this string operation for compilation, > > like KASAN. So the purpose of this porting is for the features like KASAN > > which cannot be compiled

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-22 Thread Andrey Ryabinin
On 8/7/19 10:19 AM, Nick Hu wrote: > There are some features which need this string operation for compilation, > like KASAN. So the purpose of this porting is for the features like KASAN > which cannot be compiled without it. > Compilation error can be fixed by diff bellow (I didn't test it). If

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-19 Thread Nick Hu
Hi Paul, On Thu, Aug 15, 2019 at 11:27:51AM -0700, Paul Walmsley wrote: > On Thu, 15 Aug 2019, Nick Hu wrote: > > > On Wed, Aug 14, 2019 at 10:03:39AM -0700, Paul Walmsley wrote: > > > > > Thanks for the explanation. What do you think about Palmer's idea to > > > define a generic C set of

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-14 Thread Nick Hu
Hi Paul, On Wed, Aug 14, 2019 at 10:03:39AM -0700, Paul Walmsley wrote: > Hi Nick, > > On Wed, 14 Aug 2019, Nick Hu wrote: > > > On Wed, Aug 14, 2019 at 10:22:15AM +0800, Paul Walmsley wrote: > > > On Tue, 13 Aug 2019, Palmer Dabbelt wrote: > > > > > > > On Mon, 12 Aug 2019 08:04:46 PDT

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-14 Thread Palmer Dabbelt
On Tue, 13 Aug 2019 19:22:15 PDT (-0700), Paul Walmsley wrote: On Tue, 13 Aug 2019, Palmer Dabbelt wrote: On Mon, 12 Aug 2019 08:04:46 PDT (-0700), Christoph Hellwig wrote: > On Wed, Aug 07, 2019 at 03:19:14PM +0800, Nick Hu wrote: > > There are some features which need this string operation

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-13 Thread Nick Hu
On Wed, Aug 14, 2019 at 10:22:15AM +0800, Paul Walmsley wrote: > On Tue, 13 Aug 2019, Palmer Dabbelt wrote: > > > On Mon, 12 Aug 2019 08:04:46 PDT (-0700), Christoph Hellwig wrote: > > > On Wed, Aug 07, 2019 at 03:19:14PM +0800, Nick Hu wrote: > > > > There are some features which need this

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-13 Thread Paul Walmsley
On Tue, 13 Aug 2019, Palmer Dabbelt wrote: > On Mon, 12 Aug 2019 08:04:46 PDT (-0700), Christoph Hellwig wrote: > > On Wed, Aug 07, 2019 at 03:19:14PM +0800, Nick Hu wrote: > > > There are some features which need this string operation for compilation, > > > like KASAN. So the purpose of this

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-13 Thread Palmer Dabbelt
On Mon, 12 Aug 2019 08:04:46 PDT (-0700), Christoph Hellwig wrote: On Wed, Aug 07, 2019 at 03:19:14PM +0800, Nick Hu wrote: There are some features which need this string operation for compilation, like KASAN. So the purpose of this porting is for the features like KASAN which cannot be

Re: [PATCH 1/2] riscv: Add memmove string operation.

2019-08-12 Thread Christoph Hellwig
On Wed, Aug 07, 2019 at 03:19:14PM +0800, Nick Hu wrote: > There are some features which need this string operation for compilation, > like KASAN. So the purpose of this porting is for the features like KASAN > which cannot be compiled without it. > > KASAN's string operations would replace the

[PATCH 1/2] riscv: Add memmove string operation.

2019-08-07 Thread Nick Hu
There are some features which need this string operation for compilation, like KASAN. So the purpose of this porting is for the features like KASAN which cannot be compiled without it. KASAN's string operations would replace the original string operations and call for the architecture defined