diff options
| author | Arjan van de Ven <arjan@infradead.org> | 2006-07-03 03:24:21 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-07-03 18:26:59 -0400 |
| commit | 62322d2554d2f9680c8ace7bbf1f97d8fa84ad1a (patch) | |
| tree | 904f8b8354a4079cc1daaa355fe30200dcae9dc9 /drivers/char/viotape.c | |
| parent | d459ec0baa5d22e111dfb139c46d6d788a9eac20 (diff) | |
[PATCH] make more file_operation structs static
Mark the static struct file_operations in drivers/char as const. Making
them const prevents accidental bugs, and moves them to the .rodata section
so that they no longer do any false sharing; in addition with the proper
debug option they are then protected against corruption..
[akpm@osdl.org: build fix]
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'drivers/char/viotape.c')
| -rw-r--r-- | drivers/char/viotape.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/viotape.c b/drivers/char/viotape.c index 7d42c8ec8dbc..b72b2049aaae 100644 --- a/drivers/char/viotape.c +++ b/drivers/char/viotape.c | |||
| @@ -292,7 +292,7 @@ static int proc_viotape_open(struct inode *inode, struct file *file) | |||
| 292 | return single_open(file, proc_viotape_show, NULL); | 292 | return single_open(file, proc_viotape_show, NULL); |
| 293 | } | 293 | } |
| 294 | 294 | ||
| 295 | static struct file_operations proc_viotape_operations = { | 295 | static const struct file_operations proc_viotape_operations = { |
| 296 | .open = proc_viotape_open, | 296 | .open = proc_viotape_open, |
| 297 | .read = seq_read, | 297 | .read = seq_read, |
| 298 | .llseek = seq_lseek, | 298 | .llseek = seq_lseek, |