re: CVS commit: src/sys/modules

[Paul Goyette <paul%whooppee.com@localhost>]

On Thu, 9 Jun 2016, matthew green wrote:

> "Paul Goyette" writes:
> > Module Name:	src
> > Committed By:	pgoyette
> > Date:		Thu Jun  9 04:44:19 UTC 2016
> >
> > Modified Files:
> > 	src/sys/modules: Makefile
> > Added Files:
> > 	src/sys/modules/ipl: Makefile
> >
> > Log Message:
> > New module for ipl (aka ipfilter).
>
> fwiw, the old lkm was called "if_ipl".

Well, it's really not an interface, so that's not really appropriate.

Since the functionality is accessed by opening the device (and to do 
various ioctl's), naming the module the same as the device makes the 
autoload stuff work (in spec_vnops.c).

> thanks for restoring this lost functionality since netbsd-5.

One note - when you load the module, the functionality comes up in the

	ipf -D

disabled mode.  You need to explicitly enable it with

	ipf -E

before loading the first rules.

And the module can only be unloaded if it has been disabled (with -D) 
_and_ there are no discernible outstanding accesses.


FWIW, I've passed the diffs along upstream (to Darren Reed), but I don't 
know if he'll pick them up into the master sources.  Not that there is 
much new development going on for which we might need to re-import.  :)



+------------------+--------------------------+------------------------+
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:      |
| (Retired)        | FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com   |
| Kernel Developer | 0786 F758 55DE 53BA 7731 | pgoyette at netbsd.org |
+------------------+--------------------------+------------------------+