Bug 199095 - man page for oce driver have wrong line for loader.conf
Summary: man page for oce driver have wrong line for loader.conf
Status: Closed FIXED
Alias: None
Product: Documentation
Classification: Unclassified
Component: Books & Articles (show other bugs)
Version: Latest
Hardware: amd64 Any
: --- Affects Many People
Assignee: John Baldwin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-01 17:13 UTC by Jiří Navrátil
Modified: 2015-06-05 20:43 UTC (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jiří Navrátil 2015-04-01 17:13:33 UTC
The oce driver need line

	   oce_load="YES"

in /boot/loader.conf 

but man page

https://www.freebsd.org/cgi/man.cgi?query=oce&sektion=4&manpath=FreeBSD+10.1-RELEASE

have state

	   if_oce_load="YES"

which is not working.

Please change if_oce_load="YES" to oce_load="YES"
Comment 1 John Baldwin freebsd_committer freebsd_triage 2015-04-06 20:57:59 UTC
This is probably a bug in the oce(4) driver rather than the manpage.  Network interface drivers are supposed to use an if_ prefix for their module name.  ifconfig(8) has logic that hardcodes this (it will autoload 'if_<foo>.ko' to create a foo0 interface if the interface doesn't exist).
Comment 2 John Baldwin freebsd_committer freebsd_triage 2015-04-06 20:59:29 UTC
Please try changing the KMOD line in sys/modules/oce/Makefile from 'oce' to 'if_oce'.  I think that is all that is required.
Comment 3 Jiří Navrátil 2015-04-08 08:11:00 UTC
I'm using default modules, no compilation on my side. 

For me, the 10.1 RELEASE is either including wrong man page or the prepared oce driver is not compiled properly or ...?

Please take my report as giving you something back. I leave up to you, what will fixed.

Maybe line in erradata.txt can help someone with same trouble?
Comment 4 commit-hook freebsd_committer freebsd_triage 2015-04-24 14:48:24 UTC
A commit references this bug:

Author: jhb
Date: Fri Apr 24 14:47:54 UTC 2015
New revision: 281932
URL: https://svnweb.freebsd.org/changeset/base/281932

Log:
  Rename the kld for oce(4) to if_oce.ko.  ifconfig(8) has special knowledge
  about kld filenames for network drivers that requires them to follow the
  pattern of if_<foo>.  This also fixes the existing documentation in the
  manpage which says to use if_oce_load=YES in loader.conf.

  PR:		199095
  MFC after:	1 week

Changes:
  head/sys/modules/oce/Makefile
Comment 5 commit-hook freebsd_committer freebsd_triage 2015-06-05 20:39:25 UTC
A commit references this bug:

Author: jhb
Date: Fri Jun  5 20:38:23 UTC 2015
New revision: 284045
URL: https://svnweb.freebsd.org/changeset/base/284045

Log:
  MFC 281932:
  Rename the kld for oce(4) to if_oce.ko.  ifconfig(8) has special knowledge
  about kld filenames for network drivers that requires them to follow the
  pattern of if_<foo>.  This also fixes the existing documentation in the
  manpage which says to use if_oce_load=YES in loader.conf.

  PR:		199095

Changes:
_U  stable/10/
  stable/10/sys/modules/oce/Makefile
_U  stable/9/sys/
_U  stable/9/sys/modules/
  stable/9/sys/modules/oce/Makefile