Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 21 Aug 2016 00:59:54 -0400 (EDT)
From:      "Sean C. Farley" <scf@FreeBSD.org>
To:        Alexander Motin <mav@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   Re: svn commit: r304425 - stable/10/usr.sbin/bhyve
Message-ID:  <alpine.BSF.2.20.1608210017080.78249@thor.farley.org>
In-Reply-To: <201608181156.u7IBu7OL054695@repo.freebsd.org>
References:  <201608181156.u7IBu7OL054695@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, 18 Aug 2016, Alexander Motin wrote:

> Author: mav
> Date: Thu Aug 18 11:56:07 2016
> New Revision: 304425
> URL: https://svnweb.freebsd.org/changeset/base/304425
>
> Log:
>  MFC r302504, r302666, r302668, r302932, r302933:
>  Add emulation for Intel e1000 (e82545) network adapter.
>
>  The code was successfully tested with FreeBSD, Linux, Solaris and Windows
>  guests.  This interface is predictably slower (about 2x) then virtio-net,
>  but it is very helpful for guests not supporting virtio-net by default.
>
>  Thanks to Jeremiah Lott and Peter Grehan for doing original heavy lifting.
>
> Added:
>  stable/10/usr.sbin/bhyve/pci_e82545.c
>     - copied, changed from r302504, head/usr.sbin/bhyve/pci_e82545.c
> Modified:
>  stable/10/usr.sbin/bhyve/Makefile
>  stable/10/usr.sbin/bhyve/bhyve.8
> Directory Properties:
>  stable/10/   (props changed)
>
> Modified: stable/10/usr.sbin/bhyve/Makefile
> ==============================================================================
> --- stable/10/usr.sbin/bhyve/Makefile	Thu Aug 18 11:51:14 2016	(r304424)
> +++ stable/10/usr.sbin/bhyve/Makefile	Thu Aug 18 11:56:07 2016	(r304425)
> @@ -23,6 +23,7 @@ SRCS=	\
> 	mevent.c		\
> 	mptbl.c			\
> 	pci_ahci.c		\
> +	pci_e82545.c		\
> 	pci_emul.c		\
> 	pci_hostbridge.c	\
> 	pci_irq.c		\
> @@ -48,6 +49,10 @@ SRCS+=	vmm_instruction_emul.c
> DPADD=	${LIBVMMAPI} ${LIBMD} ${LIBUTIL} ${LIBPTHREAD}
> LDADD=	-lvmmapi -lmd -lutil -lpthread
>
> +CFLAGS+= -I${BHYVE_SYSDIR}/sys/dev/e1000
> +CFLAGS+= -I${BHYVE_SYSDIR}/sys/dev/mii
> +CFLAGS+= -I${BHYVE_SYSDIR}/sys/dev/usb/controller
> +
> WARNS?=	2
>
> .include <bsd.prog.mk>

Alexander,

The MFC is a tiny bit incomplete.  I cannot build stable/10 world since 
BHYVE_SYSDIR is not defined in the usr.sbin/bhyve/Makefile as it is in 
HEAD.  The issue was only noticed because I do not have the source tree 
in /usr/src.  I presume the symlink that exists for /sys to usr/src/sys 
masks this for most people.

------------------------------------------------
===> usr.sbin/bhyve (depend)
rm -f .depend
CC='cc  ' mkdep -f .depend -a    -I/sys/dev/e1000 -I/sys/dev/mii -I/sys/dev/usb/controller -std=gnu99    /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/atkbdc.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/acpi.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/bhyverun.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/block_if.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/bootrom.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/consport.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/dbgport.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/fwctl.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/inout.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/ioapic.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/mem.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/mevent.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/mptbl.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_ahci.c /usr/FreeBSD/branches/stable/10/src/usr.sb!
 in/bhyve/pci_e82545.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_emul.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_hostbridge.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_irq.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_lpc.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_passthru.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_virtio_block.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_virtio_net.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_virtio_rnd.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_uart.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pm.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/post.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/rtc.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/smbiostbl.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/task_switch.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/uart_emul.c /usr/FreeBSD/branc!
 hes/stable/10/src/usr.sbin/bhyve/virtio.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/xmsr.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/spinup_ap.c /usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/../../sys/amd64/vmm/vmm_instruction_emul.c
/usr/FreeBSD/branches/stable/10/src/usr.sbin/bhyve/pci_e82545.c:51:10: fatal error: 'e1000_regs.h' file not found
#include "e1000_regs.h"
          ^
1 error generated.
mkdep: compile failed
*** [.depend] Error code 1
------------------------------------------------

This missing line completes the build, but I do not know if anything 
else needs merging.  The missing merge to the .PATH variable did not 
appear to affect the buildworld.

+ BHYVE_SYSDIR?=${SRCTOP}

Thank you.

Sean
-- 
scf@FreeBSD.org
From owner-svn-src-stable-10@freebsd.org  Sun Aug 21 05:08:39 2016
Return-Path: <owner-svn-src-stable-10@freebsd.org>
Delivered-To: svn-src-stable-10@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id 0C1BFBB9C76;
 Sun, 21 Aug 2016 05:08:39 +0000 (UTC)
 (envelope-from ngie@FreeBSD.org)
Received: from repo.freebsd.org (repo.freebsd.org
 [IPv6:2610:1c1:1:6068::e6a:0])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (Client did not present a certificate)
 by mx1.freebsd.org (Postfix) with ESMTPS id CC8231283;
 Sun, 21 Aug 2016 05:08:38 +0000 (UTC)
 (envelope-from ngie@FreeBSD.org)
Received: from repo.freebsd.org ([127.0.1.37])
 by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id u7L58bv8017166;
 Sun, 21 Aug 2016 05:08:37 GMT (envelope-from ngie@FreeBSD.org)
Received: (from ngie@localhost)
 by repo.freebsd.org (8.15.2/8.15.2/Submit) id u7L58bAF017165;
 Sun, 21 Aug 2016 05:08:37 GMT (envelope-from ngie@FreeBSD.org)
Message-Id: <201608210508.u7L58bAF017165@repo.freebsd.org>
X-Authentication-Warning: repo.freebsd.org: ngie set sender to
 ngie@FreeBSD.org using -f
From: Garrett Cooper <ngie@FreeBSD.org>
Date: Sun, 21 Aug 2016 05:08:37 +0000 (UTC)
To: src-committers@freebsd.org, svn-src-all@freebsd.org,
 svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject: svn commit: r304553 - stable/10/lib/libc/tests/stdlib
X-SVN-Group: stable-10
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
X-BeenThere: svn-src-stable-10@freebsd.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: SVN commit messages for only the 10-stable src tree
 <svn-src-stable-10.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/svn-src-stable-10>, 
 <mailto:svn-src-stable-10-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-stable-10/>;
List-Post: <mailto:svn-src-stable-10@freebsd.org>
List-Help: <mailto:svn-src-stable-10-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10>, 
 <mailto:svn-src-stable-10-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 21 Aug 2016 05:08:39 -0000

Author: ngie
Date: Sun Aug 21 05:08:37 2016
New Revision: 304553
URL: https://svnweb.freebsd.org/changeset/base/304553

Log:
  Unbreak the build when MK_TESTS != no after r304527
  
  - src.opts.mk should be bsd.own.mk on ^/stable/10
  - LIBADD should be DPADD/LDADD on ^/stable/10
  
  Pointyhat to:	kib
  Reported by:	Olivier Pinter <oliver.pinter@hardenedbsd.org>
  Sponsored by:	EMC / Isilon Storage Division

Modified:
  stable/10/lib/libc/tests/stdlib/Makefile

Modified: stable/10/lib/libc/tests/stdlib/Makefile
==============================================================================
--- stable/10/lib/libc/tests/stdlib/Makefile	Sun Aug 21 00:48:41 2016	(r304552)
+++ stable/10/lib/libc/tests/stdlib/Makefile	Sun Aug 21 05:08:37 2016	(r304553)
@@ -1,6 +1,6 @@
 # $FreeBSD$
 
-.include <src.opts.mk>
+.include <bsd.own.mk>
 
 ATF_TESTS_C+=		heapsort_test
 ATF_TESTS_C+=		mergesort_test
@@ -42,7 +42,8 @@ CFLAGS+=	-I${.CURDIR}
 
 CXXFLAGS.cxa_thread_atexit_test+=	-std=c++11
 CXXFLAGS.cxa_thread_atexit_nothr_test+=	-std=c++11
-LIBADD.cxa_thread_atexit_test+=		pthread
+DPADD.cxa_thread_atexit_test+=		${LIBPTHREAD}
+LDADD.cxa_thread_atexit_test+=		-lpthread
 
 .for t in h_getopt h_getopt_long
 CFLAGS.$t+=	-I${LIBNETBSD_SRCDIR} -I${SRCTOP}/contrib/netbsd-tests



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.20.1608210017080.78249>