Bug 125639

Summary: [patch] kldunloadf(2) does not mention about possible error EINVAL
Product: Documentation Reporter: Mateusz Guzik <mjguzik>
Component: Books & ArticlesAssignee: Tom Rhodes <trhodes>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Mateusz Guzik 2008-07-15 15:40:04 UTC
Function kldunloadf performs the following check before calling kern_kldunload:
 if (uap->flags != LINKER_UNLOAD_NORMAL &&
            uap->flags != LINKER_UNLOAD_FORCE)
                return (EINVAL);

But this return value is not documented.

Fix: Patch attached with submission follows:
Comment 1 Tom Rhodes freebsd_committer freebsd_triage 2008-12-25 09:15:39 UTC
State Changed
From-To: open->patched

After checking kern_linker.c, I have updated the manual 
page in CURRENT.  Thanks! 


Comment 2 Tom Rhodes freebsd_committer freebsd_triage 2008-12-25 09:15:39 UTC
Responsible Changed
From-To: freebsd-doc->trhodes

Over to me.  MFC reminder.
Comment 3 dfilter service freebsd_committer freebsd_triage 2008-12-25 09:15:45 UTC
Author: trhodes
Date: Thu Dec 25 09:15:31 2008
New Revision: 186489
URL: http://svn.freebsd.org/changeset/base/186489

Log:
  Document that kldunloadf can return EINVAL.
  
  PR:		125639

Modified:
  head/lib/libc/sys/kldunload.2

Modified: head/lib/libc/sys/kldunload.2
==============================================================================
--- head/lib/libc/sys/kldunload.2	Thu Dec 25 09:02:55 2008	(r186488)
+++ head/lib/libc/sys/kldunload.2	Thu Dec 25 09:15:31 2008	(r186489)
@@ -25,7 +25,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd March 10, 2008
+.Dd December 25, 2008
 .Dt KLDUNLOAD 2
 .Os
 .Sh NAME
@@ -69,6 +69,10 @@ You do not have access to unlink the fil
 The file was not found.
 .It Bq Er EBUSY
 You attempted to unload a file linked by the kernel.
+.It Bq Er EINVAL
+The
+.Fn kldunloadf
+system call was passed invalid flags.
 .El
 .Sh SEE ALSO
 .Xr kldfind 2 ,
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
Comment 4 dfilter service freebsd_committer freebsd_triage 2011-06-11 01:06:23 UTC
Author: gjb (doc committer)
Date: Sat Jun 11 00:06:14 2011
New Revision: 222964
URL: http://svn.freebsd.org/changeset/base/222964

Log:
  MFC 186489 [1]:
  
  Document that kldunloadf can return EINVAL.
  
  PR:		125639
  Original commit by:	trhodes [1]

Modified:
  stable/7/lib/libc/sys/kldunload.2
Directory Properties:
  stable/7/lib/libc/   (props changed)
  stable/7/lib/libc/stdtime/   (props changed)

Modified: stable/7/lib/libc/sys/kldunload.2
==============================================================================
--- stable/7/lib/libc/sys/kldunload.2	Fri Jun 10 23:52:53 2011	(r222963)
+++ stable/7/lib/libc/sys/kldunload.2	Sat Jun 11 00:06:14 2011	(r222964)
@@ -25,7 +25,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd March 10, 2008
+.Dd December 25, 2008
 .Dt KLDUNLOAD 2
 .Os
 .Sh NAME
@@ -70,6 +70,10 @@ You do not have access to unlink the fil
 The file was not found.
 .It Bq Er EBUSY
 You attempted to unload a file linked by the kernel.
+.It Bq Er EINVAL
+The
+.Fn kldunloadf
+system call was passed invalid flags.
 .El
 .Sh SEE ALSO
 .Xr kldfind 2 ,
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org"
Comment 5 Glen Barber freebsd_committer freebsd_triage 2011-06-11 01:06:26 UTC
State Changed
From-To: patched->closed

MFCd to stable/7.