From: Christoph Hellwig <hch@infradead.org>
To: Oliver Neukum <oliver@neukum.org>
Cc: Christoph Hellwig <hch@infradead.org>,
Martin Schlemmer <azarah@gentoo.org>,
Ro0tSiEgE LKML <lkml@ro0tsiege.org>,
KML <linux-kernel@vger.kernel.org>
Subject: Re: devfsd
Date: Fri, 18 Jul 2003 09:15:12 +0100 [thread overview]
Message-ID: <20030718091512.B16388@infradead.org> (raw)
In-Reply-To: <200307180952.33868.oliver@neukum.org>; from oliver@neukum.org on Fri, Jul 18, 2003 at 09:52:33AM +0200
On Fri, Jul 18, 2003 at 09:52:33AM +0200, Oliver Neukum wrote:
> > > Any way, if you are serious, what make you consider it broken (no,
> > > not talking about personal preferences/phobias 8)
> >
> > There's unsolvable design issues in the way devfsd communication works
> > (with the last two patches the holes are closed as much as possible)
>
> Could you elaborate?
lookup is called with i_sem on parent, devfs calls up to devfsd in
lookup which might again do operation that would block on the same
i_sem. To avoid the deadlock we have to drop i_sem somewhere which
always introduces races. In 2.4 and earlier 2.5 theses races where
huge and easily exploitable at least with the O(1) scheduler. In
current 2.5 they're much smaller so you usually don't trip them but
they;re not going away as long as we keep the stateful devfsd design.
> > issues so people should switch to that ASAP. That doesn't mean we
> > can simply rip it out because people started to rely on the non-standard
> > device names, but it's use is pretty much discouraged in 2.6.
>
> How does udev avoid these complications?
udev is a hotplug upcall, not a stateful deamon.
> If udev doesn't have those issues, why can't they be fixed for devfsd?
Not without changing it to a stateless design, i.e. recreating something
resembling udev..
prev parent reply other threads:[~2003-07-18 8:00 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-07-15 21:46 devfsd Ro0tSiEgE LKML
2003-07-17 15:56 ` devfsd Christoph Hellwig
2003-07-17 16:38 ` devfsd Michael Buesch
2003-07-18 5:57 ` devfsd Martin Schlemmer
2003-07-18 7:44 ` devfsd Christoph Hellwig
2003-07-18 7:52 ` devfsd Oliver Neukum
2003-07-18 8:15 ` Christoph Hellwig [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=20030718091512.B16388@infradead.org \
--to=hch@infradead.org \
--cc=azarah@gentoo.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lkml@ro0tsiege.org \
--cc=oliver@neukum.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