* [Qemu-devel] [PATCH] linux-user/syscall_defs.h: Sync the latest si_code from Linux kernel
@ 2015-09-26 4:10 gang.chen.5i5j
2015-09-26 16:06 ` Peter Maydell
0 siblings, 1 reply; 3+ messages in thread
From: gang.chen.5i5j @ 2015-09-26 4:10 UTC (permalink / raw)
To: riku.voipio, peter.maydell, rth
Cc: cmetcalf, qemu-devel, xili_gchen_5257, Chen Gang
From: Chen Gang <gang.chen.5i5j@gmail.com>
They content several new macro members, also contents TARGET_N*.
Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
---
linux-user/syscall_defs.h | 44 ++++++++++++++++++++++++++++++--------------
1 file changed, 30 insertions(+), 14 deletions(-)
diff --git a/linux-user/syscall_defs.h b/linux-user/syscall_defs.h
index cdc8db4..6bec3f2 100644
--- a/linux-user/syscall_defs.h
+++ b/linux-user/syscall_defs.h
@@ -736,14 +736,21 @@ typedef struct target_siginfo {
/*
* SIGILL si_codes
*/
-#define TARGET_ILL_ILLOPC (1) /* illegal opcode */
-#define TARGET_ILL_ILLOPN (2) /* illegal operand */
-#define TARGET_ILL_ILLADR (3) /* illegal addressing mode */
-#define TARGET_ILL_ILLTRP (4) /* illegal trap */
-#define TARGET_ILL_PRVOPC (5) /* privileged opcode */
-#define TARGET_ILL_PRVREG (6) /* privileged register */
-#define TARGET_ILL_COPROC (7) /* coprocessor error */
-#define TARGET_ILL_BADSTK (8) /* internal stack error */
+#define TARGET_ILL_ILLOPC (1) /* illegal opcode */
+#define TARGET_ILL_ILLOPN (2) /* illegal operand */
+#define TARGET_ILL_ILLADR (3) /* illegal addressing mode */
+#define TARGET_ILL_ILLTRP (4) /* illegal trap */
+#define TARGET_ILL_PRVOPC (5) /* privileged opcode */
+#define TARGET_ILL_PRVREG (6) /* privileged register */
+#define TARGET_ILL_COPROC (7) /* coprocessor error */
+#define TARGET_ILL_BADSTK (8) /* internal stack error */
+#ifdef TARGET_TILEGX
+#define TARGET_ILL_DBLFLT (9) /* double fault */
+#define TARGET_ILL_HARDWALL (10) /* user networks hardwall violation */
+#define TARGET_NSIGILL 10
+#else
+#define TARGET_NSIGILL 8
+#endif
/*
* SIGFPE si_codes
@@ -763,19 +770,28 @@ typedef struct target_siginfo {
*/
#define TARGET_SEGV_MAPERR (1) /* address not mapped to object */
#define TARGET_SEGV_ACCERR (2) /* invalid permissions for mapped object */
+#define TARGET_SEGV_BNDERR (3) /* failed address bound checks */
+#define TARGET_NSIGSEGV 3
/*
* SIGBUS si_codes
*/
-#define TARGET_BUS_ADRALN (1) /* invalid address alignment */
-#define TARGET_BUS_ADRERR (2) /* non-existent physical address */
-#define TARGET_BUS_OBJERR (3) /* object specific hardware error */
-
+#define TARGET_BUS_ADRALN (1) /* invalid address alignment */
+#define TARGET_BUS_ADRERR (2) /* non-existent physical address */
+#define TARGET_BUS_OBJERR (3) /* object specific hardware error */
+/* hardware memory error consumed on a machine check: action required */
+#define TARGET_BUS_MCEERR_AR (4)
+/* hardware memory error detected in process but not consumed: action optional*/
+#define TARGET_BUS_MCEERR_AO (5)
+#define TARGET_NSIGBUS 5
/*
* SIGTRAP si_codes
*/
-#define TARGET_TRAP_BRKPT (1) /* process breakpoint */
-#define TARGET_TRAP_TRACE (2) /* process trace trap */
+#define TARGET_TRAP_BRKPT (1) /* process breakpoint */
+#define TARGET_TRAP_TRACE (2) /* process trace trap */
+#define TARGET_TRAP_BRANCH (3) /* process taken branch trap */
+#define TARGET_TRAP_HWBKPT (4) /* hardware breakpoint/watchpoint */
+#define TARGET_NSIGTRAP 4
#endif /* defined(TARGET_I386) || defined(TARGET_ARM) */
--
1.9.3
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [Qemu-devel] [PATCH] linux-user/syscall_defs.h: Sync the latest si_code from Linux kernel
2015-09-26 4:10 [Qemu-devel] [PATCH] linux-user/syscall_defs.h: Sync the latest si_code from Linux kernel gang.chen.5i5j
@ 2015-09-26 16:06 ` Peter Maydell
2015-09-26 23:04 ` Chen Gang
0 siblings, 1 reply; 3+ messages in thread
From: Peter Maydell @ 2015-09-26 16:06 UTC (permalink / raw)
To: Chen Gang
Cc: Chris Metcalf, Riku Voipio, QEMU Developers, Chen Gang,
Richard Henderson
On 25 September 2015 at 21:10, <gang.chen.5i5j@gmail.com> wrote:
> From: Chen Gang <gang.chen.5i5j@gmail.com>
>
> They content several new macro members, also contents TARGET_N*.
>
> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
> ---
> linux-user/syscall_defs.h | 44 ++++++++++++++++++++++++++++++--------------
> 1 file changed, 30 insertions(+), 14 deletions(-)
>
> diff --git a/linux-user/syscall_defs.h b/linux-user/syscall_defs.h
> index cdc8db4..6bec3f2 100644
> --- a/linux-user/syscall_defs.h
> +++ b/linux-user/syscall_defs.h
> @@ -736,14 +736,21 @@ typedef struct target_siginfo {
> /*
> * SIGILL si_codes
> */
> -#define TARGET_ILL_ILLOPC (1) /* illegal opcode */
> -#define TARGET_ILL_ILLOPN (2) /* illegal operand */
> -#define TARGET_ILL_ILLADR (3) /* illegal addressing mode */
> -#define TARGET_ILL_ILLTRP (4) /* illegal trap */
> -#define TARGET_ILL_PRVOPC (5) /* privileged opcode */
> -#define TARGET_ILL_PRVREG (6) /* privileged register */
> -#define TARGET_ILL_COPROC (7) /* coprocessor error */
> -#define TARGET_ILL_BADSTK (8) /* internal stack error */
> +#define TARGET_ILL_ILLOPC (1) /* illegal opcode */
> +#define TARGET_ILL_ILLOPN (2) /* illegal operand */
> +#define TARGET_ILL_ILLADR (3) /* illegal addressing mode */
> +#define TARGET_ILL_ILLTRP (4) /* illegal trap */
> +#define TARGET_ILL_PRVOPC (5) /* privileged opcode */
> +#define TARGET_ILL_PRVREG (6) /* privileged register */
> +#define TARGET_ILL_COPROC (7) /* coprocessor error */
> +#define TARGET_ILL_BADSTK (8) /* internal stack error */
> +#ifdef TARGET_TILEGX
> +#define TARGET_ILL_DBLFLT (9) /* double fault */
> +#define TARGET_ILL_HARDWALL (10) /* user networks hardwall violation */
> +#define TARGET_NSIGILL 10
> +#else
> +#define TARGET_NSIGILL 8
> +#endif
Please don't change lines which haven't actually changed;
it makes it hard to see what has been added.
thanks
-- PMM
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Qemu-devel] [PATCH] linux-user/syscall_defs.h: Sync the latest si_code from Linux kernel
2015-09-26 16:06 ` Peter Maydell
@ 2015-09-26 23:04 ` Chen Gang
0 siblings, 0 replies; 3+ messages in thread
From: Chen Gang @ 2015-09-26 23:04 UTC (permalink / raw)
To: Peter Maydell, Chen Gang
Cc: Chris Metcalf, Riku Voipio, QEMU Developers, Chen Gang,
Richard Henderson
On 9/27/15 00:06, Peter Maydell wrote:
> On 25 September 2015 at 21:10, <gang.chen.5i5j@gmail.com> wrote:
>> From: Chen Gang <gang.chen.5i5j@gmail.com>
>>
>> They content several new macro members, also contents TARGET_N*.
>>
>> Signed-off-by: Chen Gang <gang.chen.5i5j@gmail.com>
>> ---
>> linux-user/syscall_defs.h | 44 ++++++++++++++++++++++++++++++--------------
>> 1 file changed, 30 insertions(+), 14 deletions(-)
>>
>> diff --git a/linux-user/syscall_defs.h b/linux-user/syscall_defs.h
>> index cdc8db4..6bec3f2 100644
>> --- a/linux-user/syscall_defs.h
>> +++ b/linux-user/syscall_defs.h
>> @@ -736,14 +736,21 @@ typedef struct target_siginfo {
>> /*
>> * SIGILL si_codes
>> */
>> -#define TARGET_ILL_ILLOPC (1) /* illegal opcode */
>> -#define TARGET_ILL_ILLOPN (2) /* illegal operand */
>> -#define TARGET_ILL_ILLADR (3) /* illegal addressing mode */
>> -#define TARGET_ILL_ILLTRP (4) /* illegal trap */
>> -#define TARGET_ILL_PRVOPC (5) /* privileged opcode */
>> -#define TARGET_ILL_PRVREG (6) /* privileged register */
>> -#define TARGET_ILL_COPROC (7) /* coprocessor error */
>> -#define TARGET_ILL_BADSTK (8) /* internal stack error */
>> +#define TARGET_ILL_ILLOPC (1) /* illegal opcode */
>> +#define TARGET_ILL_ILLOPN (2) /* illegal operand */
>> +#define TARGET_ILL_ILLADR (3) /* illegal addressing mode */
>> +#define TARGET_ILL_ILLTRP (4) /* illegal trap */
>> +#define TARGET_ILL_PRVOPC (5) /* privileged opcode */
>> +#define TARGET_ILL_PRVREG (6) /* privileged register */
>> +#define TARGET_ILL_COPROC (7) /* coprocessor error */
>> +#define TARGET_ILL_BADSTK (8) /* internal stack error */
>> +#ifdef TARGET_TILEGX
>> +#define TARGET_ILL_DBLFLT (9) /* double fault */
>> +#define TARGET_ILL_HARDWALL (10) /* user networks hardwall violation */
>> +#define TARGET_NSIGILL 10
>> +#else
>> +#define TARGET_NSIGILL 8
>> +#endif
>
> Please don't change lines which haven't actually changed;
> it makes it hard to see what has been added.
>
"scripts/checkpatch.pl" will check '\t', so when we add new lines, we
have to consider about it. And all related lines (at least in a "group")
have to changed, too.
I guess, I need mention about it in comments (but I did not originally).
So for me, please add related comments for it when you apply the patch
(if it passes review). If it is necessary to send patch v2, please let
me know.
Thanks
--
Chen Gang
Open, share, and attitude like air, water, and life which God blessed
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-09-26 23:02 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-26 4:10 [Qemu-devel] [PATCH] linux-user/syscall_defs.h: Sync the latest si_code from Linux kernel gang.chen.5i5j
2015-09-26 16:06 ` Peter Maydell
2015-09-26 23:04 ` Chen Gang
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).