* [PATCH 0/2] xtensa: Unify the system call scripts
2019-01-02 15:28 [PATCH 0/2] xtensa: Unify the system call scripts Firoz Khan
@ 2019-01-02 15:28 ` Firoz Khan
2019-01-02 15:28 ` [PATCH 1/2] xtensa: remove nargs from __SYSCALL Firoz Khan
2019-01-02 15:28 ` [PATCH 2/2] xtensa: generate uapi header and syscall table header files Firoz Khan
2 siblings, 0 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-02 15:28 UTC (permalink / raw)
To: Chris Zankel, Max Filippov, Dominik Brodowski, linux-xtensa,
Greg Kroah-Hartman, Philippe Ombredanne, Thomas Gleixner,
Kate Stewart
Cc: y2038, linux-kernel, linux-arch, arnd, deepa.kernel,
marcin.juszkiewicz, firoz.khan
System call table generation support is provided for
alpha, ia64, m68k, microblaze, mips, parisc, powerpc,
sh, sparc and xtensa architectures. The implementat-
ions are almost similar across all the above archte-
ctures. In order to reduce the source code across all
the above architectures, create common ".sh" files and
keep it in the common directory, script/. This will
be a generic scripts which can use for all the above
architectures.
This patch depends on;
https://lore.kernel.org/lkml/1546439331-18646-1-git-send-email-firoz.khan@linaro.org/
Firoz Khan (2):
xtensa: remove nargs from __SYSCALL
xtensa: generate uapi header and syscall table header files
arch/xtensa/kernel/syscall.c | 2 +-
arch/xtensa/kernel/syscalls/Makefile | 11 ++++++++--
arch/xtensa/kernel/syscalls/syscallhdr.sh | 36 -------------------------------
arch/xtensa/kernel/syscalls/syscalltbl.sh | 32 ---------------------------
4 files changed, 10 insertions(+), 71 deletions(-)
delete mode 100644 arch/xtensa/kernel/syscalls/syscallhdr.sh
delete mode 100644 arch/xtensa/kernel/syscalls/syscalltbl.sh
--
1.9.1
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-02 15:28 [PATCH 0/2] xtensa: Unify the system call scripts Firoz Khan
2019-01-02 15:28 ` Firoz Khan
@ 2019-01-02 15:28 ` Firoz Khan
2019-01-02 15:28 ` Firoz Khan
2019-01-04 2:58 ` Max Filippov
2019-01-02 15:28 ` [PATCH 2/2] xtensa: generate uapi header and syscall table header files Firoz Khan
2 siblings, 2 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-02 15:28 UTC (permalink / raw)
To: Chris Zankel, Max Filippov, Dominik Brodowski, linux-xtensa,
Greg Kroah-Hartman, Philippe Ombredanne, Thomas Gleixner,
Kate Stewart
Cc: y2038, linux-kernel, linux-arch, arnd, deepa.kernel,
marcin.juszkiewicz, firoz.khan
The __SYSCALL macro's arguments are system call number,
system call entry name and number of arguments for the
system call.
Argument- nargs in __SYSCALL(nr, entry, nargs) is neither
calculated nor used anywhere. So it would be better to
keep the implementaion as __SYSCALL(nr, entry). This will
unifies the implementation with some other architetures
too.
Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
---
arch/xtensa/kernel/syscall.c | 2 +-
arch/xtensa/kernel/syscalls/syscalltbl.sh | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/xtensa/kernel/syscall.c b/arch/xtensa/kernel/syscall.c
index 2c415fc..0665339 100644
--- a/arch/xtensa/kernel/syscall.c
+++ b/arch/xtensa/kernel/syscall.c
@@ -31,7 +31,7 @@
syscall_t sys_call_table[__NR_syscalls] /* FIXME __cacheline_aligned */= {
[0 ... __NR_syscalls - 1] = (syscall_t)&sys_ni_syscall,
-#define __SYSCALL(nr, entry, nargs)[nr] = (syscall_t)entry,
+#define __SYSCALL(nr, entry)[nr] = (syscall_t)entry,
#include <asm/syscall_table.h>
#undef __SYSCALL
};
diff --git a/arch/xtensa/kernel/syscalls/syscalltbl.sh b/arch/xtensa/kernel/syscalls/syscalltbl.sh
index 85d78d9..904b8e6 100644
--- a/arch/xtensa/kernel/syscalls/syscalltbl.sh
+++ b/arch/xtensa/kernel/syscalls/syscalltbl.sh
@@ -13,10 +13,10 @@ emit() {
t_entry="$3"
while [ $t_nxt -lt $t_nr ]; do
- printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
+ printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
t_nxt=$((t_nxt+1))
done
- printf "__SYSCALL(%s, %s, )\n" "${t_nxt}" "${t_entry}"
+ printf "__SYSCALL(%s,%s)\n" "${t_nxt}" "${t_entry}"
}
grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
--
1.9.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-02 15:28 ` [PATCH 1/2] xtensa: remove nargs from __SYSCALL Firoz Khan
@ 2019-01-02 15:28 ` Firoz Khan
2019-01-04 2:58 ` Max Filippov
1 sibling, 0 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-02 15:28 UTC (permalink / raw)
To: Chris Zankel, Max Filippov, Dominik Brodowski, linux-xtensa,
Greg Kroah-Hartman, Philippe Ombredanne, Thomas Gleixner,
Kate Stewart
Cc: y2038, linux-kernel, linux-arch, arnd, deepa.kernel,
marcin.juszkiewicz, firoz.khan
The __SYSCALL macro's arguments are system call number,
system call entry name and number of arguments for the
system call.
Argument- nargs in __SYSCALL(nr, entry, nargs) is neither
calculated nor used anywhere. So it would be better to
keep the implementaion as __SYSCALL(nr, entry). This will
unifies the implementation with some other architetures
too.
Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
---
arch/xtensa/kernel/syscall.c | 2 +-
arch/xtensa/kernel/syscalls/syscalltbl.sh | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/xtensa/kernel/syscall.c b/arch/xtensa/kernel/syscall.c
index 2c415fc..0665339 100644
--- a/arch/xtensa/kernel/syscall.c
+++ b/arch/xtensa/kernel/syscall.c
@@ -31,7 +31,7 @@
syscall_t sys_call_table[__NR_syscalls] /* FIXME __cacheline_aligned */= {
[0 ... __NR_syscalls - 1] = (syscall_t)&sys_ni_syscall,
-#define __SYSCALL(nr, entry, nargs)[nr] = (syscall_t)entry,
+#define __SYSCALL(nr, entry)[nr] = (syscall_t)entry,
#include <asm/syscall_table.h>
#undef __SYSCALL
};
diff --git a/arch/xtensa/kernel/syscalls/syscalltbl.sh b/arch/xtensa/kernel/syscalls/syscalltbl.sh
index 85d78d9..904b8e6 100644
--- a/arch/xtensa/kernel/syscalls/syscalltbl.sh
+++ b/arch/xtensa/kernel/syscalls/syscalltbl.sh
@@ -13,10 +13,10 @@ emit() {
t_entry="$3"
while [ $t_nxt -lt $t_nr ]; do
- printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
+ printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
t_nxt=$((t_nxt+1))
done
- printf "__SYSCALL(%s, %s, )\n" "${t_nxt}" "${t_entry}"
+ printf "__SYSCALL(%s,%s)\n" "${t_nxt}" "${t_entry}"
}
grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
--
1.9.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-02 15:28 ` [PATCH 1/2] xtensa: remove nargs from __SYSCALL Firoz Khan
2019-01-02 15:28 ` Firoz Khan
@ 2019-01-04 2:58 ` Max Filippov
2019-01-04 2:58 ` Max Filippov
2019-01-04 5:17 ` Firoz Khan
1 sibling, 2 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-04 2:58 UTC (permalink / raw)
To: Firoz Khan
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart, y2038, LKML,
Linux-Arch, Arnd Bergmann, Deepa Dinamani, marcin.juszkiewicz
On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
>
> The __SYSCALL macro's arguments are system call number,
> system call entry name and number of arguments for the
> system call.
>
> Argument- nargs in __SYSCALL(nr, entry, nargs) is neither
> calculated nor used anywhere. So it would be better to
> keep the implementaion as __SYSCALL(nr, entry). This will
> unifies the implementation with some other architetures
> too.
Perhaps "This will help unify the implementation with other architectures"?
> Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
> ---
> arch/xtensa/kernel/syscall.c | 2 +-
> arch/xtensa/kernel/syscalls/syscalltbl.sh | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/xtensa/kernel/syscall.c b/arch/xtensa/kernel/syscall.c
> index 2c415fc..0665339 100644
> --- a/arch/xtensa/kernel/syscall.c
> +++ b/arch/xtensa/kernel/syscall.c
> @@ -31,7 +31,7 @@
> syscall_t sys_call_table[__NR_syscalls] /* FIXME __cacheline_aligned */= {
> [0 ... __NR_syscalls - 1] = (syscall_t)&sys_ni_syscall,
>
> -#define __SYSCALL(nr, entry, nargs)[nr] = (syscall_t)entry,
> +#define __SYSCALL(nr, entry)[nr] = (syscall_t)entry,
> #include <asm/syscall_table.h>
> #undef __SYSCALL
> };
> diff --git a/arch/xtensa/kernel/syscalls/syscalltbl.sh b/arch/xtensa/kernel/syscalls/syscalltbl.sh
> index 85d78d9..904b8e6 100644
> --- a/arch/xtensa/kernel/syscalls/syscalltbl.sh
> +++ b/arch/xtensa/kernel/syscalls/syscalltbl.sh
> @@ -13,10 +13,10 @@ emit() {
> t_entry="$3"
>
> while [ $t_nxt -lt $t_nr ]; do
> - printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
> + printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
Please add space after the comma.
> t_nxt=$((t_nxt+1))
> done
> - printf "__SYSCALL(%s, %s, )\n" "${t_nxt}" "${t_entry}"
> + printf "__SYSCALL(%s,%s)\n" "${t_nxt}" "${t_entry}"
Ditto.
> }
>
> grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
> --
> 1.9.1
With the above changes:
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-04 2:58 ` Max Filippov
@ 2019-01-04 2:58 ` Max Filippov
2019-01-04 5:17 ` Firoz Khan
1 sibling, 0 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-04 2:58 UTC (permalink / raw)
To: Firoz Khan
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart, y2038, LKML,
Linux-Arch, Arnd Bergmann, Deepa Dinamani, marcin.juszkiewicz
On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
>
> The __SYSCALL macro's arguments are system call number,
> system call entry name and number of arguments for the
> system call.
>
> Argument- nargs in __SYSCALL(nr, entry, nargs) is neither
> calculated nor used anywhere. So it would be better to
> keep the implementaion as __SYSCALL(nr, entry). This will
> unifies the implementation with some other architetures
> too.
Perhaps "This will help unify the implementation with other architectures"?
> Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
> ---
> arch/xtensa/kernel/syscall.c | 2 +-
> arch/xtensa/kernel/syscalls/syscalltbl.sh | 4 ++--
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/arch/xtensa/kernel/syscall.c b/arch/xtensa/kernel/syscall.c
> index 2c415fc..0665339 100644
> --- a/arch/xtensa/kernel/syscall.c
> +++ b/arch/xtensa/kernel/syscall.c
> @@ -31,7 +31,7 @@
> syscall_t sys_call_table[__NR_syscalls] /* FIXME __cacheline_aligned */= {
> [0 ... __NR_syscalls - 1] = (syscall_t)&sys_ni_syscall,
>
> -#define __SYSCALL(nr, entry, nargs)[nr] = (syscall_t)entry,
> +#define __SYSCALL(nr, entry)[nr] = (syscall_t)entry,
> #include <asm/syscall_table.h>
> #undef __SYSCALL
> };
> diff --git a/arch/xtensa/kernel/syscalls/syscalltbl.sh b/arch/xtensa/kernel/syscalls/syscalltbl.sh
> index 85d78d9..904b8e6 100644
> --- a/arch/xtensa/kernel/syscalls/syscalltbl.sh
> +++ b/arch/xtensa/kernel/syscalls/syscalltbl.sh
> @@ -13,10 +13,10 @@ emit() {
> t_entry="$3"
>
> while [ $t_nxt -lt $t_nr ]; do
> - printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
> + printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
Please add space after the comma.
> t_nxt=$((t_nxt+1))
> done
> - printf "__SYSCALL(%s, %s, )\n" "${t_nxt}" "${t_entry}"
> + printf "__SYSCALL(%s,%s)\n" "${t_nxt}" "${t_entry}"
Ditto.
> }
>
> grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
> --
> 1.9.1
With the above changes:
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-04 2:58 ` Max Filippov
2019-01-04 2:58 ` Max Filippov
@ 2019-01-04 5:17 ` Firoz Khan
2019-01-04 5:17 ` Firoz Khan
2019-01-04 5:55 ` Max Filippov
1 sibling, 2 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 5:17 UTC (permalink / raw)
To: Max Filippov
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
Hi Max,
Thanks for your reply.
On Fri, 4 Jan 2019 at 08:28, Max Filippov <jcmvbkbc@gmail.com> wrote:
> On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> > while [ $t_nxt -lt $t_nr ]; do
> > - printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
> > + printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
>
> Please add space after the comma.
This was intentional as I was trying compile other one architecture with the
previous implementation, it was giving the error. So I modified without space
after comma.
Firoz
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-04 5:17 ` Firoz Khan
@ 2019-01-04 5:17 ` Firoz Khan
2019-01-04 5:55 ` Max Filippov
1 sibling, 0 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 5:17 UTC (permalink / raw)
To: Max Filippov
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
Hi Max,
Thanks for your reply.
On Fri, 4 Jan 2019 at 08:28, Max Filippov <jcmvbkbc@gmail.com> wrote:
> On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> > while [ $t_nxt -lt $t_nr ]; do
> > - printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
> > + printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
>
> Please add space after the comma.
This was intentional as I was trying compile other one architecture with the
previous implementation, it was giving the error. So I modified without space
after comma.
Firoz
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-04 5:17 ` Firoz Khan
2019-01-04 5:17 ` Firoz Khan
@ 2019-01-04 5:55 ` Max Filippov
2019-01-04 5:55 ` Max Filippov
2019-01-04 6:14 ` Firoz Khan
1 sibling, 2 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-04 5:55 UTC (permalink / raw)
To: Firoz Khan
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
On Thu, Jan 3, 2019 at 9:17 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> On Fri, 4 Jan 2019 at 08:28, Max Filippov <jcmvbkbc@gmail.com> wrote:
> > On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> > > while [ $t_nxt -lt $t_nr ]; do
> > > - printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
> > > + printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
> >
> > Please add space after the comma.
>
> This was intentional as I was trying compile other one architecture with the
> previous implementation, it was giving the error. So I modified without space
> after comma.
I have tested that presence or absence of space does not affect the build
for xtensa, and normally we put space after a comma, so I've mentioned that.
Since this script is anyway removed in the next patch of the series it doesn't
really matter whether the spaces are present or not.
So, with or without this change:
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-04 5:55 ` Max Filippov
@ 2019-01-04 5:55 ` Max Filippov
2019-01-04 6:14 ` Firoz Khan
1 sibling, 0 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-04 5:55 UTC (permalink / raw)
To: Firoz Khan
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
On Thu, Jan 3, 2019 at 9:17 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> On Fri, 4 Jan 2019 at 08:28, Max Filippov <jcmvbkbc@gmail.com> wrote:
> > On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> > > while [ $t_nxt -lt $t_nr ]; do
> > > - printf "__SYSCALL(%s, sys_ni_syscall, )\n" "${t_nxt}"
> > > + printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
> >
> > Please add space after the comma.
>
> This was intentional as I was trying compile other one architecture with the
> previous implementation, it was giving the error. So I modified without space
> after comma.
I have tested that presence or absence of space does not affect the build
for xtensa, and normally we put space after a comma, so I've mentioned that.
Since this script is anyway removed in the next patch of the series it doesn't
really matter whether the spaces are present or not.
So, with or without this change:
Acked-by: Max Filippov <jcmvbkbc@gmail.com>
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-04 5:55 ` Max Filippov
2019-01-04 5:55 ` Max Filippov
@ 2019-01-04 6:14 ` Firoz Khan
2019-01-04 6:14 ` Firoz Khan
1 sibling, 1 reply; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 6:14 UTC (permalink / raw)
To: Max Filippov
Cc: Kate Stewart, Chris Zankel, Linux-Arch, Arnd Bergmann,
linux-xtensa, y2038 Mailman List, Greg Kroah-Hartman, LKML,
Dominik Brodowski, Marcin Juszkiewicz, Deepa Dinamani,
Philippe Ombredanne, Thomas Gleixner
Hi Max,
On Fri, 4 Jan 2019 at 11:25, Max Filippov <jcmvbkbc@gmail.com> wrote:
> On Thu, Jan 3, 2019 at 9:17 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> > On Fri, 4 Jan 2019 at 08:28, Max Filippov <jcmvbkbc@gmail.com> wrote:
> > > On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> > This was intentional as I was trying compile other one architecture with the
> > previous implementation, it was giving the error. So I modified without space
> > after comma.
>
> I have tested that presence or absence of space does not affect the build
> for xtensa, and normally we put space after a comma, so I've mentioned that.
>
> Since this script is anyway removed in the next patch of the series it doesn't
> really matter whether the spaces are present or not.
Yes, you are right, there is no effect in xtensa but if I'm right
there is a problem
in powerpc architecture (not remembering exactly which arch) with this space
after comma.
BTW, the script from all architecture will be removed and keep it in
scripts directory
is our plan. So to satisfy powerpc (again I'm not sure which arch) I
just did this
change.
Thanks for your feedback.
Firoz
_______________________________________________
Y2038 mailing list
Y2038@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/y2038
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 1/2] xtensa: remove nargs from __SYSCALL
2019-01-04 6:14 ` Firoz Khan
@ 2019-01-04 6:14 ` Firoz Khan
0 siblings, 0 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 6:14 UTC (permalink / raw)
To: Max Filippov
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
Hi Max,
On Fri, 4 Jan 2019 at 11:25, Max Filippov <jcmvbkbc@gmail.com> wrote:
> On Thu, Jan 3, 2019 at 9:17 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> > On Fri, 4 Jan 2019 at 08:28, Max Filippov <jcmvbkbc@gmail.com> wrote:
> > > On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> > This was intentional as I was trying compile other one architecture with the
> > previous implementation, it was giving the error. So I modified without space
> > after comma.
>
> I have tested that presence or absence of space does not affect the build
> for xtensa, and normally we put space after a comma, so I've mentioned that.
>
> Since this script is anyway removed in the next patch of the series it doesn't
> really matter whether the spaces are present or not.
Yes, you are right, there is no effect in xtensa but if I'm right
there is a problem
in powerpc architecture (not remembering exactly which arch) with this space
after comma.
BTW, the script from all architecture will be removed and keep it in
scripts directory
is our plan. So to satisfy powerpc (again I'm not sure which arch) I
just did this
change.
Thanks for your feedback.
Firoz
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-02 15:28 [PATCH 0/2] xtensa: Unify the system call scripts Firoz Khan
2019-01-02 15:28 ` Firoz Khan
2019-01-02 15:28 ` [PATCH 1/2] xtensa: remove nargs from __SYSCALL Firoz Khan
@ 2019-01-02 15:28 ` Firoz Khan
2019-01-02 15:28 ` Firoz Khan
2019-01-03 22:44 ` Max Filippov
2 siblings, 2 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-02 15:28 UTC (permalink / raw)
To: Chris Zankel, Max Filippov, Dominik Brodowski, linux-xtensa,
Greg Kroah-Hartman, Philippe Ombredanne, Thomas Gleixner,
Kate Stewart
Cc: y2038, linux-kernel, linux-arch, arnd, deepa.kernel,
marcin.juszkiewicz, firoz.khan
Unified system call table generation script must be run to
generate unistd_32.h and syscall_table.h files. This patch
will have changes which will invokes the script.
This patch will generate unistd_32.h and syscall_table.h
files by the syscall table generation script invoked by
xtensa/Makefile and the generated files against the removed
files must be identical.
The generated uapi header file will be included in uapi/-
asm/unistd.h and generated system call table header file
will be included by kernel/syscall.c file.
Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
---
arch/xtensa/kernel/syscalls/Makefile | 11 ++++++++--
arch/xtensa/kernel/syscalls/syscallhdr.sh | 36 -------------------------------
arch/xtensa/kernel/syscalls/syscalltbl.sh | 32 ---------------------------
3 files changed, 9 insertions(+), 70 deletions(-)
delete mode 100644 arch/xtensa/kernel/syscalls/syscallhdr.sh
delete mode 100644 arch/xtensa/kernel/syscalls/syscalltbl.sh
diff --git a/arch/xtensa/kernel/syscalls/Makefile b/arch/xtensa/kernel/syscalls/Makefile
index 659faef..75c7421 100644
--- a/arch/xtensa/kernel/syscalls/Makefile
+++ b/arch/xtensa/kernel/syscalls/Makefile
@@ -6,8 +6,9 @@ _dummy := $(shell [ -d '$(uapi)' ] || mkdir -p '$(uapi)') \
$(shell [ -d '$(kapi)' ] || mkdir -p '$(kapi)')
syscall := $(srctree)/$(src)/syscall.tbl
-syshdr := $(srctree)/$(src)/syscallhdr.sh
-systbl := $(srctree)/$(src)/syscalltbl.sh
+syshdr := $(srctree)/scripts/syscallhdr.sh
+sysnr := $(srctree)/scripts/syscallnr.sh
+systbl := $(srctree)/scripts/syscalltbl.sh
quiet_cmd_syshdr = SYSHDR $@
cmd_syshdr = $(CONFIG_SHELL) '$(syshdr)' '$<' '$@' \
@@ -15,6 +16,12 @@ quiet_cmd_syshdr = SYSHDR $@
'$(syshdr_pfx_$(basetarget))' \
'$(syshdr_offset_$(basetarget))'
+quiet_cmd_sysnr = SYSNR $@
+ cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
+ '$(sysnr_abis_$(basetarget))' \
+ '$(sysnr_pfx_$(basetarget))' \
+ '$(sysnr_offset_$(basetarget))'
+
quiet_cmd_systbl = SYSTBL $@
cmd_systbl = $(CONFIG_SHELL) '$(systbl)' '$<' '$@' \
'$(systbl_abis_$(basetarget))' \
diff --git a/arch/xtensa/kernel/syscalls/syscallhdr.sh b/arch/xtensa/kernel/syscalls/syscallhdr.sh
deleted file mode 100644
index d37db64..0000000
--- a/arch/xtensa/kernel/syscalls/syscallhdr.sh
+++ /dev/null
@@ -1,36 +0,0 @@
-#!/bin/sh
-# SPDX-License-Identifier: GPL-2.0
-
-in="$1"
-out="$2"
-my_abis=`echo "($3)" | tr ',' '|'`
-prefix="$4"
-offset="$5"
-
-fileguard=_UAPI_ASM_XTENSA_`basename "$out" | sed \
- -e 'y/abcdefghijklmnopqrstuvwxyz/ABCDEFGHIJKLMNOPQRSTUVWXYZ/' \
- -e 's/[^A-Z0-9_]/_/g' -e 's/__/_/g'`
-grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
- printf "#ifndef %s\n" "${fileguard}"
- printf "#define %s\n" "${fileguard}"
- printf "\n"
-
- nxt=0
- while read nr abi name entry ; do
- if [ -z "$offset" ]; then
- printf "#define __NR_%s%s\t%s\n" \
- "${prefix}" "${name}" "${nr}"
- else
- printf "#define __NR_%s%s\t(%s + %s)\n" \
- "${prefix}" "${name}" "${offset}" "${nr}"
- fi
- nxt=$((nr+1))
- done
-
- printf "\n"
- printf "#ifdef __KERNEL__\n"
- printf "#define __NR_syscalls\t%s\n" "${nxt}"
- printf "#endif\n"
- printf "\n"
- printf "#endif /* %s */" "${fileguard}"
-) > "$out"
diff --git a/arch/xtensa/kernel/syscalls/syscalltbl.sh b/arch/xtensa/kernel/syscalls/syscalltbl.sh
deleted file mode 100644
index 904b8e6..0000000
--- a/arch/xtensa/kernel/syscalls/syscalltbl.sh
+++ /dev/null
@@ -1,32 +0,0 @@
-#!/bin/sh
-# SPDX-License-Identifier: GPL-2.0
-
-in="$1"
-out="$2"
-my_abis=`echo "($3)" | tr ',' '|'`
-my_abi="$4"
-offset="$5"
-
-emit() {
- t_nxt="$1"
- t_nr="$2"
- t_entry="$3"
-
- while [ $t_nxt -lt $t_nr ]; do
- printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
- t_nxt=$((t_nxt+1))
- done
- printf "__SYSCALL(%s,%s)\n" "${t_nxt}" "${t_entry}"
-}
-
-grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
- nxt=0
- if [ -z "$offset" ]; then
- offset=0
- fi
-
- while read nr abi name entry ; do
- emit $((nxt+offset)) $((nr+offset)) $entry
- nxt=$((nr+1))
- done
-) > "$out"
--
1.9.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-02 15:28 ` [PATCH 2/2] xtensa: generate uapi header and syscall table header files Firoz Khan
@ 2019-01-02 15:28 ` Firoz Khan
2019-01-03 22:44 ` Max Filippov
1 sibling, 0 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-02 15:28 UTC (permalink / raw)
To: Chris Zankel, Max Filippov, Dominik Brodowski, linux-xtensa,
Greg Kroah-Hartman, Philippe Ombredanne, Thomas Gleixner,
Kate Stewart
Cc: y2038, linux-kernel, linux-arch, arnd, deepa.kernel,
marcin.juszkiewicz, firoz.khan
Unified system call table generation script must be run to
generate unistd_32.h and syscall_table.h files. This patch
will have changes which will invokes the script.
This patch will generate unistd_32.h and syscall_table.h
files by the syscall table generation script invoked by
xtensa/Makefile and the generated files against the removed
files must be identical.
The generated uapi header file will be included in uapi/-
asm/unistd.h and generated system call table header file
will be included by kernel/syscall.c file.
Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
---
arch/xtensa/kernel/syscalls/Makefile | 11 ++++++++--
arch/xtensa/kernel/syscalls/syscallhdr.sh | 36 -------------------------------
arch/xtensa/kernel/syscalls/syscalltbl.sh | 32 ---------------------------
3 files changed, 9 insertions(+), 70 deletions(-)
delete mode 100644 arch/xtensa/kernel/syscalls/syscallhdr.sh
delete mode 100644 arch/xtensa/kernel/syscalls/syscalltbl.sh
diff --git a/arch/xtensa/kernel/syscalls/Makefile b/arch/xtensa/kernel/syscalls/Makefile
index 659faef..75c7421 100644
--- a/arch/xtensa/kernel/syscalls/Makefile
+++ b/arch/xtensa/kernel/syscalls/Makefile
@@ -6,8 +6,9 @@ _dummy := $(shell [ -d '$(uapi)' ] || mkdir -p '$(uapi)') \
$(shell [ -d '$(kapi)' ] || mkdir -p '$(kapi)')
syscall := $(srctree)/$(src)/syscall.tbl
-syshdr := $(srctree)/$(src)/syscallhdr.sh
-systbl := $(srctree)/$(src)/syscalltbl.sh
+syshdr := $(srctree)/scripts/syscallhdr.sh
+sysnr := $(srctree)/scripts/syscallnr.sh
+systbl := $(srctree)/scripts/syscalltbl.sh
quiet_cmd_syshdr = SYSHDR $@
cmd_syshdr = $(CONFIG_SHELL) '$(syshdr)' '$<' '$@' \
@@ -15,6 +16,12 @@ quiet_cmd_syshdr = SYSHDR $@
'$(syshdr_pfx_$(basetarget))' \
'$(syshdr_offset_$(basetarget))'
+quiet_cmd_sysnr = SYSNR $@
+ cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
+ '$(sysnr_abis_$(basetarget))' \
+ '$(sysnr_pfx_$(basetarget))' \
+ '$(sysnr_offset_$(basetarget))'
+
quiet_cmd_systbl = SYSTBL $@
cmd_systbl = $(CONFIG_SHELL) '$(systbl)' '$<' '$@' \
'$(systbl_abis_$(basetarget))' \
diff --git a/arch/xtensa/kernel/syscalls/syscallhdr.sh b/arch/xtensa/kernel/syscalls/syscallhdr.sh
deleted file mode 100644
index d37db64..0000000
--- a/arch/xtensa/kernel/syscalls/syscallhdr.sh
+++ /dev/null
@@ -1,36 +0,0 @@
-#!/bin/sh
-# SPDX-License-Identifier: GPL-2.0
-
-in="$1"
-out="$2"
-my_abis=`echo "($3)" | tr ',' '|'`
-prefix="$4"
-offset="$5"
-
-fileguard=_UAPI_ASM_XTENSA_`basename "$out" | sed \
- -e 'y/abcdefghijklmnopqrstuvwxyz/ABCDEFGHIJKLMNOPQRSTUVWXYZ/' \
- -e 's/[^A-Z0-9_]/_/g' -e 's/__/_/g'`
-grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
- printf "#ifndef %s\n" "${fileguard}"
- printf "#define %s\n" "${fileguard}"
- printf "\n"
-
- nxt=0
- while read nr abi name entry ; do
- if [ -z "$offset" ]; then
- printf "#define __NR_%s%s\t%s\n" \
- "${prefix}" "${name}" "${nr}"
- else
- printf "#define __NR_%s%s\t(%s + %s)\n" \
- "${prefix}" "${name}" "${offset}" "${nr}"
- fi
- nxt=$((nr+1))
- done
-
- printf "\n"
- printf "#ifdef __KERNEL__\n"
- printf "#define __NR_syscalls\t%s\n" "${nxt}"
- printf "#endif\n"
- printf "\n"
- printf "#endif /* %s */" "${fileguard}"
-) > "$out"
diff --git a/arch/xtensa/kernel/syscalls/syscalltbl.sh b/arch/xtensa/kernel/syscalls/syscalltbl.sh
deleted file mode 100644
index 904b8e6..0000000
--- a/arch/xtensa/kernel/syscalls/syscalltbl.sh
+++ /dev/null
@@ -1,32 +0,0 @@
-#!/bin/sh
-# SPDX-License-Identifier: GPL-2.0
-
-in="$1"
-out="$2"
-my_abis=`echo "($3)" | tr ',' '|'`
-my_abi="$4"
-offset="$5"
-
-emit() {
- t_nxt="$1"
- t_nr="$2"
- t_entry="$3"
-
- while [ $t_nxt -lt $t_nr ]; do
- printf "__SYSCALL(%s,sys_ni_syscall)\n" "${t_nxt}"
- t_nxt=$((t_nxt+1))
- done
- printf "__SYSCALL(%s,%s)\n" "${t_nxt}" "${t_entry}"
-}
-
-grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
- nxt=0
- if [ -z "$offset" ]; then
- offset=0
- fi
-
- while read nr abi name entry ; do
- emit $((nxt+offset)) $((nr+offset)) $entry
- nxt=$((nr+1))
- done
-) > "$out"
--
1.9.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-02 15:28 ` [PATCH 2/2] xtensa: generate uapi header and syscall table header files Firoz Khan
2019-01-02 15:28 ` Firoz Khan
@ 2019-01-03 22:44 ` Max Filippov
2019-01-03 22:44 ` Max Filippov
2019-01-04 5:13 ` Firoz Khan
1 sibling, 2 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-03 22:44 UTC (permalink / raw)
To: Firoz Khan
Cc: Kate Stewart, Chris Zankel, Linux-Arch, Arnd Bergmann,
linux-xtensa, y2038, Greg Kroah-Hartman, LKML, Dominik Brodowski,
marcin.juszkiewicz, Deepa Dinamani, Philippe Ombredanne,
Thomas Gleixner
Hi Firoz,
On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> Unified system call table generation script must be run to
> generate unistd_32.h and syscall_table.h files. This patch
> will have changes which will invokes the script.
>
> This patch will generate unistd_32.h and syscall_table.h
> files by the syscall table generation script invoked by
> xtensa/Makefile and the generated files against the removed
> files must be identical.
>
> The generated uapi header file will be included in uapi/-
> asm/unistd.h and generated system call table header file
> will be included by kernel/syscall.c file.
>
> Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
> ---
> arch/xtensa/kernel/syscalls/Makefile | 11 ++++++++--
> arch/xtensa/kernel/syscalls/syscallhdr.sh | 36 -------------------------------
> arch/xtensa/kernel/syscalls/syscalltbl.sh | 32 ---------------------------
> 3 files changed, 9 insertions(+), 70 deletions(-)
> delete mode 100644 arch/xtensa/kernel/syscalls/syscallhdr.sh
> delete mode 100644 arch/xtensa/kernel/syscalls/syscalltbl.sh
>
> diff --git a/arch/xtensa/kernel/syscalls/Makefile b/arch/xtensa/kernel/syscalls/Makefile
> index 659faef..75c7421 100644
> --- a/arch/xtensa/kernel/syscalls/Makefile
> +++ b/arch/xtensa/kernel/syscalls/Makefile
> @@ -6,8 +6,9 @@ _dummy := $(shell [ -d '$(uapi)' ] || mkdir -p '$(uapi)') \
> $(shell [ -d '$(kapi)' ] || mkdir -p '$(kapi)')
>
> syscall := $(srctree)/$(src)/syscall.tbl
> -syshdr := $(srctree)/$(src)/syscallhdr.sh
> -systbl := $(srctree)/$(src)/syscalltbl.sh
> +syshdr := $(srctree)/scripts/syscallhdr.sh
> +sysnr := $(srctree)/scripts/syscallnr.sh
> +systbl := $(srctree)/scripts/syscalltbl.sh
>
> quiet_cmd_syshdr = SYSHDR $@
> cmd_syshdr = $(CONFIG_SHELL) '$(syshdr)' '$<' '$@' \
> @@ -15,6 +16,12 @@ quiet_cmd_syshdr = SYSHDR $@
> '$(syshdr_pfx_$(basetarget))' \
> '$(syshdr_offset_$(basetarget))'
>
> +quiet_cmd_sysnr = SYSNR $@
> + cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
> + '$(sysnr_abis_$(basetarget))' \
> + '$(sysnr_pfx_$(basetarget))' \
> + '$(sysnr_offset_$(basetarget))'
> +
This patch seems to do more than it says in the description.
Perhaps adding cmd_sysnr needs a separate patch with its
own description?
--
Thanks.
-- Max
_______________________________________________
Y2038 mailing list
Y2038@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/y2038
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-03 22:44 ` Max Filippov
@ 2019-01-03 22:44 ` Max Filippov
2019-01-04 5:13 ` Firoz Khan
1 sibling, 0 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-03 22:44 UTC (permalink / raw)
To: Firoz Khan
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart, y2038, LKML,
Linux-Arch, Arnd Bergmann, Deepa Dinamani, marcin.juszkiewicz
Hi Firoz,
On Wed, Jan 2, 2019 at 7:29 AM Firoz Khan <firoz.khan@linaro.org> wrote:
> Unified system call table generation script must be run to
> generate unistd_32.h and syscall_table.h files. This patch
> will have changes which will invokes the script.
>
> This patch will generate unistd_32.h and syscall_table.h
> files by the syscall table generation script invoked by
> xtensa/Makefile and the generated files against the removed
> files must be identical.
>
> The generated uapi header file will be included in uapi/-
> asm/unistd.h and generated system call table header file
> will be included by kernel/syscall.c file.
>
> Signed-off-by: Firoz Khan <firoz.khan@linaro.org>
> ---
> arch/xtensa/kernel/syscalls/Makefile | 11 ++++++++--
> arch/xtensa/kernel/syscalls/syscallhdr.sh | 36 -------------------------------
> arch/xtensa/kernel/syscalls/syscalltbl.sh | 32 ---------------------------
> 3 files changed, 9 insertions(+), 70 deletions(-)
> delete mode 100644 arch/xtensa/kernel/syscalls/syscallhdr.sh
> delete mode 100644 arch/xtensa/kernel/syscalls/syscalltbl.sh
>
> diff --git a/arch/xtensa/kernel/syscalls/Makefile b/arch/xtensa/kernel/syscalls/Makefile
> index 659faef..75c7421 100644
> --- a/arch/xtensa/kernel/syscalls/Makefile
> +++ b/arch/xtensa/kernel/syscalls/Makefile
> @@ -6,8 +6,9 @@ _dummy := $(shell [ -d '$(uapi)' ] || mkdir -p '$(uapi)') \
> $(shell [ -d '$(kapi)' ] || mkdir -p '$(kapi)')
>
> syscall := $(srctree)/$(src)/syscall.tbl
> -syshdr := $(srctree)/$(src)/syscallhdr.sh
> -systbl := $(srctree)/$(src)/syscalltbl.sh
> +syshdr := $(srctree)/scripts/syscallhdr.sh
> +sysnr := $(srctree)/scripts/syscallnr.sh
> +systbl := $(srctree)/scripts/syscalltbl.sh
>
> quiet_cmd_syshdr = SYSHDR $@
> cmd_syshdr = $(CONFIG_SHELL) '$(syshdr)' '$<' '$@' \
> @@ -15,6 +16,12 @@ quiet_cmd_syshdr = SYSHDR $@
> '$(syshdr_pfx_$(basetarget))' \
> '$(syshdr_offset_$(basetarget))'
>
> +quiet_cmd_sysnr = SYSNR $@
> + cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
> + '$(sysnr_abis_$(basetarget))' \
> + '$(sysnr_pfx_$(basetarget))' \
> + '$(sysnr_offset_$(basetarget))'
> +
This patch seems to do more than it says in the description.
Perhaps adding cmd_sysnr needs a separate patch with its
own description?
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-03 22:44 ` Max Filippov
2019-01-03 22:44 ` Max Filippov
@ 2019-01-04 5:13 ` Firoz Khan
2019-01-04 5:13 ` Firoz Khan
2019-01-04 5:59 ` Max Filippov
1 sibling, 2 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 5:13 UTC (permalink / raw)
To: Max Filippov
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
Hi Max,
Thanks for your feedback.
On Fri, 4 Jan 2019 at 04:14, Max Filippov <jcmvbkbc@gmail.com> wrote:
> >
> > +quiet_cmd_sysnr = SYSNR $@
> > + cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
> > + '$(sysnr_abis_$(basetarget))' \
> > + '$(sysnr_pfx_$(basetarget))' \
> > + '$(sysnr_offset_$(basetarget))'
> > +
>
> This patch seems to do more than it says in the description.
> Perhaps adding cmd_sysnr needs a separate patch with its
> own description?
This build rule is unused now as I was tried to implement
something else. Hopefully I'll remove the the above change
in my v2.
Thanks
Firoz
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-04 5:13 ` Firoz Khan
@ 2019-01-04 5:13 ` Firoz Khan
2019-01-04 5:59 ` Max Filippov
1 sibling, 0 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 5:13 UTC (permalink / raw)
To: Max Filippov
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
Hi Max,
Thanks for your feedback.
On Fri, 4 Jan 2019 at 04:14, Max Filippov <jcmvbkbc@gmail.com> wrote:
> >
> > +quiet_cmd_sysnr = SYSNR $@
> > + cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
> > + '$(sysnr_abis_$(basetarget))' \
> > + '$(sysnr_pfx_$(basetarget))' \
> > + '$(sysnr_offset_$(basetarget))'
> > +
>
> This patch seems to do more than it says in the description.
> Perhaps adding cmd_sysnr needs a separate patch with its
> own description?
This build rule is unused now as I was tried to implement
something else. Hopefully I'll remove the the above change
in my v2.
Thanks
Firoz
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-04 5:13 ` Firoz Khan
2019-01-04 5:13 ` Firoz Khan
@ 2019-01-04 5:59 ` Max Filippov
2019-01-04 5:59 ` Max Filippov
2019-01-04 6:15 ` Firoz Khan
1 sibling, 2 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-04 5:59 UTC (permalink / raw)
To: Firoz Khan
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
On Thu, Jan 3, 2019 at 9:13 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> On Fri, 4 Jan 2019 at 04:14, Max Filippov <jcmvbkbc@gmail.com> wrote:
> > >
> > > +quiet_cmd_sysnr = SYSNR $@
> > > + cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
> > > + '$(sysnr_abis_$(basetarget))' \
> > > + '$(sysnr_pfx_$(basetarget))' \
> > > + '$(sysnr_offset_$(basetarget))'
> > > +
> >
> > This patch seems to do more than it says in the description.
> > Perhaps adding cmd_sysnr needs a separate patch with its
> > own description?
>
> This build rule is unused now as I was tried to implement
> something else. Hopefully I'll remove the the above change
> in my v2.
Then maybe there's a way to move all these macros to a common
makefile, as they seem to be fairly generic?
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-04 5:59 ` Max Filippov
@ 2019-01-04 5:59 ` Max Filippov
2019-01-04 6:15 ` Firoz Khan
1 sibling, 0 replies; 22+ messages in thread
From: Max Filippov @ 2019-01-04 5:59 UTC (permalink / raw)
To: Firoz Khan
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
On Thu, Jan 3, 2019 at 9:13 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> On Fri, 4 Jan 2019 at 04:14, Max Filippov <jcmvbkbc@gmail.com> wrote:
> > >
> > > +quiet_cmd_sysnr = SYSNR $@
> > > + cmd_sysnr = $(CONFIG_SHELL) '$(sysnr)' '$<' '$@' \
> > > + '$(sysnr_abis_$(basetarget))' \
> > > + '$(sysnr_pfx_$(basetarget))' \
> > > + '$(sysnr_offset_$(basetarget))'
> > > +
> >
> > This patch seems to do more than it says in the description.
> > Perhaps adding cmd_sysnr needs a separate patch with its
> > own description?
>
> This build rule is unused now as I was tried to implement
> something else. Hopefully I'll remove the the above change
> in my v2.
Then maybe there's a way to move all these macros to a common
makefile, as they seem to be fairly generic?
--
Thanks.
-- Max
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-04 5:59 ` Max Filippov
2019-01-04 5:59 ` Max Filippov
@ 2019-01-04 6:15 ` Firoz Khan
2019-01-04 6:15 ` Firoz Khan
1 sibling, 1 reply; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 6:15 UTC (permalink / raw)
To: Max Filippov
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
Hi Max,
On Fri, 4 Jan 2019 at 11:29, Max Filippov <jcmvbkbc@gmail.com> wrote:
>
> On Thu, Jan 3, 2019 at 9:13 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> > On Fri, 4 Jan 2019 at 04:14, Max Filippov <jcmvbkbc@gmail.com> wrote:
> >
> > This build rule is unused now as I was tried to implement
> > something else. Hopefully I'll remove the the above change
> > in my v2.
>
> Then maybe there's a way to move all these macros to a common
> makefile, as they seem to be fairly generic?
Good idea! Sure, I'll look into it.
Thanks
Firoz
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH 2/2] xtensa: generate uapi header and syscall table header files
2019-01-04 6:15 ` Firoz Khan
@ 2019-01-04 6:15 ` Firoz Khan
0 siblings, 0 replies; 22+ messages in thread
From: Firoz Khan @ 2019-01-04 6:15 UTC (permalink / raw)
To: Max Filippov
Cc: Chris Zankel, Dominik Brodowski, linux-xtensa, Greg Kroah-Hartman,
Philippe Ombredanne, Thomas Gleixner, Kate Stewart,
y2038 Mailman List, LKML, Linux-Arch, Arnd Bergmann,
Deepa Dinamani, Marcin Juszkiewicz
Hi Max,
On Fri, 4 Jan 2019 at 11:29, Max Filippov <jcmvbkbc@gmail.com> wrote:
>
> On Thu, Jan 3, 2019 at 9:13 PM Firoz Khan <firoz.khan@linaro.org> wrote:
> > On Fri, 4 Jan 2019 at 04:14, Max Filippov <jcmvbkbc@gmail.com> wrote:
> >
> > This build rule is unused now as I was tried to implement
> > something else. Hopefully I'll remove the the above change
> > in my v2.
>
> Then maybe there's a way to move all these macros to a common
> makefile, as they seem to be fairly generic?
Good idea! Sure, I'll look into it.
Thanks
Firoz
^ permalink raw reply [flat|nested] 22+ messages in thread