Date: Wed, 11 May 2016 19:54:04 +0000 (GMT) From: jenkins-admin@FreeBSD.org To: cem@FreeBSD.org, emaste@FreeBSD.org, bdrewery@FreeBSD.org, hselasky@FreeBSD.org, andrew@FreeBSD.org, jkim@FreeBSD.org, gonzo@FreeBSD.org, jenkins-admin@FreeBSD.org, freebsd-arm@FreeBSD.org Subject: FreeBSD_HEAD_arm64 - Build #3120 - Fixed Message-ID: <2027221716.5.1462996560013.JavaMail.jenkins@jenkins-9.freebsd.org> In-Reply-To: <1939146439.41.1462983251429.JavaMail.jenkins@jenkins-9.freebsd.org>
index | next in thread | previous in thread | raw e-mail
FreeBSD_HEAD_arm64 - Build #3120 - Fixed: Build information: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_arm64/3120/ Full change log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_arm64/3120/changes Full build log: https://jenkins.FreeBSD.org/job/FreeBSD_HEAD_arm64/3120/console Change summaries: 299477 by gonzo: Add OF_prop_free function as a counterpart for OF_*prop_alloc - Introduce new OF API function OF_prop_free to free memory allocated by OF_getprop_alloc and OF_getencprop_alloc. Current code just calls free(9) with M_OFWPROP memory class which assumes knowledge about OF_*prop_alloc functions' internals and leads to unneccessary code coupling - Convert some of the free(..., M_OFWPROP) instances to OF_prop_free Files affected by this commit are the ones I was able to test on real hardware. The rest of free(..., M_OFWPROP) instances will be handled with idividual maintainers Reviewed by: andrew Differential Revision: https://reviews.freebsd.org/D6315 299476 by cem: whois(1): Fix potential double-close and logic mistakes Close the fd the poll error was detected on, rather than the last opened fd, to fix the double-close. Use -1 to make it explict which int variables no longer own socket file descriptors. Actually shrink, rather than grow, the poll timeout to match comment. Reported by: Coverity CID: 1304860, 1305616 Sponsored by: EMC / Isilon Storage Division 299475 by gonzo: Add gpiokeys driver gpiokey driver implements functional subset of gpiokeys device-tree bindings: https://www.kernel.org/doc/Documentation/devicetree/bindings/input/gpio-keys.txt It acts as a virtual keyboard, so keys are visible through kbdmux(4) Driver maps linux scancodes for most common keys to FreeBSD scancodes and also extends spec by introducing freebsd,code property to specify FreeBSD-native scancodes. Reviewed by: mmel, jmcneill Differential Revision: https://reviews.freebsd.org/D6279 299474 by emaste: Deorbit ALLOW_SHARED_TEXTREL We want to avoid .text relocations in shared objects. libcrypto was the only consumer and it is now fixed (as of r299389). Remove the now-unused support for turning off the linker warning. Reviewed by: kib Differential Revision: https://reviews.freebsd.org/D6323 299473 by cem: whois(1): Pull out async multiple host connection code into a routine This logic was added to the whois() function in r281959, but could easily be its own routine. In this case, I think the abstraction makes both functions easier to reason about. This precedes some Coverity-suggested cleanup. Sponsored by: EMC / Isilon Storage Division 299472 by bdrewery: DIRDEPS_BUILD: Exclude host tools for Makefile.depend.host as well. Sponsored by: EMC / Isilon Storage Division 299471 by hselasky: Resolve LINT linking issue by renaming ida_init() to ida_setup(). The ida_init() symbol name is now taken for use by the LinuxKPI. Reported by: emaste @ Discussed with: mav @ 299470 by cem: mixer(8): Style: Tag no-return usage() as __dead2 Coverity really should have figured this out from the exit(3) call at the end of the routine, but just make it explicit. No functional change. Reported by: Coverity CID: 1304866 (false positive double-close of 'baz') Sponsored by: EMC / Isilon Storage Division 299469 by hselasky: Match Linux behaviour and iterate the IDR tree unlocked. The caller is responsible the IDR tree stays unmodified while iterating. MFC after: 1 week Sponsored by: Mellanox Technologies 299468 by hselasky: The idr_for_each() function is now part of the LinuxKPI. Use the LinuxKPI's idr_for_each() function instead of the local one to avoid compilation issues. Discussed with: np @ MFC after: 1 week 299467 by andrew: Add a new get_id interface to pci and pcib. This will allow us to both detect failures, and get different PCI IDs. For the former the interface returns an int to signal an error. The ID is returned at a uintptr_t * argument. For the latter there is a type argument that allows selecting the ID type. This only specifies a single type, however a MSI type will be added to handle the need to find the ID the hardware passes to the ARM GICv3 interrupt controller. A follow up commit will be made to remove pci_get_rid. Reviewed by: jhb, rstone Obtained from: ABT Systems Ltd Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D6239 299466 by cem: bsnmpd: Fix size of trapsink::comm to match other community arrays This fixes a number of possible strcpy() buffer overruns between the various community strings in trap.c. Reported by: Coverity CIDs: 1006820, 1006821, 1006822 Sponsored by: EMC / Isilon Storage Division 299465 by cem: bsnmp: Don't overrun privkey buffer by copying wrong size The 'priv_key' array is SNMP_PRIV_KEY_SIZ bytes, not SNMP_AUTH_KEY_SIZ. Reported by: Coverity CIDs: 1008326, 1009675 Sponsored by: EMC / Isilon Storage Division 299464 by emaste: libcrypto: add "Do not modify" comment to generated source files Reviewed by: jkim Differential Revision: https://reviews.freebsd.org/D6237 299463 by andrew: On arm64 always create a bus_dmamap_t object. This will be use to hold the list of memory that the kernel will need to sync when operating with a non-cache coherent DMA engine. Obtained from: ABT Systems Ltd Sponsored by: The FreeBSD Foundation 299462 by jkim: Enable linker error if libcrypto.so contains a relocation against text. It is position independent on all platforms since r299389. Submitted by: kib 299461 by cem: ffs_bswap: Copy one UFS dinode member at a time No functional change. Reported by: Coverity CIDs: 974635, 974636, 977396, 977397, 977398, 977399 Sponsored by: EMC / Isilon Storage Division 299460 by cem: fsck_ffs: Don't overrun mount device buffer Maybe this case is impossible. Either way, when attempting to "/dev/"-prefix a non-global device name, check that we do not overrun the f_mntfromname buffer. In this case, truncating (with strlcpy or similar) would not be useful, since the f_mntfromname result of getmntpt() is passed directly to open(2) later. Reported by: Coverity CID: 1006789 Sponsored by: EMC / Isilon Storage Division 299459 by cem: compat/opensolaris: Don't redefined off64_t if already defined A follow-up to r299456. Reported by: gjb Sponsored by: EMC / Isilon Storage Divisionhelp
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2027221716.5.1462996560013.JavaMail.jenkins>