diff options
author | David Miller <davem@davemloft.net> | 2008-05-14 01:06:56 -0400 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2008-05-23 16:35:57 -0400 |
commit | aa5e5ceaf52a882a29d9b86531a20733f5116066 (patch) | |
tree | edfa557db13cce863a3c68fbd72bed473f83e1f7 /kernel | |
parent | 74f4e369fc5b52433ad824cef32d3bf1304549be (diff) |
ftrace: remove packed attribute on ftrace_page.
It causes unaligned access traps on platforms like sparc
(ftrace_page may be marked packed, but once we return
a dyn_ftrace sub-object from this array to another piece
of code, the "packed" part of the typing information doesn't
propagate).
But also, it didn't serve any purpose either. Even if packed,
on 64-bit or 32-bit, it didn't give us any more dyn_ftrace
entries per-page.
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/trace/ftrace.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index af5ad8949abb..07b2a14943f8 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -177,9 +177,9 @@ static DEFINE_MUTEX(ftrace_filter_lock); | |||
177 | 177 | ||
178 | struct ftrace_page { | 178 | struct ftrace_page { |
179 | struct ftrace_page *next; | 179 | struct ftrace_page *next; |
180 | int index; | 180 | unsigned long index; |
181 | struct dyn_ftrace records[]; | 181 | struct dyn_ftrace records[]; |
182 | } __attribute__((packed)); | 182 | }; |
183 | 183 | ||
184 | #define ENTRIES_PER_PAGE \ | 184 | #define ENTRIES_PER_PAGE \ |
185 | ((PAGE_SIZE - sizeof(struct ftrace_page)) / sizeof(struct dyn_ftrace)) | 185 | ((PAGE_SIZE - sizeof(struct ftrace_page)) / sizeof(struct dyn_ftrace)) |