svn commit: r224875 - head

2011-08-15 Thread Robert Watson
Author: rwatson
Date: Mon Aug 15 07:30:48 2011
New Revision: 224875
URL: http://svn.freebsd.org/changeset/base/224875

Log:
  Bump __FreeBSD_version to reflect the availability of capabilities, but
  also capability-related changes to fget(9).  This is likely not part of
  a formal KPI, but the nvidia driver (at least) uses it.
  
  Mention /dev/{stdin,stdout,stderr} breakage that appears in certain
  kernel revisions as best avoided!
  
  Approved by:  re (xxx)

Modified:
  head/UPDATING

Modified: head/UPDATING
==
--- head/UPDATING   Mon Aug 15 03:41:48 2011(r224874)
+++ head/UPDATING   Mon Aug 15 07:30:48 2011(r224875)
@@ -22,6 +22,16 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 9.
machines to maximize performance.  (To disable malloc debugging, run
ln -s aj /etc/malloc.conf.)
 
+20110815:
+   During the merge of Capsicum features, the fget(9) KPI was modified.
+   This may require the rebuilding of out-of-tree device drivers --
+   issues have been reported specifically with the nVidia device driver.
+   __FreeBSD_version is bumped to 900041.
+
+   Also, there is a period between 20110811 and 20110814 where the
+   special devices /dev/{stdin,stdout,stderr} did not work correctly.
+   Building world from a kernel during that window may not work.
+
 20110628:
The packet filter (pf) code has been updated to OpenBSD 4.5.
You need to update userland tools to be in sync with kernel.
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


svn commit: r224876 - in head/sys: modules/portalfs ufs/ffs

2011-08-15 Thread Robert Watson
Author: rwatson
Date: Mon Aug 15 07:32:44 2011
New Revision: 224876
URL: http://svn.freebsd.org/changeset/base/224876

Log:
  Fix two cases involving opt_capsicum.h and module builds:
  
  (1) opt_capsicum.h is no longer required in ffs_alloc.c, so remove the
 #include.
  
  (2) portalfs depends on opt_capsicum.h, so have the Makefile generate one
 if required.
  
  These affect only modules built without a kernel (i.e, not buildkernel,
  but yes buildworld if the dubious MODULES_WITH_WORLD is used).
  
  Approved by:  re (bz)
  Sponsored by: Google Inc

Modified:
  head/sys/modules/portalfs/Makefile
  head/sys/ufs/ffs/ffs_alloc.c

Modified: head/sys/modules/portalfs/Makefile
==
--- head/sys/modules/portalfs/Makefile  Mon Aug 15 07:30:48 2011
(r224875)
+++ head/sys/modules/portalfs/Makefile  Mon Aug 15 07:32:44 2011
(r224876)
@@ -4,6 +4,7 @@
 
 KMOD=  portalfs
 SRCS=  vnode_if.h \
-   portal_vfsops.c portal_vnops.c
+   portal_vfsops.c portal_vnops.c \
+   opt_capsicum.h
 
 .include bsd.kmod.mk

Modified: head/sys/ufs/ffs/ffs_alloc.c
==
--- head/sys/ufs/ffs/ffs_alloc.cMon Aug 15 07:30:48 2011
(r224875)
+++ head/sys/ufs/ffs/ffs_alloc.cMon Aug 15 07:32:44 2011
(r224876)
@@ -62,7 +62,6 @@
 #include sys/cdefs.h
 __FBSDID($FreeBSD$);
 
-#include opt_capsicum.h
 #include opt_quota.h
 
 #include sys/param.h
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


Re: svn commit: r224842 - head/sys/kern

2011-08-15 Thread Robert Watson

On Sun, 14 Aug 2011, Niclas Zeising wrote:


  When falloc() was broken into separate falloc_noinstall() and finstall(),
  a bug was introduced in kern_openat() such that the error from the vnode
  open operation was overwritten before it was passed as an argument to
  dupfdopen().  This broke operations on /dev/{stdin,stdout,stderr}.  Fix
  by preserving the original error number across finstall() so that it is
  still available.


It might be worth mentioning this in UPDATING or similar, since a kernel 
built with clang (I haven't tried gcc) during this window will make it 
impossible to do a buildworld without first rebuilding the kernel with this 
fix. It also seems to break at least portsnap, and possibly other tools as 
well.


Entry now in UPDATING, thanks!  I've also sent out a warning to current@.

Robert
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


Re: svn commit: r224842 - head/sys/kern

2011-08-15 Thread Robert Watson


On Sun, 14 Aug 2011, Alexander Best wrote:

It might be worth mentioning this in UPDATING or similar, since a kernel 
built with clang (I haven't tried gcc) during this window will make it 
impossible to do a buildworld without first rebuilding the kernel with this 
fix. It also seems to break at least portsnap, and possibly other tools as 
well.


+1. please also mention the KPI change to fget()! so many people have had 
issues, where 3rd party drivers - mostly the nvidia binary driver - got 
broken.


Now in place.  However, I think it's fairly likely that nVidia binary drivers 
will see further disruption in the future.  Other than preventing the driver 
from loading when it's out of sync with __FreeBSD_version (which I believe we 
now do?), I'm not sure what further is to be done there.  In general, we don't 
tweak things like fget(9) very often, but we do make changes to fileops with 
moderate frequency, and the failure modes there are quite a bit more subtle. 
It would be useful to see a KBI analysis of the nVidia module so we can decide 
how to moderate that risk.


(We've done some work on formalising the relationship between network device 
drivers and the kernel, which has led is to have specific policies on data 
structure changes, for example -- but we haven't done symbol dependencies as 
yet.)


Robert
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


svn commit: r224879 - head/share/man/man4

2011-08-15 Thread Gleb Smirnoff
Author: glebius
Date: Mon Aug 15 12:08:41 2011
New Revision: 224879
URL: http://svn.freebsd.org/changeset/base/224879

Log:
  Describe how carp(4) status changes can be processed with
  help of devd(8).
  
  Submitted by: Alexander V. Chernikov melifaro ipfw.ru
  Approved by:  re (kib)

Modified:
  head/share/man/man4/carp.4

Modified: head/share/man/man4/carp.4
==
--- head/share/man/man4/carp.4  Mon Aug 15 09:26:53 2011(r224878)
+++ head/share/man/man4/carp.4  Mon Aug 15 12:08:41 2011(r224879)
@@ -25,7 +25,7 @@
 .\
 .\ $FreeBSD$
 .\
-.Dd January 5, 2010
+.Dd August 15, 2011
 .Dt CARP 4
 .Os
 .Sh NAME
@@ -168,6 +168,25 @@ forwarded to its destination, and destin
 than the state information is packed and synced with the second router.
 If the reply would be load balanced to second router, it will be
 dropped due to no state.
+.Sh STATE CHANGE NOTIFICATIONS
+Sometimes it is useful to get notified about
+.Nm
+status change events.
+This can be accomplished by using
+.Xr devd 8
+hooks.
+Master/slave events are signalled as
+.Nm
+interface
+.Dv LINK_UP
+or
+.Dv LINK_DOWN
+event.
+Please see
+.Xr devd.conf 5
+and 
+.Sx EXAMPLES
+section for more information.
 .Sh EXAMPLES
 For firewalls and routers with multiple interfaces, it is desirable to
 failover all of the
@@ -249,6 +268,25 @@ This way, locally connected systems will
 subsequent IP traffic will be balanced among the hosts.
 If one of the hosts fails, the other will take over the virtual MAC address,
 and begin answering ARP requests on its behalf.
+.Pp
+Processing of
+.Nm
+status change events can be set up by using the following devd.conf rules:
+.Bd -literal -offset indent
+notify 0 {
+   match system  IFNET;
+   match typeLINK_UP;
+   match subsystem   carp*;
+   action /root/carpcontrol.sh $type $subsystem;
+};
+
+notify 0 {
+   match system  IFNET;
+   match typeLINK_UP;
+   match subsystem   carp*;
+   action /root/carpcontrol.sh $type $subsystem;
+};
+.Ed
 .Sh SEE ALSO
 .Xr inet 4 ,
 .Xr pfsync 4 ,
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


Re: svn commit: r224721 - head/sys/sys

2011-08-15 Thread Alexander Best
On Mon Aug 15 11, Bruce Evans wrote:
 On Wed, 10 Aug 2011, Alexander Best wrote:
 
 On Wed Aug 10 11, Bruce Evans wrote:
 On Wed, 10 Aug 2011, Alexander Best wrote:
 any reason {TIMEVAL,TIMESPEC}_TO_{TIMESPEC,TIMEVAL}()s code is being
 executed
 in a
 
 do { ... } while (0)
 
 conditional loop?
 
 Just the usual syntactical trick for making large macros that look
 like function calls almost usable like function calls.  Without the
 ...
 thanks a lot for the in depth information. :) any reason, back in the 
 days, it
 was decided that the functionality of converting a timespec to a timeval 
 and
 vice versa should be implemented as a macro and not a function?
 
 Macros avoid some namespace pollution problems, and all the old kernel
 timeval manipulation interfaces are either extern functions or macros,
 partly because inline functions didn't exist when these interfaces were
 designed.  But the TIME* macros still have gratuitously different styles:
 1) they are spelled in upper case (which is correct since they are unsafe
macros, but this is not done for the other timeval macros which are 
almost
all unsafe)
 2) FreeBSD moved their definitions from sys/time.h (where 4.4BSD-Lite
put them) to sys/timespec.h.  This is not incorrect (since sys/time.h
is an old header that should only declare timeval interfaces (POSIX
put timespec interfaces in time.h).  However, the move became out of
date before it was done (in 2001) because sys/time.h had already
grown several other timespec interfaces which were not moved.  But these
were kernel-only, so they didn't cause namespace problems.  Now
sys/time.h has grown several more user timespec interfaces.

thanks for the explanation. :)

 
 Bruce
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


Re: svn commit: r224721 - head/sys/sys

2011-08-15 Thread Alexander Best
On Mon Aug 15 11, Alexander Best wrote:
 On Mon Aug 15 11, Bruce Evans wrote:
  On Wed, 10 Aug 2011, Alexander Best wrote:
  
  On Wed Aug 10 11, Bruce Evans wrote:
  On Wed, 10 Aug 2011, Alexander Best wrote:
  any reason {TIMEVAL,TIMESPEC}_TO_{TIMESPEC,TIMEVAL}()s code is being
  executed
  in a
  
  do { ... } while (0)
  
  conditional loop?
  
  Just the usual syntactical trick for making large macros that look
  like function calls almost usable like function calls.  Without the
  ...
  thanks a lot for the in depth information. :) any reason, back in the 
  days, it
  was decided that the functionality of converting a timespec to a timeval 
  and
  vice versa should be implemented as a macro and not a function?
  
  Macros avoid some namespace pollution problems, and all the old kernel
  timeval manipulation interfaces are either extern functions or macros,
  partly because inline functions didn't exist when these interfaces were
  designed.  But the TIME* macros still have gratuitously different styles:
  1) they are spelled in upper case (which is correct since they are unsafe
 macros, but this is not done for the other timeval macros which are 
 almost
 all unsafe)
  2) FreeBSD moved their definitions from sys/time.h (where 4.4BSD-Lite
 put them) to sys/timespec.h.  This is not incorrect (since sys/time.h
 is an old header that should only declare timeval interfaces (POSIX
 put timespec interfaces in time.h).  However, the move became out of
 date before it was done (in 2001) because sys/time.h had already
 grown several other timespec interfaces which were not moved.  But these
 were kernel-only, so they didn't cause namespace problems.  Now
 sys/time.h has grown several more user timespec interfaces.
 
 thanks for the explanation. :)

sorry. this was supposed to go to bde@ alone and not the lists.

 
  
  Bruce
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


svn commit: r224883 - head/share/man/man4

2011-08-15 Thread Gleb Smirnoff
Author: glebius
Date: Mon Aug 15 13:33:28 2011
New Revision: 224883
URL: http://svn.freebsd.org/changeset/base/224883

Log:
  Fix a couple of issues in last commit.
  
  Submitted by: maxim
  Approved by:  re (kib)

Modified:
  head/share/man/man4/carp.4

Modified: head/share/man/man4/carp.4
==
--- head/share/man/man4/carp.4  Mon Aug 15 13:33:14 2011(r224882)
+++ head/share/man/man4/carp.4  Mon Aug 15 13:33:28 2011(r224883)
@@ -282,7 +282,7 @@ notify 0 {
 
 notify 0 {
match system  IFNET;
-   match typeLINK_UP;
+   match typeLINK_DOWN;
match subsystem   carp*;
action /root/carpcontrol.sh $type $subsystem;
 };
@@ -291,6 +291,7 @@ notify 0 {
 .Xr inet 4 ,
 .Xr pfsync 4 ,
 .Xr rc.conf 5 ,
+.Xr devd.conf 5 ,
 .Xr ifconfig 8 ,
 .Xr sysctl 8
 .Sh HISTORY
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


svn commit: r224898 - head/sys/dev/puc

2011-08-15 Thread John Baldwin
Author: jhb
Date: Mon Aug 15 19:29:25 2011
New Revision: 224898
URL: http://svn.freebsd.org/changeset/base/224898

Log:
  Add device id for the Moxa CP-112UL dual-port serial adapters.
  
  Submitted by: Jan Mikkelsen  janm of transactionware com
  Approved by:  re (kib)
  MFC after:1 week

Modified:
  head/sys/dev/puc/pucdata.c

Modified: head/sys/dev/puc/pucdata.c
==
--- head/sys/dev/puc/pucdata.c  Mon Aug 15 18:49:42 2011(r224897)
+++ head/sys/dev/puc/pucdata.c  Mon Aug 15 19:29:25 2011(r224898)
@@ -524,6 +524,12 @@ const struct puc_cfg puc_pci_devices[] =
PUC_PORT_4S, 0x18, 0, 8,
},
 
+   {   0x1393, 0x1120, 0x, 0,
+   Moxa Technologies, CP-112UL,
+   DEFAULT_RCLK * 8,
+   PUC_PORT_2S, 0x18, 0, 8,
+   },
+
{   0x1393, 0x1141, 0x, 0,
Moxa Technologies, Industio CP-114,
DEFAULT_RCLK * 8,
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


Re: svn commit: r221071 - in head: . sys/amd64/conf sys/arm/conf sys/cam sys/cam/ata sys/conf sys/geom sys/i386/conf sys/mips/conf sys/modules/cam sys/pc98/conf

2011-08-15 Thread David O'Brien
On Tue, Apr 26, 2011 at 05:01:49PM +, Alexander Motin wrote:
 Log:
- Add shim to simplify migration to the CAM-based ATA. For each new adaX
   device in /dev/ create symbolic link with adY name, trying to mimic old ATA
   numbering. Imitation is not complete, but should be enough in most cases to
   mount file systems without touching /etc/fstab.
- To know what behavior to mimic, restore ATA_STATIC_ID option in cases
   where it was present before.
- Add some more details to UPDATING.

[picking a random ata(4) CAM commit]

Alexander,
What are your plans on updating the ata.4 man page for 9.0-RELEASE?

It still speaks of the old world order and doesn't provide help for folks
trying to convert over.

thanks,
-- 
-- David  (obr...@freebsd.org)
___
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to svn-src-head-unsubscr...@freebsd.org


Re: svn commit: r224875 - head

2011-08-15 Thread Peter Jeremy
On 2011-Aug-15 07:30:48 +, Robert Watson rwat...@freebsd.org wrote:
Log:
  Bump __FreeBSD_version to reflect the availability of capabilities, but
  also capability-related changes to fget(9).  This is likely not part of
  a formal KPI, but the nvidia driver (at least) uses it.

Maybe I missed it somewhere but I don't see the version bump anywhere.
SVN reports this commit only affects UPDATING.

-- 
Peter Jeremy


pgpARYKn6IgMs.pgp
Description: PGP signature