FreeBSD src-all

More graphs

2017-09-18 16:08:11.924608323719rlibby

MFC r304877,r323573:r304877 (by bapt):Remove warning on struct-overflow on gcc 5.3.0 as zic(8) dies on itr323573:gcc builds: reenable -Wstrict-overflow for bsd.sys.mkThis effectively reverts r304877, after having relegated the warningsuppression to the zic(8) makefile in r323572.Effectively this is a record-only merge of r304877 and r323573.
2017-08-08 07:29:05.407892322247avg

MFV r322246: 8508 Mounting a zpool on 32-bit platforms panicsFreeBSD note: this is a record-only commit, the actual change originatedfrom FreeBSD.illumos/illumos-gate@b11fe8c01471a5bff68e83e1fe5f809ad16b3be8https://github.com/illumos/illumos-gate/commit/b11fe8c01471a5bff68e83e1fe5f809ad16b3be8https://www.illumos.org/issues/8508Mounting a zpool on a 32-bit system triggers a panic in spa_history_log_version() due to a type format mismatch for ZPL_VERSION. ZPL_VERSION is a unsignedlong long, but the format expects an integer. On 64-bit platforms this may notbe an issue due to word size and alignment. On 32-bit platforms a word size ishalf that of a long long, causing the second word of the long long to be seenas the string pointer for utsname.nodename.Reviewed by: Matt Ahrens <mahrens@delphix.com>Approved by: Dan McDonald <danmcd@joyent.com>Author: Justin Hibbits <chmeeedalf@gmail.com>
2017-08-08 06:36:28.775235322220avg

MFV r322219: 8416 abd.h is not C++ friendlyFreeBSD note: this is a record-only commit, we had to apply the changein r320156.illumos/illumos-gate@5e2a074725cb7c16ea1c6554da11ab4d6b4e7aeehttps://github.com/illumos/illumos-gate/commit/5e2a074725cb7c16ea1c6554da11ab4d6b4e7aeehttps://www.illumos.org/issues/8416A C++ compiler fails to compile abd_is_linear(), which is an inline functiondefined in abd.h, with the following error:error: cannot initialize return object of type 'boolean_t' with anrvalue of type 'bool'That happens because a bool can not be converted to an enum in C++.That's a problem because abd.h can be visible through other header files that aC++ program that works with ZFS can include.Reviewed by: Igor Kozhukhov <igor@dilos.org>Reviewed by: Dan Kimmel <dan.kimmel@delphix.com>Reviewed by: Alek Pinchuk <pinchuk.alek@gmail.com>Approved by: Robert Mustacchi <rm@joyent.com>Author: Andriy Gapon <avg@FreeBSD.org>
2017-08-08 00:59:54.807542322214ngie

Make test scripts under tests/... non-executableExecutable bits should be set at install time instead of in the repo.Setting executable bits on files triggers false positives with Phabricator.MFC after: 2 months
2017-07-26 12:58:25.289302321564mav

MFC r318926: MFV r316919: 7885 zpool list can report 16.0e for expandszFreeBSD note: this is a record-only change, the actual change wasdirectly committed by smh.illumos/illumos-gate@c040c10cdd1e4eab0fc88203758367dd81e057b7https://github.com/illumos/illumos-gate/commit/c040c10cdd1e4eab0fc88203758367dd81e057b7https://www.illumos.org/issues/7885When a member of a RAIDZ has been replaced with a device smaller than theoriginal, then the top level vdev can report its expand size as 16.0E.The reduced child asize causes the RAIDZ to have a vdev_asize lower than itsvdev_max_asize which then results in an underflow during the calculation of theparents expand size.Also for RAIDZ vdevs the sum of their child vdev_min_asize could be smallerthan the parents vdev_min_size.Fixed by: https://github.com/openzfs/openzfs/pull/296Reviewed by: Matthew Ahrens <mahrens@delphix.com>Reviewed by: George Wilson <george.wilson@delphix.com>Approved by: Gordon Ross <gordon.w.ross@gmail.com>Author: Steven Hartland <steven.hartland@multiplay.co.uk>
2017-07-26 12:52:47.448065321560mav

MFC r318922: MFV r316927: 5379 modifying a mmap()-ed file does not update its timestampsFreeBSD note: this is a record-only merge as the FreeBSD putpages codeis quite different from the upstream.illumos/illumos-gate@80e10fd0d22bbf0d18bfdae035e06f44c68ae8e6https://github.com/illumos/illumos-gate/commit/80e10fd0d22bbf0d18bfdae035e06f44c68ae8e6https://www.illumos.org/issues/5379The following is based on a review of the illumos code and on a similar problemreported for FreeBSD where the relevant code is different.Looking at this block of code http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/fs/zfs/zfs_vnops.c#4187 I see code to set up ansa_bulk_attr_t object, I see code to set up mtime and ctime values, but I donot see code to actually apply the attributes...I would expect there to be a call to sa_bulk_update(), there is such a call inzfs_write() for instance.mmap_write.c [Magnifier] - demo (1.42 KB) Andriy Gapon, 2015-11-11 01:53 PMReviewed by: Matthew Ahrens <mahrens@delphix.com>Reviewed by: Prashanth Sreenivasa <pks@delphix.com>Reviewed by: Dan McDonald <danmcd@omniti.com>Approved by: Gordon Ross <gordon.w.ross@gmail.com>Author: Andriy Gapon <andriy.gapon@clusterhq.com>
2017-07-26 12:49:55.531004321557mav

MFC r318834: MFV r316930: 5814 bpobj_iterate_impl(): Close a refcount leak iterating on a sublist.illumos/illumos-gate@b67dde11a73a9455d641403cbbb65ec2add41b41https://github.com/illumos/illumos-gate/commit/b67dde11a73a9455d641403cbbb65ec2add41b41https://www.illumos.org/issues/5814Lets pull in this patch from freebsd:http://svnweb.freebsd.org/base?view=revision&revision=271781bpobj_iterate_impl(): Close a refcount leak iterating on a sublist.If bpobj_space() returned non-zero here, the sublist would have beenleft open, along with the bonus buffer hold it requires. This calldoes not invoke any calls to bpobj_close() itself.This bug doesn't have any known vector, but was found on inspection.MFC after: 1 weekSponsored by: Spectra LogicAffects: All ZFS versions starting 21 May 2010 (illumos cde58dbc)MFSpectraBSD: r1050998 on 2014/03/26Fix bpobj_iterate_impl() to properly call bpobj_close() if bpobj_space()returns an error.Reviewed by: Prakash Surya <prakash.surya@delphix.com>Reviewed by: Matthew Ahrens <mahrens@delphix.com>Reviewed by: Paul Dagnelie <paul.dagnelie@delphix.com>Reviewed by: Simon Klinkert <simon.klinkert@gmail.com>Approved by: Gordon Ross <gwr@nexenta.com>Author: Will Andrews <will@freebsd.org>FreeBSD note: this is a "record-only" commit as the actual change wasdirectly committed to FreeBSD by (or on behalf of) the author.
2017-07-24 08:11:48.736586321280ngie

Revert r321278I accidentally merged a mergeinfo change to this branch with a badcommit message.
2017-07-24 08:11:48.725981321278ngie

MFC r314543:Add missing section when referencing ctl(4) via .Xr macro
2017-07-24 08:11:48.263499321255ngie

Record-only merge for ^/head@r312419
2017-07-24 08:11:48.244352321254ngie

Record-only merge for ^/head@r312419
2017-07-19 12:54:57.186376321216ngie

Record merges for r319659 and r319660.
2017-07-19 12:54:47.406824321215ngie

Record merges for r319659 and r319660.
2017-07-12 20:20:55.978565320926mm

Re-tag 3.3.2
2017-07-01 23:43:47.776541320563araujo

Revert bad merge r320390, sorry for the noisy.
2017-07-01 23:35:12.220107320562araujo

MFC r320390:With r318394 seems it breaks gpart(8) in some embedded systems such like PCEngines,RPI1-B, Alix and APU2 boards as well as NanoBSD with the following message:vnode_pager_generic_getpages_done: I/O read error 5Seems the breakage was because it was missed to include acr in glabel update.Reported by: Peter Blok <pblok@bsd4all.org>, madpilot, imp and trasz.Reviewed by: traszTested by: Peter Blok and madpilot.Sponsored by: iXsystems, Inc.Differential Revision: https://reviews.freebsd.org/D11365
2017-05-26 07:26:09.396375318926avg

MFV r316919: 7885 zpool list can report 16.0e for expandszFreeBSD note: this is a record-only change, the actual change wasdirectly committed by smh.illumos/illumos-gate@c040c10cdd1e4eab0fc88203758367dd81e057b7https://github.com/illumos/illumos-gate/commit/c040c10cdd1e4eab0fc88203758367dd81e057b7https://www.illumos.org/issues/7885When a member of a RAIDZ has been replaced with a device smaller than theoriginal, then the top level vdev can report its expand size as 16.0E.The reduced child asize causes the RAIDZ to have a vdev_asize lower than itsvdev_max_asize which then results in an underflow during the calculation of theparents expand size.Also for RAIDZ vdevs the sum of their child vdev_min_asize could be smallerthan the parents vdev_min_size.Fixed by: https://github.com/openzfs/openzfs/pull/296Reviewed by: Matthew Ahrens <mahrens@delphix.com>Reviewed by: George Wilson <george.wilson@delphix.com>Approved by: Gordon Ross <gordon.w.ross@gmail.com>Author: Steven Hartland <steven.hartland@multiplay.co.uk>
2017-05-26 06:34:56.670123318922avg

MFV r316927: 5379 modifying a mmap()-ed file does not update its timestampsFreeBSD note: this is a record-only merge as the FreeBSD putpages codeis quite different from the upstream.illumos/illumos-gate@80e10fd0d22bbf0d18bfdae035e06f44c68ae8e6https://github.com/illumos/illumos-gate/commit/80e10fd0d22bbf0d18bfdae035e06f44c68ae8e6https://www.illumos.org/issues/5379The following is based on a review of the illumos code and on a similar problemreported for FreeBSD where the relevant code is different.Looking at this block of code http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/fs/zfs/zfs_vnops.c#4187 I see code to set up ansa_bulk_attr_t object, I see code to set up mtime and ctime values, but I donot see code to actually apply the attributes...I would expect there to be a call to sa_bulk_update(), there is such a call inzfs_write() for instance.mmap_write.c [Magnifier] - demo (1.42 KB) Andriy Gapon, 2015-11-11 01:53 PMReviewed by: Matthew Ahrens <mahrens@delphix.com>Reviewed by: Prashanth Sreenivasa <pks@delphix.com>Reviewed by: Dan McDonald <danmcd@omniti.com>Approved by: Gordon Ross <gordon.w.ross@gmail.com>Author: Andriy Gapon <andriy.gapon@clusterhq.com>
2017-05-24 18:41:19.477702318834avg

MFV r316930: 5814 bpobj_iterate_impl(): Close a refcount leak iterating on a sublist.illumos/illumos-gate@b67dde11a73a9455d641403cbbb65ec2add41b41https://github.com/illumos/illumos-gate/commit/b67dde11a73a9455d641403cbbb65ec2add41b41https://www.illumos.org/issues/5814Lets pull in this patch from freebsd:http://svnweb.freebsd.org/base?view=revision&revision=271781bpobj_iterate_impl(): Close a refcount leak iterating on a sublist.If bpobj_space() returned non-zero here, the sublist would have beenleft open, along with the bonus buffer hold it requires. This calldoes not invoke any calls to bpobj_close() itself.This bug doesn't have any known vector, but was found on inspection.MFC after: 1 weekSponsored by: Spectra LogicAffects: All ZFS versions starting 21 May 2010 (illumos cde58dbc)MFSpectraBSD: r1050998 on 2014/03/26Fix bpobj_iterate_impl() to properly call bpobj_close() if bpobj_space()returns an error.Reviewed by: Prakash Surya <prakash.surya@delphix.com>Reviewed by: Matthew Ahrens <mahrens@delphix.com>Reviewed by: Paul Dagnelie <paul.dagnelie@delphix.com>Reviewed by: Simon Klinkert <simon.klinkert@gmail.com>Approved by: Gordon Ross <gwr@nexenta.com>Author: Will Andrews <will@freebsd.org>FreeBSD note: this is a "record-only" commit as the actual change wasdirectly committed to FreeBSD by (or on behalf of) the author.
2017-05-23 05:20:46.767076318735n_hibma

Commit the mergeinfo for the commit to sbin/dhclient.