From: Eugene Syromiatnikov <esyr@redhat.com>
To: Firoz Khan <firoz.khan@linaro.org>
Cc: linux-parisc@vger.kernel.org,
"James E . J . Bottomley" <jejb@parisc-linux.org>,
Helge Deller <deller@gmx.de>,
Thomas Gleixner <tglx@linutronix.de>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Philippe Ombredanne <pombredanne@nexb.com>,
Kate Stewart <kstewart@linuxfoundation.org>,
y2038@lists.linaro.org, linux-kernel@vger.kernel.org,
linux-arch@vger.kernel.org, arnd@arndb.de,
deepa.kernel@gmail.com, marcin.juszkiewicz@linaro.org
Subject: Re: [PATCH v4 3/6] parisc: add system call table generation support
Date: Sun, 14 Oct 2018 03:06:58 +0200 [thread overview]
Message-ID: <20181014010658.GB717@asgard.redhat.com> (raw)
In-Reply-To: <1539337442-3676-4-git-send-email-firoz.khan@linaro.org>
On Fri, Oct 12, 2018 at 03:13:59PM +0530, Firoz Khan wrote:
> diff --git a/arch/parisc/kernel/syscalls/syscall.tbl b/arch/parisc/kernel/syscalls/syscall.tbl
> new file mode 100644
> index 0000000..7c9f268
> +86 common uselib sys_ni_syscall
Again, why is this definition left, while others have been removed?
> +348 common pwritev2 sys_pwritev2 compat_sys_pwritev2
> +349 common statx sys_statx
> +350 common io_pgetevents sys_io_pgetevents compat_sys_io_pgetevents
Tab instead of spaces.
> diff --git a/arch/parisc/kernel/syscalls/syscallhdr.sh b/arch/parisc/kernel/syscalls/syscallhdr.sh
> new file mode 100644
> index 0000000..607d4ca
> --- /dev/null
> +++ b/arch/parisc/kernel/syscalls/syscallhdr.sh
> @@ -0,0 +1,35 @@
> +#!/bin/sh
/bin/sh -efu
> +# SPDX-License-Identifier: GPL-2.0
"export LANG=C", due to usage of sed/grep/sort.
> +
> +in="$1"
> +out="$2"
> +my_abis=`echo "($3)" | tr ',' '|'`
> +prefix="$4"
> +offset="$5"
> +
> +fileguard=_UAPI_ASM_PARISC_`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 | (
grep -E '^[[:xdigit:]Xx]+[[:space:]]+'"${my_abis}" "$in"
sort -n -k1,1
> + echo "#ifndef ${fileguard}"
> + echo "#define ${fileguard}"
> + echo ""
cat <<-EOF
#ifndef ${fileguard}
#define ${fileguard}
EOF
> +
> + nxt=0
> + while read nr abi name entry compat ; do
> + if [ -z "$offset" ]; then
> + echo -e "#define __NR_${prefix}${name}\t$nr"
> + else
> + echo -e "#define __NR_${prefix}${name}\t($offset + $nr)"
> + fi
echo options are not portable; something like
printf "#define __NR_%s%s\t(%s + %s)" "${prefix}" "${name}" "${offset}" "${nr}"
> + nxt=$nr
> + let nxt=nxt+1
"let" is a bash extension, posix-conformant expression would be
nxt=$((nr + 1))
(I would also check that nr >= nxt, however)
> + done
> +
> + echo ""
> + echo "#ifdef __KERNEL__"
> + echo -e "#define __NR_syscalls\t$nxt"
> + echo "#endif"
> + echo ""
> + echo "#endif /* ${fileguard} */"
cat <<-EOF
#ifdef __KERNEL__
# define __NR_syscalls\t${nxt}
#endif
#endif /* ${fileguard} */
EOF
> +) > "$out"
> diff --git a/arch/parisc/kernel/syscalls/syscalltbl.sh b/arch/parisc/kernel/syscalls/syscalltbl.sh
> new file mode 100644
> index 0000000..04abde7
> --- /dev/null
> +++ b/arch/parisc/kernel/syscalls/syscalltbl.sh
> @@ -0,0 +1,46 @@
> +#!/bin/sh
> +# SPDX-License-Identifier: GPL-2.0
> +
> +in="$1"
> +out="$2"
> +my_abis=`echo "($3)" | tr ',' '|'`
> +offset="$4"
> +
> +emit() {
> + nxt="$1"
> + if [ -z "$offset" ]; then
> + nr="$2"
> + else
> + nr="$2"
> + nr=$((nr+offset))
> + fi
> + entry="$3"
> +
> + while [ $nxt -lt $nr ]; do
It would break nicely if nxt="-n x -o 1" or something like that.
> + echo "__SYSCALL($nxt, sys_ni_syscall, )"
> + let nxt=nxt+1
> + done
> + echo "__SYSCALL($nxt, $entry, )"
> +}
> +
> +grep -E "^[0-9A-Fa-fXx]+[[:space:]]+${my_abis}" "$in" | sort -n | (
> + if [ -z "$offset" ]; then
> + nxt=0
> + else
> + nxt=$offset
> + fi
> +
> + my_abi="$(cut -d'|' -f2 <<< $my_abis)"
my_abi="${my_abis#*|}"
But it looks like that $my_abis includes parentheses and this code
is broken.
> + while read nr abi name entry compat ; do
> + if [ $my_abi = "compat" ]; then
> + if [ -z "$compat" ]; then
> + emit $nxt $nr $entry
> + else
> + emit $nxt $nr $compat
> + fi
> + else
> + emit $nxt $nr $entry
> + fi
> + let nxt=nxt+1
> + done
> +) > "$out"
> --
> 1.9.1
>
next prev parent reply other threads:[~2018-10-14 1:06 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-12 9:43 [PATCH v4 0/6] parisc: system call table generation support Firoz Khan
2018-10-12 9:43 ` [PATCH v4 1/6] parisc: move __IGNORE* entries to non uapi header Firoz Khan
2018-10-12 9:43 ` [PATCH v4 2/6] parisc: add __NR_Linux_syscalls along with __NR_syscalls Firoz Khan
2018-10-12 9:43 ` [PATCH v4 3/6] parisc: add system call table generation support Firoz Khan
2018-10-12 10:18 ` Firoz Khan
2018-10-12 11:51 ` Arnd Bergmann
2018-10-13 15:34 ` Firoz Khan
2018-10-12 12:07 ` Rolf Eike Beer
2018-10-12 13:57 ` Firoz Khan
2018-10-12 14:03 ` Rolf Eike Beer
2018-10-15 4:48 ` Firoz Khan
2018-10-15 5:16 ` Rolf Eike Beer
2018-10-15 5:45 ` Firoz Khan
2018-10-14 1:06 ` Eugene Syromiatnikov [this message]
2018-10-15 5:12 ` Firoz Khan
2018-10-12 9:44 ` [PATCH v4 4/6] parisc: uapi header and system call table file generation Firoz Khan
2018-10-12 9:44 ` [PATCH v4 5/6] parisc: wire up rseq system call Firoz Khan
2018-10-12 9:56 ` Arnd Bergmann
2018-10-12 10:16 ` Firoz Khan
2018-10-12 11:52 ` Arnd Bergmann
2018-10-12 20:23 ` Helge Deller
2018-10-13 5:42 ` Firoz Khan
2018-10-12 9:44 ` [PATCH v4 6/6] parisc: syscalls: Ignore nfsservctl for other architectures Firoz Khan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181014010658.GB717@asgard.redhat.com \
--to=esyr@redhat.com \
--cc=arnd@arndb.de \
--cc=deepa.kernel@gmail.com \
--cc=deller@gmx.de \
--cc=firoz.khan@linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=jejb@parisc-linux.org \
--cc=kstewart@linuxfoundation.org \
--cc=linux-arch@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-parisc@vger.kernel.org \
--cc=marcin.juszkiewicz@linaro.org \
--cc=pombredanne@nexb.com \
--cc=tglx@linutronix.de \
--cc=y2038@lists.linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).