2023-10-15 15:07:59

by Guo Ren

[permalink] [raw]
Subject: [PATCH] set_thread_area.2: Add C-SKY document

From: Guo Ren <[email protected]>

C-SKY only needs set_thread_area, no need for get_thread_area, the
same as MIPS.

Signed-off-by: Guo Ren <[email protected]>
Signed-off-by: Guo Ren <[email protected]>
---
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__)
.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


2023-10-15 18:00:42

by Alejandro Colomar

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

Hi Guo,

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

Patch applied.
<https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=1c4464ae2c40318b77e125961e24710d1784df5d>

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
>

--
<https://www.alejandro-colomar.es/>


Attachments:
(No filename) (2.15 kB)
signature.asc (849.00 B)
Download all attachments

2023-10-16 00:53:58

by Guo Ren

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

On Mon, Oct 16, 2023 at 2:00 AM Alejandro Colomar <[email protected]> wrote:
>
> Hi Guo,
>
> On Sun, Oct 15, 2023 at 11:07:32AM -0400, [email protected] wrote:
> > From: Guo Ren <[email protected]>
> >
> > C-SKY only needs set_thread_area, no need for get_thread_area, the
> > same as MIPS.
> >
> > Signed-off-by: Guo Ren <[email protected]>
> > Signed-off-by: Guo Ren <[email protected]>
> > ---
>
> Patch applied.
> <https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=1c4464ae2c40318b77e125961e24710d1784df5d>
>
> 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
> >
>
> --
> <https://www.alejandro-colomar.es/>



--
Best Regards
Guo Ren