* [PATCH] powerpc: Make it clearer that systbl check errors are errors
@ 2018-04-30 3:27 Michael Ellerman
2018-05-10 14:06 ` Michael Ellerman
0 siblings, 1 reply; 2+ messages in thread
From: Michael Ellerman @ 2018-04-30 3:27 UTC (permalink / raw)
To: linuxppc-dev
If the systbl_chk.sh checks fail we print a message, but with no
indication that it's an error. That makes it hard to find in build
logs with eg. grep.
So prefix any output with "Error:".
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
---
arch/powerpc/kernel/systbl_chk.sh | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/kernel/systbl_chk.sh b/arch/powerpc/kernel/systbl_chk.sh
index 31b6e7c358ca..f2e356c2a345 100644
--- a/arch/powerpc/kernel/systbl_chk.sh
+++ b/arch/powerpc/kernel/systbl_chk.sh
@@ -16,7 +16,7 @@ awk 'BEGIN { num = -1; } # Ignore the beginning of the file
/^START_TABLE/ { num = 0; next; }
/^END_TABLE/ {
if (num != $2) {
- printf "NR_syscalls (%s) is not one more than the last syscall (%s)\n",
+ printf "Error: NR_syscalls (%s) is not one more than the last syscall (%s)\n",
$2, num - 1;
exit(1);
}
@@ -25,7 +25,7 @@ awk 'BEGIN { num = -1; } # Ignore the beginning of the file
{
if (num == -1) next;
if (($1 != -1) && ($1 != num)) {
- printf "Syscall %s out of order (expected %s)\n",
+ printf "Error: Syscall %s out of order (expected %s)\n",
$1, num;
exit(1);
};
--
2.14.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: powerpc: Make it clearer that systbl check errors are errors
2018-04-30 3:27 [PATCH] powerpc: Make it clearer that systbl check errors are errors Michael Ellerman
@ 2018-05-10 14:06 ` Michael Ellerman
0 siblings, 0 replies; 2+ messages in thread
From: Michael Ellerman @ 2018-05-10 14:06 UTC (permalink / raw)
To: Michael Ellerman, linuxppc-dev
On Mon, 2018-04-30 at 03:27:36 UTC, Michael Ellerman wrote:
> If the systbl_chk.sh checks fail we print a message, but with no
> indication that it's an error. That makes it hard to find in build
> logs with eg. grep.
>
> So prefix any output with "Error:".
>
> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Applied to powerpc next.
https://git.kernel.org/powerpc/c/53da14d0833a663cb98ef111ed91c5
cheers
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-05-10 14:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-04-30 3:27 [PATCH] powerpc: Make it clearer that systbl check errors are errors Michael Ellerman
2018-05-10 14:06 ` Michael Ellerman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.