Re: [PATCH] CDP handler for linux

Chris Crowther (chrisc@shad0w.org.uk)
Tue, 14 Aug 2001 21:54:09 +0100 (BST)


On Tue, 14 Aug 2001, Alan Cox wrote:

> It looks well written. I do have only one question, other than "because
> I wanted to learn how to do it this way", is there a reason for putting
> this into kernel space not a daemon ?

Hmm, looking at CDP there isn't really any reason to put it in
userspace either, it doesn't need to be configured, it doesn't pass
anything the way of the user, except by way of the output of (currently)
/proc/net/cdp_neighbors, and it doesn't need the user to interact with it
either - it just sits there, collecting and sending information. The only
thing you would really need in userspace, would be tools to read
information from the cdp handler if you wanted to do more than just look
at the neighbor summary. I can't see any real advantages of running it as
a daemon as opposed to a kernel component.

This said, if the consesus is that it belongs in userspace, I
shall set about porting the code over to a dameon and possibly maintaing
the kernel patch as a secondary "hobby project".

-- 
Chris "_Shad0w_" Crowther
chrisc@shad0w.org.uk
http://www.shad0w.org.uk/

- 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/