diff options
author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2008-12-28 04:34:26 -0500 |
---|---|---|
committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2008-12-31 07:13:25 -0500 |
commit | f92b982680e4b4149c559789a54e1e9db190752a (patch) | |
tree | fb24f6351be9f307d8f9001f226f8aa0ee71cfdc /fs/ubifs/ubifs.h | |
parent | 6a4a9b438fe43397f4652853838f284cddd629b5 (diff) |
UBIFS: fix checkpatch.pl warnings
These are mostly long lines and wrong indentation warning
fixes. But also there are two volatile variables and
checkpatch.pl complains about them:
WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
+ volatile int gc_seq;
WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
+ volatile int gced_lnum;
Well, we anyway use smp_wmb() for c->gc_seq and c->gced_lnum, so
these 'volatile' modifiers can be just dropped.
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Diffstat (limited to 'fs/ubifs/ubifs.h')
-rw-r--r-- | fs/ubifs/ubifs.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/fs/ubifs/ubifs.h b/fs/ubifs/ubifs.h index a17dd794ae92..3275c89a3585 100644 --- a/fs/ubifs/ubifs.h +++ b/fs/ubifs/ubifs.h | |||
@@ -481,8 +481,8 @@ struct ubifs_lprops { | |||
481 | struct ubifs_lpt_lprops { | 481 | struct ubifs_lpt_lprops { |
482 | int free; | 482 | int free; |
483 | int dirty; | 483 | int dirty; |
484 | unsigned tgc : 1; | 484 | unsigned tgc:1; |
485 | unsigned cmt : 1; | 485 | unsigned cmt:1; |
486 | }; | 486 | }; |
487 | 487 | ||
488 | /** | 488 | /** |
@@ -1322,8 +1322,8 @@ struct ubifs_info { | |||
1322 | void *sbuf; | 1322 | void *sbuf; |
1323 | struct list_head idx_gc; | 1323 | struct list_head idx_gc; |
1324 | int idx_gc_cnt; | 1324 | int idx_gc_cnt; |
1325 | volatile int gc_seq; | 1325 | int gc_seq; |
1326 | volatile int gced_lnum; | 1326 | int gced_lnum; |
1327 | 1327 | ||
1328 | struct list_head infos_list; | 1328 | struct list_head infos_list; |
1329 | struct mutex umount_mutex; | 1329 | struct mutex umount_mutex; |