diff options
Diffstat (limited to 'fs/afs')
-rw-r--r-- | fs/afs/dir.c | 12 | ||||
-rw-r--r-- | fs/afs/file.c | 2 | ||||
-rw-r--r-- | fs/afs/inode.c | 1 | ||||
-rw-r--r-- | fs/afs/proc.c | 2 | ||||
-rw-r--r-- | fs/afs/vlocation.c | 3 | ||||
-rw-r--r-- | fs/afs/volume.c | 3 |
6 files changed, 10 insertions, 13 deletions
diff --git a/fs/afs/dir.c b/fs/afs/dir.c index 2fc99877cb0d..a6ec75c56fcf 100644 --- a/fs/afs/dir.c +++ b/fs/afs/dir.c | |||
@@ -30,7 +30,7 @@ static int afs_dir_readdir(struct file *file, void *dirent, filldir_t filldir); | |||
30 | static int afs_d_revalidate(struct dentry *dentry, struct nameidata *nd); | 30 | static int afs_d_revalidate(struct dentry *dentry, struct nameidata *nd); |
31 | static int afs_d_delete(struct dentry *dentry); | 31 | static int afs_d_delete(struct dentry *dentry); |
32 | static int afs_dir_lookup_filldir(void *_cookie, const char *name, int nlen, | 32 | static int afs_dir_lookup_filldir(void *_cookie, const char *name, int nlen, |
33 | loff_t fpos, ino_t ino, unsigned dtype); | 33 | loff_t fpos, u64 ino, unsigned dtype); |
34 | 34 | ||
35 | const struct file_operations afs_dir_file_operations = { | 35 | const struct file_operations afs_dir_file_operations = { |
36 | .open = afs_dir_open, | 36 | .open = afs_dir_open, |
@@ -211,8 +211,8 @@ static int afs_dir_open(struct inode *inode, struct file *file) | |||
211 | { | 211 | { |
212 | _enter("{%lu}", inode->i_ino); | 212 | _enter("{%lu}", inode->i_ino); |
213 | 213 | ||
214 | BUG_ON(sizeof(union afs_dir_block) != 2048); | 214 | BUILD_BUG_ON(sizeof(union afs_dir_block) != 2048); |
215 | BUG_ON(sizeof(union afs_dirent) != 32); | 215 | BUILD_BUG_ON(sizeof(union afs_dirent) != 32); |
216 | 216 | ||
217 | if (AFS_FS_I(inode)->flags & AFS_VNODE_DELETED) | 217 | if (AFS_FS_I(inode)->flags & AFS_VNODE_DELETED) |
218 | return -ENOENT; | 218 | return -ENOENT; |
@@ -409,7 +409,7 @@ static int afs_dir_readdir(struct file *file, void *cookie, filldir_t filldir) | |||
409 | * uniquifier through dtype | 409 | * uniquifier through dtype |
410 | */ | 410 | */ |
411 | static int afs_dir_lookup_filldir(void *_cookie, const char *name, int nlen, | 411 | static int afs_dir_lookup_filldir(void *_cookie, const char *name, int nlen, |
412 | loff_t fpos, ino_t ino, unsigned dtype) | 412 | loff_t fpos, u64 ino, unsigned dtype) |
413 | { | 413 | { |
414 | struct afs_dir_lookup_cookie *cookie = _cookie; | 414 | struct afs_dir_lookup_cookie *cookie = _cookie; |
415 | 415 | ||
@@ -446,8 +446,8 @@ static struct dentry *afs_dir_lookup(struct inode *dir, struct dentry *dentry, | |||
446 | _enter("{%lu},%p{%s}", dir->i_ino, dentry, dentry->d_name.name); | 446 | _enter("{%lu},%p{%s}", dir->i_ino, dentry, dentry->d_name.name); |
447 | 447 | ||
448 | /* insanity checks first */ | 448 | /* insanity checks first */ |
449 | BUG_ON(sizeof(union afs_dir_block) != 2048); | 449 | BUILD_BUG_ON(sizeof(union afs_dir_block) != 2048); |
450 | BUG_ON(sizeof(union afs_dirent) != 32); | 450 | BUILD_BUG_ON(sizeof(union afs_dirent) != 32); |
451 | 451 | ||
452 | if (dentry->d_name.len > 255) { | 452 | if (dentry->d_name.len > 255) { |
453 | _leave(" = -ENAMETOOLONG"); | 453 | _leave(" = -ENAMETOOLONG"); |
diff --git a/fs/afs/file.c b/fs/afs/file.c index 67d6634101fd..2e8c42639eaa 100644 --- a/fs/afs/file.c +++ b/fs/afs/file.c | |||
@@ -16,7 +16,6 @@ | |||
16 | #include <linux/slab.h> | 16 | #include <linux/slab.h> |
17 | #include <linux/fs.h> | 17 | #include <linux/fs.h> |
18 | #include <linux/pagemap.h> | 18 | #include <linux/pagemap.h> |
19 | #include <linux/buffer_head.h> | ||
20 | #include "volume.h" | 19 | #include "volume.h" |
21 | #include "vnode.h" | 20 | #include "vnode.h" |
22 | #include <rxrpc/call.h> | 21 | #include <rxrpc/call.h> |
@@ -37,7 +36,6 @@ struct inode_operations afs_file_inode_operations = { | |||
37 | 36 | ||
38 | const struct address_space_operations afs_fs_aops = { | 37 | const struct address_space_operations afs_fs_aops = { |
39 | .readpage = afs_file_readpage, | 38 | .readpage = afs_file_readpage, |
40 | .sync_page = block_sync_page, | ||
41 | .set_page_dirty = __set_page_dirty_nobuffers, | 39 | .set_page_dirty = __set_page_dirty_nobuffers, |
42 | .releasepage = afs_file_releasepage, | 40 | .releasepage = afs_file_releasepage, |
43 | .invalidatepage = afs_file_invalidatepage, | 41 | .invalidatepage = afs_file_invalidatepage, |
diff --git a/fs/afs/inode.c b/fs/afs/inode.c index 4ebb30a50ed5..6f37754906c2 100644 --- a/fs/afs/inode.c +++ b/fs/afs/inode.c | |||
@@ -72,7 +72,6 @@ static int afs_inode_map_status(struct afs_vnode *vnode) | |||
72 | inode->i_ctime.tv_sec = vnode->status.mtime_server; | 72 | inode->i_ctime.tv_sec = vnode->status.mtime_server; |
73 | inode->i_ctime.tv_nsec = 0; | 73 | inode->i_ctime.tv_nsec = 0; |
74 | inode->i_atime = inode->i_mtime = inode->i_ctime; | 74 | inode->i_atime = inode->i_mtime = inode->i_ctime; |
75 | inode->i_blksize = PAGE_CACHE_SIZE; | ||
76 | inode->i_blocks = 0; | 75 | inode->i_blocks = 0; |
77 | inode->i_version = vnode->fid.unique; | 76 | inode->i_version = vnode->fid.unique; |
78 | inode->i_mapping->a_ops = &afs_fs_aops; | 77 | inode->i_mapping->a_ops = &afs_fs_aops; |
diff --git a/fs/afs/proc.c b/fs/afs/proc.c index 101d21b6c037..86463ec9ccb4 100644 --- a/fs/afs/proc.c +++ b/fs/afs/proc.c | |||
@@ -775,6 +775,7 @@ static int afs_proc_cell_servers_release(struct inode *inode, | |||
775 | * first item | 775 | * first item |
776 | */ | 776 | */ |
777 | static void *afs_proc_cell_servers_start(struct seq_file *m, loff_t *_pos) | 777 | static void *afs_proc_cell_servers_start(struct seq_file *m, loff_t *_pos) |
778 | __acquires(m->private->sv_lock) | ||
778 | { | 779 | { |
779 | struct list_head *_p; | 780 | struct list_head *_p; |
780 | struct afs_cell *cell = m->private; | 781 | struct afs_cell *cell = m->private; |
@@ -823,6 +824,7 @@ static void *afs_proc_cell_servers_next(struct seq_file *p, void *v, | |||
823 | * clean up after reading from the cells list | 824 | * clean up after reading from the cells list |
824 | */ | 825 | */ |
825 | static void afs_proc_cell_servers_stop(struct seq_file *p, void *v) | 826 | static void afs_proc_cell_servers_stop(struct seq_file *p, void *v) |
827 | __releases(p->private->sv_lock) | ||
826 | { | 828 | { |
827 | struct afs_cell *cell = p->private; | 829 | struct afs_cell *cell = p->private; |
828 | 830 | ||
diff --git a/fs/afs/vlocation.c b/fs/afs/vlocation.c index 331f730a1fb3..782ee7c600ca 100644 --- a/fs/afs/vlocation.c +++ b/fs/afs/vlocation.c | |||
@@ -281,11 +281,10 @@ int afs_vlocation_lookup(struct afs_cell *cell, | |||
281 | spin_unlock(&cell->vl_gylock); | 281 | spin_unlock(&cell->vl_gylock); |
282 | 282 | ||
283 | /* not in the cell's in-memory lists - create a new record */ | 283 | /* not in the cell's in-memory lists - create a new record */ |
284 | vlocation = kmalloc(sizeof(struct afs_vlocation), GFP_KERNEL); | 284 | vlocation = kzalloc(sizeof(struct afs_vlocation), GFP_KERNEL); |
285 | if (!vlocation) | 285 | if (!vlocation) |
286 | return -ENOMEM; | 286 | return -ENOMEM; |
287 | 287 | ||
288 | memset(vlocation, 0, sizeof(struct afs_vlocation)); | ||
289 | atomic_set(&vlocation->usage, 1); | 288 | atomic_set(&vlocation->usage, 1); |
290 | INIT_LIST_HEAD(&vlocation->link); | 289 | INIT_LIST_HEAD(&vlocation->link); |
291 | rwlock_init(&vlocation->lock); | 290 | rwlock_init(&vlocation->lock); |
diff --git a/fs/afs/volume.c b/fs/afs/volume.c index 0ff4b86476e3..768c6dbd323a 100644 --- a/fs/afs/volume.c +++ b/fs/afs/volume.c | |||
@@ -186,11 +186,10 @@ int afs_volume_lookup(const char *name, struct afs_cell *cell, int rwpath, | |||
186 | _debug("creating new volume record"); | 186 | _debug("creating new volume record"); |
187 | 187 | ||
188 | ret = -ENOMEM; | 188 | ret = -ENOMEM; |
189 | volume = kmalloc(sizeof(struct afs_volume), GFP_KERNEL); | 189 | volume = kzalloc(sizeof(struct afs_volume), GFP_KERNEL); |
190 | if (!volume) | 190 | if (!volume) |
191 | goto error_up; | 191 | goto error_up; |
192 | 192 | ||
193 | memset(volume, 0, sizeof(struct afs_volume)); | ||
194 | atomic_set(&volume->usage, 1); | 193 | atomic_set(&volume->usage, 1); |
195 | volume->type = type; | 194 | volume->type = type; |
196 | volume->type_force = force; | 195 | volume->type_force = force; |