All of lore.kernel.org
 help / color / mirror / Atom feed
From: gchen <gang.chen@asianux.com>
To: Jeff Layton <jlayton@redhat.com>
Cc: linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [QUESTION] about NFS sub system between Public Kernel and Red Hat Kernel.
Date: Sun, 02 Sep 2012 13:41:54 +0800	[thread overview]
Message-ID: <5042F1A2.6090407@asianux.com> (raw)
In-Reply-To: <20120831070213.6e350dd1@corrin.poochiereds.net>

于 2012年08月31日 22:02, Jeff Layton 写道:
> On Fri, 31 Aug 2012 13:40:16 +0800
> gchen <gang.chen@asianux.com> wrote:
>
>> Hi linux-nfs@vger.kernel.org
>>
>> I have 1 question, and also 2 conclusions which need confirm.
>>
>>
>> 1) Question:
>>
>> Jeff Layton said in Red Hat Bugzilla (bug 848706):
>> "Have configuration where the same host is acting as both NFS client
>> and server. That's a configuration known to cause deadlocks."
>>
>> Does it mean that the public Linux kernel (not Red Hat) also can cause
>> deadlocks if NFS client and server are under the same machine ?
>>
> Yes.

I will communicate with LTP (Linux Test Project) to confirm it, too.

And, it will be better if other members in linux-nfs@vger.kernel.org
also confirm it.

>> 2) Confirm 1: (better by Jeff Layton)
>>
>> For function nfs_commit_set_lock in ./fs/nfs/write.c
>>
>> for latest public kernel version:
>> the parameters of out_of_line_wait_on_bit_lock() are
>> (&nfsi->flags, NFS_INO_COMMIT, nfs_wait_killable, TASK_KILLABLE)
>> for Red Hat kernel version: kernel-2.6.18-308.4.1.el5
>> the parameters of out_of_line_wait_on_bit_lock() are
>> (&nfsi->flags, NFS_INO_COMMIT,
>> nfs_wait_bit_uninterruptible, TASK_UNINTERRUPTIBLE)
>>
>> It means for red hat version:
>> when deadlock occurs, we can not boot machine in normal way
>> (it is true for my test machine, the deadlock task can not be killed)
>> It means for public kernel version:
>> "Assume deadlock occurs", we can still boot machine in normal way,
>> because the task can be killed.
>>
>> Is what I said above correct ?
>>
> Not sure I understand your question. RHEL5 doesn't have support for
> TASK_KILLABLE, and I didn't backport it, hence the difference in that
> function.

I agree with what you said above.

Thanks for your confirmation.

>> 3) Confirm 2:
>>
>> Is LTP (Linux Test Project) still a suitable test tools for public kernel ?
>> (for ltp-full-20100331.gz stress test, it mounts NFS on local machine,
>> and the latest LTP ltp-full-20120401.bz2 also seems the same).
>>
> That I'm not sure of. All I can tell you is that mounts over loopback
> (or similar configurations) are easily deadlockable under load.
>

I will communicate with LTP (Linux Test Project) about these informations.


Thanks.

gchen.

Asianux Corporation.


      reply	other threads:[~2012-09-02  5:40 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-31  5:40 [QUESTION] about NFS sub system between Public Kernel and Red Hat Kernel gchen
2012-08-31 14:02 ` Jeff Layton
2012-09-02  5:41   ` gchen [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=5042F1A2.6090407@asianux.com \
    --to=gang.chen@asianux.com \
    --cc=jlayton@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nfs@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.