Re: [GSoC] Introduction + question about dmd

[Rohan Prinja <rohan.prinja@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 4062 bytes --]

Thank you, this clarified some doubts I had.

As suggested, I read through RFC 2131 and took notes. I also looked at
lwIP's RFC 2131-compliant implementation of DHCP and I think I have a fair
understanding of the DHCP protocol now. BTW, you didn't mention RFC 3315 -
does this mean that the proposed DHCP client should support only DHCPv4 for
now, with DHCPv6 being added in later?

Also, are you considering the implementation of the dmd event loop to be
part of this project idea or outside of it? As you said, once the DHCP
library is written, we could test its interop with dmd by calling its
functions within the appropriate event handlers in the event loop. So the
question is, do we incorporate the event loop into this project idea or let
it run as a parallel project and then add the tests for the DHCP library
once the event loop is done?

I'll start working on the first draft of my proposal now, and will make
sure to incorporate your feedback as I update it.

Thanks and regards,
Rohan

On 5 March 2015 at 16:21, Ludovic Courtès <ludo@gnu.org> wrote:

> Hello!
>
> Rohan Prinja <rohan.prinja@gmail.com> skribis:
>
> > The project I'm interested in working on for GNU GSoC this year is to
> > implement a DHCP client library in Guile. The end result, as I discussed
> > with Ludovic, would be a package installable via Guix. This would make
> > available a command roughly equivalent to ISC's dhclient. The library
> > should also implement a service that dmd can run.
>
> Specifically, the library’s functionality should be available in a way
> that makes it easy to integrate in some event loop.
>
> > Things I've done so far - read the dmd manual, started learning Guile
> (I've
> > programmed in Racket before, but not Guile), reading up on the DHCP
> > protocol, and started looking at ISC's implementation of dhclient (I used
> > the one obtained by apt-get source isc-dhcp-client).
>
> (Alternately, you can get it with ‘guix build -S isc-dhclient’.  ;-))
>
> This is a good start.  You might want to look at
> <https://www.ietf.org/rfc/rfc2131.txt> more than at the actual
> implementation.
>
> > @Ludovic: continuing our discussion - could you please explain what you
> > mean by the dmd event loop? I searched in the manual as well the dmd-0.2
> > source but couldn't find any reference to it. Is this something planned
> to
> > be added to dmd? How will it look like?
>
> It’s something planned.  Basically dmd has to react to various events;
> currently, the only events that happen in practice are SIGCHLD,
> connections made by the client programs (‘deco’, ‘reboot’, etc.), and
> commands sent on open connections by these clients.
>
> Currently, in lieu of an full-blown even loop, there’s just this naive
> loop in the (dmd) module:
>
>           (let next-command ()
>             (match (accept sock)
>               ((command-source . client-address)
>                (process-connection command-source))
>               (_ #f))
>             (next-command))
>
> That doesn’t scale well and isn’t safe though, because a client could
> wait forever and thus lead dmd to wait forever as well (although that’s
> slightly beyond the adversary model we would think of.)
>
> More importantly, we would like dmd to be able to handle other kinds of
> events.  For instance, I would like it to run a “REPL server” (info
> "(guile) REPL Servers"); that means it will also have to check for
> events occurring on the REPL server socket.
>
> Other types of events could include network reconfigurations (which
> could be detected via a PF_NETLINK/NETLINK_ROUTE socket), device
> additions/removal, events related to the DHCP client process, etc.
>
> To be able to handle these events, dmd needs a usual event loop.  It
> doesn’t have it yet, but that’s something I’m planning to add (I say “I”
> but anyone reading this message is welcome to help ;-)).
>
> Does it clarify things a bit?
>
> Thanks,
> Ludo’.
>

[-- Attachment #2: Type: text/html, Size: 4851 bytes --]