| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-only |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * linux/fs/nfs/dir.c |
|---|
| 3 | 4 | * |
|---|
| .. | .. |
|---|
| 57 | 58 | const struct file_operations nfs_dir_operations = { |
|---|
| 58 | 59 | .llseek = nfs_llseek_dir, |
|---|
| 59 | 60 | .read = generic_read_dir, |
|---|
| 60 | | - .iterate = nfs_readdir, |
|---|
| 61 | + .iterate_shared = nfs_readdir, |
|---|
| 61 | 62 | .open = nfs_opendir, |
|---|
| 62 | 63 | .release = nfs_closedir, |
|---|
| 63 | 64 | .fsync = nfs_fsync_dir, |
|---|
| .. | .. |
|---|
| 67 | 68 | .freepage = nfs_readdir_clear_array, |
|---|
| 68 | 69 | }; |
|---|
| 69 | 70 | |
|---|
| 70 | | -static struct nfs_open_dir_context *alloc_nfs_open_dir_context(struct inode *dir, struct rpc_cred *cred) |
|---|
| 71 | +static struct nfs_open_dir_context *alloc_nfs_open_dir_context(struct inode *dir, const struct cred *cred) |
|---|
| 71 | 72 | { |
|---|
| 72 | 73 | struct nfs_inode *nfsi = NFS_I(dir); |
|---|
| 73 | 74 | struct nfs_open_dir_context *ctx; |
|---|
| .. | .. |
|---|
| 77 | 78 | ctx->attr_gencount = nfsi->attr_gencount; |
|---|
| 78 | 79 | ctx->dir_cookie = 0; |
|---|
| 79 | 80 | ctx->dup_cookie = 0; |
|---|
| 80 | | - ctx->cred = get_rpccred(cred); |
|---|
| 81 | + ctx->cred = get_cred(cred); |
|---|
| 81 | 82 | spin_lock(&dir->i_lock); |
|---|
| 83 | + if (list_empty(&nfsi->open_files) && |
|---|
| 84 | + (nfsi->cache_validity & NFS_INO_DATA_INVAL_DEFER)) |
|---|
| 85 | + nfsi->cache_validity |= NFS_INO_INVALID_DATA | |
|---|
| 86 | + NFS_INO_REVAL_FORCED; |
|---|
| 82 | 87 | list_add(&ctx->list, &nfsi->open_files); |
|---|
| 83 | 88 | spin_unlock(&dir->i_lock); |
|---|
| 84 | 89 | return ctx; |
|---|
| .. | .. |
|---|
| 91 | 96 | spin_lock(&dir->i_lock); |
|---|
| 92 | 97 | list_del(&ctx->list); |
|---|
| 93 | 98 | spin_unlock(&dir->i_lock); |
|---|
| 94 | | - put_rpccred(ctx->cred); |
|---|
| 99 | + put_cred(ctx->cred); |
|---|
| 95 | 100 | kfree(ctx); |
|---|
| 96 | 101 | } |
|---|
| 97 | 102 | |
|---|
| .. | .. |
|---|
| 103 | 108 | { |
|---|
| 104 | 109 | int res = 0; |
|---|
| 105 | 110 | struct nfs_open_dir_context *ctx; |
|---|
| 106 | | - struct rpc_cred *cred; |
|---|
| 107 | 111 | |
|---|
| 108 | 112 | dfprintk(FILE, "NFS: open dir(%pD2)\n", filp); |
|---|
| 109 | 113 | |
|---|
| 110 | 114 | nfs_inc_stats(inode, NFSIOS_VFSOPEN); |
|---|
| 111 | 115 | |
|---|
| 112 | | - cred = rpc_lookup_cred(); |
|---|
| 113 | | - if (IS_ERR(cred)) |
|---|
| 114 | | - return PTR_ERR(cred); |
|---|
| 115 | | - ctx = alloc_nfs_open_dir_context(inode, cred); |
|---|
| 116 | + ctx = alloc_nfs_open_dir_context(inode, current_cred()); |
|---|
| 116 | 117 | if (IS_ERR(ctx)) { |
|---|
| 117 | 118 | res = PTR_ERR(ctx); |
|---|
| 118 | 119 | goto out; |
|---|
| 119 | 120 | } |
|---|
| 120 | 121 | filp->private_data = ctx; |
|---|
| 121 | 122 | out: |
|---|
| 122 | | - put_rpccred(cred); |
|---|
| 123 | 123 | return res; |
|---|
| 124 | 124 | } |
|---|
| 125 | 125 | |
|---|
| .. | .. |
|---|
| 141 | 141 | int size; |
|---|
| 142 | 142 | int eof_index; |
|---|
| 143 | 143 | u64 last_cookie; |
|---|
| 144 | | - struct nfs_cache_array_entry array[0]; |
|---|
| 144 | + struct nfs_cache_array_entry array[]; |
|---|
| 145 | 145 | }; |
|---|
| 146 | 146 | |
|---|
| 147 | | -typedef int (*decode_dirent_t)(struct xdr_stream *, struct nfs_entry *, bool); |
|---|
| 148 | 147 | typedef struct { |
|---|
| 149 | 148 | struct file *file; |
|---|
| 150 | 149 | struct page *page; |
|---|
| .. | .. |
|---|
| 153 | 152 | u64 *dir_cookie; |
|---|
| 154 | 153 | u64 last_cookie; |
|---|
| 155 | 154 | loff_t current_index; |
|---|
| 156 | | - decode_dirent_t decode; |
|---|
| 155 | + loff_t prev_index; |
|---|
| 157 | 156 | |
|---|
| 157 | + unsigned long dir_verifier; |
|---|
| 158 | 158 | unsigned long timestamp; |
|---|
| 159 | 159 | unsigned long gencount; |
|---|
| 160 | 160 | unsigned int cache_entry_index; |
|---|
| .. | .. |
|---|
| 198 | 198 | int nfs_readdir_make_qstr(struct qstr *string, const char *name, unsigned int len) |
|---|
| 199 | 199 | { |
|---|
| 200 | 200 | string->len = len; |
|---|
| 201 | | - string->name = kmemdup(name, len, GFP_KERNEL); |
|---|
| 201 | + string->name = kmemdup_nul(name, len, GFP_KERNEL); |
|---|
| 202 | 202 | if (string->name == NULL) |
|---|
| 203 | 203 | return -ENOMEM; |
|---|
| 204 | 204 | /* |
|---|
| .. | .. |
|---|
| 237 | 237 | out: |
|---|
| 238 | 238 | kunmap(page); |
|---|
| 239 | 239 | return ret; |
|---|
| 240 | +} |
|---|
| 241 | + |
|---|
| 242 | +static inline |
|---|
| 243 | +int is_32bit_api(void) |
|---|
| 244 | +{ |
|---|
| 245 | +#ifdef CONFIG_COMPAT |
|---|
| 246 | + return in_compat_syscall(); |
|---|
| 247 | +#else |
|---|
| 248 | + return (BITS_PER_LONG == 32); |
|---|
| 249 | +#endif |
|---|
| 250 | +} |
|---|
| 251 | + |
|---|
| 252 | +static |
|---|
| 253 | +bool nfs_readdir_use_cookie(const struct file *filp) |
|---|
| 254 | +{ |
|---|
| 255 | + if ((filp->f_mode & FMODE_32BITHASH) || |
|---|
| 256 | + (!(filp->f_mode & FMODE_64BITHASH) && is_32bit_api())) |
|---|
| 257 | + return false; |
|---|
| 258 | + return true; |
|---|
| 240 | 259 | } |
|---|
| 241 | 260 | |
|---|
| 242 | 261 | static |
|---|
| .. | .. |
|---|
| 288 | 307 | !nfs_readdir_inode_mapping_valid(nfsi)) { |
|---|
| 289 | 308 | ctx->duped = 0; |
|---|
| 290 | 309 | ctx->attr_gencount = nfsi->attr_gencount; |
|---|
| 291 | | - } else if (new_pos < desc->ctx->pos) { |
|---|
| 310 | + } else if (new_pos < desc->prev_index) { |
|---|
| 292 | 311 | if (ctx->duped > 0 |
|---|
| 293 | 312 | && ctx->dup_cookie == *desc->dir_cookie) { |
|---|
| 294 | 313 | if (printk_ratelimit()) { |
|---|
| .. | .. |
|---|
| 304 | 323 | ctx->dup_cookie = *desc->dir_cookie; |
|---|
| 305 | 324 | ctx->duped = -1; |
|---|
| 306 | 325 | } |
|---|
| 307 | | - desc->ctx->pos = new_pos; |
|---|
| 326 | + if (nfs_readdir_use_cookie(desc->file)) |
|---|
| 327 | + desc->ctx->pos = *desc->dir_cookie; |
|---|
| 328 | + else |
|---|
| 329 | + desc->ctx->pos = new_pos; |
|---|
| 330 | + desc->prev_index = new_pos; |
|---|
| 308 | 331 | desc->cache_entry_index = i; |
|---|
| 309 | 332 | return 0; |
|---|
| 310 | 333 | } |
|---|
| .. | .. |
|---|
| 346 | 369 | struct nfs_entry *entry, struct file *file, struct inode *inode) |
|---|
| 347 | 370 | { |
|---|
| 348 | 371 | struct nfs_open_dir_context *ctx = file->private_data; |
|---|
| 349 | | - struct rpc_cred *cred = ctx->cred; |
|---|
| 372 | + const struct cred *cred = ctx->cred; |
|---|
| 350 | 373 | unsigned long timestamp, gencount; |
|---|
| 351 | 374 | int error; |
|---|
| 352 | 375 | |
|---|
| 353 | 376 | again: |
|---|
| 354 | 377 | timestamp = jiffies; |
|---|
| 355 | 378 | gencount = nfs_inc_attr_generation_counter(); |
|---|
| 379 | + desc->dir_verifier = nfs_save_change_attribute(inode); |
|---|
| 356 | 380 | error = NFS_PROTO(inode)->readdir(file_dentry(file), cred, entry->cookie, pages, |
|---|
| 357 | 381 | NFS_SERVER(inode)->dtsize, desc->plus); |
|---|
| 358 | 382 | if (error < 0) { |
|---|
| .. | .. |
|---|
| 374 | 398 | static int xdr_decode(nfs_readdir_descriptor_t *desc, |
|---|
| 375 | 399 | struct nfs_entry *entry, struct xdr_stream *xdr) |
|---|
| 376 | 400 | { |
|---|
| 401 | + struct inode *inode = file_inode(desc->file); |
|---|
| 377 | 402 | int error; |
|---|
| 378 | 403 | |
|---|
| 379 | | - error = desc->decode(xdr, entry, desc->plus); |
|---|
| 404 | + error = NFS_PROTO(inode)->decode_dirent(xdr, entry, desc->plus); |
|---|
| 380 | 405 | if (error) |
|---|
| 381 | 406 | return error; |
|---|
| 382 | 407 | entry->fattr->time_start = desc->timestamp; |
|---|
| .. | .. |
|---|
| 449 | 474 | if (nfs_server_capable(dir, NFS_CAP_READDIRPLUS) && |
|---|
| 450 | 475 | !list_empty(&nfsi->open_files)) { |
|---|
| 451 | 476 | set_bit(NFS_INO_ADVISE_RDPLUS, &nfsi->flags); |
|---|
| 452 | | - invalidate_mapping_pages(dir->i_mapping, 0, -1); |
|---|
| 477 | + invalidate_mapping_pages(dir->i_mapping, |
|---|
| 478 | + nfsi->page_index + 1, -1); |
|---|
| 453 | 479 | } |
|---|
| 454 | 480 | } |
|---|
| 455 | 481 | |
|---|
| 456 | 482 | static |
|---|
| 457 | | -void nfs_prime_dcache(struct dentry *parent, struct nfs_entry *entry) |
|---|
| 483 | +void nfs_prime_dcache(struct dentry *parent, struct nfs_entry *entry, |
|---|
| 484 | + unsigned long dir_verifier) |
|---|
| 458 | 485 | { |
|---|
| 459 | 486 | struct qstr filename = QSTR_INIT(entry->name, entry->len); |
|---|
| 460 | | - DECLARE_SWAIT_QUEUE_HEAD_ONSTACK(wq); |
|---|
| 487 | + DECLARE_WAIT_QUEUE_HEAD_ONSTACK(wq); |
|---|
| 461 | 488 | struct dentry *dentry; |
|---|
| 462 | 489 | struct dentry *alias; |
|---|
| 463 | | - struct inode *dir = d_inode(parent); |
|---|
| 464 | 490 | struct inode *inode; |
|---|
| 465 | 491 | int status; |
|---|
| 466 | 492 | |
|---|
| .. | .. |
|---|
| 499 | 525 | if (nfs_same_file(dentry, entry)) { |
|---|
| 500 | 526 | if (!entry->fh->size) |
|---|
| 501 | 527 | goto out; |
|---|
| 502 | | - nfs_set_verifier(dentry, nfs_save_change_attribute(dir)); |
|---|
| 528 | + nfs_set_verifier(dentry, dir_verifier); |
|---|
| 503 | 529 | status = nfs_refresh_inode(d_inode(dentry), entry->fattr); |
|---|
| 504 | 530 | if (!status) |
|---|
| 505 | 531 | nfs_setsecurity(d_inode(dentry), entry->fattr, entry->label); |
|---|
| .. | .. |
|---|
| 525 | 551 | dput(dentry); |
|---|
| 526 | 552 | dentry = alias; |
|---|
| 527 | 553 | } |
|---|
| 528 | | - nfs_set_verifier(dentry, nfs_save_change_attribute(dir)); |
|---|
| 554 | + nfs_set_verifier(dentry, dir_verifier); |
|---|
| 529 | 555 | out: |
|---|
| 530 | 556 | dput(dentry); |
|---|
| 531 | 557 | } |
|---|
| .. | .. |
|---|
| 566 | 592 | count++; |
|---|
| 567 | 593 | |
|---|
| 568 | 594 | if (desc->plus) |
|---|
| 569 | | - nfs_prime_dcache(file_dentry(desc->file), entry); |
|---|
| 595 | + nfs_prime_dcache(file_dentry(desc->file), entry, |
|---|
| 596 | + desc->dir_verifier); |
|---|
| 570 | 597 | |
|---|
| 571 | 598 | status = nfs_readdir_add_to_array(entry, page); |
|---|
| 572 | 599 | if (status != 0) |
|---|
| .. | .. |
|---|
| 594 | 621 | } |
|---|
| 595 | 622 | |
|---|
| 596 | 623 | /* |
|---|
| 597 | | - * nfs_readdir_large_page will allocate pages that must be freed with a call |
|---|
| 598 | | - * to nfs_readdir_free_pagearray |
|---|
| 624 | + * nfs_readdir_alloc_pages() will allocate pages that must be freed with a call |
|---|
| 625 | + * to nfs_readdir_free_pages() |
|---|
| 599 | 626 | */ |
|---|
| 600 | 627 | static |
|---|
| 601 | 628 | int nfs_readdir_alloc_pages(struct page **pages, unsigned int npages) |
|---|
| .. | .. |
|---|
| 723 | 750 | static |
|---|
| 724 | 751 | int find_and_lock_cache_page(nfs_readdir_descriptor_t *desc) |
|---|
| 725 | 752 | { |
|---|
| 753 | + struct inode *inode = file_inode(desc->file); |
|---|
| 754 | + struct nfs_inode *nfsi = NFS_I(inode); |
|---|
| 726 | 755 | int res; |
|---|
| 727 | 756 | |
|---|
| 728 | 757 | desc->page = get_cache_page(desc); |
|---|
| .. | .. |
|---|
| 734 | 763 | res = -EAGAIN; |
|---|
| 735 | 764 | if (desc->page->mapping != NULL) { |
|---|
| 736 | 765 | res = nfs_readdir_search_array(desc); |
|---|
| 737 | | - if (res == 0) |
|---|
| 766 | + if (res == 0) { |
|---|
| 767 | + nfsi->page_index = desc->page_index; |
|---|
| 738 | 768 | return 0; |
|---|
| 769 | + } |
|---|
| 739 | 770 | } |
|---|
| 740 | 771 | unlock_page(desc->page); |
|---|
| 741 | 772 | error: |
|---|
| .. | .. |
|---|
| 751 | 782 | |
|---|
| 752 | 783 | if (desc->page_index == 0) { |
|---|
| 753 | 784 | desc->current_index = 0; |
|---|
| 785 | + desc->prev_index = 0; |
|---|
| 754 | 786 | desc->last_cookie = 0; |
|---|
| 755 | 787 | } |
|---|
| 756 | 788 | do { |
|---|
| .. | .. |
|---|
| 781 | 813 | desc->eof = true; |
|---|
| 782 | 814 | break; |
|---|
| 783 | 815 | } |
|---|
| 784 | | - desc->ctx->pos++; |
|---|
| 785 | 816 | if (i < (array->size-1)) |
|---|
| 786 | 817 | *desc->dir_cookie = array->array[i+1].cookie; |
|---|
| 787 | 818 | else |
|---|
| 788 | 819 | *desc->dir_cookie = array->last_cookie; |
|---|
| 820 | + if (nfs_readdir_use_cookie(file)) |
|---|
| 821 | + desc->ctx->pos = *desc->dir_cookie; |
|---|
| 822 | + else |
|---|
| 823 | + desc->ctx->pos++; |
|---|
| 789 | 824 | if (ctx->duped != 0) |
|---|
| 790 | 825 | ctx->duped = 1; |
|---|
| 791 | 826 | } |
|---|
| .. | .. |
|---|
| 855 | 890 | { |
|---|
| 856 | 891 | struct dentry *dentry = file_dentry(file); |
|---|
| 857 | 892 | struct inode *inode = d_inode(dentry); |
|---|
| 858 | | - nfs_readdir_descriptor_t my_desc, |
|---|
| 859 | | - *desc = &my_desc; |
|---|
| 860 | 893 | struct nfs_open_dir_context *dir_ctx = file->private_data; |
|---|
| 894 | + nfs_readdir_descriptor_t my_desc = { |
|---|
| 895 | + .file = file, |
|---|
| 896 | + .ctx = ctx, |
|---|
| 897 | + .dir_cookie = &dir_ctx->dir_cookie, |
|---|
| 898 | + .plus = nfs_use_readdirplus(inode, ctx), |
|---|
| 899 | + }, |
|---|
| 900 | + *desc = &my_desc; |
|---|
| 861 | 901 | int res = 0; |
|---|
| 862 | 902 | |
|---|
| 863 | 903 | dfprintk(FILE, "NFS: readdir(%pD2) starting at cookie %llu\n", |
|---|
| .. | .. |
|---|
| 870 | 910 | * to either find the entry with the appropriate number or |
|---|
| 871 | 911 | * revalidate the cookie. |
|---|
| 872 | 912 | */ |
|---|
| 873 | | - memset(desc, 0, sizeof(*desc)); |
|---|
| 874 | | - |
|---|
| 875 | | - desc->file = file; |
|---|
| 876 | | - desc->ctx = ctx; |
|---|
| 877 | | - desc->dir_cookie = &dir_ctx->dir_cookie; |
|---|
| 878 | | - desc->decode = NFS_PROTO(inode)->decode_dirent; |
|---|
| 879 | | - desc->plus = nfs_use_readdirplus(inode, ctx); |
|---|
| 880 | | - |
|---|
| 881 | 913 | if (ctx->pos == 0 || nfs_attribute_cache_expired(inode)) |
|---|
| 882 | 914 | res = nfs_revalidate_mapping(inode, file->f_mapping); |
|---|
| 883 | 915 | if (res < 0) |
|---|
| .. | .. |
|---|
| 923 | 955 | |
|---|
| 924 | 956 | static loff_t nfs_llseek_dir(struct file *filp, loff_t offset, int whence) |
|---|
| 925 | 957 | { |
|---|
| 926 | | - struct inode *inode = file_inode(filp); |
|---|
| 927 | 958 | struct nfs_open_dir_context *dir_ctx = filp->private_data; |
|---|
| 928 | 959 | |
|---|
| 929 | 960 | dfprintk(FILE, "NFS: llseek dir(%pD2, %lld, %d)\n", |
|---|
| .. | .. |
|---|
| 935 | 966 | case SEEK_SET: |
|---|
| 936 | 967 | if (offset < 0) |
|---|
| 937 | 968 | return -EINVAL; |
|---|
| 938 | | - inode_lock(inode); |
|---|
| 969 | + spin_lock(&filp->f_lock); |
|---|
| 939 | 970 | break; |
|---|
| 940 | 971 | case SEEK_CUR: |
|---|
| 941 | 972 | if (offset == 0) |
|---|
| 942 | 973 | return filp->f_pos; |
|---|
| 943 | | - inode_lock(inode); |
|---|
| 974 | + spin_lock(&filp->f_lock); |
|---|
| 944 | 975 | offset += filp->f_pos; |
|---|
| 945 | 976 | if (offset < 0) { |
|---|
| 946 | | - inode_unlock(inode); |
|---|
| 977 | + spin_unlock(&filp->f_lock); |
|---|
| 947 | 978 | return -EINVAL; |
|---|
| 948 | 979 | } |
|---|
| 949 | 980 | } |
|---|
| 950 | 981 | if (offset != filp->f_pos) { |
|---|
| 951 | 982 | filp->f_pos = offset; |
|---|
| 952 | | - dir_ctx->dir_cookie = 0; |
|---|
| 983 | + if (nfs_readdir_use_cookie(filp)) |
|---|
| 984 | + dir_ctx->dir_cookie = offset; |
|---|
| 985 | + else |
|---|
| 986 | + dir_ctx->dir_cookie = 0; |
|---|
| 953 | 987 | dir_ctx->duped = 0; |
|---|
| 954 | 988 | } |
|---|
| 955 | | - inode_unlock(inode); |
|---|
| 989 | + spin_unlock(&filp->f_lock); |
|---|
| 956 | 990 | return offset; |
|---|
| 957 | 991 | } |
|---|
| 958 | 992 | |
|---|
| .. | .. |
|---|
| 963 | 997 | static int nfs_fsync_dir(struct file *filp, loff_t start, loff_t end, |
|---|
| 964 | 998 | int datasync) |
|---|
| 965 | 999 | { |
|---|
| 966 | | - struct inode *inode = file_inode(filp); |
|---|
| 967 | | - |
|---|
| 968 | 1000 | dfprintk(FILE, "NFS: fsync dir(%pD2) datasync %d\n", filp, datasync); |
|---|
| 969 | 1001 | |
|---|
| 970 | | - inode_lock(inode); |
|---|
| 971 | | - nfs_inc_stats(inode, NFSIOS_VFSFSYNC); |
|---|
| 972 | | - inode_unlock(inode); |
|---|
| 1002 | + nfs_inc_stats(file_inode(filp), NFSIOS_VFSFSYNC); |
|---|
| 973 | 1003 | return 0; |
|---|
| 974 | 1004 | } |
|---|
| 975 | 1005 | |
|---|
| 976 | 1006 | /** |
|---|
| 977 | 1007 | * nfs_force_lookup_revalidate - Mark the directory as having changed |
|---|
| 978 | | - * @dir - pointer to directory inode |
|---|
| 1008 | + * @dir: pointer to directory inode |
|---|
| 979 | 1009 | * |
|---|
| 980 | 1010 | * This forces the revalidation code in nfs_lookup_revalidate() to do a |
|---|
| 981 | 1011 | * full lookup on all child dentries of 'dir' whenever a change occurs |
|---|
| 982 | 1012 | * on the server that might have invalidated our dcache. |
|---|
| 983 | 1013 | * |
|---|
| 1014 | + * Note that we reserve bit '0' as a tag to let us know when a dentry |
|---|
| 1015 | + * was revalidated while holding a delegation on its inode. |
|---|
| 1016 | + * |
|---|
| 984 | 1017 | * The caller should be holding dir->i_lock |
|---|
| 985 | 1018 | */ |
|---|
| 986 | 1019 | void nfs_force_lookup_revalidate(struct inode *dir) |
|---|
| 987 | 1020 | { |
|---|
| 988 | | - NFS_I(dir)->cache_change_attribute++; |
|---|
| 1021 | + NFS_I(dir)->cache_change_attribute += 2; |
|---|
| 989 | 1022 | } |
|---|
| 990 | 1023 | EXPORT_SYMBOL_GPL(nfs_force_lookup_revalidate); |
|---|
| 1024 | + |
|---|
| 1025 | +/** |
|---|
| 1026 | + * nfs_verify_change_attribute - Detects NFS remote directory changes |
|---|
| 1027 | + * @dir: pointer to parent directory inode |
|---|
| 1028 | + * @verf: previously saved change attribute |
|---|
| 1029 | + * |
|---|
| 1030 | + * Return "false" if the verifiers doesn't match the change attribute. |
|---|
| 1031 | + * This would usually indicate that the directory contents have changed on |
|---|
| 1032 | + * the server, and that any dentries need revalidating. |
|---|
| 1033 | + */ |
|---|
| 1034 | +static bool nfs_verify_change_attribute(struct inode *dir, unsigned long verf) |
|---|
| 1035 | +{ |
|---|
| 1036 | + return (verf & ~1UL) == nfs_save_change_attribute(dir); |
|---|
| 1037 | +} |
|---|
| 1038 | + |
|---|
| 1039 | +static void nfs_set_verifier_delegated(unsigned long *verf) |
|---|
| 1040 | +{ |
|---|
| 1041 | + *verf |= 1UL; |
|---|
| 1042 | +} |
|---|
| 1043 | + |
|---|
| 1044 | +#if IS_ENABLED(CONFIG_NFS_V4) |
|---|
| 1045 | +static void nfs_unset_verifier_delegated(unsigned long *verf) |
|---|
| 1046 | +{ |
|---|
| 1047 | + *verf &= ~1UL; |
|---|
| 1048 | +} |
|---|
| 1049 | +#endif /* IS_ENABLED(CONFIG_NFS_V4) */ |
|---|
| 1050 | + |
|---|
| 1051 | +static bool nfs_test_verifier_delegated(unsigned long verf) |
|---|
| 1052 | +{ |
|---|
| 1053 | + return verf & 1; |
|---|
| 1054 | +} |
|---|
| 1055 | + |
|---|
| 1056 | +static bool nfs_verifier_is_delegated(struct dentry *dentry) |
|---|
| 1057 | +{ |
|---|
| 1058 | + return nfs_test_verifier_delegated(dentry->d_time); |
|---|
| 1059 | +} |
|---|
| 1060 | + |
|---|
| 1061 | +static void nfs_set_verifier_locked(struct dentry *dentry, unsigned long verf) |
|---|
| 1062 | +{ |
|---|
| 1063 | + struct inode *inode = d_inode(dentry); |
|---|
| 1064 | + struct inode *dir = d_inode(dentry->d_parent); |
|---|
| 1065 | + |
|---|
| 1066 | + if (!nfs_verify_change_attribute(dir, verf)) |
|---|
| 1067 | + return; |
|---|
| 1068 | + if (inode && NFS_PROTO(inode)->have_delegation(inode, FMODE_READ)) |
|---|
| 1069 | + nfs_set_verifier_delegated(&verf); |
|---|
| 1070 | + dentry->d_time = verf; |
|---|
| 1071 | +} |
|---|
| 1072 | + |
|---|
| 1073 | +/** |
|---|
| 1074 | + * nfs_set_verifier - save a parent directory verifier in the dentry |
|---|
| 1075 | + * @dentry: pointer to dentry |
|---|
| 1076 | + * @verf: verifier to save |
|---|
| 1077 | + * |
|---|
| 1078 | + * Saves the parent directory verifier in @dentry. If the inode has |
|---|
| 1079 | + * a delegation, we also tag the dentry as having been revalidated |
|---|
| 1080 | + * while holding a delegation so that we know we don't have to |
|---|
| 1081 | + * look it up again after a directory change. |
|---|
| 1082 | + */ |
|---|
| 1083 | +void nfs_set_verifier(struct dentry *dentry, unsigned long verf) |
|---|
| 1084 | +{ |
|---|
| 1085 | + |
|---|
| 1086 | + spin_lock(&dentry->d_lock); |
|---|
| 1087 | + nfs_set_verifier_locked(dentry, verf); |
|---|
| 1088 | + spin_unlock(&dentry->d_lock); |
|---|
| 1089 | +} |
|---|
| 1090 | +EXPORT_SYMBOL_GPL(nfs_set_verifier); |
|---|
| 1091 | + |
|---|
| 1092 | +#if IS_ENABLED(CONFIG_NFS_V4) |
|---|
| 1093 | +/** |
|---|
| 1094 | + * nfs_clear_verifier_delegated - clear the dir verifier delegation tag |
|---|
| 1095 | + * @inode: pointer to inode |
|---|
| 1096 | + * |
|---|
| 1097 | + * Iterates through the dentries in the inode alias list and clears |
|---|
| 1098 | + * the tag used to indicate that the dentry has been revalidated |
|---|
| 1099 | + * while holding a delegation. |
|---|
| 1100 | + * This function is intended for use when the delegation is being |
|---|
| 1101 | + * returned or revoked. |
|---|
| 1102 | + */ |
|---|
| 1103 | +void nfs_clear_verifier_delegated(struct inode *inode) |
|---|
| 1104 | +{ |
|---|
| 1105 | + struct dentry *alias; |
|---|
| 1106 | + |
|---|
| 1107 | + if (!inode) |
|---|
| 1108 | + return; |
|---|
| 1109 | + spin_lock(&inode->i_lock); |
|---|
| 1110 | + hlist_for_each_entry(alias, &inode->i_dentry, d_u.d_alias) { |
|---|
| 1111 | + spin_lock(&alias->d_lock); |
|---|
| 1112 | + nfs_unset_verifier_delegated(&alias->d_time); |
|---|
| 1113 | + spin_unlock(&alias->d_lock); |
|---|
| 1114 | + } |
|---|
| 1115 | + spin_unlock(&inode->i_lock); |
|---|
| 1116 | +} |
|---|
| 1117 | +EXPORT_SYMBOL_GPL(nfs_clear_verifier_delegated); |
|---|
| 1118 | +#endif /* IS_ENABLED(CONFIG_NFS_V4) */ |
|---|
| 991 | 1119 | |
|---|
| 992 | 1120 | /* |
|---|
| 993 | 1121 | * A check for whether or not the parent directory has changed. |
|---|
| .. | .. |
|---|
| 1050 | 1178 | /* A NFSv4 OPEN will revalidate later */ |
|---|
| 1051 | 1179 | if (server->caps & NFS_CAP_ATOMIC_OPEN) |
|---|
| 1052 | 1180 | goto out; |
|---|
| 1053 | | - /* Fallthrough */ |
|---|
| 1181 | + fallthrough; |
|---|
| 1054 | 1182 | case S_IFDIR: |
|---|
| 1055 | 1183 | if (server->flags & NFS_MOUNT_NOCTO) |
|---|
| 1056 | 1184 | break; |
|---|
| .. | .. |
|---|
| 1071 | 1199 | if (ret != 0) |
|---|
| 1072 | 1200 | return ret; |
|---|
| 1073 | 1201 | goto out; |
|---|
| 1202 | +} |
|---|
| 1203 | + |
|---|
| 1204 | +static void nfs_mark_dir_for_revalidate(struct inode *inode) |
|---|
| 1205 | +{ |
|---|
| 1206 | + struct nfs_inode *nfsi = NFS_I(inode); |
|---|
| 1207 | + |
|---|
| 1208 | + spin_lock(&inode->i_lock); |
|---|
| 1209 | + nfsi->cache_validity |= NFS_INO_REVAL_PAGECACHE; |
|---|
| 1210 | + spin_unlock(&inode->i_lock); |
|---|
| 1074 | 1211 | } |
|---|
| 1075 | 1212 | |
|---|
| 1076 | 1213 | /* |
|---|
| .. | .. |
|---|
| 1107 | 1244 | __func__, dentry); |
|---|
| 1108 | 1245 | return 1; |
|---|
| 1109 | 1246 | case 0: |
|---|
| 1110 | | - nfs_mark_for_revalidate(dir); |
|---|
| 1111 | | - if (inode && S_ISDIR(inode->i_mode)) { |
|---|
| 1112 | | - /* Purge readdir caches. */ |
|---|
| 1113 | | - nfs_zap_caches(inode); |
|---|
| 1114 | | - /* |
|---|
| 1115 | | - * We can't d_drop the root of a disconnected tree: |
|---|
| 1116 | | - * its d_hash is on the s_anon list and d_drop() would hide |
|---|
| 1117 | | - * it from shrink_dcache_for_unmount(), leading to busy |
|---|
| 1118 | | - * inodes on unmount and further oopses. |
|---|
| 1119 | | - */ |
|---|
| 1120 | | - if (IS_ROOT(dentry)) |
|---|
| 1121 | | - return 1; |
|---|
| 1122 | | - } |
|---|
| 1247 | + /* |
|---|
| 1248 | + * We can't d_drop the root of a disconnected tree: |
|---|
| 1249 | + * its d_hash is on the s_anon list and d_drop() would hide |
|---|
| 1250 | + * it from shrink_dcache_for_unmount(), leading to busy |
|---|
| 1251 | + * inodes on unmount and further oopses. |
|---|
| 1252 | + */ |
|---|
| 1253 | + if (inode && IS_ROOT(dentry)) |
|---|
| 1254 | + return 1; |
|---|
| 1123 | 1255 | dfprintk(LOOKUPCACHE, "NFS: %s(%pd2) is invalid\n", |
|---|
| 1124 | 1256 | __func__, dentry); |
|---|
| 1125 | 1257 | return 0; |
|---|
| .. | .. |
|---|
| 1157 | 1289 | struct nfs_fh *fhandle; |
|---|
| 1158 | 1290 | struct nfs_fattr *fattr; |
|---|
| 1159 | 1291 | struct nfs4_label *label; |
|---|
| 1292 | + unsigned long dir_verifier; |
|---|
| 1160 | 1293 | int ret; |
|---|
| 1161 | 1294 | |
|---|
| 1162 | 1295 | ret = -ENOMEM; |
|---|
| .. | .. |
|---|
| 1166 | 1299 | if (fhandle == NULL || fattr == NULL || IS_ERR(label)) |
|---|
| 1167 | 1300 | goto out; |
|---|
| 1168 | 1301 | |
|---|
| 1169 | | - ret = NFS_PROTO(dir)->lookup(dir, &dentry->d_name, fhandle, fattr, label); |
|---|
| 1302 | + dir_verifier = nfs_save_change_attribute(dir); |
|---|
| 1303 | + ret = NFS_PROTO(dir)->lookup(dir, dentry, fhandle, fattr, label); |
|---|
| 1170 | 1304 | if (ret < 0) { |
|---|
| 1171 | | - if (ret == -ESTALE || ret == -ENOENT) |
|---|
| 1305 | + switch (ret) { |
|---|
| 1306 | + case -ESTALE: |
|---|
| 1307 | + case -ENOENT: |
|---|
| 1172 | 1308 | ret = 0; |
|---|
| 1309 | + break; |
|---|
| 1310 | + case -ETIMEDOUT: |
|---|
| 1311 | + if (NFS_SERVER(inode)->flags & NFS_MOUNT_SOFTREVAL) |
|---|
| 1312 | + ret = 1; |
|---|
| 1313 | + } |
|---|
| 1173 | 1314 | goto out; |
|---|
| 1174 | 1315 | } |
|---|
| 1175 | 1316 | ret = 0; |
|---|
| .. | .. |
|---|
| 1179 | 1320 | goto out; |
|---|
| 1180 | 1321 | |
|---|
| 1181 | 1322 | nfs_setsecurity(inode, fattr, label); |
|---|
| 1182 | | - nfs_set_verifier(dentry, nfs_save_change_attribute(dir)); |
|---|
| 1323 | + nfs_set_verifier(dentry, dir_verifier); |
|---|
| 1183 | 1324 | |
|---|
| 1184 | 1325 | /* set a readdirplus hint that we had a cache miss */ |
|---|
| 1185 | 1326 | nfs_force_use_readdirplus(dir); |
|---|
| .. | .. |
|---|
| 1188 | 1329 | nfs_free_fattr(fattr); |
|---|
| 1189 | 1330 | nfs_free_fhandle(fhandle); |
|---|
| 1190 | 1331 | nfs4_label_free(label); |
|---|
| 1332 | + |
|---|
| 1333 | + /* |
|---|
| 1334 | + * If the lookup failed despite the dentry change attribute being |
|---|
| 1335 | + * a match, then we should revalidate the directory cache. |
|---|
| 1336 | + */ |
|---|
| 1337 | + if (!ret && nfs_verify_change_attribute(dir, dentry->d_time)) |
|---|
| 1338 | + nfs_mark_dir_for_revalidate(dir); |
|---|
| 1191 | 1339 | return nfs_lookup_revalidate_done(dir, dentry, inode, ret); |
|---|
| 1192 | 1340 | } |
|---|
| 1193 | 1341 | |
|---|
| .. | .. |
|---|
| 1221 | 1369 | goto out_bad; |
|---|
| 1222 | 1370 | } |
|---|
| 1223 | 1371 | |
|---|
| 1224 | | - if (NFS_PROTO(dir)->have_delegation(inode, FMODE_READ)) |
|---|
| 1372 | + if (nfs_verifier_is_delegated(dentry)) |
|---|
| 1225 | 1373 | return nfs_lookup_revalidate_delegated(dir, dentry, inode); |
|---|
| 1226 | 1374 | |
|---|
| 1227 | 1375 | /* Force a full look up iff the parent directory has changed */ |
|---|
| .. | .. |
|---|
| 1230 | 1378 | error = nfs_lookup_verify_inode(inode, flags); |
|---|
| 1231 | 1379 | if (error) { |
|---|
| 1232 | 1380 | if (error == -ESTALE) |
|---|
| 1233 | | - nfs_zap_caches(dir); |
|---|
| 1381 | + nfs_mark_dir_for_revalidate(dir); |
|---|
| 1234 | 1382 | goto out_bad; |
|---|
| 1235 | 1383 | } |
|---|
| 1236 | 1384 | nfs_advise_use_readdirplus(dir); |
|---|
| .. | .. |
|---|
| 1406 | 1554 | struct nfs_fh *fhandle = NULL; |
|---|
| 1407 | 1555 | struct nfs_fattr *fattr = NULL; |
|---|
| 1408 | 1556 | struct nfs4_label *label = NULL; |
|---|
| 1557 | + unsigned long dir_verifier; |
|---|
| 1409 | 1558 | int error; |
|---|
| 1410 | 1559 | |
|---|
| 1411 | 1560 | dfprintk(VFS, "NFS: lookup(%pd2)\n", dentry); |
|---|
| .. | .. |
|---|
| 1431 | 1580 | if (IS_ERR(label)) |
|---|
| 1432 | 1581 | goto out; |
|---|
| 1433 | 1582 | |
|---|
| 1583 | + dir_verifier = nfs_save_change_attribute(dir); |
|---|
| 1434 | 1584 | trace_nfs_lookup_enter(dir, dentry, flags); |
|---|
| 1435 | | - error = NFS_PROTO(dir)->lookup(dir, &dentry->d_name, fhandle, fattr, label); |
|---|
| 1585 | + error = NFS_PROTO(dir)->lookup(dir, dentry, fhandle, fattr, label); |
|---|
| 1436 | 1586 | if (error == -ENOENT) |
|---|
| 1437 | 1587 | goto no_entry; |
|---|
| 1438 | 1588 | if (error < 0) { |
|---|
| .. | .. |
|---|
| 1454 | 1604 | goto out_label; |
|---|
| 1455 | 1605 | dentry = res; |
|---|
| 1456 | 1606 | } |
|---|
| 1457 | | - nfs_set_verifier(dentry, nfs_save_change_attribute(dir)); |
|---|
| 1607 | + nfs_set_verifier(dentry, dir_verifier); |
|---|
| 1458 | 1608 | out_label: |
|---|
| 1459 | 1609 | trace_nfs_lookup_exit(dir, dentry, flags, error); |
|---|
| 1460 | 1610 | nfs4_label_free(label); |
|---|
| .. | .. |
|---|
| 1477 | 1627 | .d_release = nfs_d_release, |
|---|
| 1478 | 1628 | }; |
|---|
| 1479 | 1629 | EXPORT_SYMBOL_GPL(nfs4_dentry_operations); |
|---|
| 1480 | | - |
|---|
| 1481 | | -static fmode_t flags_to_mode(int flags) |
|---|
| 1482 | | -{ |
|---|
| 1483 | | - fmode_t res = (__force fmode_t)flags & FMODE_EXEC; |
|---|
| 1484 | | - if ((flags & O_ACCMODE) != O_WRONLY) |
|---|
| 1485 | | - res |= FMODE_READ; |
|---|
| 1486 | | - if ((flags & O_ACCMODE) != O_RDONLY) |
|---|
| 1487 | | - res |= FMODE_WRITE; |
|---|
| 1488 | | - return res; |
|---|
| 1489 | | -} |
|---|
| 1490 | 1630 | |
|---|
| 1491 | 1631 | static struct nfs_open_context *create_nfs_open_context(struct dentry *dentry, int open_flags, struct file *filp) |
|---|
| 1492 | 1632 | { |
|---|
| .. | .. |
|---|
| 1520 | 1660 | struct file *file, unsigned open_flags, |
|---|
| 1521 | 1661 | umode_t mode) |
|---|
| 1522 | 1662 | { |
|---|
| 1523 | | - DECLARE_SWAIT_QUEUE_HEAD_ONSTACK(wq); |
|---|
| 1663 | + DECLARE_WAIT_QUEUE_HEAD_ONSTACK(wq); |
|---|
| 1524 | 1664 | struct nfs_open_context *ctx; |
|---|
| 1525 | 1665 | struct dentry *res; |
|---|
| 1526 | 1666 | struct iattr attr = { .ia_valid = ATTR_OPEN }; |
|---|
| .. | .. |
|---|
| 1677 | 1817 | if (inode == NULL) |
|---|
| 1678 | 1818 | goto full_reval; |
|---|
| 1679 | 1819 | |
|---|
| 1680 | | - if (NFS_PROTO(dir)->have_delegation(inode, FMODE_READ)) |
|---|
| 1820 | + if (nfs_verifier_is_delegated(dentry)) |
|---|
| 1681 | 1821 | return nfs_lookup_revalidate_delegated(dir, dentry, inode); |
|---|
| 1682 | 1822 | |
|---|
| 1683 | 1823 | /* NFS only supports OPEN on regular files */ |
|---|
| .. | .. |
|---|
| 1697 | 1837 | reval_dentry: |
|---|
| 1698 | 1838 | if (flags & LOOKUP_RCU) |
|---|
| 1699 | 1839 | return -ECHILD; |
|---|
| 1700 | | - return nfs_lookup_revalidate_dentry(dir, dentry, inode);; |
|---|
| 1840 | + return nfs_lookup_revalidate_dentry(dir, dentry, inode); |
|---|
| 1701 | 1841 | |
|---|
| 1702 | 1842 | full_reval: |
|---|
| 1703 | 1843 | return nfs_do_lookup_revalidate(dir, dentry, flags); |
|---|
| .. | .. |
|---|
| 1711 | 1851 | |
|---|
| 1712 | 1852 | #endif /* CONFIG_NFSV4 */ |
|---|
| 1713 | 1853 | |
|---|
| 1714 | | -/* |
|---|
| 1715 | | - * Code common to create, mkdir, and mknod. |
|---|
| 1716 | | - */ |
|---|
| 1717 | | -int nfs_instantiate(struct dentry *dentry, struct nfs_fh *fhandle, |
|---|
| 1854 | +struct dentry * |
|---|
| 1855 | +nfs_add_or_obtain(struct dentry *dentry, struct nfs_fh *fhandle, |
|---|
| 1718 | 1856 | struct nfs_fattr *fattr, |
|---|
| 1719 | 1857 | struct nfs4_label *label) |
|---|
| 1720 | 1858 | { |
|---|
| .. | .. |
|---|
| 1722 | 1860 | struct inode *dir = d_inode(parent); |
|---|
| 1723 | 1861 | struct inode *inode; |
|---|
| 1724 | 1862 | struct dentry *d; |
|---|
| 1725 | | - int error = -EACCES; |
|---|
| 1863 | + int error; |
|---|
| 1726 | 1864 | |
|---|
| 1727 | 1865 | d_drop(dentry); |
|---|
| 1728 | 1866 | |
|---|
| 1729 | | - /* We may have been initialized further down */ |
|---|
| 1730 | | - if (d_really_is_positive(dentry)) |
|---|
| 1731 | | - goto out; |
|---|
| 1732 | 1867 | if (fhandle->size == 0) { |
|---|
| 1733 | | - error = NFS_PROTO(dir)->lookup(dir, &dentry->d_name, fhandle, fattr, NULL); |
|---|
| 1868 | + error = NFS_PROTO(dir)->lookup(dir, dentry, fhandle, fattr, NULL); |
|---|
| 1734 | 1869 | if (error) |
|---|
| 1735 | 1870 | goto out_error; |
|---|
| 1736 | 1871 | } |
|---|
| .. | .. |
|---|
| 1744 | 1879 | } |
|---|
| 1745 | 1880 | inode = nfs_fhget(dentry->d_sb, fhandle, fattr, label); |
|---|
| 1746 | 1881 | d = d_splice_alias(inode, dentry); |
|---|
| 1747 | | - if (IS_ERR(d)) { |
|---|
| 1748 | | - error = PTR_ERR(d); |
|---|
| 1749 | | - goto out_error; |
|---|
| 1750 | | - } |
|---|
| 1751 | | - dput(d); |
|---|
| 1752 | 1882 | out: |
|---|
| 1753 | 1883 | dput(parent); |
|---|
| 1754 | | - return 0; |
|---|
| 1884 | + return d; |
|---|
| 1755 | 1885 | out_error: |
|---|
| 1756 | | - nfs_mark_for_revalidate(dir); |
|---|
| 1757 | | - dput(parent); |
|---|
| 1758 | | - return error; |
|---|
| 1886 | + d = ERR_PTR(error); |
|---|
| 1887 | + goto out; |
|---|
| 1888 | +} |
|---|
| 1889 | +EXPORT_SYMBOL_GPL(nfs_add_or_obtain); |
|---|
| 1890 | + |
|---|
| 1891 | +/* |
|---|
| 1892 | + * Code common to create, mkdir, and mknod. |
|---|
| 1893 | + */ |
|---|
| 1894 | +int nfs_instantiate(struct dentry *dentry, struct nfs_fh *fhandle, |
|---|
| 1895 | + struct nfs_fattr *fattr, |
|---|
| 1896 | + struct nfs4_label *label) |
|---|
| 1897 | +{ |
|---|
| 1898 | + struct dentry *d; |
|---|
| 1899 | + |
|---|
| 1900 | + d = nfs_add_or_obtain(dentry, fhandle, fattr, label); |
|---|
| 1901 | + if (IS_ERR(d)) |
|---|
| 1902 | + return PTR_ERR(d); |
|---|
| 1903 | + |
|---|
| 1904 | + /* Callers don't care */ |
|---|
| 1905 | + dput(d); |
|---|
| 1906 | + return 0; |
|---|
| 1759 | 1907 | } |
|---|
| 1760 | 1908 | EXPORT_SYMBOL_GPL(nfs_instantiate); |
|---|
| 1761 | 1909 | |
|---|
| .. | .. |
|---|
| 1858 | 2006 | |
|---|
| 1859 | 2007 | trace_nfs_rmdir_enter(dir, dentry); |
|---|
| 1860 | 2008 | if (d_really_is_positive(dentry)) { |
|---|
| 1861 | | -#ifdef CONFIG_PREEMPT_RT_BASE |
|---|
| 1862 | | - down(&NFS_I(d_inode(dentry))->rmdir_sem); |
|---|
| 1863 | | -#else |
|---|
| 1864 | 2009 | down_write(&NFS_I(d_inode(dentry))->rmdir_sem); |
|---|
| 1865 | | -#endif |
|---|
| 1866 | 2010 | error = NFS_PROTO(dir)->rmdir(dir, &dentry->d_name); |
|---|
| 1867 | 2011 | /* Ensure the VFS deletes this inode */ |
|---|
| 1868 | 2012 | switch (error) { |
|---|
| .. | .. |
|---|
| 1872 | 2016 | case -ENOENT: |
|---|
| 1873 | 2017 | nfs_dentry_handle_enoent(dentry); |
|---|
| 1874 | 2018 | } |
|---|
| 1875 | | -#ifdef CONFIG_PREEMPT_RT_BASE |
|---|
| 1876 | | - up(&NFS_I(d_inode(dentry))->rmdir_sem); |
|---|
| 1877 | | -#else |
|---|
| 1878 | 2019 | up_write(&NFS_I(d_inode(dentry))->rmdir_sem); |
|---|
| 1879 | | -#endif |
|---|
| 1880 | 2020 | } else |
|---|
| 1881 | 2021 | error = NFS_PROTO(dir)->rmdir(dir, &dentry->d_name); |
|---|
| 1882 | 2022 | trace_nfs_rmdir_exit(dir, dentry, error); |
|---|
| .. | .. |
|---|
| 2188 | 2328 | static LIST_HEAD(nfs_access_lru_list); |
|---|
| 2189 | 2329 | static atomic_long_t nfs_access_nr_entries; |
|---|
| 2190 | 2330 | |
|---|
| 2191 | | -static unsigned long nfs_access_max_cachesize = ULONG_MAX; |
|---|
| 2331 | +static unsigned long nfs_access_max_cachesize = 4*1024*1024; |
|---|
| 2192 | 2332 | module_param(nfs_access_max_cachesize, ulong, 0644); |
|---|
| 2193 | 2333 | MODULE_PARM_DESC(nfs_access_max_cachesize, "NFS access maximum total cache length"); |
|---|
| 2194 | 2334 | |
|---|
| 2195 | 2335 | static void nfs_access_free_entry(struct nfs_access_entry *entry) |
|---|
| 2196 | 2336 | { |
|---|
| 2197 | | - put_rpccred(entry->cred); |
|---|
| 2337 | + put_cred(entry->cred); |
|---|
| 2198 | 2338 | kfree_rcu(entry, rcu_head); |
|---|
| 2199 | 2339 | smp_mb__before_atomic(); |
|---|
| 2200 | 2340 | atomic_long_dec(&nfs_access_nr_entries); |
|---|
| .. | .. |
|---|
| 2320 | 2460 | } |
|---|
| 2321 | 2461 | EXPORT_SYMBOL_GPL(nfs_access_zap_cache); |
|---|
| 2322 | 2462 | |
|---|
| 2323 | | -static struct nfs_access_entry *nfs_access_search_rbtree(struct inode *inode, struct rpc_cred *cred) |
|---|
| 2463 | +static struct nfs_access_entry *nfs_access_search_rbtree(struct inode *inode, const struct cred *cred) |
|---|
| 2324 | 2464 | { |
|---|
| 2325 | 2465 | struct rb_node *n = NFS_I(inode)->access_cache.rb_node; |
|---|
| 2326 | | - struct nfs_access_entry *entry; |
|---|
| 2327 | 2466 | |
|---|
| 2328 | 2467 | while (n != NULL) { |
|---|
| 2329 | | - entry = rb_entry(n, struct nfs_access_entry, rb_node); |
|---|
| 2468 | + struct nfs_access_entry *entry = |
|---|
| 2469 | + rb_entry(n, struct nfs_access_entry, rb_node); |
|---|
| 2470 | + int cmp = cred_fscmp(cred, entry->cred); |
|---|
| 2330 | 2471 | |
|---|
| 2331 | | - if (cred < entry->cred) |
|---|
| 2472 | + if (cmp < 0) |
|---|
| 2332 | 2473 | n = n->rb_left; |
|---|
| 2333 | | - else if (cred > entry->cred) |
|---|
| 2474 | + else if (cmp > 0) |
|---|
| 2334 | 2475 | n = n->rb_right; |
|---|
| 2335 | 2476 | else |
|---|
| 2336 | 2477 | return entry; |
|---|
| .. | .. |
|---|
| 2338 | 2479 | return NULL; |
|---|
| 2339 | 2480 | } |
|---|
| 2340 | 2481 | |
|---|
| 2341 | | -static int nfs_access_get_cached(struct inode *inode, struct rpc_cred *cred, struct nfs_access_entry *res, bool may_block) |
|---|
| 2482 | +static int nfs_access_get_cached_locked(struct inode *inode, const struct cred *cred, u32 *mask, bool may_block) |
|---|
| 2342 | 2483 | { |
|---|
| 2343 | 2484 | struct nfs_inode *nfsi = NFS_I(inode); |
|---|
| 2344 | 2485 | struct nfs_access_entry *cache; |
|---|
| .. | .. |
|---|
| 2356 | 2497 | /* Found an entry, is our attribute cache valid? */ |
|---|
| 2357 | 2498 | if (!nfs_check_cache_invalid(inode, NFS_INO_INVALID_ACCESS)) |
|---|
| 2358 | 2499 | break; |
|---|
| 2500 | + if (!retry) |
|---|
| 2501 | + break; |
|---|
| 2359 | 2502 | err = -ECHILD; |
|---|
| 2360 | 2503 | if (!may_block) |
|---|
| 2361 | 2504 | goto out; |
|---|
| 2362 | | - if (!retry) |
|---|
| 2363 | | - goto out_zap; |
|---|
| 2364 | 2505 | spin_unlock(&inode->i_lock); |
|---|
| 2365 | 2506 | err = __nfs_revalidate_inode(NFS_SERVER(inode), inode); |
|---|
| 2366 | 2507 | if (err) |
|---|
| .. | .. |
|---|
| 2368 | 2509 | spin_lock(&inode->i_lock); |
|---|
| 2369 | 2510 | retry = false; |
|---|
| 2370 | 2511 | } |
|---|
| 2371 | | - res->cred = cache->cred; |
|---|
| 2372 | | - res->mask = cache->mask; |
|---|
| 2512 | + *mask = cache->mask; |
|---|
| 2373 | 2513 | list_move_tail(&cache->lru, &nfsi->access_cache_entry_lru); |
|---|
| 2374 | 2514 | err = 0; |
|---|
| 2375 | 2515 | out: |
|---|
| .. | .. |
|---|
| 2381 | 2521 | return -ENOENT; |
|---|
| 2382 | 2522 | } |
|---|
| 2383 | 2523 | |
|---|
| 2384 | | -static int nfs_access_get_cached_rcu(struct inode *inode, struct rpc_cred *cred, struct nfs_access_entry *res) |
|---|
| 2524 | +static int nfs_access_get_cached_rcu(struct inode *inode, const struct cred *cred, u32 *mask) |
|---|
| 2385 | 2525 | { |
|---|
| 2386 | 2526 | /* Only check the most recently returned cache entry, |
|---|
| 2387 | 2527 | * but do it without locking. |
|---|
| .. | .. |
|---|
| 2394 | 2534 | rcu_read_lock(); |
|---|
| 2395 | 2535 | if (nfsi->cache_validity & NFS_INO_INVALID_ACCESS) |
|---|
| 2396 | 2536 | goto out; |
|---|
| 2397 | | - lh = rcu_dereference(nfsi->access_cache_entry_lru.prev); |
|---|
| 2537 | + lh = rcu_dereference(list_tail_rcu(&nfsi->access_cache_entry_lru)); |
|---|
| 2398 | 2538 | cache = list_entry(lh, struct nfs_access_entry, lru); |
|---|
| 2399 | 2539 | if (lh == &nfsi->access_cache_entry_lru || |
|---|
| 2400 | | - cred != cache->cred) |
|---|
| 2540 | + cred_fscmp(cred, cache->cred) != 0) |
|---|
| 2401 | 2541 | cache = NULL; |
|---|
| 2402 | 2542 | if (cache == NULL) |
|---|
| 2403 | 2543 | goto out; |
|---|
| 2404 | 2544 | if (nfs_check_cache_invalid(inode, NFS_INO_INVALID_ACCESS)) |
|---|
| 2405 | 2545 | goto out; |
|---|
| 2406 | | - res->cred = cache->cred; |
|---|
| 2407 | | - res->mask = cache->mask; |
|---|
| 2546 | + *mask = cache->mask; |
|---|
| 2408 | 2547 | err = 0; |
|---|
| 2409 | 2548 | out: |
|---|
| 2410 | 2549 | rcu_read_unlock(); |
|---|
| 2411 | 2550 | return err; |
|---|
| 2412 | 2551 | } |
|---|
| 2552 | + |
|---|
| 2553 | +int nfs_access_get_cached(struct inode *inode, const struct cred *cred, |
|---|
| 2554 | + u32 *mask, bool may_block) |
|---|
| 2555 | +{ |
|---|
| 2556 | + int status; |
|---|
| 2557 | + |
|---|
| 2558 | + status = nfs_access_get_cached_rcu(inode, cred, mask); |
|---|
| 2559 | + if (status != 0) |
|---|
| 2560 | + status = nfs_access_get_cached_locked(inode, cred, mask, |
|---|
| 2561 | + may_block); |
|---|
| 2562 | + |
|---|
| 2563 | + return status; |
|---|
| 2564 | +} |
|---|
| 2565 | +EXPORT_SYMBOL_GPL(nfs_access_get_cached); |
|---|
| 2413 | 2566 | |
|---|
| 2414 | 2567 | static void nfs_access_add_rbtree(struct inode *inode, struct nfs_access_entry *set) |
|---|
| 2415 | 2568 | { |
|---|
| .. | .. |
|---|
| 2418 | 2571 | struct rb_node **p = &root_node->rb_node; |
|---|
| 2419 | 2572 | struct rb_node *parent = NULL; |
|---|
| 2420 | 2573 | struct nfs_access_entry *entry; |
|---|
| 2574 | + int cmp; |
|---|
| 2421 | 2575 | |
|---|
| 2422 | 2576 | spin_lock(&inode->i_lock); |
|---|
| 2423 | 2577 | while (*p != NULL) { |
|---|
| 2424 | 2578 | parent = *p; |
|---|
| 2425 | 2579 | entry = rb_entry(parent, struct nfs_access_entry, rb_node); |
|---|
| 2580 | + cmp = cred_fscmp(set->cred, entry->cred); |
|---|
| 2426 | 2581 | |
|---|
| 2427 | | - if (set->cred < entry->cred) |
|---|
| 2582 | + if (cmp < 0) |
|---|
| 2428 | 2583 | p = &parent->rb_left; |
|---|
| 2429 | | - else if (set->cred > entry->cred) |
|---|
| 2584 | + else if (cmp > 0) |
|---|
| 2430 | 2585 | p = &parent->rb_right; |
|---|
| 2431 | 2586 | else |
|---|
| 2432 | 2587 | goto found; |
|---|
| .. | .. |
|---|
| 2450 | 2605 | if (cache == NULL) |
|---|
| 2451 | 2606 | return; |
|---|
| 2452 | 2607 | RB_CLEAR_NODE(&cache->rb_node); |
|---|
| 2453 | | - cache->cred = get_rpccred(set->cred); |
|---|
| 2608 | + cache->cred = get_cred(set->cred); |
|---|
| 2454 | 2609 | cache->mask = set->mask; |
|---|
| 2455 | 2610 | |
|---|
| 2456 | 2611 | /* The above field assignments must be visible |
|---|
| .. | .. |
|---|
| 2514 | 2669 | } |
|---|
| 2515 | 2670 | EXPORT_SYMBOL_GPL(nfs_access_set_mask); |
|---|
| 2516 | 2671 | |
|---|
| 2517 | | -static int nfs_do_access(struct inode *inode, struct rpc_cred *cred, int mask) |
|---|
| 2672 | +static int nfs_do_access(struct inode *inode, const struct cred *cred, int mask) |
|---|
| 2518 | 2673 | { |
|---|
| 2519 | 2674 | struct nfs_access_entry cache; |
|---|
| 2520 | 2675 | bool may_block = (mask & MAY_NOT_BLOCK) == 0; |
|---|
| 2521 | | - int cache_mask; |
|---|
| 2676 | + int cache_mask = -1; |
|---|
| 2522 | 2677 | int status; |
|---|
| 2523 | 2678 | |
|---|
| 2524 | 2679 | trace_nfs_access_enter(inode); |
|---|
| 2525 | 2680 | |
|---|
| 2526 | | - status = nfs_access_get_cached_rcu(inode, cred, &cache); |
|---|
| 2527 | | - if (status != 0) |
|---|
| 2528 | | - status = nfs_access_get_cached(inode, cred, &cache, may_block); |
|---|
| 2681 | + status = nfs_access_get_cached(inode, cred, &cache.mask, may_block); |
|---|
| 2529 | 2682 | if (status == 0) |
|---|
| 2530 | 2683 | goto out_cached; |
|---|
| 2531 | 2684 | |
|---|
| .. | .. |
|---|
| 2537 | 2690 | * Determine which access bits we want to ask for... |
|---|
| 2538 | 2691 | */ |
|---|
| 2539 | 2692 | cache.mask = NFS_ACCESS_READ | NFS_ACCESS_MODIFY | NFS_ACCESS_EXTEND; |
|---|
| 2693 | + if (nfs_server_capable(inode, NFS_CAP_XATTR)) { |
|---|
| 2694 | + cache.mask |= NFS_ACCESS_XAREAD | NFS_ACCESS_XAWRITE | |
|---|
| 2695 | + NFS_ACCESS_XALIST; |
|---|
| 2696 | + } |
|---|
| 2540 | 2697 | if (S_ISDIR(inode->i_mode)) |
|---|
| 2541 | 2698 | cache.mask |= NFS_ACCESS_DELETE | NFS_ACCESS_LOOKUP; |
|---|
| 2542 | 2699 | else |
|---|
| .. | .. |
|---|
| 2545 | 2702 | status = NFS_PROTO(inode)->access(inode, &cache); |
|---|
| 2546 | 2703 | if (status != 0) { |
|---|
| 2547 | 2704 | if (status == -ESTALE) { |
|---|
| 2548 | | - nfs_zap_caches(inode); |
|---|
| 2549 | 2705 | if (!S_ISDIR(inode->i_mode)) |
|---|
| 2550 | | - set_bit(NFS_INO_STALE, &NFS_I(inode)->flags); |
|---|
| 2706 | + nfs_set_inode_stale(inode); |
|---|
| 2707 | + else |
|---|
| 2708 | + nfs_zap_caches(inode); |
|---|
| 2551 | 2709 | } |
|---|
| 2552 | 2710 | goto out; |
|---|
| 2553 | 2711 | } |
|---|
| .. | .. |
|---|
| 2557 | 2715 | if ((mask & ~cache_mask & (MAY_READ | MAY_WRITE | MAY_EXEC)) != 0) |
|---|
| 2558 | 2716 | status = -EACCES; |
|---|
| 2559 | 2717 | out: |
|---|
| 2560 | | - trace_nfs_access_exit(inode, status); |
|---|
| 2718 | + trace_nfs_access_exit(inode, mask, cache_mask, status); |
|---|
| 2561 | 2719 | return status; |
|---|
| 2562 | 2720 | } |
|---|
| 2563 | 2721 | |
|---|
| .. | .. |
|---|
| 2578 | 2736 | return mask; |
|---|
| 2579 | 2737 | } |
|---|
| 2580 | 2738 | |
|---|
| 2581 | | -int nfs_may_open(struct inode *inode, struct rpc_cred *cred, int openflags) |
|---|
| 2739 | +int nfs_may_open(struct inode *inode, const struct cred *cred, int openflags) |
|---|
| 2582 | 2740 | { |
|---|
| 2583 | 2741 | return nfs_do_access(inode, cred, nfs_open_permission_mask(openflags)); |
|---|
| 2584 | 2742 | } |
|---|
| .. | .. |
|---|
| 2603 | 2761 | |
|---|
| 2604 | 2762 | int nfs_permission(struct inode *inode, int mask) |
|---|
| 2605 | 2763 | { |
|---|
| 2606 | | - struct rpc_cred *cred; |
|---|
| 2764 | + const struct cred *cred = current_cred(); |
|---|
| 2607 | 2765 | int res = 0; |
|---|
| 2608 | 2766 | |
|---|
| 2609 | 2767 | nfs_inc_stats(inode, NFSIOS_VFSACCESS); |
|---|
| .. | .. |
|---|
| 2635 | 2793 | if (!NFS_PROTO(inode)->access) |
|---|
| 2636 | 2794 | goto out_notsup; |
|---|
| 2637 | 2795 | |
|---|
| 2638 | | - /* Always try fast lookups first */ |
|---|
| 2639 | | - rcu_read_lock(); |
|---|
| 2640 | | - cred = rpc_lookup_cred_nonblock(); |
|---|
| 2641 | | - if (!IS_ERR(cred)) |
|---|
| 2642 | | - res = nfs_do_access(inode, cred, mask|MAY_NOT_BLOCK); |
|---|
| 2643 | | - else |
|---|
| 2644 | | - res = PTR_ERR(cred); |
|---|
| 2645 | | - rcu_read_unlock(); |
|---|
| 2646 | | - if (res == -ECHILD && !(mask & MAY_NOT_BLOCK)) { |
|---|
| 2647 | | - /* Fast lookup failed, try the slow way */ |
|---|
| 2648 | | - cred = rpc_lookup_cred(); |
|---|
| 2649 | | - if (!IS_ERR(cred)) { |
|---|
| 2650 | | - res = nfs_do_access(inode, cred, mask); |
|---|
| 2651 | | - put_rpccred(cred); |
|---|
| 2652 | | - } else |
|---|
| 2653 | | - res = PTR_ERR(cred); |
|---|
| 2654 | | - } |
|---|
| 2796 | + res = nfs_do_access(inode, cred, mask); |
|---|
| 2655 | 2797 | out: |
|---|
| 2656 | 2798 | if (!res && (mask & MAY_EXEC)) |
|---|
| 2657 | 2799 | res = nfs_execute_ok(inode, mask); |
|---|