Re: [PATCH 0/2] NFS: Use complete() instead complete_all()

From	Anna Schumaker <Anna.Schumaker@netapp.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 0/2] NFS: Use complete() instead complete_all()
Date	2016-09-26 15:40 +0200
Message-ID	<slEit-4nH-5@gated-at.bofh.it> (permalink)
References	<skaPv-5PF-9@gated-at.bofh.it> <skz1w-4hK-61@gated-at.bofh.it> <slwNX-88R-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 09/26/2016 01:33 AM, Daniel Wagner wrote:
> Hi Anna,
> 
> On 09/23/2016 03:48 PM, Anna Schumaker wrote:
>>> Besides trying to analys all the code paths to the wait_for_completion()
>>> call and convince myself that there is only one waiter, I also run
>>> a few tests:
>>>
>>>  - some fio benchmarks
>>>  - pynfs
>>>   -cthon04
>>
>> Thanks for the patches, and for the extensive testing!  I haven't
>> tried them with xfstests yet, but They look okay to me otherwise.
>> Assuming I don't see any new failures there I'll plan on adding them
>> for v4.9.
> 
> I tried a few tests from xfstests but I was not sure which make sense to run, that's why I went for the more NFS specific tests.
> 
> I'll see what happens when I run the generic tests from xfstests. Should all of them pass?

No, there are a handful that we know don't pass.  When I run on NFS v4.1, I get:

        Failures: generic/035 generic/071 generic/128 generic/294 generic/306 generic/309 generic/310 generic/313 generic/314 generic/315 generic/316 generic/319 generic/337 generic/360 generic/375
        Failed 15 of 92 tests

Thanks,
Anna

> 
> cheers,
> daniel

Thread

Re: [PATCH 0/2] NFS: Use complete() instead complete_all() Anna Schumaker <Anna.Schumaker@netapp.com> - 2016-09-23 15:50 +0200
  Re: [PATCH 0/2] NFS: Use complete() instead complete_all() Daniel Wagner <daniel.wagner@bmw-carit.de> - 2016-09-26 07:40 +0200
    Re: [PATCH 0/2] NFS: Use complete() instead complete_all() Anna Schumaker <Anna.Schumaker@netapp.com> - 2016-09-26 15:40 +0200
      Re: [PATCH 0/2] NFS: Use complete() instead complete_all() Daniel Wagner <daniel.wagner@bmw-carit.de> - 2016-09-27 10:40 +0200
        Re: [PATCH 0/2] NFS: Use complete() instead complete_all() Anna Schumaker <Anna.Schumaker@netapp.com> - 2016-09-27 19:20 +0200

csiph-web