From: "J. Bruce Fields" <bfields@fieldses.org>
To: Benny Halevy <bhalevy@panasas.com>
Cc: Andy Adamson <andros@netapp.com>,
NFS list <linux-nfs@vger.kernel.org>,
pNFS Mailing List <pnfs@linux-nfs.org>
Subject: Re: [pnfs] [PATCH 0/21] nfsd41-for-2.6.30 review cleanup
Date: Thu, 2 Apr 2009 21:58:37 -0400 [thread overview]
Message-ID: <20090403015837.GA30324@fieldses.org> (raw)
In-Reply-To: <49D55D9A.9070504@panasas.com>
On Fri, Apr 03, 2009 at 03:51:38AM +0300, Benny Halevy wrote:
> On Apr. 03, 2009, 3:45 +0300, Benny Halevy <bhalevy@panasas.com> wrote:
> > On Apr. 03, 2009, 1:33 +0300, Benny Halevy <bhalevy@panasas.com> wrote:
> >> Bruce,
> >>
> >> The following patches fix non-DRC review comments.
> >> They are untested yet, but I wanted to send them out ASAP
> >
> > Passed cthon tests over nfs4 so far from several clients:
> > same kernel, nfs41-for-2.6.30 from today, and from 2.6.27.19-78.2.30.fc9.x86_64.
> > The latter hit a single unreproducible failure in the bigfile test.
> >
> > No pynfs regressions.
>
> Oh, and I forgot to mention that I saw these suspicious messages
> on /var/log/messages a while after my last cthon test finished and the
> test server was essentially idle:
>
> Apr 3 03:19:05 tl1 kernel: nfsd: non-standard errno: -9
> Apr 3 03:25:52 tl1 kernel: nfsd: inode locked twice during operation.
> Apr 3 03:25:54 tl1 kernel: RPC: multiple fragments per record not supported
>
> I'm not what caused them and if they show up with the nfsd41 patches
> or not.
The last two look normal for pynfs runs.
The first I don't recall seeing before. Hm, -9 is EBADF. I'd be
curious where that's coming from.
Probably it's nothing serious--pynfs does weird stuff.
--b.
prev parent reply other threads:[~2009-04-03 1:58 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-02 22:33 [PATCH 0/21] nfsd41-for-2.6.30 review cleanup Benny Halevy
2009-04-02 22:37 ` [PATCH 01/21] SQUASHME: nfsd41: conditionally decode_sequence in nfs4_xdr_dec_cb_recall Benny Halevy
2009-04-02 22:37 ` [PATCH 02/21] nfsd41: control nfsv4.1 svc via /proc/fs/nfsd/versions Benny Halevy
2009-04-02 22:37 ` [PATCH 03/21] SQUASHME: nfsd41: disable support for minorversion by default Benny Halevy
2009-04-02 22:37 ` [PATCH 04/21] SQUASHME: nfs41: reverse EXCHGID4_INVAL_FLAG_MASK_{A,R} Benny Halevy
2009-04-02 22:37 ` [PATCH 05/21] SQUASHME: nfsd41: reverse use of EXCHGID4_INVAL_FLAG_MASK_A Benny Halevy
2009-04-02 22:37 ` [PATCH 06/21] SQUASHME: nfsd41: simplify match_clientid_establishment logic Benny Halevy
2009-04-02 22:38 ` [PATCH 07/21] SQUASHME: simplify nfsd4_encode_sequence error handling Benny Halevy
2009-04-02 22:38 ` [PATCH 08/21] SQUASHME: simplify nfsd4_encode_create_session " Benny Halevy
2009-04-02 22:38 ` [PATCH 09/21] SQUASHME: simplify nfsd4_encode_exchange_id " Benny Halevy
2009-04-02 22:38 ` [PATCH 10/21] SQUASHME: nfsd41: document unenforced nfs41 compound ordering rules Benny Halevy
2009-04-02 22:38 ` [PATCH 11/21] SQUASHME: nfsd41: embed an xdr_netobj in nfsd4_exchange_id Benny Halevy
2009-04-02 22:38 ` [PATCH 12/21] SQUASHME: nfsd41: return nfserr_serverfault for spa_how == SP4_MACH_CRED Benny Halevy
2009-04-02 22:38 ` [PATCH 13/21] SQUASHME: nfsd41: fix comment style in init_forechannel_attrs Benny Halevy
2009-04-02 22:38 ` [PATCH 14/21] SQUASHME: nfsd41: allocate struct nfsd4_session and slot table in one piece Benny Halevy
2009-04-02 22:38 ` [PATCH 15/21] SQUASHME: nfsd41: no need to INIT_LIST_HEAD in alloc_init_session just prior to list_add Benny Halevy
2009-04-02 22:38 ` [PATCH 16/21] SQUASHME: nfsd41: pass nfsd4_compoundres * to nfsd4_process_open1 Benny Halevy
2009-04-02 22:38 ` [PATCH 17/21] nfsd: pass nfsd4_compound_state* to nfs4_preprocess_{state,seq}id_op Benny Halevy
2009-04-02 22:38 ` [PATCH 18/21] SQUASHME: nfsd41: calculate HAS_SESSION in nfs4_preprocess_stateid_op Benny Halevy
2009-04-02 22:38 ` [PATCH 19/21] SQUASHME: nfsd41: calculate HAS_SESSION in nfs4_preprocess_seqid_op Benny Halevy
2009-04-02 22:38 ` [PATCH 20/21] SQUASHME: Revert "nfsd41: Add Kconfig symbols for NFSv4.1" Benny Halevy
2009-04-02 22:39 ` [PATCH 21/21] SQUASHME: get rid of CONFIG_NFSD_V4_1 Benny Halevy
2009-04-03 0:45 ` [pnfs] [PATCH 0/21] nfsd41-for-2.6.30 review cleanup Benny Halevy
2009-04-03 0:51 ` Benny Halevy
2009-04-03 1:58 ` J. Bruce Fields [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090403015837.GA30324@fieldses.org \
--to=bfields@fieldses.org \
--cc=andros@netapp.com \
--cc=bhalevy@panasas.com \
--cc=linux-nfs@vger.kernel.org \
--cc=pnfs@linux-nfs.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox