Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1488822
| Path | csiph.com!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod |
|---|---|
| From | Daniel Wagner <wagi@monom.org> |
| Newsgroups | linux.kernel |
| Subject | [PATCH 0/2] NFS: Use complete() instead complete_all() |
| Date | Thu, 22 Sep 2016 14:00:02 +0200 |
| Message-ID | <skaPv-5PF-9@gated-at.bofh.it> (permalink) |
| X-Original-To | linux-nfs@vger.kernel.org |
| X-Mailer | git-send-email 2.7.4 |
| Sender | robomod@news.nic.it |
| List-ID | <linux-kernel.vger.kernel.org> |
| X-Mailing-List | linux-kernel@vger.kernel.org |
| Approved | robomod@news.nic.it |
| Lines | 247 |
| Organization | linux.* mail to news gateway |
| X-Original-Cc | Anna Schumaker <anna.schumaker@netapp.com>, Trond Myklebust <trond.myklebust@primarydata.com>, linux-kernel@vger.kernel.org, Daniel Wagner <daniel.wagner@bmw-carit.de> |
| X-Original-Date | Thu, 22 Sep 2016 13:54:27 +0200 |
| X-Original-Message-ID | <1474545269-8694-1-git-send-email-wagi@monom.org> |
| X-Original-Sender | linux-kernel-owner@vger.kernel.org |
| Xref | csiph.com linux.kernel:1488822 |
Show key headers only | View raw
From: Daniel Wagner <daniel.wagner@bmw-carit.de>
Hi,
Using complete_all() is not wrong per se but it suggest that there
might be more than one waiter. For -rt I am reviewing all
complete_all() users and would like to leave only the real ones in the
tree. The main problem for -rt about complete_all() is that it can be
uses inside IRQ context and that can lead to unbounded amount work
inside the interrupt handler. That is a no no for -rt.
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
Non of them exposed any blocking threads etc. pynfs did showed some
failures but I am guessing that is just missing implementation:
pynfs/nfs4.1/testserver.py random:/home/nfs-test --force -v all
EID50 st_exchange_id.testSSV : FAILURE
NFS4Error: OP_EXCHANGE_ID should return NFS4_OK,
instead got NFS4ERR_ENCR_ALG_UNSUPP
LKPP1a st_lookupp.testLink : RUNNING
LKPP1a st_lookupp.testLink : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_SYMLINK, instead got NFS4ERR_NOENT
LKPP1b st_lookupp.testBlock : RUNNING
LKPP1b st_lookupp.testBlock : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1c st_lookupp.testChar : RUNNING
LKPP1c st_lookupp.testChar : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1d st_lookupp.testLookupp : RUNNING
LKPP1d st_lookupp.testLookupp : PASS
LKPP1f st_lookupp.testFifo : RUNNING
LKPP1f st_lookupp.testFifo : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1r st_lookupp.testFile : RUNNING
LKPP1r st_lookupp.testFile : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1s st_lookupp.testSock : RUNNING
LKPP1s st_lookupp.testSock : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
PUTFH1a st_putfh.testLink : RUNNING
PUTFH1a st_putfh.testLink : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1b st_putfh.testBlock : RUNNING
PUTFH1b st_putfh.testBlock : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1c st_putfh.testChar : RUNNING
PUTFH1c st_putfh.testChar : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1d st_putfh.testDir : RUNNING
PUTFH1d st_putfh.testDir : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1f st_putfh.testFifo : RUNNING
PUTFH1f st_putfh.testFifo : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1r st_putfh.testFile : RUNNING
PUTFH1r st_putfh.testFile : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1s st_putfh.testSocket : RUNNING
PUTFH1s st_putfh.testSocket : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM1b st_rename.testValidBlock : RUNNING
RNM1b st_rename.testValidBlock : FAILURE
OP_CREATE should return NFS4_OK, instead got
NFS4ERR_PERM
RNM1c st_rename.testValidChar : RUNNING
RNM1c st_rename.testValidChar : FAILURE
OP_CREATE should return NFS4_OK, instead got
NFS4ERR_PERM
RNM2a st_rename.testSfhLink : RUNNING
RNM2a st_rename.testSfhLink : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2b st_rename.testSfhBlock : RUNNING
RNM2b st_rename.testSfhBlock : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2c st_rename.testSfhChar : RUNNING
RNM2c st_rename.testSfhChar : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2f st_rename.testSfhFifo : RUNNING
RNM2f st_rename.testSfhFifo : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2r st_rename.testSfhFile : RUNNING
RNM2r st_rename.testSfhFile : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2s st_rename.testSfhSocket : RUNNING
RNM2s st_rename.testSfhSocket : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3a st_rename.testCfhLink : RUNNING
RNM3a st_rename.testCfhLink : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3b st_rename.testCfhBlock : RUNNING
RNM3b st_rename.testCfhBlock : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3c st_rename.testCfhChar : RUNNING
RNM3c st_rename.testCfhChar : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3f st_rename.testCfhFifo : RUNNING
RNM3f st_rename.testCfhFifo : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3r st_rename.testCfhFile : RUNNING
RNM3r st_rename.testCfhFile : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3s st_rename.testCfhSocket : RUNNING
RNM3s st_rename.testCfhSocket : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
EID50 st_exchange_id.testSSV : FAILURE
NFS4Error: OP_EXCHANGE_ID should return NFS4_OK,
instead got NFS4ERR_ENCR_ALG_UNSUPP
LKPP1a st_lookupp.testLink : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_SYMLINK, instead got NFS4ERR_NOENT
LKPP1b st_lookupp.testBlock : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1c st_lookupp.testChar : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1f st_lookupp.testFifo : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1r st_lookupp.testFile : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
LKPP1s st_lookupp.testSock : FAILURE
LOOKUPP with non-dir <cfh> should return
NFS4ERR_NOTDIR, instead got NFS4ERR_NOENT
PUTFH1a st_putfh.testLink : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1b st_putfh.testBlock : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1c st_putfh.testChar : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1d st_putfh.testDir : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1f st_putfh.testFifo : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1r st_putfh.testFile : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
PUTFH1s st_putfh.testSocket : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM1b st_rename.testValidBlock : FAILURE
OP_CREATE should return NFS4_OK, instead got
NFS4ERR_PERM
RNM1c st_rename.testValidChar : FAILURE
OP_CREATE should return NFS4_OK, instead got
NFS4ERR_PERM
RNM2a st_rename.testSfhLink : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2b st_rename.testSfhBlock : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2c st_rename.testSfhChar : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2f st_rename.testSfhFifo : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2r st_rename.testSfhFile : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM2s st_rename.testSfhSocket : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3a st_rename.testCfhLink : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3b st_rename.testCfhBlock : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3c st_rename.testCfhChar : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3f st_rename.testCfhFifo : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3r st_rename.testCfhFile : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
RNM3s st_rename.testCfhSocket : FAILURE
OP_LOOKUP should return NFS4_OK, instead got
NFS4ERR_NOENT
VF1r st_verify.testMandFile : FAILURE
cheers,
daniel
Daniel Wagner (2):
NFS: direct: use complete() instead of complete_all()
NFS: cache_lib: use complete() instead of complete_all()
fs/nfs/cache_lib.c | 2 +-
fs/nfs/direct.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
2.7.4
Back to linux.kernel | Previous | Next | Find similar | Unroll thread
[PATCH 0/2] NFS: Use complete() instead complete_all() Daniel Wagner <wagi@monom.org> - 2016-09-22 14:00 +0200
csiph-web