lkml.org 
[lkml]   [2006]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 2.6.15] Adding kernel-level identd dispatcher
From
Date
On Tue, 2006-03-28 at 02:14 -0800, Edward Chernenko wrote:
> --- Trond Myklebust <trond.myklebust@fys.uio.no>
> wrote:
> >
> > Justification, please.
> >
> > You haven't even tried to explain to us what is so
> > broken about the
> > userland identd that it needs to be replaced with a
> > kernel version.
> >
>
> My point is that everything which follows this
> conditions should be moved into kernel:
> - must dispatch requests in a fixed time
> - must work rarely, sleep most time
> - must depend on internal kernel variables (for
> example, established connections table)
>
> Don't forget that many years ago there was echo daemon
> in userspace. But as it's highly effective to dispatch
> all echo requests in kernel, it was moved into
> low-level TCP implementation.
>
> I think that ident protocol also matches this
> criteria.

Most servers are designed for low latency. A lot of them sleep a lot,
and a fair number of them also go poking around the kernel variables
in /proc (which exists precisely in order to export internal kernel
variables to userspace programs). I'll bet even your average Oracle
database application fits those criteria.

Echo made sense to move into the kernel because in addition to the above
it is a required feature on all Internet hosts, is pretty much stateless
(and/or depends only on internal IP stack state), and needs extra low
latency because it is designed to be used for timing purposes by
clients.
The same criteria hardly apply to identd.

Cheers,
Trond

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-03-28 15:40    [W:0.162 / U:0.880 seconds]
©2003-2014 Jasper Spaans. Advertise on this site