netdev
[Top] [All Lists]

Re: Kernel connector - userspace <-> kernelspace "linker".

To: Alan Cox <alan@xxxxxxxxxxxxxxxxxxx>
Subject: Re: Kernel connector - userspace <-> kernelspace "linker".
From: Buddy Lucas <buddy.lucas@xxxxxxxxx>
Date: Tue, 21 Sep 2004 16:21:18 +0200
Cc: Evgeniy Polyakov <johnpol@xxxxxxxxxxx>, netdev@xxxxxxxxxxx, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>
In-reply-to: <1095769354.30743.64.camel@localhost.localdomain>
References: <1095331899.18219.58.camel@uganda> <20040921124623.GA6942@uganda.factory.vocord.ru> <1095769354.30743.64.camel@localhost.localdomain>
Reply-to: Buddy Lucas <buddy.lucas@xxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
On Tue, 21 Sep 2004 13:22:35 +0100, Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:
> On Maw, 2004-09-21 at 13:46, Evgeniy Polyakov wrote:
> > Connector driver adds possibility to connect various agents using
> > netlink based network.
> > One must register callback and identificator. When driver receives
> > special netlink message with appropriate identificator, appropriate
> > callback will be called.
> 
> Looks sane enough to me - and it seems to fit the mentality d-bus and
> HAL want to have.
> 
> Alan
> 
> ps: only trivial item (and really trivial) is that the printk messages
> should be "waiting for %s".
> 

Actually, it should be "waiting for %s to become", not "became". ;-)

Also -- yes, I'm really nitpicking but Evgeniy asked for it ;-) --
brace after for() should not be on a new line. And
s/allocte/allocate/.

That's it for me, nice one, Evgeniy. ;-)


Cheers,
Buddy

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

<Prev in Thread] Current Thread [Next in Thread>