On 15 Dec 2016, at 17:40, Trond Myklebust wrote:
On Dec 9, 2016, at 08:41, Benjamin Coddington <bcodding@xxxxxxxxxx>
wrote:
@@ -519,13 +508,7 @@ void nfs_prime_dcache(struct dentry *parent,
struct nfs_entry *entry)
filename.hash = full_name_hash(parent, filename.name, filename.len);
dentry = d_lookup(parent, &filename);
-again:
- if (!dentry) {
- dentry = d_alloc_parallel(parent, &filename, &wq);
- if (IS_ERR(dentry))
- return;
- }
- if (!d_in_lookup(dentry)) {
+ if (dentry != NULL) {
This all looks like it is reverting to using an obsolete VFS API.
I’d
prefer an ACK from Al as to whether or not this is allowed. Please
note
that the rest of the lookup code is still parallelised.
I should've made sure the revert wasn't going to jump back to older VFS
usage. I'll go back over this to make sure that's not the case.
Al, are you hoping to get rid of ->iterate completely? If so, better to
work on this another way.
Ben
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
