Skip to content

Commit 0e2815d

Browse files
Peter Zijlstraingomolnar
authored andcommitted
x86/headers: Clean up too long lines
Signed-off-by: Peter Zijlstra <peterz@infradead.org> Cc: Linus Torvalds <torvalds@linux-foundation.org> Cc: Thomas Gleixner <tglx@linutronix.de> Cc: bp@alien8.de Cc: brgerst@gmail.com Cc: dvlasenk@redhat.com Cc: luto@amacapital.net Cc: mikko.rapeli@iki.fi Cc: oleg@redhat.com Link: http://lkml.kernel.org/r/20150909071244.GM3644@twins.programming.kicks-ass.net Signed-off-by: Ingo Molnar <mingo@kernel.org>
1 parent decb4c4 commit 0e2815d

1 file changed

Lines changed: 31 additions & 26 deletions

File tree

arch/x86/include/uapi/asm/sigcontext.h

Lines changed: 31 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -2,16 +2,16 @@
22
#define _UAPI_ASM_X86_SIGCONTEXT_H
33

44
/*
5-
* Linux signal context definitions. The sigcontext includes a complex hierarchy of CPU
6-
* and FPU state, available to user-space (on the stack) when a signal handler is
7-
* executed.
5+
* Linux signal context definitions. The sigcontext includes a complex
6+
* hierarchy of CPU and FPU state, available to user-space (on the stack) when
7+
* a signal handler is executed.
88
*
9-
* As over the years this ABI grew from its very simple roots towards supporting more and
10-
* more CPU state organically, some of the details (which were rather clever hacks back
11-
* in the days) became a bit quirky by today.
9+
* As over the years this ABI grew from its very simple roots towards
10+
* supporting more and more CPU state organically, some of the details (which
11+
* were rather clever hacks back in the days) became a bit quirky by today.
1212
*
13-
* The current ABI includes flexible provisions for future extensions, so we won't have
14-
* to grow new quirks for quite some time. Promise!
13+
* The current ABI includes flexible provisions for future extensions, so we
14+
* won't have to grow new quirks for quite some time. Promise!
1515
*/
1616

1717
#include <linux/compiler.h>
@@ -23,27 +23,32 @@
2323

2424
/*
2525
* Bytes 464..511 in the current 512-byte layout of the FXSAVE/FXRSTOR frame
26-
* are reserved for SW usage. On CPUs supporting XSAVE/XRSTOR, these bytes
27-
* are used to extend the fpstate pointer in the sigcontext, which now
28-
* includes the extended state information along with fpstate information.
26+
* are reserved for SW usage. On CPUs supporting XSAVE/XRSTOR, these bytes are
27+
* used to extend the fpstate pointer in the sigcontext, which now includes the
28+
* extended state information along with fpstate information.
2929
*
30-
* If sw_reserved.magic1 == FP_XSTATE_MAGIC1 then there's a sw_reserved.extended_size
31-
* bytes large extended context area present. (The last 32-bit word of this extended
32-
* area (at the fpstate+extended_size-FP_XSTATE_MAGIC2_SIZE address) is set to
30+
* If sw_reserved.magic1 == FP_XSTATE_MAGIC1 then there's a
31+
* sw_reserved.extended_size bytes large extended context area present. (The
32+
* last 32-bit word of this extended area (at the
33+
* fpstate+extended_size-FP_XSTATE_MAGIC2_SIZE address) is set to
3334
* FP_XSTATE_MAGIC2 so that you can sanity check your size calculations.)
3435
*
35-
* This extended area typically grows with newer CPUs that have larger and larger
36-
* XSAVE areas.
36+
* This extended area typically grows with newer CPUs that have larger and
37+
* larger XSAVE areas.
3738
*/
3839
struct _fpx_sw_bytes {
39-
/* If set to FP_XSTATE_MAGIC1 then this is an xstate context. 0 if a legacy frame. */
40+
/*
41+
* If set to FP_XSTATE_MAGIC1 then this is an xstate context.
42+
* 0 if a legacy frame.
43+
*/
4044
__u32 magic1;
4145

4246
/*
4347
* Total size of the fpstate area:
4448
*
4549
* - if magic1 == 0 then it's sizeof(struct _fpstate)
46-
* - if magic1 == FP_XSTATE_MAGIC1 then it's sizeof(struct _xstate) plus extensions (if any)
50+
* - if magic1 == FP_XSTATE_MAGIC1 then it's sizeof(struct _xstate)
51+
* plus extensions (if any)
4752
*/
4853
__u32 extended_size;
4954

@@ -66,13 +71,13 @@ struct _fpx_sw_bytes {
6671
/*
6772
* As documented in the iBCS2 standard:
6873
*
69-
* The first part of "struct _fpstate" is just the normal i387
70-
* hardware setup, the extra "status" word is used to save the
71-
* coprocessor status word before entering the handler.
74+
* The first part of "struct _fpstate" is just the normal i387 hardware setup,
75+
* the extra "status" word is used to save the coprocessor status word before
76+
* entering the handler.
7277
*
73-
* The FPU state data structure has had to grow to accommodate the
74-
* extended FPU state required by the Streaming SIMD Extensions.
75-
* There is no documented standard to accomplish this at the moment.
78+
* The FPU state data structure has had to grow to accommodate the extended FPU
79+
* state required by the Streaming SIMD Extensions. There is no documented
80+
* standard to accomplish this at the moment.
7681
*/
7782

7883
/* 10-byte legacy floating point register: */
@@ -137,8 +142,8 @@ struct _fpstate_32 {
137142
* 'struct _fpstate' so that you can always assume the _fpstate portion
138143
* exists so that you can check the magic value.
139144
*
140-
* Note2: Reserved fields may someday contain valuable data. Always save/restore
141-
* them when you change signal frames.
145+
* Note2: Reserved fields may someday contain valuable data. Always
146+
* save/restore them when you change signal frames.
142147
*/
143148
struct _fpstate_64 {
144149
__u16 cwd;

0 commit comments

Comments
 (0)