libnl 1.0-pre7 is out

[Luis R. Rodriguez]

On Nov 13, 2007 9:44 AM, Dan Williams <dcbw at redhat.com> wrote:
>
> On Sun, 2007-11-11 at 18:51 -0500, Luis R. Rodriguez wrote:
> > On Nov 11, 2007 4:18 PM, Luis R. Rodriguez <mcgrof at gmail.com> wrote:
> > >
> > > On Nov 11, 2007 4:02 PM, Luis R. Rodriguez <mcgrof at gmail.com> wrote:
> > > > CC'ing Kyle and John.
> > > >
> > > > On Nov 11, 2007 3:32 PM, Michael Biebl <biebl at debian.org> wrote:
> > > > > Luis R. Rodriguez schrieb:
> > > > >
> > > > > > Hey folks, so libnl-1.0-pre7 is out.
> > > > > >
> > > > > > http://www.suug.ch/~tgr/libnl/
> > > > > >
> > > > > > This guy adds libnl support which we need for nl80211 support (Linux
> > > > > > wireless). Just sending a friendly reminder out in hopes we can start
> > > > > > moving to it soon.
> > > > >
> > > > > NetworkManager does not compile against libnl-1.0-pre7. I'll have to
> > > > > investigate that first and update NetworkManager accordingly before I go
> > > > > on uploading the new libnl version.
> > > >
> > > > Hm, good to know, thanks.
> > > >
> > > >   Luis
> > > >
> > >
> > > Oh Let me add Dan Williams on this while at it, maybe he has a fix already.
> >
> > Attached patch fixes it, doesn't seem we support setting the pid
> > explicitly anymore.
>
> Ok, so does that mean that libnl sets the PID itself then?  _Something_
> has to set that flag.

libnl used to set the PID by itself before, it used to initialize it as follows:

handle->h_local.nl_pid = getpid();

The set_pid stuff was added in case you had multiple handles so the
PIDs wouldn't conflict. Anyway, now the pid is now initialized as
follows:

handle->h_local.nl_pid = generate_local_port();

generate_local_port() does the magic. It allows 1024 unique ports per
application. There is a bug there if you want more than 1024 handles
but I suppose Network Manager won't use them up.

CC'ing Thomas in case I've erred.

  Luis