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