From: "J. Bruce Fields" <bfields@fieldses.org>
To: "Tomasz Kłoczko"
<kloczek-HJEY2/a9Cdz90TmaoATp/9Ui29CwQgaX@public.gmane.org>
Cc: Jeff Layton <jlayton@redhat.com>,
linux-nfs@vger.kernel.org, nfsv4@linux-nfs.org,
linux-kernel@vger.kernel.org, richterd@citi.umich.edu
Subject: Re: [PATCH 0/7] Intro: convert lockd to kthread and fix use-after-free (try #3)
Date: Sat, 5 Jan 2008 11:44:03 -0500 [thread overview]
Message-ID: <20080105164403.GA12928@fieldses.org> (raw)
In-Reply-To: <Pine.BSO.4.63.0712192004140.3875-HJEY2/a9Cdz90TmaoATp/9Ui29CwQgaX@public.gmane.org>
On Wed, Dec 19, 2007 at 08:15:43PM +0100, Tomasz K=C5=82oczko wrote:
> On Tue, 18 Dec 2007, Jeff Layton wrote:
>
>> This is the third patchset to fix the use-after-free problem in lock=
d,
>> and to convert lockd to use the kthread API instead of kernel_thread=
=2E
>> The main change from the last patchset is the elimination of the
>> svc_create_kthread helper function, and having lockd_up call kthread=
_run
>> directly. I've also made some changes to the function prototype for
>> the lockd function to eliminate the need for casting the function
>> pointer.
>
> BTW kernel, NFS and lockd: SuSE kernel IIRC is distributed with patch=
which=20
> adds kernel-based status monitor (statd).
>
> Q to NFS people: any plans about merge this in main kernel tree ?
No. There was opposition to the idea. I can't remember the argument,
though, and a quick google search isn't turning it up.
> And another question related to NFS: any progress on add clear NFS=20
> statistics in kernel (nfsstat -z) ?
No. We do have some patches that add roughly equivalent functionality
in userspace.
--b.
WARNING: multiple messages have this Message-ID (diff)
From: "J. Bruce Fields" <bfields@fieldses.org>
To: "Tomasz Kłoczko" <kloczek@rudy.mif.pg.gda.pl>
Cc: Jeff Layton <jlayton@redhat.com>,
linux-nfs@vger.kernel.org, nfsv4@linux-nfs.org,
linux-kernel@vger.kernel.org, richterd@citi.umich.edu
Subject: Re: [PATCH 0/7] Intro: convert lockd to kthread and fix use-after-free (try #3)
Date: Sat, 5 Jan 2008 11:44:03 -0500 [thread overview]
Message-ID: <20080105164403.GA12928@fieldses.org> (raw)
In-Reply-To: <Pine.BSO.4.63.0712192004140.3875@rudy.mif.pg.gda.pl>
On Wed, Dec 19, 2007 at 08:15:43PM +0100, Tomasz Kłoczko wrote:
> On Tue, 18 Dec 2007, Jeff Layton wrote:
>
>> This is the third patchset to fix the use-after-free problem in lockd,
>> and to convert lockd to use the kthread API instead of kernel_thread.
>> The main change from the last patchset is the elimination of the
>> svc_create_kthread helper function, and having lockd_up call kthread_run
>> directly. I've also made some changes to the function prototype for
>> the lockd function to eliminate the need for casting the function
>> pointer.
>
> BTW kernel, NFS and lockd: SuSE kernel IIRC is distributed with patch which
> adds kernel-based status monitor (statd).
>
> Q to NFS people: any plans about merge this in main kernel tree ?
No. There was opposition to the idea. I can't remember the argument,
though, and a quick google search isn't turning it up.
> And another question related to NFS: any progress on add clear NFS
> statistics in kernel (nfsstat -z) ?
No. We do have some patches that add roughly equivalent functionality
in userspace.
--b.
next prev parent reply other threads:[~2008-01-05 16:44 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-12-18 20:19 [PATCH 0/7] Intro: convert lockd to kthread and fix use-after-free (try #3) Jeff Layton
2007-12-18 20:19 ` Jeff Layton
2007-12-18 20:19 ` [PATCH 1/7] SUNRPC: Allow svc_pool_map_set_cpumask to work with any task Jeff Layton
2007-12-18 20:19 ` [PATCH 2/7] SUNRPC: spin svc_rqst initialization to its own function Jeff Layton
2007-12-18 20:19 ` Jeff Layton
2007-12-18 20:19 ` [PATCH 3/7] SUNRPC: export svc_sock_update_bufs Jeff Layton
2007-12-18 20:19 ` [PATCH 4/7] NLM: Initialize completion variable in lockd_up Jeff Layton
2007-12-18 20:19 ` [PATCH 5/7] NLM: Have lockd call try_to_freeze Jeff Layton
2007-12-18 20:19 ` [PATCH 6/7] NLM: Convert lockd to use kthreads Jeff Layton
2007-12-18 20:19 ` [PATCH 7/7] NLM: Add reference counting to lockd Jeff Layton
2007-12-19 19:15 ` [PATCH 0/7] Intro: convert lockd to kthread and fix use-after-free (try #3) Tomasz Kłoczko
[not found] ` <Pine.BSO.4.63.0712192004140.3875-HJEY2/a9Cdz90TmaoATp/9Ui29CwQgaX@public.gmane.org>
2008-01-05 16:44 ` J. Bruce Fields [this message]
2008-01-05 16:44 ` J. Bruce Fields
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=20080105164403.GA12928@fieldses.org \
--to=bfields@fieldses.org \
--cc=jlayton@redhat.com \
--cc=kloczek-HJEY2/a9Cdz90TmaoATp/9Ui29CwQgaX@public.gmane.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=nfsv4@linux-nfs.org \
--cc=richterd@citi.umich.edu \
/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.