lkml.org 
[lkml]   [2017]   [Jan]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] NFS: switch back to ->iterate()
Date

On 15 Dec 2016, at 17:40, Trond Myklebust wrote:

>> On Dec 9, 2016, at 08:41, Benjamin Coddington <bcodding@redhat.com>
>> 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

\
 
 \ /
  Last update: 2017-01-05 18:06    [W:0.049 / U:2.296 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site