tech-kern archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: Mark module tun obsolete
Hmmm, checking the cvs history it seems that I added this dummy module
for a specific reason. :)
Although creating an interface properly attempts to load the if_* module
(if the device doesn't already exist), there's nothing that adds the
prefix when one attempts to open /dev/tun; the autoload code in specfs
simply tries to load a module named tun.
Since _both_ methods of triggering the autoload need to work, we will
need to have both modules.
So I won't be removing this after all.
On Sun, 24 Mar 2019, Paul Goyette wrote:
While looking into creating a new module for the tap pseudo-device, I noticed
that for tun(4) we create both an 'if_tun' module and another 'tun' module.
The latter of these seems to have no purpose other than to require the
former.
Perhaps there was a time when the autoload code for network drivers would not
add the 'if_' prefix to the interface name (ie, autoload would only load
'tun'). But in any case that would no longer be necessary, as creating a tun
interface now properly autoloads the 'if_tun' module.
Therefore, I propose to remove the tun module from the distribution.
Any objections?
+--------------------+--------------------------+-----------------------+
| Paul Goyette | PGP Key fingerprint: | E-mail addresses: |
| (Retired) | FA29 0E3B 35AF E8AE 6651 | paul%whooppee.com@localhost |
| Software Developer | 0786 F758 55DE 53BA 7731 | pgoyette%netbsd.org@localhost |
+--------------------+--------------------------+-----------------------+
!DSPAM:5c96e6ca101409665740574!
+--------------------+--------------------------+-----------------------+
| Paul Goyette | PGP Key fingerprint: | E-mail addresses: |
| (Retired) | FA29 0E3B 35AF E8AE 6651 | paul%whooppee.com@localhost |
| Software Developer | 0786 F758 55DE 53BA 7731 | pgoyette%netbsd.org@localhost |
+--------------------+--------------------------+-----------------------+
Home |
Main Index |
Thread Index |
Old Index