Re: svn commit: r277737 - in head: etc/pam.d tools/build/mk

2015-01-26 Thread Dag-Erling Smørgrav
Garrett Cooper n...@freebsd.org writes:
 Log:
   Honor MK_ACCT with etc/pam.d/atrun

The correct idiom would be

Index: etc/pam.d/Makefile
===
--- etc/pam.d/Makefile  (revision 277747)
+++ etc/pam.d/Makefile  (working copy)
@@ -5,6 +5,7 @@
 NO_OBJ=
 
 FILES= README \
+   ${_atrun} \
cron \
ftpd \
imap \
@@ -17,7 +18,7 @@
xdm
 
 .if ${MK_AT} != no
-FILES+=atrun
+_atrun=atrun
 .endif
 
 FILESDIR=  /etc/pam.d

DES
-- 
Dag-Erling Smørgrav - d...@des.no
___
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-unsubscr...@freebsd.org

svn commit: r277749 - stable/10/usr.sbin/ctld

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:21:30 2015
New Revision: 277749
URL: https://svnweb.freebsd.org/changeset/base/277749

Log:
  MFC r274791:
  
  Add missing error checking for kernel_port_{add,remove}().  Both can fail
  for reasons yet unknown; don't make it increment cumulated_error as a kind
  of temporary workaround.
  
  MFC r275399:
  
  Fix null pointer dereference.
  
  Sponsored by: The FreeBSD Foundation

Modified:
  stable/10/usr.sbin/ctld/ctld.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/usr.sbin/ctld/ctld.c
==
--- stable/10/usr.sbin/ctld/ctld.c  Mon Jan 26 13:17:20 2015
(r277748)
+++ stable/10/usr.sbin/ctld/ctld.c  Mon Jan 26 13:21:30 2015
(r277749)
@@ -1680,7 +1680,16 @@ conf_apply(struct conf *oldconf, struct 
cumulated_error++;
}
}
-   kernel_port_remove(oldtarg);
+   error = kernel_port_remove(oldtarg);
+   if (error != 0) {
+   log_warnx(failed to remove target %s,
+   oldtarg-t_name);
+   /*
+* XXX: Uncomment after fixing the root cause.
+*
+* cumulated_error++;
+*/
+   }
continue;
}
 
@@ -1814,8 +1823,18 @@ conf_apply(struct conf *oldconf, struct 
cumulated_error++;
}
}
-   if (oldtarg == NULL)
-   kernel_port_add(newtarg);
+   if (oldtarg == NULL) {
+   error = kernel_port_add(newtarg);
+   if (error != 0) {
+   log_warnx(failed to add target %s,
+   newtarg-t_name);
+   /*
+* XXX: Uncomment after fixing the root cause.
+*
+* cumulated_error++;
+*/
+   }
+   }
}
 
/*
___
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-unsubscr...@freebsd.org


svn commit: r277752 - stable/10/share/man/man4

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:32:07 2015
New Revision: 277752
URL: https://svnweb.freebsd.org/changeset/base/277752

Log:
  MFC r272165 by gavin@:
  
  Add very basic outline man page for smsc(4).

Added:
  stable/10/share/man/man4/smsc.4
 - copied unchanged from r272165, head/share/man/man4/smsc.4
Modified:
  stable/10/share/man/man4/Makefile
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/man/man4/Makefile
==
--- stable/10/share/man/man4/Makefile   Mon Jan 26 13:29:43 2015
(r277751)
+++ stable/10/share/man/man4/Makefile   Mon Jan 26 13:32:07 2015
(r277752)
@@ -440,6 +440,7 @@ MAN=aac.4 \
smb.4 \
smbus.4 \
smp.4 \
+   smsc.4 \
sn.4 \
snd_ad1816.4 \
snd_als4000.4 \

Copied: stable/10/share/man/man4/smsc.4 (from r272165, 
head/share/man/man4/smsc.4)
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ stable/10/share/man/man4/smsc.4 Mon Jan 26 13:32:07 2015
(r277752, copy of r272165, head/share/man/man4/smsc.4)
@@ -0,0 +1,89 @@
+.\ Copyright (c) 2014 Gavin Atkinson
+.\ All rights reserved.
+.\
+.\ Redistribution and use in source and binary forms, with or without
+.\ modification, are permitted provided that the following conditions
+.\ are met:
+.\
+.\- Redistributions of source code must retain the above copyright
+.\  notice, this list of conditions and the following disclaimer.
+.\- Redistributions in binary form must reproduce the above
+.\  copyright notice, this list of conditions and the following
+.\  disclaimer in the documentation and/or other materials provided
+.\  with the distribution.
+.\
+.\ THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+.\ AS IS AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+.\ LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+.\ FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+.\ COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+.\ INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
+.\ BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+.\ LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+.\ CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+.\ LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
+.\ ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+.\ POSSIBILITY OF SUCH DAMAGE.
+.\
+.\ $FreeBSD$
+.\
+.Dd September 26, 2014
+.Dt SMSC 4
+.Os
+.Sh NAME
+.Nm smsc
+.Nd USB SMSC LAN9xxx Fast Ethernet driver
+.Sh SYNOPSIS
+To load the driver as a module at boot time, place the
+following line in
+.Xr loader.conf 5 :
+.Bd -literal -offset indent
+if_smsc_load=YES
+.Ed
+.Pp
+Alternatively, to compile this driver into the kernel, place the
+following lines in your kernel configuration file:
+.Bd -ragged -offset indent
+.Cd device uhci
+.Cd device ohci
+.Cd device usb
+.Cd device smsc
+.Ed
+.Sh DESCRIPTION
+The
+.Nm
+device driver provides support for USB Fast Ethernet adapters based
+on the SMSC LAN9xxx chipsets.
+.Pp
+For more information on configuring this device, see
+.Xr ifconfig 8 .
+.Sh HARDWARE
+The following devices are supported by the
+.Nm
+driver:
+.Pp
+.Bl -bullet -compact
+.It
+SMSC LAN9500, LAN9500A, LAN9505 and LAN9505A based Ethernet adapters
+.It
+SMSC LAN89530, LAN9530 and LAN9730 based Ethernet adapters
+.It
+SMSC LAN951x Ethernet adapters with integrated USB hub
+.El
+.Sh SEE ALSO
+.Xr arp 4 ,
+.Xr intro 4 ,
+.Xr netintro 4 ,
+.Xr usb 4 ,
+.Xr ifconfig 8
+.Sh HISTORY
+The
+.Nm
+device driver first appeared in
+.Fx 10.0 .
+.Sh AUTHORS
+.An -nosplit
+The
+.Nm
+driver was written by
+.An Ben Gray Aq Mt bg...@freebsd.org .
___
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-unsubscr...@freebsd.org


svn commit: r277754 - stable/10/share/man/man4

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:35:45 2015
New Revision: 277754
URL: https://svnweb.freebsd.org/changeset/base/277754

Log:
  MFC r272171 by pluknet@:
  
  Add smsc(4) to the list of drivers use the miibus interface.

Modified:
  stable/10/share/man/man4/miibus.4
  stable/10/share/man/man4/smsc.4
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/man/man4/miibus.4
==
--- stable/10/share/man/man4/miibus.4   Mon Jan 26 13:34:21 2015
(r277753)
+++ stable/10/share/man/man4/miibus.4   Mon Jan 26 13:35:45 2015
(r277754)
@@ -8,7 +8,7 @@
 .\
 .\ $FreeBSD$
 .\
-.Dd January 15, 2011
+.Dd September 26, 2014
 .Dt MIIBUS 4
 .Os
 .Sh NAME
@@ -105,6 +105,8 @@ Silicon Integrated Systems SiS190/191 Et
 Silicon Integrated Systems SiS 900/SiS 7016
 .It Xr sk 4
 SysKonnect SK-984x and SK-982x Gigabit Ethernet
+.It Xr smsc 4
+SMSC LAN9xxx USB Fast Ethernet
 .It Xr ste 4
 Sundance ST201 (D-Link DFE-550TX)
 .It Xr stge 4
@@ -169,6 +171,7 @@ but as a result are not well behaved new
 .Xr sge 4 ,
 .Xr sis 4 ,
 .Xr sk 4 ,
+.Xr smsc 4 ,
 .Xr ste 4 ,
 .Xr stge 4 ,
 .Xr tl 4 ,

Modified: stable/10/share/man/man4/smsc.4
==
--- stable/10/share/man/man4/smsc.4 Mon Jan 26 13:34:21 2015
(r277753)
+++ stable/10/share/man/man4/smsc.4 Mon Jan 26 13:35:45 2015
(r277754)
@@ -47,6 +47,7 @@ following lines in your kernel configura
 .Cd device uhci
 .Cd device ohci
 .Cd device usb
+.Cd device miibus
 .Cd device smsc
 .Ed
 .Sh DESCRIPTION
@@ -73,6 +74,7 @@ SMSC LAN951x Ethernet adapters with inte
 .Sh SEE ALSO
 .Xr arp 4 ,
 .Xr intro 4 ,
+.Xr miibus 4 ,
 .Xr netintro 4 ,
 .Xr usb 4 ,
 .Xr ifconfig 8
___
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-unsubscr...@freebsd.org


svn commit: r277755 - stable/10/share/man/man4

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:37:18 2015
New Revision: 277755
URL: https://svnweb.freebsd.org/changeset/base/277755

Log:
  MFC r272294 by gavin@:
  
  Make clear in the ipheth(4) hardware notes that this driver is for the
  tethering functionality only.  Add a bugs section to give a pointer
  to usbconfig set_config if the device isn't automatically detected.

Modified:
  stable/10/share/man/man4/ipheth.4
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/man/man4/ipheth.4
==
--- stable/10/share/man/man4/ipheth.4   Mon Jan 26 13:35:45 2015
(r277754)
+++ stable/10/share/man/man4/ipheth.4   Mon Jan 26 13:37:18 2015
(r277755)
@@ -27,12 +27,12 @@
 .\
 .\ $FreeBSD$
 .\
-.Dd September 25, 2014
+.Dd September 30, 2014
 .Dt IPHETH 4
 .Os
 .Sh NAME
 .Nm ipheth
-.Nd USB Apple iPhone/iPad Ethernet driver
+.Nd USB Apple iPhone/iPad tethered Ethernet driver
 .Sh SYNOPSIS
 To load the driver as a module at boot time, place the
 following line in
@@ -61,6 +61,7 @@ In most cases this must be explicitly en
 .Pp
 For more information on configuring this device, see
 .Xr ifconfig 8 .
+The device does not support different media types or options.
 .Sh HARDWARE
 The following devices are supported by the
 .Nm
@@ -68,9 +69,9 @@ driver:
 .Pp
 .Bl -bullet -compact
 .It
-Apple iPhone (all models)
+Apple iPhone tethering (all models)
 .It
-Apple iPad (all models)
+Apple iPad tethering (all models)
 .El
 .Sh SEE ALSO
 .Xr arp 4 ,
@@ -80,6 +81,7 @@ Apple iPad (all models)
 .Xr urndis 4 ,
 .Xr usb 4 ,
 .Xr ifconfig 8
+.Xr usbconfig 8
 .Sh HISTORY
 The
 .Nm
@@ -91,3 +93,14 @@ The
 .Nm
 driver was written by
 .An Hans Petter Selasky Aq Mt hsela...@freebsd.org .
+.Sh BUGS
+Some devices may need to be manually configured to use an alternative
+configuration with the
+.Xr usbconfig 8
+utility.
+A command similar to
+.Dl usbconfig -u 1 -a 2 set_config 3
+may be required if the device is not recognised automatically by
+.Nm
+after it is connected.
+
___
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-unsubscr...@freebsd.org


svn commit: r277753 - stable/10/share/man/man4

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:34:21 2015
New Revision: 277753
URL: https://svnweb.freebsd.org/changeset/base/277753

Log:
  MFC r272168 by gavin@:
  
  Add MLINKS for if_ipheth(4) and if_smsc(4).

Modified:
  stable/10/share/man/man4/Makefile
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/man/man4/Makefile
==
--- stable/10/share/man/man4/Makefile   Mon Jan 26 13:32:07 2015
(r277752)
+++ stable/10/share/man/man4/Makefile   Mon Jan 26 13:34:21 2015
(r277753)
@@ -660,6 +660,7 @@ MLINKS+=ip.4 rawip.4
 MLINKS+=ipfirewall.4 ipaccounting.4 \
ipfirewall.4 ipacct.4 \
ipfirewall.4 ipfw.4
+MLINKS+=ipheth.4 if_ipheth.4
 MLINKS+=ipw.4 if_ipw.4
 MLINKS+=iwi.4 if_iwi.4
 MLINKS+=iwn.4 if_iwn.4
@@ -715,6 +716,7 @@ MLINKS+=sge.4 if_sge.4
 MLINKS+=sis.4 if_sis.4
 MLINKS+=sk.4 if_sk.4
 MLINKS+=smp.4 SMP.4
+MLINKS+=smsc.4 if_smsc.4
 MLINKS+=sn.4 if_sn.4
 MLINKS+=snd_envy24.4 snd_ak452x.4
 MLINKS+=snd_sbc.4 snd_sb16.4 \
___
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-unsubscr...@freebsd.org


svn commit: r277748 - stable/10/usr.sbin/autofs

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:17:20 2015
New Revision: 277748
URL: https://svnweb.freebsd.org/changeset/base/277748

Log:
  MFC r275746:
  
  Fix markup.
  
  Sponsored by: The FreeBSD Foundation

Modified:
  stable/10/usr.sbin/autofs/autounmountd.8
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/usr.sbin/autofs/autounmountd.8
==
--- stable/10/usr.sbin/autofs/autounmountd.8Mon Jan 26 11:49:06 2015
(r277747)
+++ stable/10/usr.sbin/autofs/autounmountd.8Mon Jan 26 13:17:20 2015
(r277748)
@@ -27,7 +27,7 @@
 .\
 .\ $FreeBSD$
 .\
-.Dd April 20, 2014
+.Dd December 13, 2014
 .Dt AUTOUNMOUNTD 8
 .Os
 .Sh NAME
@@ -36,8 +36,8 @@
 .Sh SYNOPSIS
 .Nm
 .Op Fl d
-.Op Fl r time
-.Op Fl t time
+.Op Fl r Ar time
+.Op Fl t Ar time
 .Op Fl v
 .Sh DESCRIPTION
 The
___
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-unsubscr...@freebsd.org


svn commit: r277750 - stable/10/share/man/man4

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:28:13 2015
New Revision: 277750
URL: https://svnweb.freebsd.org/changeset/base/277750

Log:
  MFC r272106 by gavin@:
  
  Add basic man page for ipheth(4).

Added:
  stable/10/share/man/man4/ipheth.4
 - copied unchanged from r272106, head/share/man/man4/ipheth.4
Modified:
  stable/10/share/man/man4/Makefile
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/man/man4/Makefile
==
--- stable/10/share/man/man4/Makefile   Mon Jan 26 13:21:30 2015
(r277749)
+++ stable/10/share/man/man4/Makefile   Mon Jan 26 13:28:13 2015
(r277750)
@@ -201,6 +201,7 @@ MAN=aac.4 \
ip.4 \
ip6.4 \
ipfirewall.4 \
+   ipheth.4 \
${_ipmi.4} \
ips.4 \
ipsec.4 \

Copied: stable/10/share/man/man4/ipheth.4 (from r272106, 
head/share/man/man4/ipheth.4)
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ stable/10/share/man/man4/ipheth.4   Mon Jan 26 13:28:13 2015
(r277750, copy of r272106, head/share/man/man4/ipheth.4)
@@ -0,0 +1,93 @@
+.\ Copyright (c) 2014 Gavin Atkinson
+.\ All rights reserved.
+.\
+.\ Redistribution and use in source and binary forms, with or without
+.\ modification, are permitted provided that the following conditions
+.\ are met:
+.\
+.\- Redistributions of source code must retain the above copyright
+.\  notice, this list of conditions and the following disclaimer.
+.\- Redistributions in binary form must reproduce the above
+.\  copyright notice, this list of conditions and the following
+.\  disclaimer in the documentation and/or other materials provided
+.\  with the distribution.
+.\
+.\ THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+.\ AS IS AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+.\ LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+.\ FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+.\ COPYRIGHT HOLDERS OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+.\ INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
+.\ BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
+.\ LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+.\ CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+.\ LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
+.\ ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+.\ POSSIBILITY OF SUCH DAMAGE.
+.\
+.\ $FreeBSD$
+.\
+.Dd September 25, 2014
+.Dt IPHETH 4
+.Os
+.Sh NAME
+.Nm ipheth
+.Nd USB Apple iPhone/iPad Ethernet driver
+.Sh SYNOPSIS
+To load the driver as a module at boot time, place the
+following line in
+.Xr loader.conf 5 :
+.Bd -literal -offset indent
+if_ipheth_load=YES
+.Ed
+.Pp
+Alternatively, to compile this driver into the kernel, place the
+following lines in your kernel configuration file:
+.Bd -ragged -offset indent
+.Cd device uhci
+.Cd device ohci
+.Cd device usb
+.Cd device ipheth
+.Ed
+.Sh DESCRIPTION
+The
+.Nm
+driver provides support for network access through Apple
+iPhone and iPad devices, often referred to as USB tethering.
+.Pp
+.Nm
+should work with any Apple iPhone or iPad device.
+In most cases this must be explicitly enabled on the device first.
+.Pp
+For more information on configuring this device, see
+.Xr ifconfig 8 .
+.Sh HARDWARE
+The following devices are supported by the
+.Nm
+driver:
+.Pp
+.Bl -bullet -compact
+.It
+Apple iPhone (all models)
+.It
+Apple iPad (all models)
+.El
+.Sh SEE ALSO
+.Xr arp 4 ,
+.Xr cdce 4 ,
+.Xr intro 4 ,
+.Xr netintro 4 ,
+.Xr urndis 4 ,
+.Xr usb 4 ,
+.Xr ifconfig 8
+.Sh HISTORY
+The
+.Nm
+device driver first appeared in
+.Fx 8.2 .
+.Sh AUTHORS
+.An -nosplit
+The
+.Nm
+driver was written by
+.An Hans Petter Selasky Aq Mt hsela...@freebsd.org .
___
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-unsubscr...@freebsd.org


svn commit: r277756 - stable/10/share/man/man4

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:39:35 2015
New Revision: 277756
URL: https://svnweb.freebsd.org/changeset/base/277756

Log:
  MFC r274496:
  
  Fix mdoc warning by removing empty lines.
  
  Sponsored by: The FreeBSD Foundation

Modified:
  stable/10/share/man/man4/ipheth.4
  stable/10/share/man/man4/iscsi_initiator.4
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/man/man4/ipheth.4
==
--- stable/10/share/man/man4/ipheth.4   Mon Jan 26 13:37:18 2015
(r277755)
+++ stable/10/share/man/man4/ipheth.4   Mon Jan 26 13:39:35 2015
(r277756)
@@ -103,4 +103,3 @@ A command similar to
 may be required if the device is not recognised automatically by
 .Nm
 after it is connected.
-

Modified: stable/10/share/man/man4/iscsi_initiator.4
==
--- stable/10/share/man/man4/iscsi_initiator.4  Mon Jan 26 13:37:18 2015
(r277755)
+++ stable/10/share/man/man4/iscsi_initiator.4  Mon Jan 26 13:39:35 2015
(r277756)
@@ -54,7 +54,6 @@ Users are advised to use
 instead.
 .Ef
 .Pp
-
 The
 .Nm
 implements the kernel side of the Internet SCSI (iSCSI) network
___
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-unsubscr...@freebsd.org


svn commit: r277751 - stable/10/share/man/man4

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 13:29:43 2015
New Revision: 277751
URL: https://svnweb.freebsd.org/changeset/base/277751

Log:
  MFC r272107 by gavin@:
  
  Cross reference cdce(4), ipheth(4) and urndis(4) from each other.

Modified:
  stable/10/share/man/man4/cdce.4
  stable/10/share/man/man4/urndis.4
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/share/man/man4/cdce.4
==
--- stable/10/share/man/man4/cdce.4 Mon Jan 26 13:28:13 2015
(r277750)
+++ stable/10/share/man/man4/cdce.4 Mon Jan 26 13:29:43 2015
(r277751)
@@ -28,7 +28,7 @@
 .\ $NetBSD: cdce.4,v 1.4 2004/12/08 18:35:56 peter Exp $
 .\ $FreeBSD$
 .\
-.Dd September 17, 2005
+.Dd September 25, 2014
 .Dt CDCE 4
 .Os
 .Sh NAME
@@ -114,9 +114,10 @@ is running low on mbufs.
 .Sh SEE ALSO
 .Xr arp 4 ,
 .Xr intro 4 ,
+.Xr ipheth 4 ,
 .Xr netintro 4 ,
+.Xr urndis 4 ,
 .Xr usb 4 ,
-.\ .Xr hostname.if 5 ,
 .Xr ifconfig 8
 .Rs
 .%T Universal Serial Bus Class Definitions for Communication Devices

Modified: stable/10/share/man/man4/urndis.4
==
--- stable/10/share/man/man4/urndis.4   Mon Jan 26 13:28:13 2015
(r277750)
+++ stable/10/share/man/man4/urndis.4   Mon Jan 26 13:29:43 2015
(r277751)
@@ -29,7 +29,7 @@
 .\
 .\ $FreeBSD$
 .\
-.Dd April 21, 2014
+.Dd September 25, 2014
 .Dt URNDIS 4
 .Os
 .Sh NAME
@@ -70,6 +70,8 @@ For more information on configuring this
 .Xr ifconfig 8 .
 .Sh SEE ALSO
 .Xr arp 4 ,
+.Xr cdce 4 ,
+.Xr ipheth 4 ,
 .Xr netintro 4 ,
 .Xr usb 4 ,
 .Xr ifconfig 8
___
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-unsubscr...@freebsd.org


svn commit: r277757 - stable/10/cddl/contrib/opensolaris/cmd/zpool

2015-01-26 Thread Steven Hartland
Author: smh
Date: Mon Jan 26 13:59:39 2015
New Revision: 277757
URL: https://svnweb.freebsd.org/changeset/base/277757

Log:
  MFC r276226:
  Enhancements to zpool upgrade processing
  
  Sponsored by: Multiplay

Modified:
  stable/10/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c
==
--- stable/10/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c   Mon Jan 26 
13:39:35 2015(r277756)
+++ stable/10/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c   Mon Jan 26 
13:59:39 2015(r277757)
@@ -4509,11 +4509,12 @@ zpool_do_status(int argc, char **argv)
 }
 
 typedef struct upgrade_cbdata {
-   int cb_first;
-   charcb_poolname[ZPOOL_MAXNAMELEN];
-   int cb_argc;
-   uint64_t cb_version;
-   char**cb_argv;
+   boolean_t   cb_first;
+   boolean_t   cb_unavail;
+   charcb_poolname[ZPOOL_MAXNAMELEN];
+   int cb_argc;
+   uint64_tcb_version;
+   char**cb_argv;
 } upgrade_cbdata_t;
 
 #ifdef __FreeBSD__
@@ -4631,7 +4632,8 @@ upgrade_cb(zpool_handle_t *zhp, void *ar
 
if (zpool_get_state(zhp) == POOL_STATE_UNAVAIL) {
(void) fprintf(stderr, gettext(cannot upgrade '%s': pool is 
-   currently unavailable\n\n), zpool_get_name(zhp));
+   currently unavailable.\n\n), zpool_get_name(zhp));
+   cbp-cb_unavail = B_TRUE;
/* Allow iteration to continue. */
return (0);
}
@@ -4697,12 +4699,41 @@ upgrade_cb(zpool_handle_t *zhp, void *ar
 }
 
 static int
+upgrade_list_unavail(zpool_handle_t *zhp, void *arg)
+{
+   upgrade_cbdata_t *cbp = arg;
+
+   if (zpool_get_state(zhp) == POOL_STATE_UNAVAIL) {
+   if (cbp-cb_first) {
+   (void) fprintf(stderr, gettext(The following pools 
+   are unavailable and cannot be upgraded as this 
+   time.\n\n));
+   (void) fprintf(stderr, gettext(POOL\n));
+   (void) fprintf(stderr, gettext(\n));
+   cbp-cb_first = B_FALSE;
+   }
+   (void) printf(gettext(%s\n), zpool_get_name(zhp));
+   cbp-cb_unavail = B_TRUE;
+   }
+   return (0);
+}
+
+static int
 upgrade_list_older_cb(zpool_handle_t *zhp, void *arg)
 {
upgrade_cbdata_t *cbp = arg;
nvlist_t *config;
uint64_t version;
 
+   if (zpool_get_state(zhp) == POOL_STATE_UNAVAIL) {
+   /*
+* This will have been reported by upgrade_list_unavail so
+* just allow iteration to continue.
+*/
+   cbp-cb_unavail = B_TRUE;
+   return (0);
+   }
+
config = zpool_get_config(zhp, NULL);
verify(nvlist_lookup_uint64(config, ZPOOL_CONFIG_VERSION,
version) == 0);
@@ -4737,10 +4768,11 @@ upgrade_list_disabled_cb(zpool_handle_t 
uint64_t version;
 
if (zpool_get_state(zhp) == POOL_STATE_UNAVAIL) {
-   (void) fprintf(stderr, gettext(cannot check supported 
-   features on '%s': pool is currently unavailable\n\n),
-   zpool_get_name(zhp));
-   /* Allow iteration to continue. */
+   /*
+* This will have been reported by upgrade_list_unavail so
+* just allow iteration to continue.
+*/
+   cbp-cb_unavail = B_TRUE;
return (0);
}
 
@@ -4797,10 +4829,17 @@ upgrade_one(zpool_handle_t *zhp, void *d
uint64_t cur_version;
int ret;
 
+   if (zpool_get_state(zhp) == POOL_STATE_UNAVAIL) {
+   (void) fprintf(stderr, gettext(cannot upgrade '%s': pool is 
+   is currently unavailable.\n\n), zpool_get_name(zhp));
+   cbp-cb_unavail = B_TRUE;
+   return (1);
+   }
+
if (strcmp(log, zpool_get_name(zhp)) == 0) {
(void) printf(gettext('log' is now a reserved word\n
Pool 'log' must be renamed using export and import
-to upgrade.\n));
+to upgrade.\n\n));
return (1);
}
 
@@ -4844,7 +4883,7 @@ upgrade_one(zpool_handle_t *zhp, void *d
 #endif /* __FreeBSD __*/
} else if (cur_version == SPA_VERSION) {
(void) printf(gettext(Pool '%s' already has all 
-   supported features enabled.\n),
+   supported features enabled.\n\n),
zpool_get_name(zhp));
}
}
@@ -5001,11 +5040,13 @@ zpool_do_upgrade(int argc, char **argv)
ret = zpool_iter(g_zfs, 

svn commit: r277758 - head/sys/cam/scsi

2015-01-26 Thread Alexander Motin
Author: mav
Date: Mon Jan 26 15:47:08 2015
New Revision: 277758
URL: https://svnweb.freebsd.org/changeset/base/277758

Log:
  Fix several potential overflows in UNMAP code.
  
  MFC after:1 week

Modified:
  head/sys/cam/scsi/scsi_da.c

Modified: head/sys/cam/scsi/scsi_da.c
==
--- head/sys/cam/scsi/scsi_da.c Mon Jan 26 13:59:39 2015(r277757)
+++ head/sys/cam/scsi/scsi_da.c Mon Jan 26 15:47:08 2015(r277758)
@@ -1910,18 +1910,18 @@ dadeletemaxsize(struct da_softc *softc, 
sectors = (off_t)ATA_DSM_RANGE_MAX * softc-trim_max_ranges;
break;
case DA_DELETE_WS16:
-   sectors = (off_t)min(softc-ws_max_blks, WS16_MAX_BLKS);
+   sectors = omin(softc-ws_max_blks, WS16_MAX_BLKS);
break;
case DA_DELETE_ZERO:
case DA_DELETE_WS10:
-   sectors = (off_t)min(softc-ws_max_blks, WS10_MAX_BLKS);
+   sectors = omin(softc-ws_max_blks, WS10_MAX_BLKS);
break;
default:
return 0;
}
 
return (off_t)softc-params.secsize *
-   min(sectors, (off_t)softc-params.sectors);
+   omin(sectors, softc-params.sectors);
 }
 
 static void
@@ -2684,7 +2684,7 @@ da_delete_trim(struct cam_periph *periph
 
/* Try to extend the previous range. */
if (lba == lastlba) {
-   c = min(count, ATA_DSM_RANGE_MAX - lastcount);
+   c = omin(count, ATA_DSM_RANGE_MAX - lastcount);
lastcount += c;
off = (ranges - 1) * 8;
buf[off + 6] = lastcount  0xff;
@@ -2694,7 +2694,7 @@ da_delete_trim(struct cam_periph *periph
}
 
while (count  0) {
-   c = min(count, ATA_DSM_RANGE_MAX);
+   c = omin(count, ATA_DSM_RANGE_MAX);
off = ranges * 8;
 
buf[off + 0] = lba  0xff;
@@ -2770,7 +2770,7 @@ da_delete_ws(struct cam_periph *periph, 
%s issuing short delete %ld  %ld\n,
da_delete_method_desc[softc-delete_method],
count, ws_max_blks);
-   count = min(count, ws_max_blks);
+   count = omin(count, ws_max_blks);
break;
}
bp1 = bioq_first(softc-delete_queue);
___
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-unsubscr...@freebsd.org


svn commit: r277760 - stable/10/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

2015-01-26 Thread Alexander Motin
Author: mav
Date: Mon Jan 26 16:29:07 2015
New Revision: 277760
URL: https://svnweb.freebsd.org/changeset/base/277760

Log:
  MFC r277096: Skip extra bcopy() when scrubbing vdev without redundancy.
  
  According to profiler, this bcopy() can use about 10% of CPU time.

Modified:
  stable/10/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_mirror.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_mirror.c
==
--- stable/10/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_mirror.c  
Mon Jan 26 16:26:28 2015(r277759)
+++ stable/10/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_mirror.c  
Mon Jan 26 16:29:07 2015(r277760)
@@ -441,7 +441,8 @@ vdev_mirror_io_start(zio_t *zio)
mm = vdev_mirror_map_init(zio);
 
if (zio-io_type == ZIO_TYPE_READ) {
-   if ((zio-io_flags  ZIO_FLAG_SCRUB)  !mm-mm_replacing) {
+   if ((zio-io_flags  ZIO_FLAG_SCRUB)  !mm-mm_replacing 
+   mm-mm_children  1) {
/*
 * For scrubbing reads we need to allocate a read
 * buffer for each child and issue reads to all
___
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-unsubscr...@freebsd.org


svn commit: r277761 - head/sys/dev/cxgbe/tom

2015-01-26 Thread John Baldwin
Author: jhb
Date: Mon Jan 26 16:29:14 2015
New Revision: 277761
URL: https://svnweb.freebsd.org/changeset/base/277761

Log:
  - Update a disabled KASSERT() to use sbused() instead of accessing
the no-longer existant sb_cc sockbuf member.
  - Use sbavail() instead of sbused() in t4_soreceive_ddp() to match the
usage in soreceive_stream() on which it is based.
  
  Discussed with:   glebius (2)

Modified:
  head/sys/dev/cxgbe/tom/t4_ddp.c

Modified: head/sys/dev/cxgbe/tom/t4_ddp.c
==
--- head/sys/dev/cxgbe/tom/t4_ddp.c Mon Jan 26 16:29:07 2015
(r277760)
+++ head/sys/dev/cxgbe/tom/t4_ddp.c Mon Jan 26 16:29:14 2015
(r277761)
@@ -852,9 +852,9 @@ handle_ddp(struct socket *so, struct uio
SOCKBUF_LOCK_ASSERT(sb);
 
 #if 0
-   if (sb-sb_cc + sc-tt.ddp_thres  uio-uio_resid) {
+   if (sbused(sb) + sc-tt.ddp_thres  uio-uio_resid) {
CTR4(KTR_CXGBE, %s: sb_cc %d, threshold %d, resid %d,
-   __func__, sb-sb_cc, sc-tt.ddp_thres, uio-uio_resid);
+   __func__, sbused(sb), sc-tt.ddp_thres, uio-uio_resid);
}
 #endif
 
@@ -1081,8 +1081,8 @@ restart:
 
/* uio should be just as it was at entry */
KASSERT(oresid == uio-uio_resid,
-   (%s: oresid = %d, uio_resid = %zd, sbused = %d,
-   __func__, oresid, uio-uio_resid, sbused(sb)));
+   (%s: oresid = %d, uio_resid = %zd, sbavail = %d,
+   __func__, oresid, uio-uio_resid, sbavail(sb)));
 
error = handle_ddp(so, uio, flags, 0);
ddp_handled = 1;
@@ -1092,7 +1092,7 @@ restart:
 
/* Abort if socket has reported problems. */
if (so-so_error) {
-   if (sbused(sb))
+   if (sbavail(sb))
goto deliver;
if (oresid  uio-uio_resid)
goto out;
@@ -1104,32 +1104,32 @@ restart:
 
/* Door is closed.  Deliver what is left, if any. */
if (sb-sb_state  SBS_CANTRCVMORE) {
-   if (sbused(sb))
+   if (sbavail(sb))
goto deliver;
else
goto out;
}
 
/* Socket buffer is empty and we shall not block. */
-   if (sbused(sb) == 0 
+   if (sbavail(sb) == 0 
((so-so_state  SS_NBIO) || (flags  (MSG_DONTWAIT|MSG_NBIO {
error = EAGAIN;
goto out;
}
 
/* Socket buffer got some data that we shall deliver now. */
-   if (sbused(sb)  !(flags  MSG_WAITALL) 
+   if (sbavail(sb)  0  !(flags  MSG_WAITALL) 
((so-so_state  SS_NBIO) ||
 (flags  (MSG_DONTWAIT|MSG_NBIO)) ||
-sbused(sb) = sb-sb_lowat ||
-sbused(sb) = uio-uio_resid ||
-sbused(sb) = sb-sb_hiwat) ) {
+sbavail(sb) = sb-sb_lowat ||
+sbavail(sb) = uio-uio_resid ||
+sbavail(sb) = sb-sb_hiwat) ) {
goto deliver;
}
 
/* On MSG_WAITALL we must wait until all data or error arrives. */
if ((flags  MSG_WAITALL) 
-   (sbused(sb) = uio-uio_resid || sbused(sb) = sb-sb_lowat))
+   (sbavail(sb) = uio-uio_resid || sbavail(sb) = sb-sb_lowat))
goto deliver;
 
/*
@@ -1148,7 +1148,7 @@ restart:
 
 deliver:
SOCKBUF_LOCK_ASSERT(so-so_rcv);
-   KASSERT(sbused(sb)  0, (%s: sockbuf empty, __func__));
+   KASSERT(sbavail(sb)  0, (%s: sockbuf empty, __func__));
KASSERT(sb-sb_mb != NULL, (%s: sb_mb == NULL, __func__));
 
if (sb-sb_flags  SB_DDP_INDICATE  !ddp_handled)
@@ -1159,7 +1159,7 @@ deliver:
uio-uio_td-td_ru.ru_msgrcv++;
 
/* Fill uio until full or current end of socket buffer is reached. */
-   len = min(uio-uio_resid, sbused(sb));
+   len = min(uio-uio_resid, sbavail(sb));
if (mp0 != NULL) {
/* Dequeue as many mbufs as possible. */
if (!(flags  MSG_PEEK)  len = sb-sb_mb-m_len) {
___
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-unsubscr...@freebsd.org


svn commit: r277763 - head/sys/dev/cxgbe/tom

2015-01-26 Thread John Baldwin
Author: jhb
Date: Mon Jan 26 16:32:41 2015
New Revision: 277763
URL: https://svnweb.freebsd.org/changeset/base/277763

Log:
  Lock the socket buffer before jumping to the 'out' label if sblock()
  fails in t4_soreceive_ddp().

Modified:
  head/sys/dev/cxgbe/tom/t4_ddp.c

Modified: head/sys/dev/cxgbe/tom/t4_ddp.c
==
--- head/sys/dev/cxgbe/tom/t4_ddp.c Mon Jan 26 16:30:17 2015
(r277762)
+++ head/sys/dev/cxgbe/tom/t4_ddp.c Mon Jan 26 16:32:41 2015
(r277763)
@@ -1057,9 +1057,9 @@ t4_soreceive_ddp(struct socket *so, stru
 
/* Prevent other readers from entering the socket. */
error = sblock(sb, SBLOCKWAIT(flags));
+   SOCKBUF_LOCK(sb);
if (error)
goto out;
-   SOCKBUF_LOCK(sb);
 
/* Easy one, no space to copyout anything. */
if (uio-uio_resid == 0) {
___
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-unsubscr...@freebsd.org


Re: svn commit: r277710 - head/sys/dev/pci

2015-01-26 Thread John Baldwin
On 1/26/15 9:19 AM, Alexey Dokuchaev wrote:
 On Sun, Jan 25, 2015 at 07:53:10PM +, John Baldwin wrote:
 New Revision: 277710
 URL: https://svnweb.freebsd.org/changeset/base/277710

 Log:
   Pass a valid Dx state variable to PCIB_POWER_FOR_SLEEP() in pcib_resume()
   instead of NULL.
 
 Can this be possibly related to this message I got trying to resume a laptop
 running recent -current [1]: ?
 
   pcib0: failed to set ACPI power state D2 on \134_SB_.PCI0: AE_BAD_PARAMETER
 
 ./danfe
 
 [1] 
 http://lists.freebsd.org/pipermail/freebsd-wireless/2015-January/005365.html

Well, the patch came up from debugging something along those lines, but
it will not fix it.

-- 
John Baldwin
___
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-unsubscr...@freebsd.org


svn commit: r277759 - in head/sys: dev/cxgbe kern

2015-01-26 Thread John Baldwin
Author: jhb
Date: Mon Jan 26 16:26:28 2015
New Revision: 277759
URL: https://svnweb.freebsd.org/changeset/base/277759

Log:
  Fix a couple of panics when detaching from a cxgbe/cxl interface that was
  never brought up:
  - Allow NULL to be passed to sglist_free().
  - Don't try to stop an interface that was never fully initialized.
  
  Reviewed by:  np

Modified:
  head/sys/dev/cxgbe/t4_main.c
  head/sys/kern/subr_sglist.c

Modified: head/sys/dev/cxgbe/t4_main.c
==
--- head/sys/dev/cxgbe/t4_main.cMon Jan 26 15:47:08 2015
(r277758)
+++ head/sys/dev/cxgbe/t4_main.cMon Jan 26 16:26:28 2015
(r277759)
@@ -3292,6 +3292,12 @@ cxgbe_uninit_synchronized(struct port_in
 
ASSERT_SYNCHRONIZED_OP(sc);
 
+   if (!(pi-flags  PORT_INIT_DONE)) {
+   KASSERT(!(ifp-if_drv_flags  IFF_DRV_RUNNING),
+   (uninited port is running));
+   return (0);
+   }
+
/*
 * Disable the VI so that all its data in either direction is discarded
 * by the MPS.  Leave everything else (the queues, interrupts, and 1Hz

Modified: head/sys/kern/subr_sglist.c
==
--- head/sys/kern/subr_sglist.c Mon Jan 26 15:47:08 2015(r277758)
+++ head/sys/kern/subr_sglist.c Mon Jan 26 16:26:28 2015(r277759)
@@ -216,6 +216,9 @@ void
 sglist_free(struct sglist *sg)
 {
 
+   if (sg == NULL)
+   return;
+
if (refcount_release(sg-sg_refs))
free(sg, M_SGLIST);
 }
___
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-unsubscr...@freebsd.org


svn commit: r277762 - stable/10/sys/cam

2015-01-26 Thread Alexander Motin
Author: mav
Date: Mon Jan 26 16:30:17 2015
New Revision: 277762
URL: https://svnweb.freebsd.org/changeset/base/277762

Log:
  MFC r277385: Remove extra mtx_unlock().
  
  Submitted by: Dmitry Luhtionov dmitryluhtio...@gmail.com

Modified:
  stable/10/sys/cam/cam_xpt.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/cam/cam_xpt.c
==
--- stable/10/sys/cam/cam_xpt.c Mon Jan 26 16:29:14 2015(r277761)
+++ stable/10/sys/cam/cam_xpt.c Mon Jan 26 16:30:17 2015(r277762)
@@ -897,7 +897,6 @@ xpt_init(void *dummy)
if ((status = xpt_create_path(path, NULL, CAM_XPT_PATH_ID,
  CAM_TARGET_WILDCARD,
  CAM_LUN_WILDCARD)) != CAM_REQ_CMP) {
-   mtx_unlock(xsoftc.xpt_lock);
printf(xpt_init: xpt_create_path failed with status %#x,
failing attach\n, status);
return (EINVAL);
___
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-unsubscr...@freebsd.org


svn commit: r277768 - stable/10/sys/dev/acpica

2015-01-26 Thread Jung-uk Kim
Author: jkim
Date: Mon Jan 26 19:27:30 2015
New Revision: 277768
URL: https://svnweb.freebsd.org/changeset/base/277768

Log:
  MFC:  r277594
  
  Simplify retry loops.

Modified:
  stable/10/sys/dev/acpica/acpi_ec.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/dev/acpica/acpi_ec.c
==
--- stable/10/sys/dev/acpica/acpi_ec.c  Mon Jan 26 19:25:35 2015
(r277767)
+++ stable/10/sys/dev/acpica/acpi_ec.c  Mon Jan 26 19:27:30 2015
(r277768)
@@ -648,10 +648,8 @@ EcGpeQueryHandler(void *Context)
Status = EcCommand(sc, EC_COMMAND_QUERY);
if (ACPI_SUCCESS(Status))
break;
-   if (ACPI_SUCCESS(EcCheckStatus(sc, retr_check,
+   if (ACPI_FAILURE(EcCheckStatus(sc, retr_check,
EC_EVENT_INPUT_BUFFER_EMPTY)))
-   continue;
-   else
break;
 }
 sc-ec_sci_pend = FALSE;
@@ -966,15 +964,13 @@ EcRead(struct acpi_ec_softc *sc, UINT8 A
gen_count = sc-ec_gencount;
EC_SET_DATA(sc, Address);
status = EcWaitEvent(sc, EC_EVENT_OUTPUT_BUFFER_FULL, gen_count);
-   if (ACPI_FAILURE(status)) {
-   if (ACPI_SUCCESS(EcCheckStatus(sc, retr_check,
-   EC_EVENT_INPUT_BUFFER_EMPTY)))
-   continue;
-   else
-   break;
+   if (ACPI_SUCCESS(status)) {
+   *Data = EC_GET_DATA(sc);
+   return (AE_OK);
}
-   *Data = EC_GET_DATA(sc);
-   return (AE_OK);
+   if (ACPI_FAILURE(EcCheckStatus(sc, retr_check,
+   EC_EVENT_INPUT_BUFFER_EMPTY)))
+   break;
 }
 device_printf(sc-ec_dev, EcRead: failed waiting to get data\n);
 return (status);
___
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-unsubscr...@freebsd.org


Re: svn commit: r277710 - head/sys/dev/pci

2015-01-26 Thread Alexey Dokuchaev
On Sun, Jan 25, 2015 at 07:53:10PM +, John Baldwin wrote:
 New Revision: 277710
 URL: https://svnweb.freebsd.org/changeset/base/277710
 
 Log:
   Pass a valid Dx state variable to PCIB_POWER_FOR_SLEEP() in pcib_resume()
   instead of NULL.

Can this be possibly related to this message I got trying to resume a laptop
running recent -current [1]: ?

  pcib0: failed to set ACPI power state D2 on \134_SB_.PCI0: AE_BAD_PARAMETER

./danfe

[1] http://lists.freebsd.org/pipermail/freebsd-wireless/2015-January/005365.html
___
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-unsubscr...@freebsd.org


svn commit: r277767 - stable/10/sys/dev/acpica

2015-01-26 Thread Jung-uk Kim
Author: jkim
Date: Mon Jan 26 19:25:35 2015
New Revision: 277767
URL: https://svnweb.freebsd.org/changeset/base/277767

Log:
  MFC:  r277579
  
  Revert r216942.  This commit was premature and caused too many complaints.

Modified:
  stable/10/sys/dev/acpica/acpi_ec.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/dev/acpica/acpi_ec.c
==
--- stable/10/sys/dev/acpica/acpi_ec.c  Mon Jan 26 19:01:06 2015
(r277766)
+++ stable/10/sys/dev/acpica/acpi_ec.c  Mon Jan 26 19:25:35 2015
(r277767)
@@ -623,7 +623,7 @@ EcGpeQueryHandler(void *Context)
 struct acpi_ec_softc   *sc = (struct acpi_ec_softc *)Context;
 UINT8  Data;
 ACPI_STATUSStatus;
-intretry, sci_enqueued;
+intretry;
 char   qxx[5];
 
 ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__);
@@ -644,7 +644,6 @@ EcGpeQueryHandler(void *Context)
  * that may arise from running the query from causing another query
  * to be queued, we clear the pending flag only after running it.
  */
-sci_enqueued = sc-ec_sci_pend;
 for (retry = 0; retry  2; retry++) {
Status = EcCommand(sc, EC_COMMAND_QUERY);
if (ACPI_SUCCESS(Status))
@@ -684,14 +683,6 @@ EcGpeQueryHandler(void *Context)
device_printf(sc-ec_dev, evaluation of query method %s failed: %s\n,
qxx, AcpiFormatException(Status));
 }
-
-/* Reenable runtime GPE if its execution was deferred. */
-if (sci_enqueued) {
-   Status = AcpiFinishGpe(sc-ec_gpehandle, sc-ec_gpebit);
-   if (ACPI_FAILURE(Status))
-   device_printf(sc-ec_dev, reenabling runtime GPE failed: %s\n,
-   AcpiFormatException(Status));
-}
 }
 
 /*
@@ -725,10 +716,9 @@ EcGpeHandler(ACPI_HANDLE GpeDevice, UINT
 if ((EcStatus  EC_EVENT_SCI)  !sc-ec_sci_pend) {
CTR0(KTR_ACPI, ec gpe queueing query handler);
Status = AcpiOsExecute(OSL_GPE_HANDLER, EcGpeQueryHandler, Context);
-   if (ACPI_SUCCESS(Status)) {
+   if (ACPI_SUCCESS(Status))
sc-ec_sci_pend = TRUE;
-   return (0);
-   } else
+   else
printf(EcGpeHandler: queuing GPE query handler failed\n);
 }
 return (ACPI_REENABLE_GPE);
___
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-unsubscr...@freebsd.org


svn commit: r277770 - stable/9/sys/dev/acpica

2015-01-26 Thread Jung-uk Kim
Author: jkim
Date: Mon Jan 26 19:33:35 2015
New Revision: 20
URL: https://svnweb.freebsd.org/changeset/base/20

Log:
  MFC:  r277579
  
  Revert r216942.  This commit was premature and caused too many complaints.

Modified:
  stable/9/sys/dev/acpica/acpi_ec.c
Directory Properties:
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/acpica/acpi_ec.c
==
--- stable/9/sys/dev/acpica/acpi_ec.c   Mon Jan 26 19:31:14 2015
(r277769)
+++ stable/9/sys/dev/acpica/acpi_ec.c   Mon Jan 26 19:33:35 2015
(r20)
@@ -623,7 +623,7 @@ EcGpeQueryHandler(void *Context)
 struct acpi_ec_softc   *sc = (struct acpi_ec_softc *)Context;
 UINT8  Data;
 ACPI_STATUSStatus;
-intretry, sci_enqueued;
+intretry;
 char   qxx[5];
 
 ACPI_FUNCTION_TRACE((char *)(uintptr_t)__func__);
@@ -644,7 +644,6 @@ EcGpeQueryHandler(void *Context)
  * that may arise from running the query from causing another query
  * to be queued, we clear the pending flag only after running it.
  */
-sci_enqueued = sc-ec_sci_pend;
 for (retry = 0; retry  2; retry++) {
Status = EcCommand(sc, EC_COMMAND_QUERY);
if (ACPI_SUCCESS(Status))
@@ -684,14 +683,6 @@ EcGpeQueryHandler(void *Context)
device_printf(sc-ec_dev, evaluation of query method %s failed: %s\n,
qxx, AcpiFormatException(Status));
 }
-
-/* Reenable runtime GPE if its execution was deferred. */
-if (sci_enqueued) {
-   Status = AcpiFinishGpe(sc-ec_gpehandle, sc-ec_gpebit);
-   if (ACPI_FAILURE(Status))
-   device_printf(sc-ec_dev, reenabling runtime GPE failed: %s\n,
-   AcpiFormatException(Status));
-}
 }
 
 /*
@@ -725,10 +716,9 @@ EcGpeHandler(ACPI_HANDLE GpeDevice, UINT
 if ((EcStatus  EC_EVENT_SCI)  !sc-ec_sci_pend) {
CTR0(KTR_ACPI, ec gpe queueing query handler);
Status = AcpiOsExecute(OSL_GPE_HANDLER, EcGpeQueryHandler, Context);
-   if (ACPI_SUCCESS(Status)) {
+   if (ACPI_SUCCESS(Status))
sc-ec_sci_pend = TRUE;
-   return (0);
-   } else
+   else
printf(EcGpeHandler: queuing GPE query handler failed\n);
 }
 return (ACPI_REENABLE_GPE);
___
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-unsubscr...@freebsd.org


svn commit: r277769 - head/sys/modules/iscsi

2015-01-26 Thread Edward Tomasz Napierala
Author: trasz
Date: Mon Jan 26 19:31:14 2015
New Revision: 277769
URL: https://svnweb.freebsd.org/changeset/base/277769

Log:
  Remove cruft; ICL_RDMA was never actually working, and will be redone
  in a completely different manner.
  
  MFC after:1 month
  Sponsored by: The FreeBSD Foundation

Modified:
  head/sys/modules/iscsi/Makefile

Modified: head/sys/modules/iscsi/Makefile
==
--- head/sys/modules/iscsi/Makefile Mon Jan 26 19:27:30 2015
(r277768)
+++ head/sys/modules/iscsi/Makefile Mon Jan 26 19:31:14 2015
(r277769)
@@ -4,22 +4,12 @@
 KMOD=  iscsi
 
 SRCS=  iscsi.c
-.if defined(ICL_RDMA)
-SRCS+= icl_rdma.c
-.else
 SRCS+= icl.c
-.endif
 SRCS+= icl_proxy.c
 SRCS+= opt_cam.h
 SRCS+= bus_if.h
 SRCS+= device_if.h
 
-# Those below are required for RDMA.
-SRCS+= vnode_if.h
-SRCS+= opt_inet.h
-SRCS+= opt_inet6.h
-
-CFLAGS+= -I${.CURDIR}/../../ofed/include
 #CFLAGS+=-DICL_KERNEL_PROXY
 
 .include bsd.kmod.mk
___
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-unsubscr...@freebsd.org


Re: svn commit: r277694 - head/sys/amd64/conf

2015-01-26 Thread Alfred Perlstein
Don't have a strong opinion on this because I'm not in the know lately, but was 
wondering:

1) if we wanted to keep NE1000/2000 support for FreeBSD in emulators?  Probably 
not I assume?
2) Does Linux nuke these devices?  why/why-not?


On Jan 25, 2015, at 9:58 AM, Warner Losh wrote:

 
 On Jan 25, 2015, at 7:54 AM, Garrett Cooper yaneurab...@gmail.com wrote:
 
 
 On Jan 25, 2015, at 04:43, Sergey Kandaurov pluk...@freebsd.org wrote:
 
 On 25 January 2015 at 15:02, Dag-Erling Smørgrav d...@freebsd.org wrote:
 Author: des
 Date: Sun Jan 25 12:02:38 2015
 New Revision: 277694
 URL: https://svnweb.freebsd.org/changeset/base/277694
 
 Log:
 Remove ISA NICs.  Anyone still using these on amd64 can build their
 own kernel.
 
 Modified:
 head/sys/amd64/conf/GENERIC
 
 If so, what about i386? (I'd rather not pc98)
 What about device isa in DEFAULTS?
 
 isa is still needed in some scenarios. I just don't remember the full 
 details offhand (something about internal buses iirc... And IPMI for 
 starters...)
 
 isa” in this context should be read as “mainbus” not as “ISA slots”. You 
 can’t
 remove it without some significant work, and even then the interface changes
 to long established interfaces isn’t worth the pain.
 
 Warner
 
 

___
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-unsubscr...@freebsd.org


Re: svn commit: r277694 - head/sys/amd64/conf

2015-01-26 Thread Alfred Perlstein
Never mind, apologies, thought this was i386+amd64.  I'm going to guess 
anything emulating amd64 has pci NICs in it. :)

On Jan 26, 2015, at 9:21 AM, Alfred Perlstein wrote:

 Don't have a strong opinion on this because I'm not in the know lately, but 
 was wondering:
 
 1) if we wanted to keep NE1000/2000 support for FreeBSD in emulators?  
 Probably not I assume?
 2) Does Linux nuke these devices?  why/why-not?
 
 
 On Jan 25, 2015, at 9:58 AM, Warner Losh wrote:
 
 
 On Jan 25, 2015, at 7:54 AM, Garrett Cooper yaneurab...@gmail.com wrote:
 
 
 On Jan 25, 2015, at 04:43, Sergey Kandaurov pluk...@freebsd.org wrote:
 
 On 25 January 2015 at 15:02, Dag-Erling Smørgrav d...@freebsd.org wrote:
 Author: des
 Date: Sun Jan 25 12:02:38 2015
 New Revision: 277694
 URL: https://svnweb.freebsd.org/changeset/base/277694
 
 Log:
 Remove ISA NICs.  Anyone still using these on amd64 can build their
 own kernel.
 
 Modified:
 head/sys/amd64/conf/GENERIC
 
 If so, what about i386? (I'd rather not pc98)
 What about device isa in DEFAULTS?
 
 isa is still needed in some scenarios. I just don't remember the full 
 details offhand (something about internal buses iirc... And IPMI for 
 starters...)
 
 isa” in this context should be read as “mainbus” not as “ISA slots”. You 
 can’t
 remove it without some significant work, and even then the interface changes
 to long established interfaces isn’t worth the pain.
 
 Warner
 
 
 

___
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-unsubscr...@freebsd.org


svn commit: r277764 - in head/usr.sbin/pw: . tests

2015-01-26 Thread Baptiste Daroussin
Author: bapt
Date: Mon Jan 26 16:50:42 2015
New Revision: 277764
URL: https://svnweb.freebsd.org/changeset/base/277764

Log:
  Revert r277652
  
  uid and gid are never and should never be negative. The pw(8) manpage clearly
  states the -u and -g arguments are for uids/gids, hence using negative values 
is
  abusing a bug in former versions of pw(8)

Deleted:
  head/usr.sbin/pw/tests/pw_groupshow.sh
  head/usr.sbin/pw/tests/pw_usershow.sh
Modified:
  head/usr.sbin/pw/pw_group.c
  head/usr.sbin/pw/pw_user.c
  head/usr.sbin/pw/tests/Makefile

Modified: head/usr.sbin/pw/pw_group.c
==
--- head/usr.sbin/pw/pw_group.c Mon Jan 26 16:32:41 2015(r277763)
+++ head/usr.sbin/pw/pw_group.c Mon Jan 26 16:50:42 2015(r277764)
@@ -68,11 +68,7 @@ pw_group(struct userconf * cnf, int mode
};
 
if (a_gid != NULL) {
-   const char *teststr;
-   teststr = a_gid-val;
-   if (*teststr == '-')
-   teststr++;
-   if (strspn(teststr, 0123456789) != strlen(teststr))
+   if (strspn(a_gid-val, 0123456789) != strlen(a_gid-val))
errx(EX_USAGE, -g expects a number);
}
 

Modified: head/usr.sbin/pw/pw_user.c
==
--- head/usr.sbin/pw/pw_user.c  Mon Jan 26 16:32:41 2015(r277763)
+++ head/usr.sbin/pw/pw_user.c  Mon Jan 26 16:50:42 2015(r277764)
@@ -322,10 +322,7 @@ pw_user(struct userconf * cnf, int mode,
a_name = NULL;
}
} else {
-   const char *teststr = a_uid-val;
-   if (*teststr == '-')
-   teststr++;
-   if (strspn(teststr, 0123456789) != strlen(teststr))
+   if (strspn(a_uid-val, 0123456789) != strlen(a_uid-val))
errx(EX_USAGE, -u expects a number);
}
 

Modified: head/usr.sbin/pw/tests/Makefile
==
--- head/usr.sbin/pw/tests/Makefile Mon Jan 26 16:32:41 2015
(r277763)
+++ head/usr.sbin/pw/tests/Makefile Mon Jan 26 16:50:42 2015
(r277764)
@@ -9,11 +9,9 @@ ATF_TESTS_SH=  pw_etcdir \
pw_lock \
pw_groupdel \
pw_groupmod \
-   pw_groupshow \
pw_useradd \
pw_userdel \
-   pw_usermod \
-   pw_usershow
+   pw_usermod
 
 .for tp in ${ATF_TESTS_SH}
 TEST_METADATA.${tp}+=  required_user=root
___
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-unsubscr...@freebsd.org


svn commit: r277775 - in head/contrib/llvm/tools/clang: include/clang/Driver lib/Driver

2015-01-26 Thread Dimitry Andric
Author: dim
Date: Mon Jan 26 21:19:24 2015
New Revision: 25
URL: https://svnweb.freebsd.org/changeset/base/25

Log:
  Pull in r227062 from upstream clang trunk (by Renato Golin):
  
Allows Clang to use LLVM's fixes-x18 option
  
This patch allows clang to have llvm reserve the x18
platform register on AArch64. FreeBSD will use this in the kernel for
per-cpu data but has no need to reserve this register in userland so
will need this flag to reserve it.
  
This uses llvm r226664 to allow this register to be reserved.
  
Patch by Andrew Turner.
  
  Requested by: andrew

Modified:
  head/contrib/llvm/tools/clang/include/clang/Driver/Options.td
  head/contrib/llvm/tools/clang/lib/Driver/Tools.cpp

Modified: head/contrib/llvm/tools/clang/include/clang/Driver/Options.td
==
--- head/contrib/llvm/tools/clang/include/clang/Driver/Options.td   Mon Jan 
26 21:17:14 2015(r24)
+++ head/contrib/llvm/tools/clang/include/clang/Driver/Options.td   Mon Jan 
26 21:19:24 2015(r25)
@@ -1141,6 +1141,9 @@ def mno_long_calls : Flag[-], mno-lo
 def mgeneral_regs_only : Flag[-], mgeneral-regs-only, 
Groupm_aarch64_Features_Group,
   HelpTextGenerate code which only uses the general purpose registers 
(AArch64 only);
 
+def ffixed_x18 : Flag[-], ffixed-x18, Groupm_aarch64_Features_Group,
+  HelpTextReserve the x18 register (AArch64 only);
+
 def mvsx : Flag[-], mvsx, Groupm_ppc_Features_Group;
 def mno_vsx : Flag[-], mno-vsx, Groupm_ppc_Features_Group;
 def mfprnd : Flag[-], mfprnd, Groupm_ppc_Features_Group;

Modified: head/contrib/llvm/tools/clang/lib/Driver/Tools.cpp
==
--- head/contrib/llvm/tools/clang/lib/Driver/Tools.cpp  Mon Jan 26 21:17:14 
2015(r24)
+++ head/contrib/llvm/tools/clang/lib/Driver/Tools.cpp  Mon Jan 26 21:19:24 
2015(r25)
@@ -887,6 +887,11 @@ void Clang::AddAArch64TargetArgs(const A
 if (A-getOption().matches(options::OPT_mno_global_merge))
   CmdArgs.push_back(-mno-global-merge);
   }
+
+  if (Args.hasArg(options::OPT_ffixed_x18)) {
+CmdArgs.push_back(-backend-option);
+CmdArgs.push_back(-aarch64-reserve-x18);
+  }
 }
 
 // Get CPU and ABI names. They are not independent
___
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-unsubscr...@freebsd.org


Re: svn commit: r277714 - head/sbin/ipfw

2015-01-26 Thread Alexander V . Chernikov
   �

   �

   26.01.2015, 23:35, Olivier Cochard-Labbé oliv...@cochard.me:

   On Sun, Jan 25, 2015 at 9:37 PM, John Baldwin [1]j...@freebsd.org
   wrote:

 Author: jhb
 Date: Sun Jan 25 20:37:32 2015
 New Revision: 277714
 URL: [2]https://svnweb.freebsd.org/changeset/base/277714
 Log:
 �  natd(8) will work with an unconfigured interface and effectively
 not do
 �  anything until the interface is assigned an address.�  This fixes
 �  ipfw_nat to do the same by using an IP of INADDR_ANY instead of
 �  aborting the nat setup if the requested interface is not yet
 configured.
 �

   Hi,
   I've still a problem with ipfw_nat and unconfigured interface:
   On my setup I'm using ipfw with NAT rules using an OpenVPN tunnel
   interface as source address for NATting.
   During the machine startup, ipfw is started before openvpn (hopefully)
   and its configuration mention do to NAT using tun0 IP address.
   Then OpenVPN start and create a tun0 and set an IP address on it.
   = But no unicast traffic is allowed on this tun0 interface until I
   restart ipfw.
   If I correctly understand the log of this commit: This behavior should
   be fixed by this commit, right ?

   As far as I understand, nat instance is created with an unresolved ip
   (0.0.0.0 propagated to libalias) and tun0 interface name. After
   tun0 creation and address assignment, kernel ipfw_nat ifaddr hook
   should take action and update libalias address to primary? IPv4
   interface address.

   �

References

   1. mailto:j...@freebsd.org
   2. https://svnweb.freebsd.org/changeset/base/277714
___
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-unsubscr...@freebsd.org

svn commit: r277774 - head/contrib/llvm/lib/Target/AArch64

2015-01-26 Thread Dimitry Andric
Author: dim
Date: Mon Jan 26 21:17:14 2015
New Revision: 24
URL: https://svnweb.freebsd.org/changeset/base/24

Log:
  Pull in r226664 from upstream llvm trunk (by Tim Northover):
  
AArch64: add backend option to reserve x18 (platform register)
  
AAPCS64 says that it's up to the platform to specify whether x18 is
reserved, and a first step on that way is to add a flag controlling
it.
  
From: Andrew Turner and...@fubar.geek.nz
  
  Requested by: andrew

Modified:
  head/contrib/llvm/lib/Target/AArch64/AArch64RegisterInfo.cpp

Modified: head/contrib/llvm/lib/Target/AArch64/AArch64RegisterInfo.cpp
==
--- head/contrib/llvm/lib/Target/AArch64/AArch64RegisterInfo.cppMon Jan 
26 19:42:39 2015(r23)
+++ head/contrib/llvm/lib/Target/AArch64/AArch64RegisterInfo.cppMon Jan 
26 21:17:14 2015(r24)
@@ -33,6 +33,10 @@ using namespace llvm;
 #define GET_REGINFO_TARGET_DESC
 #include AArch64GenRegisterInfo.inc
 
+static cl::optbool
+ReserveX18(aarch64-reserve-x18, cl::Hidden,
+  cl::desc(Reserve X18, making it unavailable as GPR));
+
 AArch64RegisterInfo::AArch64RegisterInfo(const AArch64InstrInfo *tii,
  const AArch64Subtarget *sti)
 : AArch64GenRegisterInfo(AArch64::LR), TII(tii), STI(sti) {}
@@ -90,7 +94,7 @@ AArch64RegisterInfo::getReservedRegs(con
 Reserved.set(AArch64::W29);
   }
 
-  if (STI-isTargetDarwin()) {
+  if (STI-isTargetDarwin() || ReserveX18) {
 Reserved.set(AArch64::X18); // Platform register
 Reserved.set(AArch64::W18);
   }
@@ -117,7 +121,7 @@ bool AArch64RegisterInfo::isReservedReg(
 return true;
   case AArch64::X18:
   case AArch64::W18:
-return STI-isTargetDarwin();
+return STI-isTargetDarwin() || ReserveX18;
   case AArch64::FP:
   case AArch64::W29:
 return TFI-hasFP(MF) || STI-isTargetDarwin();
@@ -379,7 +383,7 @@ unsigned AArch64RegisterInfo::getRegPres
   case AArch64::GPR64commonRegClassID:
 return 32 - 1  // XZR/SP
- (TFI-hasFP(MF) || STI-isTargetDarwin()) // FP
-   - STI-isTargetDarwin() // X18 reserved as platform register
+   - (STI-isTargetDarwin() || ReserveX18) // X18 reserved as platform 
register
- hasBasePointer(MF);   // X19
   case AArch64::FPR8RegClassID:
   case AArch64::FPR16RegClassID:
___
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-unsubscr...@freebsd.org


svn commit: r277776 - head/contrib/llvm/patches

2015-01-26 Thread Dimitry Andric
Author: dim
Date: Mon Jan 26 21:24:04 2015
New Revision: 26
URL: https://svnweb.freebsd.org/changeset/base/26

Log:
  Add llvm and clang patches corresponding to r24 and r25.

Added:
  head/contrib/llvm/patches/patch-29-llvm-r226664-aarch64-x18.diff
  head/contrib/llvm/patches/patch-30-clang-r227062-fixes-x18.diff

Added: head/contrib/llvm/patches/patch-29-llvm-r226664-aarch64-x18.diff
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/contrib/llvm/patches/patch-29-llvm-r226664-aarch64-x18.diffMon Jan 
26 21:24:04 2015(r26)
@@ -0,0 +1,83 @@
+Pull in r226664 from upstream llvm trunk (by Tim Northover):
+
+  AArch64: add backend option to reserve x18 (platform register)
+
+  AAPCS64 says that it's up to the platform to specify whether x18 is
+  reserved, and a first step on that way is to add a flag controlling
+  it.
+
+  From: Andrew Turner and...@fubar.geek.nz
+
+Introduced here: http://svnweb.freebsd.org/changeset/base/24
+
+Index: lib/Target/AArch64/AArch64RegisterInfo.cpp
+===
+--- lib/Target/AArch64/AArch64RegisterInfo.cpp
 lib/Target/AArch64/AArch64RegisterInfo.cpp
+@@ -33,6 +33,10 @@ using namespace llvm;
+ #define GET_REGINFO_TARGET_DESC
+ #include AArch64GenRegisterInfo.inc
+ 
++static cl::optbool
++ReserveX18(aarch64-reserve-x18, cl::Hidden,
++  cl::desc(Reserve X18, making it unavailable as GPR));
++
+ AArch64RegisterInfo::AArch64RegisterInfo(const AArch64InstrInfo *tii,
+  const AArch64Subtarget *sti)
+ : AArch64GenRegisterInfo(AArch64::LR), TII(tii), STI(sti) {}
+@@ -90,7 +94,7 @@ AArch64RegisterInfo::getReservedRegs(const Machine
+ Reserved.set(AArch64::W29);
+   }
+ 
+-  if (STI-isTargetDarwin()) {
++  if (STI-isTargetDarwin() || ReserveX18) {
+ Reserved.set(AArch64::X18); // Platform register
+ Reserved.set(AArch64::W18);
+   }
+@@ -117,7 +121,7 @@ bool AArch64RegisterInfo::isReservedReg(const Mach
+ return true;
+   case AArch64::X18:
+   case AArch64::W18:
+-return STI-isTargetDarwin();
++return STI-isTargetDarwin() || ReserveX18;
+   case AArch64::FP:
+   case AArch64::W29:
+ return TFI-hasFP(MF) || STI-isTargetDarwin();
+@@ -379,7 +383,7 @@ unsigned AArch64RegisterInfo::getRegPressureLimit(
+   case AArch64::GPR64commonRegClassID:
+ return 32 - 1  // XZR/SP
+- (TFI-hasFP(MF) || STI-isTargetDarwin()) // FP
+-   - STI-isTargetDarwin() // X18 reserved as platform register
++   - (STI-isTargetDarwin() || ReserveX18) // X18 reserved as 
platform register
+- hasBasePointer(MF);   // X19
+   case AArch64::FPR8RegClassID:
+   case AArch64::FPR16RegClassID:
+Index: test/CodeGen/AArch64/arm64-platform-reg.ll
+===
+--- test/CodeGen/AArch64/arm64-platform-reg.ll
 test/CodeGen/AArch64/arm64-platform-reg.ll
+@@ -1,4 +1,5 @@
+-; RUN: llc -mtriple=arm64-apple-ios -o - %s | FileCheck %s 
--check-prefix=CHECK-DARWIN
++; RUN: llc -mtriple=arm64-apple-ios -o - %s | FileCheck %s 
--check-prefix=CHECK-RESERVE-X18
++; RUN: llc -mtriple=arm64-freebsd-gnu -aarch64-reserve-x18 -o - %s | 
FileCheck %s --check-prefix=CHECK-RESERVE-X18
+ ; RUN: llc -mtriple=arm64-linux-gnu -o - %s | FileCheck %s
+ 
+ ; x18 is reserved as a platform register on Darwin but not on other
+@@ -16,11 +17,11 @@ define void @keep_live() {
+ ; CHECK: ldr x18
+ ; CHECK: str x18
+ 
+-; CHECK-DARWIN-NOT: ldr fp
+-; CHECK-DARWIN-NOT: ldr x18
+-; CHECK-DARWIN: Spill
+-; CHECK-DARWIN-NOT: ldr fp
+-; CHECK-DARWIN-NOT: ldr x18
+-; CHECK-DARWIN: ret
++; CHECK-RESERVE-X18-NOT: ldr fp
++; CHECK-RESERVE-X18-NOT: ldr x18
++; CHECK-RESERVE-X18: Spill
++; CHECK-RESERVE-X18-NOT: ldr fp
++; CHECK-RESERVE-X18-NOT: ldr x18
++; CHECK-RESERVE-X18: ret
+   ret void
+ }

Added: head/contrib/llvm/patches/patch-30-clang-r227062-fixes-x18.diff
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/contrib/llvm/patches/patch-30-clang-r227062-fixes-x18.diff Mon Jan 
26 21:24:04 2015(r26)
@@ -0,0 +1,54 @@
+Pull in r227062 from upstream clang trunk (by Renato Golin):
+
+  Allows Clang to use LLVM's fixes-x18 option
+
+  This patch allows clang to have llvm reserve the x18
+  platform register on AArch64. FreeBSD will use this in the kernel for
+  per-cpu data but has no need to reserve this register in userland so
+  will need this flag to reserve it.
+
+  This uses llvm r226664 to allow this register to be reserved.
+
+  Patch by Andrew Turner.
+
+Introduced here: http://svnweb.freebsd.org/changeset/base/25
+
+Index: tools/clang/include/clang/Driver/Options.td

Re: svn commit: r277652 - in head/usr.sbin/pw: . tests

2015-01-26 Thread Devin Teske

 On Jan 25, 2015, at 7:31 AM, Bruce Evans b...@optusnet.com.au wrote:
 
 On Sun, 25 Jan 2015, Slawa Olhovchenkov wrote:
 
 On Sun, Jan 25, 2015 at 04:56:24PM +1100, Bruce Evans wrote:
 
 Negative ids have historical abuses in places like mountd.

Which paves the way for the “accepted practice” argument
and backed up by “in-the-field usage” statement(s).



  mountd still
 hard-codes -2 and -2 for the default uid and gid of an unprivileged user.
 It at least casts these values to uid_t and gid_t before using them.
 This gives the ids the non-random values of UINT32_MAX-1 if uid_t and
 gid_t are uint32_t.  (If uid_t and gid_t were signed, then it would
 leave the values as negative, so invalid.)  These magic values may work
 better than when ids were 16 bits, since there is less risk of them
 conflicting with a normal id.  However, the non-conflict is probably
 a bug.  FreeBSD uses the magic ids of 65534 for user nobody: group
 nobody.  These would have been (id_t)-2 with 16-bit ids.  They no
 longer match, so ls displays (id_t)-2 numerically.  FreeBSD also has
 a group nogroup = 65553 that doesn't match the nfs usage.  However2,
 in FreeBSD-1 wher ids were 16-bits, nobody was 32767 and nogroup was
 32766. so they didn't match nfs for other reasons.  The 2 non-groups
 now seem to be just a bug -- FreeBSD-1 didn't have group nobody.
 4.4BSD-Lite2 has the same values as FreeBSD-1.
 
 This is not full true for ZFS case.
 On ZFS nobody is 2^32-2.
 
 File systems don't get to decide this.

+1 (and thanks for the historical account, bruce — sincerely)

However, I still want to make the argument that:

a. Because we’ve supported mapping negative inputs to unsigned values in pw 
*for over a decade*, that…

b. We should either revert or make a relnotes submission to note that we’re 
changing the long-standing accepted practice.

Changing the accepted practice broke code internally, it would have likely 
broken some external code as well — and people deserve to know about said 
change else we should continue to support accepted practice that is decade(s) 
old.
— 
Devin

___
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-unsubscr...@freebsd.org

Re: svn commit: r277714 - head/sbin/ipfw

2015-01-26 Thread John Baldwin
On Monday, January 26, 2015 09:34:39 PM Olivier Cochard-Labbé wrote:
 On Sun, Jan 25, 2015 at 9:37 PM, John Baldwin j...@freebsd.org wrote:
  Author: jhb
  Date: Sun Jan 25 20:37:32 2015
  New Revision: 277714
  URL: https://svnweb.freebsd.org/changeset/base/277714
  
  Log:
natd(8) will work with an unconfigured interface and effectively not do
anything until the interface is assigned an address.  This fixes
ipfw_nat to do the same by using an IP of INADDR_ANY instead of
aborting the nat setup if the requested interface is not yet configured.
 
 Hi,
 
 I've still a problem with ipfw_nat and unconfigured interface:
 On my setup I'm using ipfw with NAT rules using an OpenVPN tunnel interface
 as source address for NATting.
 
 During the machine startup, ipfw is started before openvpn (hopefully) and
 its configuration mention do to NAT using tun0 IP address.
 Then OpenVPN start and create a tun0 and set an IP address on it.
 = But no unicast traffic is allowed on this tun0 interface until I restart
 ipfw.
 
 If I correctly understand the log of this commit: This behavior should be
 fixed by this commit, right ?

It might.  What happened for me is that I was using nat over wlan0 for VM's
on my laptop to reach the outside world, but wlan0 doesn't get an IP until
later in the boot after it associates.  As a result, wlan0 wasn't passing any
IP traffic until this fix (or if I reloaded ipfw after wlan0 was configured).

-- 
John Baldwin
___
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-unsubscr...@freebsd.org


Re: svn commit: r277714 - head/sbin/ipfw

2015-01-26 Thread Nick Hibma

 On 26 Jan 2015, at 22:24, John Baldwin j...@freebsd.org wrote:
 
 On Monday, January 26, 2015 09:34:39 PM Olivier Cochard-Labbé wrote:
 On Sun, Jan 25, 2015 at 9:37 PM, John Baldwin j...@freebsd.org wrote:
 Author: jhb
 Date: Sun Jan 25 20:37:32 2015
 New Revision: 277714
 URL: https://svnweb.freebsd.org/changeset/base/277714
 
 Log:
  natd(8) will work with an unconfigured interface and effectively not do
  anything until the interface is assigned an address.  This fixes
  ipfw_nat to do the same by using an IP of INADDR_ANY instead of
  aborting the nat setup if the requested interface is not yet configured.
 
 Hi,
 
 I've still a problem with ipfw_nat and unconfigured interface:
 On my setup I'm using ipfw with NAT rules using an OpenVPN tunnel interface
 as source address for NATting.
 
 During the machine startup, ipfw is started before openvpn (hopefully) and
 its configuration mention do to NAT using tun0 IP address.
 Then OpenVPN start and create a tun0 and set an IP address on it.
 = But no unicast traffic is allowed on this tun0 interface until I restart
 ipfw.
 
 If I correctly understand the log of this commit: This behavior should be
 fixed by this commit, right ?
 
 It might.  What happened for me is that I was using nat over wlan0 for VM's
 on my laptop to reach the outside world, but wlan0 doesn't get an IP until
 later in the boot after it associates.  As a result, wlan0 wasn't passing any
 IP traffic until this fix (or if I reloaded ipfw after wlan0 was configured).

I don’t think it does. The interface is not available until openvpn is 
started.You need to clone the interface during boot by adding

cloned_interfaces=‘tun0’

in your /etc/rc.conf. Initialisation is then done later by openvpn.

Let me know if that works for you.

Nick
___
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-unsubscr...@freebsd.org


svn commit: r277787 - in head: contrib/sendmail/cf/m4 etc/sendmail

2015-01-26 Thread Gregory Neil Shapiro
Author: gshapiro
Date: Tue Jan 27 04:06:47 2015
New Revision: 277787
URL: https://svnweb.freebsd.org/changeset/base/277787

Log:
  Remove build specific details from sendmail.cf/submit.cf in support of
  https://wiki.freebsd.org/ReproducibleBuilds
  
  The contrib/sendmail change will be made in the upstream source for a
  future sendmail release.
  
  Reviewed by:  des
  MFC after:3 days

Modified:
  head/contrib/sendmail/cf/m4/cfhead.m4
  head/etc/sendmail/Makefile

Modified: head/contrib/sendmail/cf/m4/cfhead.m4
==
--- head/contrib/sendmail/cf/m4/cfhead.m4   Tue Jan 27 03:37:58 2015
(r277786)
+++ head/contrib/sendmail/cf/m4/cfhead.m4   Tue Jan 27 04:06:47 2015
(r277787)
@@ -20,7 +20,7 @@
 ifdef(`__win32__', `dnl', `dnl
 ifdef(`TEMPFILE', `dnl', `define(`TEMPFILE', maketemp(/tmp/cfXX))dnl
 syscmd(sh _CF_DIR_`'sh/makeinfo.sh _CF_DIR_  TEMPFILE)dnl
-include(TEMPFILE)dnl
+ifdef(`_NO_MAKEINFO_',, `include(TEMPFILE)')dnl
 syscmd(rm -f TEMPFILE)dnl')')
 #
 ##

Modified: head/etc/sendmail/Makefile
==
--- head/etc/sendmail/Makefile  Tue Jan 27 03:37:58 2015(r277786)
+++ head/etc/sendmail/Makefile  Tue Jan 27 04:06:47 2015(r277787)
@@ -17,7 +17,8 @@ M4FILES!= find ${SENDMAIL_CF_DIR} -type 
 
 .mc.cf: ${M4FILES}
${RM} ${.TARGET}
-   ${M4} -D_CF_DIR_=${SENDMAIL_CF_DIR}/ ${SENDMAIL_M4_FLAGS} \
+   ${M4} -D_CF_DIR_=${SENDMAIL_CF_DIR}/ -D_NO_MAKEINFO_ \
+   ${SENDMAIL_M4_FLAGS} \
${SENDMAIL_CF_DIR}/m4/cf.m4 ${.IMPSRC}  ${.TARGET}
${CHMOD} ${ROMODE} ${.TARGET}
 
___
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-unsubscr...@freebsd.org


Re: svn commit: r277788 - head/sys/dev/virtio/block

2015-01-26 Thread Mark Johnston
On Tue, Jan 27, 2015 at 05:34:47AM +, Bryan Venteicher wrote:
 Author: bryanv
 Date: Tue Jan 27 05:34:46 2015
 New Revision: 277788
 URL: https://svnweb.freebsd.org/changeset/base/277788
 
 Log:
   Rework vtblk dump handling of in flight requests
   
   Previously, the driver resets the device and abandon the requests that
   are caught in flight when the dump was initiated. This was problematic
   if the system is resumed after the dump is completed.
   
   While that is probably not the typical action, it is simple to rework
   the driver to very likely have the device usable after the dump without
   making it more likely for the dump to fail. The in flight requests are
   simply queued for completion once the dump is finished.
   
   Requested by:   markj
   MFC after:  1 month

Thank you!
___
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-unsubscr...@freebsd.org


svn commit: r277789 - in stable/10/sys: netinet netinet6

2015-01-26 Thread Bryan Venteicher
Author: bryanv
Date: Tue Jan 27 06:19:30 2015
New Revision: 277789
URL: https://svnweb.freebsd.org/changeset/base/277789

Log:
  MFC r272886:
  
Add context pointer and source address to the UDP tunnel callback
  
These are needed for the forthcoming vxlan implementation. The context
pointer means we do not have to use a spare pointer field in the inpcb,
and the source address is required to populate vxlan's forwarding table.

Modified:
  stable/10/sys/netinet/sctputil.c
  stable/10/sys/netinet/udp_usrreq.c
  stable/10/sys/netinet/udp_var.h
  stable/10/sys/netinet6/udp6_usrreq.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/netinet/sctputil.c
==
--- stable/10/sys/netinet/sctputil.cTue Jan 27 05:34:46 2015
(r277788)
+++ stable/10/sys/netinet/sctputil.cTue Jan 27 06:19:30 2015
(r277789)
@@ -6832,7 +6832,8 @@ sctp_log_trace(uint32_t subsys, const ch
 
 #endif
 static void
-sctp_recv_udp_tunneled_packet(struct mbuf *m, int off, struct inpcb *ignored)
+sctp_recv_udp_tunneled_packet(struct mbuf *m, int off, struct inpcb *ignored,
+const struct sockaddr *sa SCTP_UNUSED, void *ctx SCTP_UNUSED)
 {
struct ip *iph;
 
@@ -6968,7 +6969,7 @@ sctp_over_udp_start(void)
}
/* Call the special UDP hook. */
if ((ret = udp_set_kernel_tunneling(SCTP_BASE_INFO(udp4_tun_socket),
-   sctp_recv_udp_tunneled_packet))) {
+   sctp_recv_udp_tunneled_packet, NULL))) {
sctp_over_udp_stop();
return (ret);
}
@@ -6992,7 +6993,7 @@ sctp_over_udp_start(void)
}
/* Call the special UDP hook. */
if ((ret = udp_set_kernel_tunneling(SCTP_BASE_INFO(udp6_tun_socket),
-   sctp_recv_udp_tunneled_packet))) {
+   sctp_recv_udp_tunneled_packet, NULL))) {
sctp_over_udp_stop();
return (ret);
}

Modified: stable/10/sys/netinet/udp_usrreq.c
==
--- stable/10/sys/netinet/udp_usrreq.c  Tue Jan 27 05:34:46 2015
(r277788)
+++ stable/10/sys/netinet/udp_usrreq.c  Tue Jan 27 06:19:30 2015
(r277789)
@@ -303,7 +303,8 @@ udp_append(struct inpcb *inp, struct ip 
 */
up = intoudpcb(inp);
if (up-u_tun_func != NULL) {
-   (*up-u_tun_func)(n, off, inp);
+   (*up-u_tun_func)(n, off, inp, (struct sockaddr *)udp_in,
+   up-u_tun_ctx);
return;
}
 
@@ -1624,7 +1625,7 @@ udp_attach(struct socket *so, int proto,
 #endif /* INET */
 
 int
-udp_set_kernel_tunneling(struct socket *so, udp_tun_func_t f)
+udp_set_kernel_tunneling(struct socket *so, udp_tun_func_t f, void *ctx)
 {
struct inpcb *inp;
struct udpcb *up;
@@ -1640,6 +1641,7 @@ udp_set_kernel_tunneling(struct socket *
return (EBUSY);
}
up-u_tun_func = f;
+   up-u_tun_ctx = ctx;
INP_WUNLOCK(inp);
return (0);
 }

Modified: stable/10/sys/netinet/udp_var.h
==
--- stable/10/sys/netinet/udp_var.h Tue Jan 27 05:34:46 2015
(r277788)
+++ stable/10/sys/netinet/udp_var.h Tue Jan 27 06:19:30 2015
(r277789)
@@ -55,7 +55,8 @@ struct udpiphdr {
 struct inpcb;
 struct mbuf;
 
-typedef void(*udp_tun_func_t)(struct mbuf *, int off, struct inpcb *);
+typedef void(*udp_tun_func_t)(struct mbuf *, int off, struct inpcb *,
+ const struct sockaddr *, void *);
 
 /*
  * UDP control block; one per udp.
@@ -65,6 +66,7 @@ struct udpcb {
u_int   u_flags;/* Generic UDP flags. */
uint16_tu_rxcslen;  /* Coverage for incoming datagrams. */
uint16_tu_txcslen;  /* Coverage for outgoing datagrams. */
+   void*u_tun_ctx; /* Tunneling callback context. */
 };
 
 #defineintoudpcb(ip)   ((struct udpcb *)(ip)-inp_ppcb)
@@ -176,7 +178,8 @@ voidudplite_input(struct mbuf *, int);
 struct inpcb   *udp_notify(struct inpcb *inp, int errno);
 intudp_shutdown(struct socket *so);
 
-intudp_set_kernel_tunneling(struct socket *so, udp_tun_func_t f);
+intudp_set_kernel_tunneling(struct socket *so, udp_tun_func_t f,
+void *ctx);
 
 #endif /* _KERNEL */
 

Modified: stable/10/sys/netinet6/udp6_usrreq.c
==
--- stable/10/sys/netinet6/udp6_usrreq.cTue Jan 27 05:34:46 2015
(r277788)
+++ stable/10/sys/netinet6/udp6_usrreq.cTue Jan 27 06:19:30 2015
(r277789)
@@ -148,7 +148,8 @@ udp6_append(struct inpcb *inp, struct mb
 */
up = intoudpcb(inp);
if (up-u_tun_func != NULL) {
-   

svn commit: r277782 - vendor/tcpdump/dist

2015-01-26 Thread Pedro F. Giffuni
Author: pfg
Date: Tue Jan 27 01:39:42 2015
New Revision: 277782
URL: https://svnweb.freebsd.org/changeset/base/277782

Log:
  Merge some cherry-picked fixes originating in OpenBSD
  
  Check whether the version field is available before looking at it.
  While we're at it, use ND_TCHECK(), rather than a hand-rolled check, to
  check whether we have the full fixed-length portion of the IPv4 header.
  
  commit c67afe913011138a2504ec4d3d423b48e73b12f3
  
  Do more length checking. From OpenBSD.
  
  commit d7516761f9c4877bcb05bb6543be3543e165249

Modified:
  vendor/tcpdump/dist/print-ip.c
  vendor/tcpdump/dist/print-sl.c

Modified: vendor/tcpdump/dist/print-ip.c
==
--- vendor/tcpdump/dist/print-ip.c  Mon Jan 26 23:45:10 2015
(r277781)
+++ vendor/tcpdump/dist/print-ip.c  Tue Jan 27 01:39:42 2015
(r277782)
@@ -529,6 +529,7 @@ ip_print(netdissect_options *ndo,
struct protoent *proto;
 
ipds-ip = (const struct ip *)bp;
+   ND_TCHECK(ipds-ip-ip_vhl);
if (IP_V(ipds-ip) != 4) { /* print version if != 4 */
ND_PRINT((ndo, IP%u , IP_V(ipds-ip)));
if (IP_V(ipds-ip) == 6)
@@ -537,10 +538,7 @@ ip_print(netdissect_options *ndo,
else if (!ndo-ndo_eflag)
ND_PRINT((ndo, IP ));
 
-   if ((u_char *)(ipds-ip + 1)  ndo-ndo_snapend) {
-   ND_PRINT((ndo, %s, tstr));
-   return;
-   }
+   ND_TCHECK(*ipds-ip);
if (length  sizeof (struct ip)) {
ND_PRINT((ndo, truncated-ip %u, length));
return;
@@ -669,6 +667,11 @@ ip_print(netdissect_options *ndo,
ND_PRINT((ndo,  ip-proto-%d, ipds-ip-ip_p));
}
}
+   return;
+
+trunc:
+   ND_PRINT((ndo, %s, tstr));
+   return;
 }
 
 void

Modified: vendor/tcpdump/dist/print-sl.c
==
--- vendor/tcpdump/dist/print-sl.c  Mon Jan 26 23:45:10 2015
(r277781)
+++ vendor/tcpdump/dist/print-sl.c  Tue Jan 27 01:39:42 2015
(r277782)
@@ -62,7 +62,7 @@ sl_if_print(netdissect_options *ndo,
register u_int length = h-len;
register const struct ip *ip;
 
-   if (caplen  SLIP_HDRLEN) {
+   if (caplen  SLIP_HDRLEN || length  SLIP_HDRLEN) {
ND_PRINT((ndo, %s, tstr));
return (caplen);
}
___
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-unsubscr...@freebsd.org


Re: svn commit: r277737 - in head: etc/pam.d tools/build/mk

2015-01-26 Thread Garrett Cooper
On Jan 26, 2015, at 4:23, Dag-Erling Smørgrav d...@des.no wrote:

 Garrett Cooper n...@freebsd.org writes:
 Log:
  Honor MK_ACCT with etc/pam.d/atrun
 
 The correct idiom would be

I disagree for a few reasons:
1. style.Makefile(9) makes no mention of having to use an idiom over the other.
2. The likelihood of typos creeping into Makefiles is greater with the _foo= 
idiom.
3. It creates unnecessary local variables — especially when appending is so 
easy to do.
4. It’s more difficult when backporting changes to do it with the variable 
method — especially if context around the line modified has changed a great 
deal.


signature.asc
Description: Message signed with OpenPGP using GPGMail


svn commit: r277790 - stable/10/sys/sys

2015-01-26 Thread Bryan Venteicher
Author: bryanv
Date: Tue Jan 27 06:24:33 2015
New Revision: 277790
URL: https://svnweb.freebsd.org/changeset/base/277790

Log:
  Bump __FreeBSD_version after r277789: Changes to the UDP tunneling callback

Modified:
  stable/10/sys/sys/param.h

Modified: stable/10/sys/sys/param.h
==
--- stable/10/sys/sys/param.h   Tue Jan 27 06:19:30 2015(r277789)
+++ stable/10/sys/sys/param.h   Tue Jan 27 06:24:33 2015(r277790)
@@ -58,7 +58,7 @@
  * in the range 5 to 9.
  */
 #undef __FreeBSD_version
-#define __FreeBSD_version 1001506  /* Master, propagated to newvers */
+#define __FreeBSD_version 1001507  /* Master, propagated to newvers */
 
 /*
  * __FreeBSD_kernel__ indicates that this system uses the kernel of FreeBSD,
___
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-unsubscr...@freebsd.org


svn commit: r277788 - head/sys/dev/virtio/block

2015-01-26 Thread Bryan Venteicher
Author: bryanv
Date: Tue Jan 27 05:34:46 2015
New Revision: 277788
URL: https://svnweb.freebsd.org/changeset/base/277788

Log:
  Rework vtblk dump handling of in flight requests
  
  Previously, the driver resets the device and abandon the requests that
  are caught in flight when the dump was initiated. This was problematic
  if the system is resumed after the dump is completed.
  
  While that is probably not the typical action, it is simple to rework
  the driver to very likely have the device usable after the dump without
  making it more likely for the dump to fail. The in flight requests are
  simply queued for completion once the dump is finished.
  
  Requested by: markj
  MFC after:1 month

Modified:
  head/sys/dev/virtio/block/virtio_blk.c

Modified: head/sys/dev/virtio/block/virtio_blk.c
==
--- head/sys/dev/virtio/block/virtio_blk.c  Tue Jan 27 04:06:47 2015
(r277787)
+++ head/sys/dev/virtio/block/virtio_blk.c  Tue Jan 27 05:34:46 2015
(r277788)
@@ -76,9 +76,8 @@ struct vtblk_softc {
 #define VTBLK_FLAG_READONLY0x0002
 #define VTBLK_FLAG_DETACH  0x0004
 #define VTBLK_FLAG_SUSPEND 0x0008
-#define VTBLK_FLAG_DUMPING 0x0010
-#define VTBLK_FLAG_BARRIER 0x0020
-#define VTBLK_FLAG_WC_CONFIG   0x0040
+#define VTBLK_FLAG_BARRIER 0x0010
+#define VTBLK_FLAG_WC_CONFIG   0x0020
 
struct virtqueue*vtblk_vq;
struct sglist   *vtblk_sglist;
@@ -95,6 +94,7 @@ struct vtblk_softc {
int  vtblk_request_count;
enum vtblk_cache_modevtblk_write_cache;
 
+   struct bio_queue vtblk_dump_queue;
struct vtblk_request vtblk_dump_request;
 };
 
@@ -162,7 +162,7 @@ static void vtblk_queue_completed(struct
struct bio_queue *);
 static voidvtblk_done_completed(struct vtblk_softc *,
struct bio_queue *);
-static voidvtblk_drain_vq(struct vtblk_softc *, int);
+static voidvtblk_drain_vq(struct vtblk_softc *);
 static voidvtblk_drain(struct vtblk_softc *);
 
 static voidvtblk_startio(struct vtblk_softc *);
@@ -177,9 +177,10 @@ static int vtblk_quiesce(struct vtblk_so
 static voidvtblk_vq_intr(void *);
 static voidvtblk_stop(struct vtblk_softc *);
 
-static voidvtblk_dump_prepare(struct vtblk_softc *);
+static voidvtblk_dump_quiesce(struct vtblk_softc *);
 static int vtblk_dump_write(struct vtblk_softc *, void *, off_t, size_t);
 static int vtblk_dump_flush(struct vtblk_softc *);
+static voidvtblk_dump_complete(struct vtblk_softc *);
 
 static voidvtblk_set_write_cache(struct vtblk_softc *, int);
 static int vtblk_write_cache_enabled(struct vtblk_softc *sc,
@@ -302,6 +303,7 @@ vtblk_attach(device_t dev)
sc-vtblk_dev = dev;
VTBLK_LOCK_INIT(sc, device_get_nameunit(dev));
bioq_init(sc-vtblk_bioq);
+   TAILQ_INIT(sc-vtblk_dump_queue);
TAILQ_INIT(sc-vtblk_req_free);
TAILQ_INIT(sc-vtblk_req_ready);
 
@@ -506,25 +508,19 @@ vtblk_dump(void *arg, void *virtual, vm_
int error;
 
dp = arg;
+   error = 0;
 
if ((sc = dp-d_drv1) == NULL)
return (ENXIO);
 
VTBLK_LOCK(sc);
 
-   if ((sc-vtblk_flags  VTBLK_FLAG_DUMPING) == 0) {
-   vtblk_dump_prepare(sc);
-   sc-vtblk_flags |= VTBLK_FLAG_DUMPING;
-   }
+   vtblk_dump_quiesce(sc);
 
if (length  0)
error = vtblk_dump_write(sc, virtual, offset, length);
-   else if (virtual == NULL  offset == 0)
-   error = vtblk_dump_flush(sc);
-   else {
-   error = EINVAL;
-   sc-vtblk_flags = ~VTBLK_FLAG_DUMPING;
-   }
+   if (error || (virtual == NULL  offset == 0))
+   vtblk_dump_complete(sc);
 
VTBLK_UNLOCK(sc);
 
@@ -996,7 +992,7 @@ vtblk_done_completed(struct vtblk_softc 
 }
 
 static void
-vtblk_drain_vq(struct vtblk_softc *sc, int skip_done)
+vtblk_drain_vq(struct vtblk_softc *sc)
 {
struct virtqueue *vq;
struct vtblk_request *req;
@@ -1006,9 +1002,7 @@ vtblk_drain_vq(struct vtblk_softc *sc, i
last = 0;
 
while ((req = virtqueue_drain(vq, last)) != NULL) {
-   if (!skip_done)
-   vtblk_bio_done(sc, req-vbr_bp, ENXIO);
-
+   vtblk_bio_done(sc, req-vbr_bp, ENXIO);
vtblk_request_enqueue(sc, req);
}
 
@@ -1031,7 +1025,7 @@ vtblk_drain(struct vtblk_softc *sc)
vtblk_queue_completed(sc, queue);
vtblk_done_completed(sc, queue);
 
-   vtblk_drain_vq(sc, 0);
+   vtblk_drain_vq(sc);
}
 
while ((req = vtblk_request_next_ready(sc)) != NULL) {
@@ -1256,31 +1250,16 @@ vtblk_stop(struct vtblk_softc *sc)
 }
 
 static void
-vtblk_dump_prepare(struct vtblk_softc *sc)
+vtblk_dump_quiesce(struct vtblk_softc *sc)
 {
-   

Re: svn commit: r276423 - stable/10/share/mk

2015-01-26 Thread Garrett Cooper
On Dec 31, 2014, at 7:54, Julio Merino j...@meroh.net wrote:

 On Dec 30, 2014, at 21:46, Garrett Cooper n...@freebsd.org wrote:
 
 Author: ngie
 Date: Tue Dec 30 20:46:01 2014
 New Revision: 276423
 URL: https://svnweb.freebsd.org/changeset/base/276423
 
 Log:
 MFC r274077:
 
   As an optimization (in simple, well used cases) only call cat ${.ALLSRC} 
 when
   generating files from various sources instead of calling cat ${.ALLSRC} | 
 sed
 
 Is there any visible effect to this optimization?  Otherwise, the extra 
 complexity does not seem warranted.

Hi Julio!

There are a few things to gain from this:
1. Errors with cat’ing the files are caught when sed transformations aren’t 
applied to the files.
2. Less resources are used in generating the files if sed isn’t needed:
i. Less forked processes.
ii. No pipelines need to be created.
I don’t have numbers for how much faster it is (probably negligible if done 
once, potentially more noticeable if done as a micro benchmark), but I could do 
this if you like.

Thank you!


signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: svn commit: r277737 - in head: etc/pam.d tools/build/mk

2015-01-26 Thread Alexey Dokuchaev
On Mon, Jan 26, 2015 at 06:26:54PM -0800, Garrett Cooper wrote:
 On Jan 26, 2015, at 4:23, Dag-Erling Smorgrav d...@des.no wrote:
  Garrett Cooper n...@freebsd.org writes:
  Log:
   Honor MK_ACCT with etc/pam.d/atrun
  
  The correct idiom would be
 
 I disagree for a few reasons: [...]
 2. The likelihood of typos creeping into Makefiles is greater with the
 _foo= idiom.
 3. It creates unnecessary local variables - especially when appending is
 so easy to do.
 4. It's more difficult when backporting changes to do it with the variable
 method - especially if context around the line modified has changed a
 great deal.

As someone who writes makefile daily I would agree, but there should be a
reason why it was always done with _foo.  One obvious one is that it shows
up front all possible values, including conditional ones, which can also
help not to forget something when backporting changes somewhere else. ;-)

./danfe
___
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-unsubscr...@freebsd.org


svn commit: r277792 - head/sys/powerpc/pseries

2015-01-26 Thread Nathan Whitehorn
Author: nwhitehorn
Date: Tue Jan 27 07:20:00 2015
New Revision: 277792
URL: https://svnweb.freebsd.org/changeset/base/277792

Log:
  Fix bug in mapppings of multiple pages exposed by updates to the VSCSI
  support in QEMU. Each page of a many page mapping was getting mapped to
  the same physical address, which is not the desired behavior.
  
  MFC after:1 week

Modified:
  head/sys/powerpc/pseries/plpar_iommu.c

Modified: head/sys/powerpc/pseries/plpar_iommu.c
==
--- head/sys/powerpc/pseries/plpar_iommu.c  Tue Jan 27 06:50:01 2015
(r277791)
+++ head/sys/powerpc/pseries/plpar_iommu.c  Tue Jan 27 07:20:00 2015
(r277792)
@@ -191,13 +191,13 @@ phyp_iommu_map(device_t dev, bus_dma_seg
 
tce = trunc_page(segs[i].ds_addr);
tce |= 0x3; /* read/write */
-   if (papr_supports_stuff_tce) {
-   error = phyp_hcall(H_STUFF_TCE, window-map-iobn,
-   alloced, tce, allocsize/PAGE_SIZE);
-   } else {
-   for (j = 0; j  allocsize; j += PAGE_SIZE)
-   error = phyp_hcall(H_PUT_TCE, window-map-iobn,
-   alloced + j, tce + j);
+   for (j = 0; j  allocsize; j += PAGE_SIZE) {
+   error = phyp_hcall(H_PUT_TCE, window-map-iobn,
+   alloced + j, tce + j);
+   if (error  0) {
+   panic(IOMMU mapping error: %d\n, error);
+   return (ENOMEM);
+   }
}
 
segs[i].ds_addr = alloced + (segs[i].ds_addr  PAGE_MASK);
___
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-unsubscr...@freebsd.org


Re: svn commit: r277681 - in head/sys: conf modules

2015-01-26 Thread Garrett Cooper
On Jan 25, 2015, at 9:46, Warner Losh i...@bsdimp.com wrote:

 The number of MK_foo opts that are intruding into the kernel module builds is
 starting to get out of hand. :( Especially since they are duplicative of the
 MODULES_OVERRIDE functionality and are very unevenly applied.

I don’t really like it either to be honest, but the problem is that building 
MK_CUSE == no (for instance) should produce a working default set of SUBDIRs so 
both the kernel and userland infrastructure are compiled with the build. 
Telling someone to use MODULES_OVERRIDE to just remove cuse (for instance), is 
a user unfriendly system..

If there was a way to pick up a subset of src.opts.mk though and apply it to 
kern.opts.mk, I would be incredibly keen on making that change.

Thank you!


signature.asc
Description: Message signed with OpenPGP using GPGMail


Re: svn commit: r277710 - head/sys/dev/pci

2015-01-26 Thread Alexey Dokuchaev
On Mon, Jan 26, 2015 at 11:06:27AM -0500, John Baldwin wrote:
 On 1/26/15 9:19 AM, Alexey Dokuchaev wrote:
  On Sun, Jan 25, 2015 at 07:53:10PM +, John Baldwin wrote:
  New Revision: 277710
  URL: https://svnweb.freebsd.org/changeset/base/277710
 
  Log:
Pass a valid Dx state variable to PCIB_POWER_FOR_SLEEP() in
pcib_resume() instead of NULL.
  
  Can this be possibly related to this message I got trying to resume a
  laptop running recent -current [...] ?
 
 Well, the patch came up from debugging something along those lines, but
 it will not fix it.

OK; still nice to hear someone is seeing these as well.  Shall there by any
other patches that could benefit from more testing, I'd gladly help.

./danfe
___
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-unsubscr...@freebsd.org


svn commit: r277783 - head/contrib/tcpdump

2015-01-26 Thread Pedro F. Giffuni
Author: pfg
Date: Tue Jan 27 01:45:47 2015
New Revision: 277783
URL: https://svnweb.freebsd.org/changeset/base/277783

Log:
  MFV   r277782:
  
  Merge some cherry-picked fixes originating in OpenBSD
  
  Check whether the version field is available before looking at it.
  While we're at it, use ND_TCHECK(), rather than a hand-rolled check, to
  check whether we have the full fixed-length portion of the IPv4 header.
  
  commit c67afe913011138a2504ec4d3d423b48e73b12f3
  
  Do more length checking. From OpenBSD.
  
  commit d7516761f9c4877bcb05bb6543be3543e165249

Modified:
  head/contrib/tcpdump/print-ip.c
  head/contrib/tcpdump/print-sl.c
Directory Properties:
  head/contrib/tcpdump/   (props changed)

Modified: head/contrib/tcpdump/print-ip.c
==
--- head/contrib/tcpdump/print-ip.c Tue Jan 27 01:39:42 2015
(r277782)
+++ head/contrib/tcpdump/print-ip.c Tue Jan 27 01:45:47 2015
(r277783)
@@ -537,6 +537,7 @@ ip_print(netdissect_options *ndo,
struct protoent *proto;
 
ipds-ip = (const struct ip *)bp;
+   ND_TCHECK(ipds-ip-ip_vhl);
if (IP_V(ipds-ip) != 4) { /* print version if != 4 */
ND_PRINT((ndo, IP%u , IP_V(ipds-ip)));
if (IP_V(ipds-ip) == 6)
@@ -545,10 +546,7 @@ ip_print(netdissect_options *ndo,
else if (!ndo-ndo_eflag)
ND_PRINT((ndo, IP ));
 
-   if ((u_char *)(ipds-ip + 1)  ndo-ndo_snapend) {
-   ND_PRINT((ndo, %s, tstr));
-   return;
-   }
+   ND_TCHECK(*ipds-ip);
if (length  sizeof (struct ip)) {
ND_PRINT((ndo, truncated-ip %u, length));
return;
@@ -677,6 +675,11 @@ ip_print(netdissect_options *ndo,
ND_PRINT((ndo,  ip-proto-%d, ipds-ip-ip_p));
}
}
+   return;
+
+trunc:
+   ND_PRINT((ndo, %s, tstr));
+   return;
 }
 
 void

Modified: head/contrib/tcpdump/print-sl.c
==
--- head/contrib/tcpdump/print-sl.c Tue Jan 27 01:39:42 2015
(r277782)
+++ head/contrib/tcpdump/print-sl.c Tue Jan 27 01:45:47 2015
(r277783)
@@ -64,7 +64,7 @@ sl_if_print(netdissect_options *ndo,
register u_int length = h-len;
register const struct ip *ip;
 
-   if (caplen  SLIP_HDRLEN) {
+   if (caplen  SLIP_HDRLEN || length  SLIP_HDRLEN) {
ND_PRINT((ndo, %s, tstr));
return (caplen);
}
___
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-unsubscr...@freebsd.org


svn commit: r277743 - head/sys/arm/arm

2015-01-26 Thread Konstantin Belousov
Author: kib
Date: Mon Jan 26 10:11:55 2015
New Revision: 277743
URL: https://svnweb.freebsd.org/changeset/base/277743

Log:
  Ensure that _tmppt KVA is used exclusively by providing exclusive sx
  lock around the mapping and uiomove().  Before r277643, it was
  partially protected by Giant (but potential sleeping in fault from
  uiomove() would still allow other thread to reuse the mapping).
  
  Noted by: ian
  Reviewed by:  alc, ian
  Sponsored by: The FreeBSD Foundation
  MFC after:3 weeks

Modified:
  head/sys/arm/arm/mem.c

Modified: head/sys/arm/arm/mem.c
==
--- head/sys/arm/arm/mem.c  Mon Jan 26 09:53:41 2015(r277742)
+++ head/sys/arm/arm/mem.c  Mon Jan 26 10:11:55 2015(r277743)
@@ -55,6 +55,7 @@ __FBSDID($FreeBSD$);
 #include sys/mutex.h
 #include sys/proc.h
 #include sys/signalvar.h
+#include sys/sx.h
 #include sys/systm.h
 #include sys/uio.h
 
@@ -72,6 +73,9 @@ MALLOC_DEFINE(M_MEMDESC, memdesc, mem
 
 struct mem_range_softc mem_range_softc;
 
+static struct sx tmppt_lock;
+SX_SYSINIT(tmppt, tmppt_lock, mem4map);
+
 /* ARGSUSED */
 int
 memrw(struct cdev *dev, struct uio *uio, int flags)
@@ -107,6 +111,7 @@ memrw(struct cdev *dev, struct uio *uio,
}
if (!address_valid)
return (EINVAL);
+   sx_xlock(tmppt_lock);
pmap_kenter((vm_offset_t)_tmppt, v);
o = (int)uio-uio_offset  PAGE_MASK;
c = (u_int)(PAGE_SIZE - ((int)iov-iov_base  
PAGE_MASK));
@@ -114,6 +119,7 @@ memrw(struct cdev *dev, struct uio *uio,
c = min(c, (u_int)iov-iov_len);
error = uiomove((caddr_t)_tmppt[o], (int)c, uio);
pmap_qremove((vm_offset_t)_tmppt, 1);
+   sx_xunlock(tmppt_lock);
continue;
}
else if (dev2unit(dev) == CDEV_MINOR_KMEM) {
___
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-unsubscr...@freebsd.org


svn commit: r277741 - in head: etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 09:51:21 2015
New Revision: 277741
URL: https://svnweb.freebsd.org/changeset/base/277741

Log:
  Respect MK_KERBEROS with etc/rc.d/ipropd_master and etc/rc.d/ipropd_slave
  
  MFC after: 2 weeks
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 09:43:08 2015(r277740)
+++ head/etc/rc.d/Makefile  Mon Jan 26 09:51:21 2015(r277741)
@@ -54,8 +54,6 @@ FILES=DAEMON \
ipfw \
ipmon \
ipnat \
-   ipropd_master \
-   ipropd_slave \
ipsec \
${_kadmind} \
${_kdc} \
@@ -213,6 +211,8 @@ _nscd=  nscd
 .endif
 
 .if ${MK_KERBEROS} != no
+FILES+=ipropd_master
+FILES+=ipropd_slave
 _kadmind=  kadmind
 _kdc=  kdc
 _kfd=  kfd

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 09:43:08 
2015(r277740)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 09:51:21 
2015(r277741)
@@ -2379,6 +2379,8 @@ OLD_FILES+=usr/share/man/man8/jls.8.gz
 .endif
 
 .if ${MK_KERBEROS} == no
+OLD_FILES+=etc/rc.d/ipropd_master
+OLD_FILES+=etc/rc.d/ipropd_slave
 OLD_FILES+=usr/bin/compile_et
 OLD_FILES+=usr/bin/hxtool
 OLD_FILES+=usr/bin/kadmin
___
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-unsubscr...@freebsd.org


svn commit: r277745 - in head/sys: arm/arm boot/fdt/dts/arm

2015-01-26 Thread Ruslan Bukin
Author: br
Date: Mon Jan 26 10:31:17 2015
New Revision: 277745
URL: https://svnweb.freebsd.org/changeset/base/277745

Log:
  Add a generic way for hooking PMC interrupt.

Added:
  head/sys/arm/arm/pmu.c   (contents, props changed)
Modified:
  head/sys/boot/fdt/dts/arm/am335x.dtsi

Added: head/sys/arm/arm/pmu.c
==
--- /dev/null   00:00:00 1970   (empty, because file is newly added)
+++ head/sys/arm/arm/pmu.c  Mon Jan 26 10:31:17 2015(r277745)
@@ -0,0 +1,157 @@
+/*-
+ * Copyright (c) 2015 Ruslan Bukin b...@bsdpad.com
+ * All rights reserved.
+ *
+ * This software was developed by SRI International and the University of
+ * Cambridge Computer Laboratory under DARPA/AFRL contract (FA8750-10-C-0237)
+ * (CTSRD), as part of the DARPA CRASH research programme.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *notice, this list of conditions and the following disclaimer in the
+ *documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+
+/*
+ * Performance Monitoring Unit
+ */
+
+#include sys/cdefs.h
+__FBSDID($FreeBSD$);
+
+#include opt_hwpmc_hooks.h
+
+#include sys/param.h
+#include sys/systm.h
+#include sys/bus.h
+#include sys/kernel.h
+#include sys/module.h
+#include sys/malloc.h
+#include sys/rman.h
+#include sys/timeet.h
+#include sys/timetc.h
+#include sys/pmc.h
+#include sys/pmckern.h
+
+#include dev/fdt/fdt_common.h
+#include dev/ofw/openfirm.h
+#include dev/ofw/ofw_bus.h
+#include dev/ofw/ofw_bus_subr.h
+
+#include machine/bus.h
+#include machine/fdt.h
+#include machine/cpu.h
+#include machine/intr.h
+
+struct pmu_softc {
+   struct resource *res[1];
+   device_tdev;
+   void*ih;
+};
+
+static struct ofw_compat_data compat_data[] = {
+   {arm,cortex-a17-pmu,  1},
+   {arm,cortex-a15-pmu,  1},
+   {arm,cortex-a12-pmu,  1},
+   {arm,cortex-a9-pmu,   1},
+   {arm,cortex-a8-pmu,   1},
+   {arm,cortex-a7-pmu,   1},
+   {arm,cortex-a5-pmu,   1},
+   {arm,arm11mpcore-pmu, 1},
+   {arm,arm1176-pmu, 1},
+   {arm,arm1136-pmu, 1},
+   {qcom,krait-pmu,  1},
+   {NULL,  0}
+};
+
+static struct resource_spec pmu_spec[] = {
+   { SYS_RES_IRQ,  0,  RF_ACTIVE },
+   { -1, 0 }
+};
+
+static int
+pmu_intr(void *arg)
+{
+   struct trapframe *tf;
+
+   tf = arg;
+
+#ifdef HWPMC_HOOKS
+   if (pmc_intr)
+   (*pmc_intr)(PCPU_GET(cpuid), tf);
+#endif
+
+   return (FILTER_HANDLED);
+}
+
+static int
+pmu_probe(device_t dev)
+{
+
+   if (!ofw_bus_status_okay(dev))
+   return (ENXIO);
+
+   if (ofw_bus_search_compatible(dev, compat_data)-ocd_data != 0) {
+   device_set_desc(dev, Performance Monitoring Unit);
+   return (BUS_PROBE_DEFAULT);
+   }
+
+   return (ENXIO);
+}
+
+static int
+pmu_attach(device_t dev)
+{
+   struct pmu_softc *sc;
+   int err;
+
+   sc = device_get_softc(dev);
+   sc-dev = dev;
+
+   if (bus_alloc_resources(dev, pmu_spec, sc-res)) {
+   device_printf(dev, could not allocate resources\n);
+   return (ENXIO);
+   }
+
+   /* Setup interrupt handler */
+   err = bus_setup_intr(dev, sc-res[0], INTR_MPSAFE | INTR_TYPE_MISC,
+   pmu_intr, NULL, NULL, sc-ih);
+   if (err) {
+   device_printf(dev, Unable to setup interrupt handler.\n);
+   return (ENXIO);
+   }
+
+   return (0);
+}
+
+static device_method_t pmu_methods[] = {
+   DEVMETHOD(device_probe, pmu_probe),
+   DEVMETHOD(device_attach,pmu_attach),
+   { 0, 0 }
+};
+
+static driver_t pmu_driver = {
+   pmu,
+   pmu_methods,
+   sizeof(struct pmu_softc),
+};
+
+static devclass_t pmu_devclass;
+

svn commit: r277730 - in head: etc/devd etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 08:04:38 2015
New Revision: 277730
URL: https://svnweb.freebsd.org/changeset/base/277730

Log:
  Honor MK_ACPI in etc/devd and etc/rc.d
  
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/devd/Makefile
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/devd/Makefile
==
--- head/etc/devd/Makefile  Mon Jan 26 07:24:18 2015(r277729)
+++ head/etc/devd/Makefile  Mon Jan 26 08:04:38 2015(r277730)
@@ -2,12 +2,16 @@
 
 .include src.opts.mk
 
+FILES=
+
 .if ${MACHINE} == powerpc
 FILES+=apple.conf
 .endif
 
 .if ${MACHINE} == amd64 || ${MACHINE} == i386
+.if ${MK_ACPI} != no
 FILES+=asus.conf
+.endif
 .if ${MK_HYPERV} != no
 FILES+=hyperv.conf
 .endif

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 07:24:18 2015(r277729)
+++ head/etc/rc.d/Makefile  Mon Jan 26 08:04:38 2015(r277730)
@@ -103,7 +103,6 @@ FILES=  DAEMON \
pflog \
pfsync \
powerd \
-   power_profile \
ppp \
pppoed \
pwcheck \
@@ -155,6 +154,10 @@ FILES= DAEMON \
zfs \
zvol
 
+.if ${MK_ACPI} != no
+FILES+=power_profile
+.endif
+
 .if ${MK_AUTOFS} != no
 FILES+=automount
 FILES+=automountd

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 07:24:18 
2015(r277729)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:04:38 
2015(r277730)
@@ -14,6 +14,8 @@ OLD_FILES+=usr/share/man/man8/sa.8.gz
 .endif
 
 .if ${MK_ACPI} == no
+OLD_FILES+=etc/devd/asus.conf
+OLD_FILES+=etc/rc.d/power_profile
 OLD_FILES+=usr/sbin/acpiconf
 OLD_FILES+=usr/sbin/acpidb
 OLD_FILES+=usr/sbin/acpidump
___
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-unsubscr...@freebsd.org


svn commit: r277735 - head/sys/amd64/amd64

2015-01-26 Thread Roger Pau Monné
Author: royger
Date: Mon Jan 26 08:42:47 2015
New Revision: 277735
URL: https://svnweb.freebsd.org/changeset/base/277735

Log:
  amd64: allow base memory segment to start at address different than 0
  
  Current code requires that the first physical memory segment starts at 0,
  but this is not really needed. We only need to make sure the bootstrap code
  and page tables for APs are allocated below 4GB.
  
  This patch removes this requirement and allows booting a Dell R710 from
  UEFI, where the first physical memory segment starts at 0x1.
  
  Sponsored by: Citrix Systems RD
  Reviewed by: jhb
  Differential Revision: https://reviews.freebsd.org/D1417

Modified:
  head/sys/amd64/amd64/machdep.c

Modified: head/sys/amd64/amd64/machdep.c
==
--- head/sys/amd64/amd64/machdep.c  Mon Jan 26 08:30:24 2015
(r277734)
+++ head/sys/amd64/amd64/machdep.c  Mon Jan 26 08:42:47 2015
(r277735)
@@ -1355,8 +1355,10 @@ add_physmap_entry(uint64_t base, uint64_
/*
 * Find insertion point while checking for overlap.  Start off by
 * assuming the new entry will be added to the end.
+*
+* NB: physmap_idx points to the next free slot.
 */
-   insert_idx = physmap_idx + 2;
+   insert_idx = physmap_idx;
for (i = 0; i = physmap_idx; i += 2) {
if (base  physmap[i + 1]) {
if (base + length = physmap[i]) {
@@ -1394,7 +1396,7 @@ add_physmap_entry(uint64_t base, uint64_
 * Move the last 'N' entries down to make room for the new
 * entry if needed.
 */
-   for (i = physmap_idx; i  insert_idx; i -= 2) {
+   for (i = (physmap_idx - 2); i  insert_idx; i -= 2) {
physmap[i] = physmap[i - 2];
physmap[i + 1] = physmap[i - 1];
}
@@ -1580,23 +1582,27 @@ getmemsize(caddr_t kmdp, u_int64_t first
int page_counter;
 
bzero(physmap, sizeof(physmap));
-   basemem = 0;
physmap_idx = 0;
 
init_ops.parse_memmap(kmdp, physmap, physmap_idx);
+   physmap_idx -= 2;
 
/*
 * Find the 'base memory' segment for SMP
 */
basemem = 0;
for (i = 0; i = physmap_idx; i += 2) {
-   if (physmap[i] == 0x) {
+   if (physmap[i] = 0xA) {
basemem = physmap[i + 1] / 1024;
break;
}
}
-   if (basemem == 0)
-   panic(BIOS smap did not include a basemem segment!);
+   if (basemem == 0 || basemem  640) {
+   if (bootverbose)
+   printf(
+   Memory map doesn't contain a basemem segment, faking it);
+   basemem = 640;
+   }
 
/*
 * Make hole for AP - long mode bootstrap code.  The
@@ -1604,8 +1610,12 @@ getmemsize(caddr_t kmdp, u_int64_t first
 * is configured to support APs and APs for the system start
 * in 32bit mode (e.g. SMP bare metal).
 */
-   if (init_ops.mp_bootaddress)
+   if (init_ops.mp_bootaddress) {
+   if (physmap[1] = 0x1)
+   panic(
+   Basemem segment is not suitable for AP bootstrap code!);
physmap[1] = init_ops.mp_bootaddress(physmap[1] / 1024);
+   }
 
/*
 * Maxmem isn't the maximum memory, it's one larger than the
@@ -1657,12 +1667,14 @@ getmemsize(caddr_t kmdp, u_int64_t first
 */
physmem_start = (vm_guest  VM_GUEST_NO ? 1 : 16)  PAGE_SHIFT;
TUNABLE_ULONG_FETCH(hw.physmem.start, physmem_start);
-   if (physmem_start  PAGE_SIZE)
-   physmap[0] = PAGE_SIZE;
-   else if (physmem_start = physmap[1])
-   physmap[0] = round_page(physmap[1] - PAGE_SIZE);
-   else
-   physmap[0] = round_page(physmem_start);
+   if (physmap[0]  physmem_start) {
+   if (physmem_start  PAGE_SIZE)
+   physmap[0] = PAGE_SIZE;
+   else if (physmem_start = physmap[1])
+   physmap[0] = round_page(physmap[1] - PAGE_SIZE);
+   else
+   physmap[0] = round_page(physmem_start);
+   }
pa_indx = 0;
da_indx = 1;
phys_avail[pa_indx++] = physmap[0];
___
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-unsubscr...@freebsd.org


svn commit: r277737 - in head: etc/pam.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 08:50:12 2015
New Revision: 277737
URL: https://svnweb.freebsd.org/changeset/base/277737

Log:
  Honor MK_ACCT with etc/pam.d/atrun
  
  MFC after: 2 weeks
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/pam.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/pam.d/Makefile
==
--- head/etc/pam.d/Makefile Mon Jan 26 08:46:26 2015(r277736)
+++ head/etc/pam.d/Makefile Mon Jan 26 08:50:12 2015(r277737)
@@ -1,9 +1,10 @@
 # $FreeBSD$
 
+.include src.opts.mk
+
 NO_OBJ=
 
 FILES= README \
-   atrun \
cron \
ftpd \
imap \
@@ -15,6 +16,10 @@ FILES=   README \
telnetd \
xdm
 
+.if ${MK_AT} != no
+FILES+=atrun
+.endif
+
 FILESDIR=  /etc/pam.d
 FILESMODE= 644
 FILESMODE_README=  444

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:46:26 
2015(r277736)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:50:12 
2015(r277737)
@@ -62,6 +62,7 @@ OLD_FILES+=usr/share/man/man8/amd64/apmc
 .endif
 
 .if ${MK_AT} == no
+OLD_FILES+=etc/pam.d/atrun
 OLD_FILES+=usr/bin/at
 OLD_FILES+=usr/bin/atq
 OLD_FILES+=usr/bin/atrm
___
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-unsubscr...@freebsd.org


Re: svn commit: r277737 - in head: etc/pam.d tools/build/mk

2015-01-26 Thread Garrett Cooper
On Jan 26, 2015, at 0:50, Garrett Cooper n...@freebsd.org wrote:

 Author: ngie
 Date: Mon Jan 26 08:50:12 2015
 New Revision: 277737
 URL: https://svnweb.freebsd.org/changeset/base/277737
 
 Log:
  Honor MK_ACCT with etc/pam.d/atrun
 
  MFC after: 2 weeks
  Sponsored by: EMC / Isilon Storage Division

I meant MK_AT


signature.asc
Description: Message signed with OpenPGP using GPGMail


svn commit: r277739 - in head: etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 09:37:14 2015
New Revision: 277739
URL: https://svnweb.freebsd.org/changeset/base/277739

Log:
  Honor MK_JAIL with etc/rc.d/jail
  
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 09:31:48 2015(r277738)
+++ head/etc/rc.d/Makefile  Mon Jan 26 09:37:14 2015(r277739)
@@ -57,7 +57,6 @@ FILES=DAEMON \
ipropd_master \
ipropd_slave \
ipsec \
-   jail \
${_kadmind} \
${_kdc} \
${_kfd} \
@@ -201,6 +200,10 @@ FILES+=iscsictl
 FILES+=iscsid
 .endif
 
+.if ${MK_JAIL} != no
+FILES+=jail
+.endif
+
 .if ${MK_LPR} != no
 FILES+=lpd
 .endif

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 09:31:48 
2015(r277738)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 09:37:14 
2015(r277739)
@@ -2369,6 +2369,7 @@ OLD_FILES+=usr/share/man/man8/iscsid.8.g
 .endif
 
 .if ${MK_JAIL} == no
+OLD_FILES+=etc/rc.d/jail
 OLD_FILES+=usr/sbin/jail
 OLD_FILES+=usr/sbin/jexec
 OLD_FILES+=usr/sbin/jls
___
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-unsubscr...@freebsd.org


svn commit: r277736 - in head: etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 08:46:26 2015
New Revision: 277736
URL: https://svnweb.freebsd.org/changeset/base/277736

Log:
  Honor MK_ACCT with etc/rc.d/accounting
  
  MFC after: 2 weeks
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 08:42:47 2015(r277735)
+++ head/etc/rc.d/Makefile  Mon Jan 26 08:46:26 2015(r277736)
@@ -150,6 +150,10 @@ FILES= DAEMON \
zfs \
zvol
 
+.if ${MK_ACCT} != no
+FILES+=accounting
+.endif
+
 .if ${MK_ACPI} != no
 FILES+=power_profile
 .endif

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:42:47 
2015(r277735)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:46:26 
2015(r277736)
@@ -6,6 +6,7 @@
 #
 
 .if ${MK_ACCT} == no
+OLD_FILES+=etc/rc.d/accounting
 OLD_FILES+=etc/periodic/daily/310.accounting
 OLD_FILES+=usr/sbin/accton
 OLD_FILES+=usr/sbin/sa
___
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-unsubscr...@freebsd.org


svn commit: r277738 - in head: etc/defaults tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 09:31:48 2015
New Revision: 277738
URL: https://svnweb.freebsd.org/changeset/base/277738

Log:
  Honor MK_BLUETOOTH with etc/defaults/bluetooth.device.conf
  
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/defaults/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/defaults/Makefile
==
--- head/etc/defaults/Makefile  Mon Jan 26 08:50:12 2015(r277737)
+++ head/etc/defaults/Makefile  Mon Jan 26 09:31:48 2015(r277738)
@@ -1,7 +1,13 @@
 # $FreeBSD$
 
-FILES= bluetooth.device.conf devfs.rules periodic.conf rc.conf
+.include src.opts.mk
+
+FILES= devfs.rules periodic.conf rc.conf
 NO_OBJ=
 FILESDIR= /etc/defaults
 
+.if ${MK_BLUETOOTH} != no
+FILES+=bluetooth.device.conf
+.endif
+
 .include bsd.prog.mk

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:50:12 
2015(r277737)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 09:31:48 
2015(r277738)
@@ -185,6 +185,7 @@ OLD_DIRS+=usr/share/examples/bhyve
 OLD_FILES+=etc/bluetooth/hcsecd.conf
 OLD_FILES+=etc/bluetooth/hosts
 OLD_FILES+=etc/bluetooth/protocols
+OLD_FILES+=etc/defaults/bluetooth.device.conf
 OLD_DIRS+=etc/bluetooth
 OLD_FILES+=usr/bin/bthost
 OLD_FILES+=usr/bin/btsockstat
___
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-unsubscr...@freebsd.org


svn commit: r277740 - in head: etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 09:43:08 2015
New Revision: 277740
URL: https://svnweb.freebsd.org/changeset/base/277740

Log:
  Honor MK_WIRELESS with etc/rc.d/hostapd and etc/rc.d/wpa_supplicant
  
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 09:37:14 2015(r277739)
+++ head/etc/rc.d/Makefile  Mon Jan 26 09:43:08 2015(r277740)
@@ -247,6 +247,11 @@ _unbound=  local_unbound
 _utx=  utx
 .endif
 
+.if ${MK_WIRELESS} != no
+FILES+=hostapd
+FILES+=wpa_supplicant
+.endif
+
 FILESDIR=  /etc/rc.d
 FILESMODE= ${BINMODE}
 

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 09:37:14 
2015(r277739)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 09:43:08 
2015(r277740)
@@ -4981,6 +4981,8 @@ OLD_FILES+=usr/share/man/man8/utx.8.gz
 
 .if ${MK_WIRELESS} == no
 OLD_FILES+=etc/regdomain.xml
+OLD_FILES+=etc/rc.d/hostapd
+OLD_FILES+=etc/rc.d/wpa_supplicant
 OLD_FILES+=usr/sbin/ancontrol
 OLD_FILES+=usr/sbin/hostapd
 OLD_FILES+=usr/sbin/hostapd_cli
___
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-unsubscr...@freebsd.org


svn commit: r277731 - in head: etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 08:20:51 2015
New Revision: 277731
URL: https://svnweb.freebsd.org/changeset/base/277731

Log:
  Honor MK_LPR with etc/rc.d/lpd
  
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 08:04:38 2015(r277730)
+++ head/etc/rc.d/Makefile  Mon Jan 26 08:20:51 2015(r277731)
@@ -192,6 +192,10 @@ FILES+=iscsictl
 FILES+=iscsid
 .endif
 
+.if ${MK_LPR} != no
+FILES+=lpd
+.endif
+
 .if ${MK_NS_CACHING} != no
 _nscd= nscd
 .endif

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:04:38 
2015(r277730)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:20:51 
2015(r277731)
@@ -3512,6 +3512,7 @@ OLD_FILES+=usr/share/man/man8/updatedb.8
 .if ${MK_LPR} == no
 OLD_FILES+=etc/hosts.lpd
 OLD_FILES+=etc/printcap
+OLD_FILES+=etc/rc.d/lpd
 OLD_FILES+=usr/bin/lp
 OLD_FILES+=usr/bin/lpq
 OLD_FILES+=usr/bin/lpr
___
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-unsubscr...@freebsd.org


svn commit: r277732 - in head: etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 08:23:36 2015
New Revision: 277732
URL: https://svnweb.freebsd.org/changeset/base/277732

Log:
  Honor MK_API in etc/rc.d
  
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 08:20:51 2015(r277731)
+++ head/etc/rc.d/Makefile  Mon Jan 26 08:23:36 2015(r277732)
@@ -12,8 +12,6 @@ FILES=DAEMON \
addswap \
adjkerntz \
amd \
-   apm \
-   apmd \
archdep \
atm1 \
atm2 \
@@ -158,6 +156,11 @@ FILES= DAEMON \
 FILES+=power_profile
 .endif
 
+.if ${MK_APM} != no
+FILES+=apm
+FILES+=apmd
+.endif
+
 .if ${MK_AUTOFS} != no
 FILES+=automount
 FILES+=automountd

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:20:51 
2015(r277731)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:23:36 
2015(r277732)
@@ -50,6 +50,8 @@ OLD_FILES+=usr/share/man/man8/wire-test.
 .endif
 
 .if ${MK_APM} == no
+OLD_FILES+=etc/rc.d/apm
+OLD_FILES+=etc/rc.d/apmd
 OLD_FILES+=etc/apmd.conf
 OLD_FILES+=usr/sbin/apm
 OLD_FILES+=usr/share/examples/etc/apmd.conf
___
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-unsubscr...@freebsd.org


svn commit: r277734 - head/etc/rc.d

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 08:30:24 2015
New Revision: 277734
URL: https://svnweb.freebsd.org/changeset/base/277734

Log:
  Remove explicit inclusion of lpd from FILES
  
  X-MFC with: 277731
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 08:28:51 2015(r277733)
+++ head/etc/rc.d/Makefile  Mon Jan 26 08:30:24 2015(r277734)
@@ -68,7 +68,6 @@ FILES=DAEMON \
local \
localpkg \
lockd \
-   lpd \
mixer \
motd \
mountcritlocal \
___
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-unsubscr...@freebsd.org


svn commit: r277733 - in head: etc/rc.d tools/build/mk

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 08:28:51 2015
New Revision: 277733
URL: https://svnweb.freebsd.org/changeset/base/277733

Log:
  Honor MK_AMD with etc/rc.d/amd
  
  MFC after: 1 week
  Sponsored by: EMC / Isilon Storage Division

Modified:
  head/etc/rc.d/Makefile
  head/tools/build/mk/OptionalObsoleteFiles.inc

Modified: head/etc/rc.d/Makefile
==
--- head/etc/rc.d/Makefile  Mon Jan 26 08:23:36 2015(r277732)
+++ head/etc/rc.d/Makefile  Mon Jan 26 08:28:51 2015(r277733)
@@ -11,7 +11,6 @@ FILES=DAEMON \
accounting \
addswap \
adjkerntz \
-   amd \
archdep \
atm1 \
atm2 \
@@ -156,6 +155,10 @@ FILES= DAEMON \
 FILES+=power_profile
 .endif
 
+.if ${MK_AMD} != no
+FILES+=amd
+.endif
+
 .if ${MK_APM} != no
 FILES+=apm
 FILES+=apmd

Modified: head/tools/build/mk/OptionalObsoleteFiles.inc
==
--- head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:23:36 
2015(r277732)
+++ head/tools/build/mk/OptionalObsoleteFiles.inc   Mon Jan 26 08:28:51 
2015(r277733)
@@ -28,6 +28,7 @@ OLD_FILES+=usr/share/man/man8/iasl.8.gz
 
 .if ${MK_AMD} == no
 OLD_FILES+=etc/amd.map
+OLD_FILES+=etc/rc.d/amd
 OLD_FILES+=usr/bin/pawd
 OLD_FILES+=usr/sbin/amd
 OLD_FILES+=usr/sbin/amq
___
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-unsubscr...@freebsd.org


svn commit: r277746 - stable/10/sys/fs/devfs

2015-01-26 Thread Konstantin Belousov
Author: kib
Date: Mon Jan 26 11:37:16 2015
New Revision: 277746
URL: https://svnweb.freebsd.org/changeset/base/277746

Log:
  MFC r277390:
  Ignore devfs directory entries for devices being destroyed.

Modified:
  stable/10/sys/fs/devfs/devfs_devs.c
  stable/10/sys/fs/devfs/devfs_vnops.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/fs/devfs/devfs_devs.c
==
--- stable/10/sys/fs/devfs/devfs_devs.c Mon Jan 26 10:31:17 2015
(r277745)
+++ stable/10/sys/fs/devfs/devfs_devs.c Mon Jan 26 11:37:16 2015
(r277746)
@@ -186,6 +186,16 @@ devfs_find(struct devfs_dirent *dd, cons
continue;
if (type != 0  type != de-de_dirent-d_type)
continue;
+
+   /*
+* The race with finding non-active name is not
+* completely closed by the check, but it is similar
+* to the devfs_allocv() in making it unlikely enough.
+*/
+   if (de-de_dirent-d_type == DT_CHR 
+   (de-de_cdp-cdp_flags  CDP_ACTIVE) == 0)
+   continue;
+
if (bcmp(name, de-de_dirent-d_name, namelen) != 0)
continue;
break;

Modified: stable/10/sys/fs/devfs/devfs_vnops.c
==
--- stable/10/sys/fs/devfs/devfs_vnops.cMon Jan 26 10:31:17 2015
(r277745)
+++ stable/10/sys/fs/devfs/devfs_vnops.cMon Jan 26 11:37:16 2015
(r277746)
@@ -1023,6 +1023,9 @@ devfs_mknod(struct vop_mknod_args *ap)
TAILQ_FOREACH(de, dd-de_dlist, de_list) {
if (cnp-cn_namelen != de-de_dirent-d_namlen)
continue;
+   if (de-de_dirent-d_type == DT_CHR 
+   (de-de_cdp-cdp_flags  CDP_ACTIVE) == 0)
+   continue;
if (bcmp(cnp-cn_nameptr, de-de_dirent-d_name,
de-de_dirent-d_namlen) != 0)
continue;
___
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-unsubscr...@freebsd.org


svn commit: r277747 - in stable/10: bin/expr bin/expr/tests etc/mtree

2015-01-26 Thread Garrett Cooper
Author: ngie
Date: Mon Jan 26 11:49:06 2015
New Revision: 277747
URL: https://svnweb.freebsd.org/changeset/base/277747

Log:
  MFC r277358:
  
  r277358 (by ngie):
  
Integrate contrib/netbsd-tests/bin/expr into the build/kyua as 
bin/expr/tests
  
Sponsored by: EMC / Isilon Storage Division

Added:
  stable/10/bin/expr/tests/
 - copied from r277358, head/bin/expr/tests/
Modified:
  stable/10/bin/expr/Makefile
  stable/10/etc/mtree/BSD.tests.dist
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/bin/expr/Makefile
==
--- stable/10/bin/expr/Makefile Mon Jan 26 11:37:16 2015(r277746)
+++ stable/10/bin/expr/Makefile Mon Jan 26 11:49:06 2015(r277747)
@@ -1,9 +1,15 @@
 # $FreeBSD$
 
+.include bsd.own.mk
+
 PROG=  expr
 SRCS=  expr.y
 YFLAGS=
 
 NO_WMISSING_VARIABLE_DECLARATIONS=
 
+.if ${MK_TESTS} != no
+SUBDIR+=   tests
+.endif
+
 .include bsd.prog.mk

Modified: stable/10/etc/mtree/BSD.tests.dist
==
--- stable/10/etc/mtree/BSD.tests.dist  Mon Jan 26 11:37:16 2015
(r277746)
+++ stable/10/etc/mtree/BSD.tests.dist  Mon Jan 26 11:49:06 2015
(r277747)
@@ -10,6 +10,8 @@
 ..
 date
 ..
+expr
+..
 mv
 ..
 pax
___
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-unsubscr...@freebsd.org


Re: svn commit: r277714 - head/sbin/ipfw

2015-01-26 Thread Olivier Cochard-Labbé
On Mon, Jan 26, 2015 at 10:50 PM, Nick Hibma n...@van-laarhoven.org wrote:


  On 26 Jan 2015, at 22:24, John Baldwin j...@freebsd.org wrote:
 
 
  It might.  What happened for me is that I was using nat over wlan0 for
 VM's
  on my laptop to reach the outside world, but wlan0 doesn't get an IP
 until
  later in the boot after it associates.  As a result, wlan0 wasn't
 passing any
  IP traffic until this fix (or if I reloaded ipfw after wlan0 was
 configured).


With a FreeBSD 11-current 277728 I still have the ipfw_nat problem on
unconfigured/unexistant interface.


 I don't think it does. The interface is not available until openvpn is
 started.You need to clone the interface during boot by adding

 cloned_interfaces='tun0'

 in your /etc/rc.conf. Initialisation is then done later by openvpn.

 Let me know if that works for you.


I've tried with cloned_interfaces too: but same problem.

Here is the status of ipfw just after a boot (ipfw loaded before tun0 IP
setup):

[]~ ipfw show
00100   0 0 allow ip from any to any via lo0
00200   0 0 allow ip from any to any via lo1
00300   0 0 allow ip from any to any via vtnet6
00400   0 0 allow ip from any to any via wlan0
00500 172 21355 nat 1 ip from any to any in via vtnet4
00600  62  4264 nat 2 ip from any to any in via tun0
00700   0 0 check-state
00800   0 0 allow udp from 0.0.0.0 68 to 255.255.255.255 dst-port 67
out via vtnet4
00900   0 0 allow udp from any 67 to me dst-port 68 in via vtnet4
01000   0 0 allow udp from any 67 to 255.255.255.255 dst-port 68 in via
vtnet4
01100   0 0 allow icmp from me to any out via vtnet4 keep-state
01200  11   756 allow udp from me to any dst-port 53 out via vtnet4
keep-state
01300   4   304 allow udp from me to any dst-port 123 out via vtnet4
keep-state
01400 172 21725 allow udp from me to any dst-port 1195 out via vtnet4
keep-state
01500   0 0 nat 1 ip from 10.6.1.0/24,10.6.2.0/24 to any out via vtnet4
01600   0 0 nat 2 udp from me to 2.2.2.2 dst-port 1812,1813 out via tun0
01700  68  4452 allow ip from any to any via tun0
65535   0 0 deny ip from any to any

= All rules are present (even the nat 2 table) and seems ok.
= No packet seems to be denied
But my OSPF adjacency didn't came up on the tun0 interface:

[]~ tcpdump -pni tun0
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on tun0, link-type NULL (BSD loopback), capture size 262144 bytes
capability mode sandbox enabled
21:25:09.555746 IP 10.0.3.2  224.0.0.5: OSPFv2, Hello, length 44
21:25:10.595286 IP 10.0.3.1  224.0.0.5: OSPFv2, Hello, length 48
^C
2 packets captured
2 packets received by filter
0 packets dropped by kernel

= tcpdump shows only some multicast traffic: It's the problem because ifpw
is filtering all unicast traffic on tun0 in its current state.
For solving this problem I just had to reload ipfw:

[]~ service ipfw restart
net.inet.ip.fw.enable: 1 - 0
net.inet6.ip6.fw.enable: 1 - 0
Firewall rules loaded.

= This fix the problem, unicast traffic are now allowed:

[]~ tcpdump -pni tun0
tcpdump: verbose output suppressed, use -v or -vv for full protocol decode
listening on tun0, link-type NULL (BSD loopback), capture size 262144 bytes
capability mode sandbox enabled
21:25:34.772225 IP 10.0.3.2  10.0.3.1: OSPFv2, Database Description,
length 32
21:25:35.784449 IP 10.0.3.1  10.0.3.2: OSPFv2, Database Description,
length 32
21:25:35.784550 IP 10.0.3.2  10.0.3.1: OSPFv2, Database Description,
length 52
21:25:35.785904 IP 10.0.3.1  10.0.3.2: OSPFv2, Database Description,
length 192
21:25:35.786007 IP 10.0.3.2  10.0.3.1: OSPFv2, Database Description,
length 32
___
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-unsubscr...@freebsd.org


Re: svn commit: r277678 - in head: etc/rc.d sbin share/man/man4 share/mk sys/conf sys/modules/geom tools/build/mk tools/build/options

2015-01-26 Thread Gleb Smirnoff
On Sun, Jan 25, 2015 at 04:52:49AM +, Garrett Cooper wrote:
G Author: ngie
G Date: Sun Jan 25 04:52:48 2015
G New Revision: 277678
G URL: https://svnweb.freebsd.org/changeset/base/277678
G 
G Log:
G   Add MK_CCD knob for building and installing ccd(4), ccdconfig, etc
G   
G   MFC after: 2 weeks
G   Sponsored by: EMC / Isilon Storage Division

Does ccd(4) has any benefits over gstripe? Looks like abandonware to
be retired.

-- 
Totus tuus, Glebius.
___
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-unsubscr...@freebsd.org


Re: svn commit: r277714 - head/sbin/ipfw

2015-01-26 Thread Olivier Cochard-Labbé
On Sun, Jan 25, 2015 at 9:37 PM, John Baldwin j...@freebsd.org wrote:

 Author: jhb
 Date: Sun Jan 25 20:37:32 2015
 New Revision: 277714
 URL: https://svnweb.freebsd.org/changeset/base/277714

 Log:
   natd(8) will work with an unconfigured interface and effectively not do
   anything until the interface is assigned an address.  This fixes
   ipfw_nat to do the same by using an IP of INADDR_ANY instead of
   aborting the nat setup if the requested interface is not yet configured.



Hi,

I've still a problem with ipfw_nat and unconfigured interface:
On my setup I'm using ipfw with NAT rules using an OpenVPN tunnel interface
as source address for NATting.

During the machine startup, ipfw is started before openvpn (hopefully) and
its configuration mention do to NAT using tun0 IP address.
Then OpenVPN start and create a tun0 and set an IP address on it.
= But no unicast traffic is allowed on this tun0 interface until I restart
ipfw.

If I correctly understand the log of this commit: This behavior should be
fixed by this commit, right ?
___
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-unsubscr...@freebsd.org