Date: Tue, 16 Feb 2016 08:03:47 +0000 (GMT) From: jenkins-admin@FreeBSD.org To: dteske@FreeBSD.org, emaste@FreeBSD.org, skra@FreeBSD.org, pfg@FreeBSD.org, mmel@FreeBSD.org, feld@FreeBSD.org, cem@FreeBSD.org, hselasky@FreeBSD.org, des@FreeBSD.org, bdrewery@FreeBSD.org, ian@FreeBSD.org, cperciva@FreeBSD.org, jimharris@FreeBSD.org, zbb@FreeBSD.org, jhibbits@FreeBSD.org, bjk@FreeBSD.org, andrew@FreeBSD.org, jhb@FreeBSD.org, mav@FreeBSD.org, royger@FreeBSD.org, wma@FreeBSD.org, tuexen@FreeBSD.org, sephe@FreeBSD.org, vangyzen@FreeBSD.org, adrian@FreeBSD.org, markj@FreeBSD.org, glebius@FreeBSD.org, kib@FreeBSD.org, kevlo@FreeBSD.org, br@FreeBSD.org, cy@FreeBSD.org, np@FreeBSD.org, jenkins-admin@FreeBSD.org, freebsd-current@FreeBSD.org, freebsd-i386@FreeBSD.org Subject: FreeBSD_HEAD_i386 - Build #2349 - Failure Message-ID: <1948561587.1.1455609836828.JavaMail.jenkins@jenkins-9.freebsd.org>
next in thread | raw e-mail | index | archive | help
FreeBSD_HEAD_i386 - Build #2349 - Failure: Build information: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2349/ Full change log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2349/changes Full build log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_i386/2349/console Change summaries: 295651 by vangyzen: ichsmb: add PCI device ID for Intel Sunrise Point-H SMBus controller MFC after: 3 days Sponsored by: Dell Inc. 295650 by bdrewery: DIRDEPS_BUILD: Update dependencies. Sponsored by: EMC / Isilon Storage Division 295649 by bdrewery: Filemon: Fix panic when fork1() is called from kproc_create(). MFC after: 2 weeks Sponsored by: EMC / Isilon Storage Division 295648 by bdrewery: Remove the need for overriding depend. Sponsored by: EMC / Isilon Storage Division 295647 by bdrewery: Add missing dependency on sys/conf/newvers.sh. Sponsored by: EMC / Isilon Storage Division 295646 by bdrewery: Add an .ORDER for all/install. Sponsored by: EMC / Isilon Storage Division 295645 by bdrewery: Use built-in :tA here rather than realpath(1). Sponsored by: EMC / Isilon Storage Division 295644 by bdrewery: Don't descend into test directories for distribution->installconfig. Sponsored by: EMC / Isilon Storage Division 295643 by bdrewery: Test directories can build in parallel fine. Sponsored by: EMC / Isilon Storage Division 295642 by jhibbits: Fix a panic bug that cropped up in the PTE rewrite. PTE was getting overwritten by just the flags. Pointy-hat to: jhibbits 295641 by bdrewery: Consolidate common beforebuild logic. Sponsored by: EMC / Isilon Storage Division 295638 by pfg: fputs: Return the number of bytes written. Fix r295631: wrong value. Pointy hat: pfg (me) Pointed out by: bde 295637 by jhb: Correct the ABI name for Linux/i386 binaries under FreeBSD/i386. This allows truss to work for these binaries again after r288424. MFC after: 3 days 295636 by jhb: Sign extend the error value for failing Linux/i386 system calls. This restores the mapping of Linux errors to native FreeBSD errno values after the refactoring in r288424. 295635 by andrew: Add support for the Allwinner DMA controller. This will be used by the at least the audio codec driver currently in review. Submitted by: Jared McNeill <jmcneill@invisible.ca> Differential Revision: https://reviews.freebsd.org/D5050 295634 by andrew: Add support for the AXP209 Power System Management IC. This allows boards with this and an Allwinner SoC to power off. Submitted by: Emmanuel Vadot <manu@bidouilliste.com> Differential Revision: https://reviews.freebsd.org/D4954 295633 by andrew: Set sc->clkfreq removed in r295470 Reported by: Jared McNeill <jmcneill@invisible.ca> Pointy-hat to: andrew 295632 by pfg: getln: We cannot expand the buffer beyond INT_MAX. In such cases return ENOMEM. This is a limitation of our implementation, alternatively you may consider getline(3). Differential Revision: https://reviews.freebsd.org/D442 (Partial) Obtained from: Apple Inc. (Libc 997.90.3) Relnotes: yes 295631 by pfg: fputs: Return the number of bytes written. POSIX.1-2008 requires that successful completion simply return a non-negative integer. We have regularly returned a constant value. Another, equally valid, implementation convention implies returning the number of bytes written. Adopt this last convention to be in line with what Apple's libc does. POSIX also explicitly notes: Note that this implementation convention cannot be adhered to for strings longer than {INT_MAX} bytes as the value would not be representable in the return type of the function. For backwards-compatibility, implementations can return the number of bytes for strings of up to {INT_MAX} bytes, and return {INT_MAX} for all longer strings. Developers shouldn't depend specifically on either convention but the change may help port software from Apple. Differential Revision: https://reviews.freebsd.org/D442 (Partial) Obtained from: Apple Inc. (Libc 997.90.3 with changes) Relnotes: yes 295630 by andrew: Fix the spelling of OF_getencprop. It will fix the data correctly for the endian of the CPU so there is no need to call fdt32_to_cpu. Sponsored by: ABT Systems Ltd 295629 by andrew: Remove an unused FDT header, fdt_common.h should only be needed in a few places, mostly in sys/dev/fdt and legacy code. Sponsored by: ABT Systems Ltd 295628 by andrew: Remove an unused static inline function. Sponsored by: ABT Systems Ltd 295627 by skra: Initial OMAP4 WUGEN pass-through driver. SPI interrupts are passed through WUGEN to GIC. Hardware initialization is left in state after reset as well as before. This is needed after an update of Linux dts files. 295626 by andrew: Add support for the Allwinner i2c device. This is similar to the existing Marvell twsi part, however uses different register locations, as such split the existing driver into Marvell and Allwinner attachments. While here clean a few style issues. Submitted by: Emmanuel Vadot <manu@bidouilliste.com> Differential Revision: https://reviews.freebsd.org/D4846 295625 by adrian: Allow MIPS INTRNG code to be built without FDT support. This patch allows the newly imported INTRNG code to be built without necessarily having FDT support in the kernel. This may be useful for some MIPS platforms that wish to move to INTRNG, but not to FDT at the same time. Basically all the code is already within ifdef's where FDT is concerned, it's just the headers that aren't. Submitted by: Stanislav Galabov <sgalabov@gmail.com> Differential Revision: https://reviews.freebsd.org/D5249 295622 by andrew: Move the twsi driver source to be under iicbus. It is in a separate directory as it is expected multiple attachments will be added for the SoC families that use this hardware. Sponsored by: ABT Systems Ltd 295621 by andrew: Create the twsi device option in preparation to move the driver to a common location. The device is also found in Allwinner SoCs. Sponsored by: ABT Systems Ltd 295618 by cem: NTB: workaround for high traffic hardware hang This patch comes from Dave Jiang's Linux tree, davejiang/ntb. It hasn't been accepted into Linus' tree, so I do not have an authoritative SHA1 to point at. Original commit log: ===================================================================== A hardware errata causes the NTB to hang when heavy bi-directional traffic in addition to the usage of BAR0/1 (where the registers reside, including the doorbell registers to trigger interrupts). This workaround is only available on Haswell and Broadwell platform. The workaround is to enable split BAR in the BIOS to allow the 64bit BAR4 to be split into two 32bit BAR4 and BAR5. The BAR4 shall be pointed to LAPIC region of the remote host. We will bypass the db mechanism and directly trigger the MSIX interrupts. The offsets and vectors are exchanged during transport scratch pad negotiation. The scratch pads are now overloaded in order to allow the exchange of the information. This gets around using the doorbell and prevents the lockup with additional pcode changes in BIOS. Signed-off-by: Dave Jiang <dave.jiang@intel.com> ===================================================================== Notable changes in the FreeBSD version of this patch: * The MSIX BAR is configurable, like hw.ntb.b2b_mw_idx (msix_mw_idx). The Linux version of the patch only uses BAR4. * MSIX negotiation aborts if the link goes down. Obtained from: Linux (Dual BSD/GPL driver) Sponsored by: EMC / Isilon Storage Division 295616 by pfg: ext2fs: Remove panics for rename() race conditions. Sync with r84642 from UFS: The panics are inappropriate because the IN_RENAME flag only fixes a few of the huge number of race conditions that can result in the source path becoming invalid even prior to the VOP_RENAME() call. Found accidentally while checking an issue from PVS Static Analysis. MFC after: 3 days 295612 by andrew: Remove an unused include. 295611 by hselasky: Restore maximum number of host keys to 64. Discussed with: adrian @ and avos @ 295610 by kevlo: Fix a bug that caused nothing to be skipped when skipping exactly the number of bytes present in a regular file was requested. Obtained from: OpenBSD 295609 by cperciva: Don't dereference a pointer immediately after determining that it is equal to NULL. [1] While I'm here, s/xb/xbd/ (the name changed a long time ago but this instance wasn't corrected). Reported by: PVS-Studio [1] 295608 by hselasky: Fix variable assignment inside if-clause in the smsc driver. Found by D5245 / PVS. MFC after: 1 week 295607 by hselasky: Reduce the number of supported WLAN keys in the rum driver, else we risk bit shifting overflows. Found by D5245 / PVS. MFC after: 1 week 295606 by sephe: hyperv/hn: Fix typo in comment Noticed by: avos Reviewed by: adrian, avos, Hongjiang Zhang <honzhan microsoft com> Approved by: adrian MFC after: 1 week Sponsored by: Microsoft OSTC Differential Revision: https://reviews.freebsd.org/D5199 295605 by cem: ioat(4): On error detected in ithread, defer HW reset to taskqueue The I/OAT HW reset process may sleep, so it is invalid to perform a channel reset from the software interrupt thread. Sponsored by: EMC / Isilon Storage Division 295604 by cem: ioat(4): Also check for errors if the channel is suspended Sponsored by: EMC / Isilon Storage Division 295603 by cem: ioat(4): Decode/define more capabilities, operations These are defined in the Intel Haswell EDS volume 2 (registers) (507849 v2.1). Sponsored by: EMC / Isilon Storage Division 295602 by kib: Hide ucontext_t and mcontext_t when neither POSIX nor XSI features are enabled in the compilation environment, i.e. for ANSI C use of #include <signal.h>. Requested and reviewed by: bde Sponsored by: The FreeBSD Foundation MFC after: 13 days 295588 by cem: ioat(4): Recheck status register on zero-descriptor wakeups Errors that halt the channel don't necessarily result in a completion update, apparently. Sponsored by: EMC / Isilon Storage Division 295587 by dteske: Fix missing description for command usage While "jng" or "jib" without arguments told you what each sub-command does, sub-command usage didn't tell you (e.g., "jng bridge" or "jib addm" gave only usage and not description). 295586 by pfg: cd9660: More "check for NULL" cleaunps. Cleanup some checks for NULL. Most of these were always unnecessary and starting with r294954 brelse() doesn't need any NULL checks at all. For now keep the checks somewhat consistent with NetBSD in case we want to merge the cleanups to older versions. 295584 by markj: Add a missing newline to a log message. MFC after: 1 week 295583 by markj: Rename the flags field of struct nd_defrouter to "raflags". This field contains the flags inherited from the corresponding router advertisement message and is not for storing private state. MFC after: 1 week 295582 by emaste: elfcopy: enable PE and EFI support Sponsored by: The FreeBSD Foundation 295581 by emaste: Add libpe for elfcopy(1) PE/COFF support Sponsored by: The FreeBSD Foundation 295580 by markj: Simplify defrtrlist_update() slightly in preparation for future changes. No functional change intended. MFC after: 1 week 295579 by markj: Remove a bogus comment from nd6_na_input(). The splnet() call that it refers to has been removed, and a lock for the default router list is in fact needed. MFC after: 1 week 295578 by markj: Remove superfluous return statements from the neighbour discovery code. MFC after: 1 week 295577 by emaste: Update ELF Tool Chain to upstream rev 3400 Some notable improvements include: readelf: - Add AArch64 relocation definitions. - Report value of unknown relocation types. elfcopy: - Consider symbols with STB_GNU_UNIQUE binding as global symbols. - Fixed support for VMA adjustment for loadable sections found in relocatable objects. - Handle nameless global symbols. - Improve wildcard matching for !-prefixed symbols. - Add PE/COFF support. elfdump: - Improve section type reporting. - Add MIPS-specific section types. This update also includes a significant number of bug fixes. PR: 207091 [exp-run] Sponsored by: The FreeBSD Foundation 295576 by markj: Fix style around allocations from M_IP6NDP. - Don't cast the return value of malloc(9). - Use M_ZERO instead of explicitly calling bzero(9). MFC after: 1 week 295575 by markj: Remove some unreferenced NDP debug variable definitions. MFC after: 1 week 295574 by markj: Clear the cookie pointer on error in tmpfs_readdir(). It is otherwise left dangling, and callers that request cookies always free the cookie buffer, even when VOP_READDIR(9) returns an error. This results in a double free if tmpfs_readdir() returns an error to the NFS server or the Linux getdents(2) emulation code. Reported by: pho MFC after: 1 week Security: double free of malloc(9)-backed memory Sponsored by: EMC / Isilon Storage Division 295573 by np: Remove duplicate definition (CPL_TRACE_PKT_T5). 295568 by feld: Document the new jail_reverse_stop parameter While here clean up the documentation for jail_list PR: 196152 Approved by: jamie, wblock MFC after: 1 week, with r295471 Differential Revision: https://reviews.freebsd.org/D5243 295567 by andrew: Stop adding an extra null terminator to strings, OF_getprop already returns a null terminated string. 295566 by bjk: Update .Dd for r295565 295565 by ian: Clarify the difference between 7- and 8-bit i2c addresses, used in FDT versus hints-based configuration, respectively. Reported by: Jukka Ukkonen <jau789@gmail.com> 295564 by br: Use __uint64_t type for floating point registers as compiler don't know about __uint128_t yet. Discussed with: theraven, kib 295563 by andrew: Only update curthread and curpcb after we have finished using the old values. If switching from a thread that used floating-point registers to a thread that is still running, but holding the blocked_lock lock we would switch the curthread to the new (running) thread, then call critical_enter. This will non-atomically increment td_critnest, and later call critical_exit to non-atomically decrement this value. This can happen at the same time as the new thread is still running on the old core, also calling these functions. In this case there will be a race between these non-atomic operations. This can be an issue as we could loose one of these operations leading to the value to not return to zero. If, later on, we then hit a data abort we check if the td_critnest is zero. If this check fails we will panic the kernel. This has been observed when running pcmstat on a Cavium ThunderX. The pcm thread will use the blocked_lock lock and there is a high chance userspace will use the floating-point registers. When, later on, pmcstat triggers a data abort we will hit this panic. The fix is to update these values after storing the floating-point state. This means we use the correct curthread while storing the state so it will not be an issue that the changes to td_critnest are non-atomic. Sponsored by: ABT Systems Ltd 295562 by mav: Don't check CmdSN for initial login request. This fixes connection errors for some initiators not starting CmdSN from zero. While there, fix wrong status details reported for couple errors. MFC after: 3 days 295561 by kib: POSIX states that #include <signal.h> shall make both mcontext_t and ucontext_t available. Our code even has XXX comment about this. Add a bit of compliance by moving struct __ucontext definition into sys/_ucontext.h and including it into signal.h and sys/ucontext.h. Several machine/ucontext.h headers were changed to use namespace-safe types (like uint64_t->__uint64_t) to not depend on sys/types.h. struct __stack_t from sys/signal.h is made always visible in private namespace to satisfy sys/_ucontext.h requirements. Apparently mips _types.h pollutes global namespace with f_register_t type definition. This commit does not try to fix the issue. PR: 207079 Reported and tested by: Ting-Wei Lan <lantw44@gmail.com> Sponsored by: The FreeBSD Foundation MFC after: 2 weeks 295560 by kib: If full width writes to the performance monitoring counters are supported, use full-width aliases MSRs for writes. This fixes the "[pmc,X] negative increment" assertion on the context switch when clipped counter value is sign-extended. Add definitions for the MSR IA32_PERF_CAPABILITIES needed to detect the feature. PR: 207068 Submitted by: joss.upton@yahoo.com MFC after: 2 weeks 295559 by kib: Adjust the size of PMC_DEFAULT_DEBUG_FLAGS to match the size of the structure. PR: 207068 Submitted by: joss.upton@yahoo.com MFC after: 2 weeks 295558 by kib: Remove tautological cast. PR: 207068 Submitted by: joss.upton@yahoo.com MFC after: 2 weeks 295557 by mmel: UART: Fix spurious interrupts generated by ns8250 and lpc drivers: - don't enable transmitter empty interrupt before filling TX FIFO. - add missing uart_barrier() call in interrupt service routine 295556 by dteske: Comments and fix small bug Reduce differences between jib/jng and fix a bug that would prevent additional interfaces from being created if the first of many already existed (counter wasn't incremented before calling only continue). 295555 by dteske: Interpret vnet_interface/vnet.interface as array 295554 by dteske: Add syntax to disable MAC allocation Adding `!' before an interface name will disable MAC allocation, falling back to driver mechanics. Alternatively adding `=' before an interface name causes the MAC address to be cloned (for ng_bridge(4) back-end only). While here, disable the auto-detection of wlan* since this knocks the host off; requiring the host that defines the jail to explicitly enable this feature by preceding the interface with `='. 295553 by dteske: Refactor conversion to hex 295549 by tuexen: Loopback addresses are 127.0.0.0/8, not 127.0.0.1/32. MFC after: 1 week 295548 by dteske: Add support for bridging iwn(4) based wlan(4) Documented in iwn(4), "Only one virtual interface may be configured at any time." However, netgraph with a cloned MAC address is able to communicate over an ng_eiface attached to an ng_bridge linked to the wlan(4) interface. While here, introduce syntax to specify the MAC address is to be cloned if the named interface begins with equals [=]. 295547 by glebius: o Gather all mbuf(9) allocation functions into kern_mbuf.c, and all mbuf(9) manipulation functions into uipc_mbuf.c. This looks like the initial intent, but had diffused in the last decade. o Gather all declarations in mbuf.h in one place and sort them. o Uninline m_clget() and m_cljget(). There are no functional changes in this patch. The patch comes from a larger version, where all mbuf(9) allocation was uninlined, which allowed to make mbuf(9) UMA zones private to kern_mbuf.c. The performance impact of the total uninlining is still unclear, so we are holding on now with larger version. Together with: melifaro, olivier 295546 by dteske: Centralize MAC derivation formula 295542 by dteske: Add example VIMAGE config required by vnet jails 295541 by tuexen: Use 4 spaces instead of a tab. 295540 by andrew: Enable ARM_INTRNG on all kernel configs where we use the Generic Interrupt Controller. Sponsored by: ABT Systems Ltd 295536 by des: Fix double-free error: r289419 moved all error handling in http_connect() to the end of the function, but did not remove a fetch_close() call which was made redundant by the one in the shared error-handling code. PR: 206774 Submitted by: Christian Heckendorf <heckendorfc@gmail.com> MFC after: 3 days 295535 by des: Use the new insecure-lan-zones option instead of listing each AS112 zone separately. MFC after: 3 days 295533 by des: Remove unbound-contrl-setup since we use a local control socket which does not require keys. MFC after: 3 days Relnotes: yes 295532 by jimharris: nvme: avoid duplicate SET_NUM_QUEUES commands nvme(4) issues a SET_NUM_QUEUES command during device initialization to ensure enough I/O queues exists for each of the MSI-X vectors we have allocated. The SET_NUM_QUEUES command is then issued again during nvme_ctrlr_start(), to ensure that is properly set after any controller reset. At least one NVMe drive exists which fails this second SET_NUM_QUEUES command during device initialization. So change nvme_ctrlr_start() to only issue its SET_NUM_QUEUES command when it is coming out of a reset - avoiding the duplicate SET_NUM_QUEUES during device initialization. Reported by: gallatin MFC after: 3 days Sponsored by: Intel 295529 by dteske: Merge SVN r295220 (bz) from projects/vnet/ Fix a panic that occurs when a vnet interface is unavailable at the time the vnet jail referencing said interface is stopped. Sponsored by: FIS Global, Inc. 295527 by andrew: Include the correct header to get a phandle_t needed by ofw_bus_if.h. While here only include opt_platform.h once. Sponsored by: ABT Systems Ltd 295523 by pfg: Ext4: Use boolean type instead of '0' and '1' There are precedents of uses of bool in the kernel and it is incorrect style to use integers as replacement for a boolean type. 295522 by emaste: Explain that boot1.efi prefers ZFS over UFS Reported by: smh 295521 by br: o Move non-generic kernel configuration out from GENERIC. o Add kernel configuration for QEMU. Both SPIKE and QEMU kernel configs are temporary (until we will be able to obtain DTB from loader). Sponsored by: DARPA, AFRL Sponsored by: HEIF5 295520 by jhibbits: Migrate the PTE format for book-e to standardize on the 'indirect PTE' format Summary: The revised Book-E spec, adding the specification for the MMUv2 and e6500, includes a hardware PTE layout for indirect page tables. In order to support this in the future, migrate the PTE format to match the MMUv2 hardware PTE format. Test Plan: Boot tested on a P5020 board. Booted to multiuser mode. Differential Revision: https://reviews.freebsd.org/D5224 295519 by zbb: Support interrupts binding in GICv3 and ITS - Add MOVI command and routine for the LPI migration - Allow to search for the ITS device descriptor using not only devID but also LPI number. - Bind SPIs in the Distributor - Don't bind its_dev to collection. Keep track of the collection IDs for each LPI. Reviewed by: wma Obtained from: Semihalf Sponsored by: Cavium Differential Revision: https://reviews.freebsd.org/D5231 295518 by zbb: Implement finer locking in ITS - Change locks' names to be more suitable - Don't use blocking mutex. Lock only basic operations such as lists or bitmaps modifications. Reviewed by: wma Obtained from: Semihalf Sponsored by: Cavium Differential Revision: https://reviews.freebsd.org/D5230 295517 by zbb: Initially bind all interrupts to the boot CPU when using GICv3 This should be done by routing all interrupts to CPU0, different assignment will be induced by either interrupts shuffling or bus_bind_intr(). Reviewed by: wma Obtained from: Semihalf Sponsored by: Cavium Differential Revision: https://reviews.freebsd.org/D5229 295516 by zbb: Call pmc_hook() correctly in the ARM64 interrupt handler pmc_hook() was called only in case of the stray interrupt but should rather be called on each interrupt. Move in to the arm_cpu_intr() handler, out of the critical section too. Reviewed by: br Obtained from: Semihalf Sponsored by: Cavium Differential Revision: https://reviews.freebsd.org/D5161 295515 by zbb: Introduce bus_bind_intr method for ARM64 It can be used to bind specific interrupt to a particular CPU. Requires PIC support for interrupts binding. Reviewed by: wma Obtained from: Semihalf Sponsored by: Cavium Differential Revision: https://reviews.freebsd.org/D5122 295514 by zbb: Fix bugs in interrupts allocation on ARM64 Separate interrupt descriptors lookup from allocation. It was possible to perform config on non-existing interrupt simply by allocating spurious descriptor. Must lock the interrupt descriptors table lookup to avoid mismatches. This ought to prevent trouble while setting up new interrupt and dispatching existing one. Use spin mutex rather than sleep mutex. This is mainly due to lock in arm_dispatch_intr. This should be eventually changed to a lock-less solution without walking through a linked list on each interrupt. Reviewed by: andrew, wma Obtained from: Semihalf Sponsored by: Cavium Differential Revision: https://reviews.freebsd.org/D5121 295513 by royger: xen-netfront: remove useless NULL check in netif_free xn_ifp is allocated in create_netdev with if_alloc(IFT_ETHER). According to the current arrangement it can't be NULL. Coverity ID: 1349805 Submitted by: Wei Liu <wei.liu2@citrix.com> Reviewed by: royger Sponsored by: Citrix Systems R&D Differential revision: https://reviews.freebsd.org/D5252 295512 by zbb: Minor clean-ups for ARM64 GICv3 and GIC drivers GICv3: - move ICC_SGI1R_EL1 definitions to armreg.h and use proper system register's names GIC: - remove unused functions Reviewed by: andrew Obtained from: Semihalf Sponsored by: Cavium Differential Revision: https://reviews.freebsd.org/D5119 295511 by royger: xen-netfront: rearrange error paths in setup_txqs Coverity spotted double free errors in error path. Fix that by removing the extraneous calls. Coverity ID: 1349798 Submitted by: Wei Liu <wei.liu2@citrix.com> Reviewed by: royger Sponsored by: Citrix Systems R&D Differential revision: https://reviews.freebsd.org/D5251 295510 by royger: xen-netfront: remove pointless assignment in xn_ioctl The variable error is assigned to 0 before entering the switch. Assigning error to 0 before break pointless rewrites the real error value that should be returned. Coverity ID: 1304974 Submitted by: Wei Liu <wei.liu2@citrix.com> Reviewed by: royger Sponsored by: Citrix Systems R&D Differential revision: https://reviews.freebsd.org/D5250 295509 by andrew: Stop defining fdt_pic_table when building for ARM_INTRNG. 295508 by br: Stop device enumeration when we see first empty slot. This fixes operation in QEMU and saves some booting time as well. Pointed out by: Sagar Karandikar <skarandikar@berkeley.edu> Sponsored by: DARPA, AFRL Sponsored by: HEIF5 295507 by andrew: Remove unused Zedboard config files. files.zedboard was empty other than comments, and std.zedboard just pointed to files.zedboard and std.xilinx. 295506 by hselasky: Use a pair of ifs when comparing the 32-bit flowid integers so that the sign bit doesn't cause an overflow. The overflow manifests itself as a sorting index wrap around in the middle of the sorted array, which is not a problem for the LRO code, but might be a problem for the logic inside qsort(). Reviewed by: gnn @ Sponsored by: Mellanox Technologies Differential Revision: https://reviews.freebsd.org/D5239 295505 by wma: ARM64 disassembler: support for LDR instructions Implemented disassembly for a whole bunch of various ldr instructions. Obtained from: Semihalf Sponsored by: Cavium Approved by: cognet (mentor) Reviewed by: zbb Differential revision: https://reviews.freebsd.org/D5217 295504 by adrian: Make bus_space_generic properly map/unmap memory (using pmap_mapdev and pmap_unmapdev respectively) so that resources are properly managed. This is work originally done by kan@. Stanislav picked it up as part of his Mediatek SoC work. Tested: * Carambola2, AR933x SoC Submitted by: Stanislav Galabov <sgalabov@gmail.com> Reviewed by: kan Differential Revision: https://reviews.freebsd.org/D5184 295503 by adrian: Migrate the other MIPS24K SoC cores to use the CPU_MIPS24K option. 295502 by adrian: Missing commit - remove MIPS fdt bus space. Differential Revision: https://reviews.freebsd.org/D5184 295501 by adrian: Remove bus space fdt for MIPS. This was originall done by kan@. Submitted by: Stanislav Galabov <sgalabov@gmail.com> Reviewed by: kan Differential Revision: https://reviews.freebsd.org/D5184 295500 by adrian: Convert MIPS nexus and mips_pic to BUS_PASS Submitted by: Stanislav Galabov <sgalabov@gmail.com> Reviewed by: kan Differential Revision: https://reviews.freebsd.org/D5196 295499 by adrian: Teach the MIPS ticker to attach itself properly when using INTRNG. Submitted by: Stanislav Galabov <sgalabov@gmail.com> Reviewed by: kan Differential Revision: https://reviews.freebsd.org/D5183 295498 by adrian: Begin the MIPS_INTRNG support. This is a prelude to intr-ng support for MIPS boards that need it - notably the CI20 port from kan@ that's upcoming, but also work that Stanislav is doing for the Mediatek platforms. This is the initial platform dependent bits in include/intr.h, some #defines for the nexus code for the intrng initialisation/runtime bits, some changed naming (which I'll fix later to be the same, much like what I did for ARM intr-ng) in exception.S, and the first cut at a PIC. Stanislav and I refactored out the common code for intrng support, so the mips intrng definitions are quite small (sys/mips/include/intr.h.) This is all work done by kan@, which stanislav has been cherry picking into common code for his mediatek chipset work. Tested: * Carambola2 - no regressions (not intr-ng though!) Submitted by: Stanislav Galabov <sgalabov@gmail.com> Reviewed by: kan (original author) Differential Revision: https://reviews.freebsd.org/D5182 295497 by emaste: Update uefi.8 for ZFS and multi device boot support As of r294068 boot1.efi can load loader.efi from ZFS. As of r295320 boot1.efi prefers to load loader.efi from the same device it was loaded from. MFC after: 3 days Sponsored by: The FreeBSD Foundation 295496 by emaste: Document boot1.efi's handling of /boot.config, added in r294768 MFC after: 3 days Sponsored by: The FreeBSD Foundation 295495 by cy: Update leapsecond file in non-chroot environments. PR: 207095 Submitted by: madpilot MFC after: 3 days The end of the build log: [...truncated 83381 lines...] ===> usr.bin/bmake/tests/shell/path_select (depend) --- depend_subdir_usr.sbin --- 1 error generated. --- depend_subdir_usr.bin --- --- depend_subdir_suffixes --- --- depend_subdir_basic --- ===> usr.bin/bmake/tests/suffixes/basic (depend) --- depend_subdir_usr.sbin --- In file included from /usr/src/usr.sbin/amd/libamu/../../../contrib/amd/libamu/wire.c:57: /usr/src/usr.sbin/amd/libamu/../include/config.h:12:10: fatal error: 'config_local.h' file not found #include "config_local.h" ^ --- depend_subdir_usr.bin --- --- depend_subdir_shell --- --- depend_subdir_replace --- ===> usr.bin/bmake/tests/shell/replace (depend) --- depend_subdir_suffixes --- --- depend_subdir_src_wild1 --- ===> usr.bin/bmake/tests/suffixes/src_wild1 (depend) --- depend_subdir_shell --- --- depend_subdir_select --- ===> usr.bin/bmake/tests/shell/select (depend) --- depend_subdir_suffixes --- --- depend_subdir_src_wild2 --- --- depend_subdir_dc --- --- depend_subdir_usr.sbin --- 1 error generated. --- depend_subdir_usr.bin --- --- depend_subdir_bmake --- ===> usr.bin/bmake/tests/suffixes/src_wild2 (depend) --- depend_subdir_dc --- ===> usr.bin/dc (depend) --- depend_subdir_usr.sbin --- In file included from /usr/src/usr.sbin/amd/libamu/../../../contrib/amd/libamu/xutil.c:49: /usr/src/usr.sbin/amd/libamu/../include/config.h:12:10: fatal error: 'config_local.h' file not found #include "config_local.h" ^ --- depend_subdir_usr.bin --- --- depend_subdir_bmake --- --- depend_subdir_syntax --- ===> usr.bin/bmake/tests/syntax (depend) --- depend_subdir_dc --- --- .depend --- rm -f .depend --- depend_subdir_bmake --- --- depend_subdir_directive-t0 --- ===> usr.bin/bmake/tests/syntax/directive-t0 (depend) --- depend_subdir_dc --- CC='cc' mkdep -f .depend -a -std=gnu99 /usr/src/usr.bin/dc/dc.c /usr/src/usr.bin/dc/bcode.c /usr/src/usr.bin/dc/inout.c /usr/src/usr.bin/dc/mem.c /usr/src/usr.bin/dc/stack.c --- depend_subdir_usr.sbin --- 1 error generated. --- depend_subdir_usr.bin --- --- depend_subdir_bmake --- --- depend_subdir_enl --- ===> usr.bin/bmake/tests/syntax/enl (depend) --- depend_subdir_usr.sbin --- In file included from /usr/src/usr.sbin/amd/libamu/../../../contrib/amd/conf/transp/transp_sockets.c:47: /usr/src/usr.sbin/amd/libamu/../include/config.h:12:10: fatal error: 'config_local.h' file not found #include "config_local.h" ^ --- depend_subdir_usr.bin --- --- depend_subdir_funny-targets --- ===> usr.bin/bmake/tests/syntax/funny-targets (depend) --- depend_subdir_semi --- ===> usr.bin/bmake/tests/syntax/semi (depend) --- depend_subdir_usr.sbin --- 1 error generated. --- depend_subdir_usr.bin --- --- depend_subdir_sysmk --- ===> usr.bin/bmake/tests/sysmk (depend) --- depend_subdir_usr.sbin --- In file included from /usr/src/usr.sbin/amd/libamu/../../../contrib/amd/conf/mtab/mtab_bsd.c:51: /usr/src/usr.sbin/amd/libamu/../include/config.h:12:10: fatal error: 'config_local.h' file not found #include "config_local.h" ^ --- depend_subdir_usr.bin --- --- depend_subdir_t0 --- ===> usr.bin/bmake/tests/sysmk/t0 (depend) --- depend_subdir_mk --- ===> usr.bin/bmake/tests/sysmk/t0/mk (depend) --- depend_subdir_2 --- ===> usr.bin/bmake/tests/sysmk/t0/2 (depend) --- depend_subdir_1 --- ===> usr.bin/bmake/tests/sysmk/t0/2/1 (depend) --- depend_subdir_usr.sbin --- 1 error generated. --- depend_subdir_usr.bin --- --- depend_subdir_t1 --- ===> usr.bin/bmake/tests/sysmk/t1 (depend) --- depend_subdir_usr.sbin --- In file included from /usr/src/usr.sbin/amd/libamu/../../../contrib/amd/conf/umount/umount_bsd44.c:49: /usr/src/usr.sbin/amd/libamu/../include/config.h:12:10: fatal error: 'config_local.h' file not found #include "config_local.h" ^ --- depend_subdir_usr.bin --- --- depend_subdir_mk --- ===> usr.bin/bmake/tests/sysmk/t1/mk (depend) --- depend_subdir_2 --- ===> usr.bin/bmake/tests/sysmk/t1/2 (depend) --- depend_subdir_1 --- ===> usr.bin/bmake/tests/sysmk/t1/2/1 (depend) --- depend_subdir_usr.sbin --- 1 error generated. --- depend_subdir_usr.bin --- --- depend_subdir_t2 --- ===> usr.bin/bmake/tests/sysmk/t2 (depend) --- depend_subdir_mk --- ===> usr.bin/bmake/tests/sysmk/t2/mk (depend) --- depend_subdir_dc --- echo dc.full: /usr/obj/usr/src/tmp/usr/lib/libc.a /usr/obj/usr/src/tmp/usr/lib/libcrypto.a >> .depend --- depend_subdir_bmake --- --- depend_subdir_2 --- ===> usr.bin/bmake/tests/sysmk/t2/2 (depend) --- depend_subdir_variables --- ===> usr.bin/bmake/tests/variables (depend) --- depend_subdir_sysmk --- --- depend_subdir_1 --- ===> usr.bin/bmake/tests/sysmk/t2/2/1 (depend) --- depend_subdir_usr.sbin --- In file included from xdr_func_%undef.c:48: /usr/src/usr.sbin/amd/libamu/../include/config.h:12:10: fatal error: 'config_local.h' file not found #include "config_local.h" ^ --- depend_subdir_usr.bin --- --- depend_subdir_variables --- --- depend_subdir_modifier_M --- ===> usr.bin/bmake/tests/variables/modifier_M (depend) --- depend_subdir_newkey --- ===> usr.bin/newkey (depend) --- depend_subdir_bmake --- --- depend_subdir_modifier_t --- ===> usr.bin/bmake/tests/variables/modifier_t (depend) --- depend_subdir_usr.sbin --- 1 error generated. mkdep: compile failed *** [.depend] Error code 1 make[5]: stopped in /usr/src/usr.sbin/amd/libamu 1 error make[5]: stopped in /usr/src/usr.sbin/amd/libamu --- depend_subdir_usr.bin --- --- depend_subdir_newkey --- --- .depend --- --- depend_subdir_usr.sbin --- *** [depend_subdir_libamu] Error code 2 make[4]: stopped in /usr/src/usr.sbin/amd 1 error make[4]: stopped in /usr/src/usr.sbin/amd *** [depend_subdir_amd] Error code 2 make[3]: stopped in /usr/src/usr.sbin --- depend_subdir_usr.bin --- rm -f .depend CC='cc' mkdep -f .depend -a -DYP -std=gnu99 /usr/src/usr.bin/newkey/newkey.c /usr/src/usr.bin/newkey/generic.c /usr/src/usr.bin/newkey/update.c --- depend_subdir_bmake --- A failure has been detected in another branch of the parallel make make[7]: stopped in /usr/src/usr.bin/bmake/tests/variables/modifier_t *** [depend_subdir_modifier_t] Error code 2 make[6]: stopped in /usr/src/usr.bin/bmake/tests/variables 1 error make[6]: stopped in /usr/src/usr.bin/bmake/tests/variables *** [depend_subdir_variables] Error code 2 make[5]: stopped in /usr/src/usr.bin/bmake/tests 1 error make[5]: stopped in /usr/src/usr.bin/bmake/tests *** [depend] Error code 2 make[4]: stopped in /usr/src/usr.bin/bmake 1 error make[4]: stopped in /usr/src/usr.bin/bmake *** [depend_subdir_bmake] Error code 2 make[3]: stopped in /usr/src/usr.bin --- depend_subdir_newkey --- echo newkey.full: /usr/obj/usr/src/tmp/usr/lib/libc.a /usr/obj/usr/src/tmp/usr/lib/librpcsvc.a /usr/obj/usr/src/tmp/usr/lib/libmp.a >> .depend A failure has been detected in another branch of the parallel make make[4]: stopped in /usr/src/usr.bin/newkey *** [depend_subdir_newkey] Error code 2 make[3]: stopped in /usr/src/usr.bin 2 errors make[3]: stopped in /usr/src/usr.bin *** [depend_subdir_usr.bin] Error code 2 make[2]: stopped in /usr/src --- depend_subdir_usr.sbin --- --- depend_subdir_tcpdump --- echo tcpdump.full: /usr/obj/usr/src/tmp/usr/lib/libc.a /usr/obj/usr/src/tmp/usr/lib/libl.a /usr/obj/usr/src/tmp/usr/lib/libpcap.a /usr/obj/usr/src/tmp/usr/lib/libcapsicum.a /usr/obj/usr/src/tmp/usr/lib/libcrypto.a >> .depend A failure has been detected in another branch of the parallel make make[5]: stopped in /usr/src/usr.sbin/tcpdump/tcpdump *** [depend] Error code 2 make[4]: stopped in /usr/src/usr.sbin/tcpdump 1 error make[4]: stopped in /usr/src/usr.sbin/tcpdump *** [depend_subdir_tcpdump] Error code 2 make[3]: stopped in /usr/src/usr.sbin 2 errors make[3]: stopped in /usr/src/usr.sbin *** [depend_subdir_usr.sbin] Error code 2 make[2]: stopped in /usr/src 2 errors make[2]: stopped in /usr/src *** [_depend] Error code 2 make[1]: stopped in /usr/src 1 error make[1]: stopped in /usr/src *** [buildworld] Error code 2 make: stopped in /usr/src 1 error make: stopped in /usr/src Build step 'Execute shell' marked build as failure [PostBuildScript] - Execution post build scripts. [FreeBSD_HEAD_i386] $ /bin/sh -xe /tmp/hudson7985524070429360485.sh + export 'PATH=/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin:/usr/local/bin' + export 'jname=FreeBSD_HEAD_i386' + echo 'clean up jail FreeBSD_HEAD_i386' clean up jail FreeBSD_HEAD_i386 + sudo jail -r FreeBSD_HEAD_i386 + sudo ifconfig igb0 inet6 2610:1c1:1:607c::106:1 -alias + sudo umount FreeBSD_HEAD_i386/usr/src + sudo umount FreeBSD_HEAD_i386/dev + sudo rm -fr FreeBSD_HEAD_i386 + true + sudo chflags -R noschg FreeBSD_HEAD_i386 + sudo rm -fr FreeBSD_HEAD_i386 Email was triggered for: Failure - Any Sending email for trigger: Failure - Any
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1948561587.1.1455609836828.JavaMail.jenkins>