{"push_id":543328273,"size":11,"distinct_size":11,"ref":"refs/heads/master","head":"d958324f97f4668a2a6e4a6ce3e5ca09b71b31d9","before":"c944be5d7e619ad7259fdb5802f0f9bd90a22488","commits":[{"sha":"1b95fd5d708bcf7de79740a0cd08de43caa0d847","author":{"email":"d0afa59274c36873fd5ce292d38af03808072dd9@druber.com","name":"Dan Swartzendruber"},"message":"Improve systemd script to not leave stale sharetab\n\nThe systemd script zfs-share.service does 'zfs share -a' to share\nany required datasets.  Unfortunately, /etc/dfs/sharetab is stale\nfrom the previous boot.  Delete it before we share.\n\nSigned-off-by: Dan Swartzendruber <dswartz@druber.com>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2883","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/1b95fd5d708bcf7de79740a0cd08de43caa0d847"},{"sha":"c66989baaeee5bb4ac06cebfad513e1253f22564","author":{"email":"dc0cd76ff5fa4640b2de9f8b4d5534da2b635680@gmail.com","name":"Jacek Fefliński"},"message":"zpool upgrade return errors to stderr instead of stdout\n\nSigned-off-by: Jacek Feflinski <feflik@gmail.com>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2955","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/c66989baaeee5bb4ac06cebfad513e1253f22564"},{"sha":"c0ba93dee6cb62ccbb2c9b3103f15ddbf4680d6e","author":{"email":"64b2b6d12bfe4baae7dad3d018f8cbf6b0e7a044@thwartedefforts.org","name":"Andy Bakun"},"message":"Fix typo in %post scriptlet lines\n\nMissing space made the %post directive be part of the package\n%description and not have a %post scriptlet defined.\n\nSigned-off-by: Andy Bakun <github@thwartedefforts.org>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2961","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/c0ba93dee6cb62ccbb2c9b3103f15ddbf4680d6e"},{"sha":"9063f65476b7b7d78ccf096fec890b8727117e2a","author":{"email":"3f3625d010735cc4c47c94b1e81ebdd197a99021@actifio.com","name":"Boris Protopopov"},"message":"Correct error returns to unify cross-pool operation error handling\n\nSigned-off-by: Boris Protopopov <boris.protopopov@actifio.com>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2911","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/9063f65476b7b7d78ccf096fec890b8727117e2a"},{"sha":"5c7afad448325b5761601f574fc937476d3a77d1","author":{"email":"32d50ad1e384dcb9a5bf7e40fa470df5dabaf348@llnl.gov","name":"Brian Behlendorf"},"message":"Fix cstyle issue from c66989b\n\nCommit c66989b accidentally introduced a cstyle issue which went\nunnoticed.  This tiny patch corrects that oversight.\n\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/5c7afad448325b5761601f574fc937476d3a77d1"},{"sha":"2d9d57b0fbe17b56a1bdd09505400144778cb189","author":{"email":"b8c8dd8f747f1a510525ee56a3584fdc28e4c351@llnl.gov","name":"Ned Bass"},"message":"vdev_id: use mawk-compatible regular expression\n\nSlot mapping in vdev_id doesn't work on systems using mawk as the 'awk'\nalternative. A regular expression in map_slot() contains an unquoted\nempty string following the alternation (|) operator, which results in an\n\"missing operand\" error with mawk. The solution is to rearrange the\nexpression so the alternation has two operands.\n\nSigned-off-by: Ned Bass <bass6@llnl.gov>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses zfsonlinux/pkg-zfs#136\nCloses zfsonlinux/zfs#2965","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/2d9d57b0fbe17b56a1bdd09505400144778cb189"},{"sha":"74328ee18f94d27f9c802d29fdd311018dab2adf","author":{"email":"32d50ad1e384dcb9a5bf7e40fa470df5dabaf348@llnl.gov","name":"Brian Behlendorf"},"message":"Fix zfs_putpage() lock inversion\n\nThere exists a lock inversions involving the zfs range lock and the\nindividual page writeback bits which can result in a deadlock.  To\nprevent this we must always manipulate the writeback bit while\nholding the range lock.  The exact deadlock is as follows:\n\n------ Process A ------        ------ Process B ------\nzpl_writepages                 zpl_fallocate\nwrite_cache_pages              zpl_fallocate_common\nzpl_putpage                    zfs_space\nzfs_putpage (set bit)          zfs_freesp\nzfs_range_lock (wait on lock)  zfs_free_range (take lock)\n[has not yet initiated I/O,    truncate_inode_pages_range\nthe bit will not be cleared]   wait_on_page_writeback (wait on bit)\n\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nSigned-off-by: Tim Chase <tim@chase2k.com>\nSigned-off-by: Richard Yao <richard.yao@clusterhq.com>\nIssue #2976","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/74328ee18f94d27f9c802d29fdd311018dab2adf"},{"sha":"49ee64e5e6ec7240bc63d96064f2feeb8f84ed9d","author":{"email":"b8c8dd8f747f1a510525ee56a3584fdc28e4c351@llnl.gov","name":"Ned Bass"},"message":"Remove duplicate typedefs from trace.h\n\nOlder versions of GCC (e.g. GCC 4.4.7 on RHEL6) do not allow duplicate\ntypedef declarations with the same type. The trace.h header contains\nsome typedefs to avoid 'unknown type' errors for C files that haven't\ndeclared the type in question. But this causes build failures for C\nfiles that have already declared the type. Newer versions of GCC (e.g.\nv4.6) allow duplicate typedefs with the same type unless pedantic error\nchecking is in force. To support the older versions we need to remove\nthe duplicate typedefs.\n\nRemoval of the typedefs means we can't built tracepoints code using\nthose types unless the required headers have been included. To\nfacilitate this, all tracepoint event declarations have been moved out\nof trace.h into separate headers. Each new header is explicitly included\nfrom the C file that uses the events defined therein. The trace.h header\nis still indirectly included form zfs_context.h and provides the\nimplementation of the dprintf(), dbgmsg(), and SET_ERROR() interfaces.\nThis makes those interfaces readily available throughout the code base.\nThe macros that redefine DTRACE_PROBE* to use Linux tracepoints are also\nstill provided by trace.h, so it is a prerequisite for the other\ntrace_*.h headers.\n\nThese new Linux implementation-specific headers do introduce a small\ndivergence from upstream ZFS in several core C files, but this should\nnot present a significant maintenance burden.\n\nSigned-off-by: Ned Bass <bass6@llnl.gov>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nIssue #2953","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/49ee64e5e6ec7240bc63d96064f2feeb8f84ed9d"},{"sha":"4e30e68cafdd558463be2352849e707bcc36aa3c","author":{"email":"b8c8dd8f747f1a510525ee56a3584fdc28e4c351@llnl.gov","name":"Ned Bass"},"message":"Don't use AC_LANG_SOURCE for conftest.h source\n\nUsing AC_LANG_SOURCE with some versions of autoconf is problematic if\nthe given source is to be written to a header file. Such versions assume\nthe contents are to be written to conftest.c and generate shell code to\nthat effect. The contents of the test program to detect support for\nLinux tracepoints were consequently malformed (containing the source for\nconftest.h) so the build system incorrectly disabled tracepoints\nsupport. Fix this in ZFS_LINUX_TRY_COMPILE_HEADER by passing the header\nsource directly to ZFS_LINUX_COMPILE_IFELSE.\n\nSigned-off-by: Ned Bass <bass6@llnl.gov>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nIssue #2953","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/4e30e68cafdd558463be2352849e707bcc36aa3c"},{"sha":"33b6dbbc51c790fc8048e2e18cd9bc10be384233","author":{"email":"b8c8dd8f747f1a510525ee56a3584fdc28e4c351@llnl.gov","name":"Ned Bass"},"message":"Document zfs_flags module parameter\n\nAdd a table describing the debugging flags that can be set in the zfs_flags\nmodule parameter.  Also change the module_param type to 'uint' so users aren't\nshown a negative value. The updated man page text is reproduced below for\nconvenience.\n\nzfs_flags (int)\n            Set  additional debugging flags. The following flags may be\n            bitwise-or'd together.\n\n            +-------------------------------------------------------+\n            |Value   Symbolic Name                                  |\n            |        Description                                    |\n            +-------------------------------------------------------+\n            |    1   ZFS_DEBUG_DPRINTF                              |\n            |        Enable dprintf entries in the debug log.       |\n            +-------------------------------------------------------+\n            |    2   ZFS_DEBUG_DBUF_VERIFY *                        |\n            |        Enable extra dbuf verifications.               |\n            +-------------------------------------------------------+\n            |    4   ZFS_DEBUG_DNODE_VERIFY *                       |\n            |        Enable extra dnode verifications.              |\n            +-------------------------------------------------------+\n            |    8   ZFS_DEBUG_SNAPNAMES                            |\n            |        Enable snapshot name verification.             |\n            +-------------------------------------------------------+\n            |   16   ZFS_DEBUG_MODIFY                               |\n            |        Check for illegally modified ARC buffers.      |\n            +-------------------------------------------------------+\n            |   32   ZFS_DEBUG_SPA                                  |\n            |        Enable spa_dbgmsg entries in the debug log.    |\n            +-------------------------------------------------------+\n            |   64   ZFS_DEBUG_ZIO_FREE                             |\n            |        Enable verification of block frees.            |\n            +-------------------------------------------------------+\n            |  128   ZFS_DEBUG_HISTOGRAM_VERIFY                     |\n            |        Enable extra spacemap histogram verifications. |\n            +-------------------------------------------------------+\n            * Requires debug build.\n\n            Default value: 0.\n\nSigned-off-by: Ned Bass <bass6@llnl.gov>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2988","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/33b6dbbc51c790fc8048e2e18cd9bc10be384233"},{"sha":"d958324f97f4668a2a6e4a6ce3e5ca09b71b31d9","author":{"email":"32d50ad1e384dcb9a5bf7e40fa470df5dabaf348@llnl.gov","name":"Brian Behlendorf"},"message":"Fix zfs_putpage() lock inversion (again)\n\nThis is a follow up commit to 74328ee which correctly resolved a lock\ninversion between zfs_putpage() and zfs_free_range().  Unfortunately,\nin the process it accidentally introduced another inversion between\nzfs_putpage() and zfs_read().  The page must be unlocked before taking\nthe range lock.  This patch corrects that issue.\n\nIn addition, because the locking rules here are subtle a block comment\nhas been added clearly explaining why the ordering here is critical.\n\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nSigned-off-by: Ned Bass <bass6@llnl.gov>\nIssue #2976","distinct":true,"url":"https://api.github.com/repos/odeke-em/zfs/commits/d958324f97f4668a2a6e4a6ce3e5ca09b71b31d9"}]}