to den 31.03.2005 Klokka 06:58 (-0500) skreiv Trond Myklebust: > > @@ -563,11 +566,14 @@ static int > nfs_scan_commit(struct inode *inode, struct list_head *dst, unsigned long > idx_start, unsigned int npages) > { > struct nfs_inode *nfsi = NFS_I(inode); > - int res; > - res = nfs_scan_list(&nfsi->commit, dst, idx_start, npages); > - nfsi->ncommit -= res; > - if ((nfsi->ncommit == 0) != list_empty(&nfsi->commit)) > - printk(KERN_ERR "NFS: desynchronized value of > nfs_i.ncommit.\n"); > + int res; ^^^^^^^ Should be "int res = 0;" > + > + if (nfsi->ncommit != 0) { > + res = nfs_scan_list(&nfsi->commit, dst, idx_start, npages); > + nfsi->ncommit -= res; > + if ((nfsi->ncommit == 0) != list_empty(&nfsi->commit)) > + printk(KERN_ERR "NFS: desynchronized value of > nfs_i.ncommit.\n"); > + } > return res; > } > #endif
Cheers, Trond -- Trond Myklebust <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/