diff options
author | Eric Ren <zren@suse.com> | 2016-07-26 18:21:29 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-07-26 19:19:19 -0400 |
commit | 191df2b51303d62f95ccd8552e09dcd78eaa1a05 (patch) | |
tree | 8d33ca1b236baf6a635b32cb9956783e07977509 /fs/ocfs2/stackglue.c | |
parent | 17359a80b9315670f56cd016a92c083665e80ae1 (diff) |
ocfs2: fix a redundant re-initialization
Obviously, memset() has zeroed the whole struct locking_max_version.
So, it's no need to zero its two fields individually.
Link: http://lkml.kernel.org/r/1463970605-18354-1-git-send-email-zren@suse.com
Signed-off-by: Eric Ren <zren@suse.com>
Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
Reviewed-by: Gang He <ghe@suse.com>
Cc: Mark Fasheh <mfasheh@suse.de>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Junxiao Bi <junxiao.bi@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ocfs2/stackglue.c')
-rw-r--r-- | fs/ocfs2/stackglue.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/ocfs2/stackglue.c b/fs/ocfs2/stackglue.c index 13219ed73e1d..52c07346bea3 100644 --- a/fs/ocfs2/stackglue.c +++ b/fs/ocfs2/stackglue.c | |||
@@ -735,8 +735,6 @@ static void __exit ocfs2_stack_glue_exit(void) | |||
735 | { | 735 | { |
736 | memset(&locking_max_version, 0, | 736 | memset(&locking_max_version, 0, |
737 | sizeof(struct ocfs2_protocol_version)); | 737 | sizeof(struct ocfs2_protocol_version)); |
738 | locking_max_version.pv_major = 0; | ||
739 | locking_max_version.pv_minor = 0; | ||
740 | ocfs2_sysfs_exit(); | 738 | ocfs2_sysfs_exit(); |
741 | if (ocfs2_table_header) | 739 | if (ocfs2_table_header) |
742 | unregister_sysctl_table(ocfs2_table_header); | 740 | unregister_sysctl_table(ocfs2_table_header); |