Re: [PATCH] set_thread_area.2: Add C-SKY document

[Guo Ren]

On Mon, Oct 16, 2023 at 2:00 AM Alejandro Colomar  wrote:
>
> Hi Guo,
>
> On Sun, Oct 15, 2023 at 11:07:32AM -0400, guo...@kernel.org wrote:
> > From: Guo Ren 
> >
> > C-SKY only needs set_thread_area, no need for get_thread_area, the
> > same as MIPS.
> >
> > Signed-off-by: Guo Ren 
> > Signed-off-by: Guo Ren 
> > ---
>
> Patch applied.
> 
>
> Thanks!
> Alex
>
> >  man2/set_thread_area.2 | 10 +-
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/man2/set_thread_area.2 b/man2/set_thread_area.2
> > index 02f65e0418f2..c43a92eb447a 100644
> > --- a/man2/set_thread_area.2
> > +++ b/man2/set_thread_area.2
> > @@ -26,7 +26,7 @@ Standard C library
> >  .B "int syscall(SYS_get_thread_area);"
> >  .BI "int syscall(SYS_set_thread_area, unsigned long " tp );
> >  .PP
> > -.B #elif defined __mips__
> > +.B #elif defined(__mips__ || defined __csky__)
>
> I removed the parentheses here.
Oops, thanks.

>
> >  .PP
> >  .BI "int syscall(SYS_set_thread_area, unsigned long " addr );
> >  .PP
> > @@ -42,17 +42,17 @@ These calls provide architecture-specific support for a 
> > thread-local storage
> >  implementation.
> >  At the moment,
> >  .BR set_thread_area ()
> > -is available on m68k, MIPS, and x86 (both 32-bit and 64-bit variants);
> > +is available on m68k, MIPS, C-SKY, and x86 (both 32-bit and 64-bit 
> > variants);
> >  .BR get_thread_area ()
> >  is available on m68k and x86.
> >  .PP
> > -On m68k and MIPS,
> > +On m68k, MIPS and C-SKY,
> >  .BR set_thread_area ()
> >  allows storing an arbitrary pointer (provided in the
> >  .B tp
> >  argument on m68k and in the
> >  .B addr
> > -argument on MIPS)
> > +argument on MIPS and C-SKY)
> >  in the kernel data structure associated with the calling thread;
> >  this pointer can later be retrieved using
> >  .BR get_thread_area ()
> > @@ -139,7 +139,7 @@ return 0 on success, and \-1 on failure, with
> >  .I errno
> >  set to indicate the error.
> >  .PP
> > -On MIPS and m68k,
> > +On C-SKY, MIPS and m68k,
> >  .BR set_thread_area ()
> >  always returns 0.
> >  On m68k,
> > --
> > 2.36.1
> >
>
> --
> 



-- 
Best Regards
 Guo Ren

Re: [PATCH] set_thread_area.2: Add C-SKY document

[Alejandro Colomar]

Hi Guo,

On Sun, Oct 15, 2023 at 11:07:32AM -0400, guo...@kernel.org wrote:
> From: Guo Ren 
> 
> C-SKY only needs set_thread_area, no need for get_thread_area, the
> same as MIPS.
> 
> Signed-off-by: Guo Ren 
> Signed-off-by: Guo Ren 
> ---

Patch applied.


Thanks!
Alex

>  man2/set_thread_area.2 | 10 +-
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/man2/set_thread_area.2 b/man2/set_thread_area.2
> index 02f65e0418f2..c43a92eb447a 100644
> --- a/man2/set_thread_area.2
> +++ b/man2/set_thread_area.2
> @@ -26,7 +26,7 @@ Standard C library
>  .B "int syscall(SYS_get_thread_area);"
>  .BI "int syscall(SYS_set_thread_area, unsigned long " tp );
>  .PP
> -.B #elif defined __mips__
> +.B #elif defined(__mips__ || defined __csky__)

I removed the parentheses here.

>  .PP
>  .BI "int syscall(SYS_set_thread_area, unsigned long " addr );
>  .PP
> @@ -42,17 +42,17 @@ These calls provide architecture-specific support for a 
> thread-local storage
>  implementation.
>  At the moment,
>  .BR set_thread_area ()
> -is available on m68k, MIPS, and x86 (both 32-bit and 64-bit variants);
> +is available on m68k, MIPS, C-SKY, and x86 (both 32-bit and 64-bit variants);
>  .BR get_thread_area ()
>  is available on m68k and x86.
>  .PP
> -On m68k and MIPS,
> +On m68k, MIPS and C-SKY,
>  .BR set_thread_area ()
>  allows storing an arbitrary pointer (provided in the
>  .B tp
>  argument on m68k and in the
>  .B addr
> -argument on MIPS)
> +argument on MIPS and C-SKY)
>  in the kernel data structure associated with the calling thread;
>  this pointer can later be retrieved using
>  .BR get_thread_area ()
> @@ -139,7 +139,7 @@ return 0 on success, and \-1 on failure, with
>  .I errno
>  set to indicate the error.
>  .PP
> -On MIPS and m68k,
> +On C-SKY, MIPS and m68k,
>  .BR set_thread_area ()
>  always returns 0.
>  On m68k,
> -- 
> 2.36.1
> 

-- 



signature.asc
Description: PGP signature