Hello!
On Apr 29, 2014, at 7:02 AM, Dan Carpenter wrote:
>> diff --git a/drivers/staging/lustre/lustre/llite/statahead.c
>> b/drivers/staging/lustre/lustre/llite/statahead.c
>> index 51c5327..1b47774 100644
>> --- a/drivers/staging/lustre/lustre/llite/statahead.c
>> +++ b/drivers/staging/lustre/lustre/llite/statahead.c
>> @@ -1230,9 +1230,7 @@ do_it:
>> */
>> ll_release_page(page, le32_to_cpu(dp->ldp_flags) &
>> LDF_COLLIDE);
>> - sai->sai_in_readpage = 1;
>> page = ll_get_dir_page(dir, pos, &chain);
>> - sai->sai_in_readpage = 0;
>> } else {
>> LASSERT(le32_to_cpu(dp->ldp_flags) & LDF_COLLIDE);
>> ll_release_page(page, 1);
>> @@ -1563,12 +1561,6 @@ int do_statahead_enter(struct inode *dir, struct
>> dentry **dentryp,
>> return entry ? 1 : -EAGAIN;
>> }
>>
>> - /* if statahead is busy in readdir, help it do post-work */
>> - while (!ll_sa_entry_stated(entry) &&
>> - sai->sai_in_readpage &&
>> - !sa_received_empty(sai))
>> - ll_post_statahead(sai);
>> -
>> if (!ll_sa_entry_stated(entry)) {
>> sai->sai_index_wait = entry->se_index;
>> lwi = LWI_TIMEOUT_INTR(cfs_time_seconds(30), NULL,
>
> What is this change about really? I've already waded through 1271 lines
> of random changes at this point and now I have to figure out what
> ll_post_statahead() does and why we don't need to call it now?
>
> Anyways, please explain this change.
Ok, so after some digging.
The first hunk of the above was gone in our code with the move of a lot of
directory
processing down onto mdc layer as part of support for multiple metadata servers
(a new feature).
Then the second hunk was the cleanup sine nobody was setting the
sai_in_readpage ever.
sai_in_readpage in itself originally was devised as performance optimization
for the case where the
statahead thread is waiting for READDIR RPC to return while there are already
some GETATTR RPCs
that have already returned and are waiting to be interpreted.
This does not affect correctness in any way, though. And removing it brings the
code closer to
our current tree which is hopefully a good thing.
I can rediff this patch with this particular part dropped if there are any
concerns and it's possible
to reintegrate it in a changed form.
Bye,
Oleg--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
