From: Tejun Heo <tj@kernel.org>
To: ericvh@gmail.com, rminnich@sandia.gov,
v9fs-developer@lists.sourceforge.net,
linux-kernel@vger.kernel.org
Subject: [PATCHSET] 9p: clean up a bit and use single poller for trans_fd
Date: Tue, 26 Aug 2008 17:50:50 +0900 [thread overview]
Message-ID: <1219740656-26458-1-git-send-email-tj@kernel.org> (raw)
This patchset cleans up 9p and 9p-trans_fd a bit and convert trans_fd
to use single poller instead of poll of pollers and contains the
following six patches.
0001-9p-implement-proper-trans-module-refcounting-and-un.patch
0002-9p-trans_fd-fix-trans_fd-p9_conn_destroy.patch
0003-9p-trans_fd-clean-up-p9_conn_create.patch
0004-9p-trans_fd-don-t-do-fs-segment-mangling-in-p9_fd_p.patch
0005-9p-trans_fd-fix-and-clean-up-module-init-exit-paths.patch
0006-9p-trans_fd-use-single-poller.patch
0001 fixes trans module registration and unregistration. 0002-0005
fix a few bugs in and clean up trans_fd. 0006 converts trans_fd to
use single poller instead of pool of pollers.
Although this patchset fixes a few problem cases but there still are
other synchronization issues in trans_fd. Most notably, a request
which is being flushed can be freed before r/w works are done with
them. What's necessary is probably flushing r/w works before actually
destroying the request from flush. Well, I guess that's for another
day.
This patchset is on top of the current linus#master (399d7f6b) and the
combined diffstat follows.
include/net/9p/9p.h | 1
include/net/9p/transport.h | 9 -
net/9p/client.c | 10 +
net/9p/mod.c | 92 ++++++++----
net/9p/trans_fd.c | 340 ++++++++++++++-------------------------------
net/9p/trans_virtio.c | 2
6 files changed, 198 insertions(+), 256 deletions(-)
Thanks.
--
tejun
next reply other threads:[~2008-08-26 8:53 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-26 8:50 Tejun Heo [this message]
2008-08-26 8:50 ` [PATCH 1/6] 9p: implement proper trans module refcounting and unregistration Tejun Heo
2008-08-26 8:50 ` [PATCH 2/6] 9p-trans_fd: fix trans_fd::p9_conn_destroy() Tejun Heo
2008-08-26 8:50 ` [PATCH 3/6] 9p-trans_fd: clean up p9_conn_create() Tejun Heo
2008-08-26 8:50 ` [PATCH 4/6] 9p-trans_fd: don't do fs segment mangling in p9_fd_poll() Tejun Heo
2008-08-26 8:50 ` [PATCH 5/6] 9p-trans_fd: fix and clean up module init/exit paths Tejun Heo
2008-08-26 8:50 ` [PATCH 6/6] 9p-trans_fd: use single poller Tejun Heo
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=1219740656-26458-1-git-send-email-tj@kernel.org \
--to=tj@kernel.org \
--cc=ericvh@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rminnich@sandia.gov \
--cc=v9fs-developer@lists.sourceforge.net \
/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.