{"push_id":3128490239,"size":56,"distinct_size":5,"ref":"refs/heads/master","head":"fa61e72340c1b47f33b5290e5eeb7e05e62eabf0","before":"9042f6033a72e2ffbd4f92af7de83ad0f81f1dfc","commits":[{"sha":"82c0a050fcaa50c2d35e562858fa5c95e05caecd","author":{"name":"Brian Behlendorf","email":"32d50ad1e384dcb9a5bf7e40fa470df5dabaf348@llnl.gov"},"message":"Linux 4.20 compat: current_kernel_time()\n\nCommit torvalds/linux@976516404 removed the current_kernel_time()\r\nfunction (and several others).  All callers are expected to use\r\ncurrent_kernel_time64().  Update the gethrestime_sec() wrapper\r\naccordingly.\r\n\r\nReviewed-by: Olaf Faaland <faaland1@llnl.gov>\r\nReviewed-by: Tony Hutter <hutter2@llnl.gov>\r\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nCloses #8074","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/82c0a050fcaa50c2d35e562858fa5c95e05caecd"},{"sha":"9553c533a635bf39697b0b54f4751580f6d2590d","author":{"name":"Matthew Ahrens","email":"0962b5d2f5c1117073cf6f982c188f029b03d89e@delphix.com"},"message":"bpobj_enqueue_subobj() should copy small subobj's\n\nWhen we delete a snapshot, we consolidate some bpobj's together because\r\nwe no longer need to keep their entries in separate buckets.  This is\r\ndone in constant time by including the \"sub\" bpobj by reference in the\r\nparent bpobj.\r\n\r\nAfter many snapshots have been deleted, we may have many sub-bpobj's.\r\nUsually, most sub-bpobj's don't contain many BP's.  Compared to this\r\nsmall payload, the sub-bpobj is relatively heavyweight since it is a\r\nobject in the MOS.  A common scenario on a long-lived pool is for the\r\nvast majority of MOS objects to be small sub-bpobj's.\r\n\r\nTo improve this situation, when consolidating bpobj's together,\r\nbpobj_enqueue_subobj() can copy the contents of small bpobj's into the\r\nparent, and then delete the enqueued bpobj, rather than including it by\r\nreference.  Since this copying is limited in size (to one block), the\r\nconsolidation is still constant time, though with a larger constant due\r\nto reading in the one block of the enqueued bpobj.\r\n\r\nThis idea and mechanism are similar to how we handle \"sub-subobj's\".\r\nWhen including a sub-bpobj by reference, if the sub-bpobj itself has\r\nless than a block of sub-sub-bpobj's, the list of sub-sub-bpobj's is\r\ncopied to the parent bpobj's list of sub-bpobj's.\r\n\r\nReviewed-by: Serapheim Dimitropoulos <serapheim@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nReviewed-by: Paul Zuchowski <pzuchowski@datto.com>\r\nSigned-off-by: Matthew Ahrens <mahrens@delphix.com>\r\nCloses #8053 \r\nIssue #7908","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/9553c533a635bf39697b0b54f4751580f6d2590d"},{"sha":"58aeb87a8f69019487457460a5d7c823910fc8ff","author":{"name":"George Melikov","email":"1d6e1cf70ec6f9ab28d3ea4b27a49a77654d370e@gmelikov.ru"},"message":"ZTS: change `$(cat)` to `$(<)` for speedup\n\nIt's better to use ksh/bash built in methods,\r\nrather than spawn new processes every time.\r\n\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nReviewed-by: John Wren Kennedy <john.kennedy@delphix.com>\r\nSigned-off-by: George Melikov <mail@gmelikov.ru>\r\nCloses #8071","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/58aeb87a8f69019487457460a5d7c823910fc8ff"},{"sha":"04a88fc00cce9f0744d7e815cbe0ced4dd009b75","author":{"name":"Paul Zuchowski","email":"1e078106fca44e97b3c018aaa947030e333ae9e2@users.noreply.github.com"},"message":"ZTS: Fix posix ACL tests that should pass\n\nMake sure tests have proper include files.  Make sure underlying\r\n\"chmod\" style permissions don't interfere with ACLs.\r\n\r\nReviewed-by: John Kennedy <john.kennedy@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Paul Zuchowski <pzuchowski@datto.com>\r\nCloses #8069","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/04a88fc00cce9f0744d7e815cbe0ced4dd009b75"},{"sha":"6644e5bb6e1a6c25c5006c819abf93c7bb662e80","author":{"name":"Richard Elling","email":"b9e258106bdd63c1cd9d263657eafb302b952285@RichardElling.com"},"message":"Update zfs-events.5 with info from PSARC 2009/497\n\nUpdate zfs-events.5 with info from PSARC 2009/497 regarding ereport fields.\r\nAlso updates ZIO_STAGE_* and ZIO_FLAG_* descriptions to match current source.\r\n\r\nReviewed by: loli10K <ezomori.nozomu@gmail.com>\r\nReviewed-by: George Melikov <mail@gmelikov.ru>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Richard Elling <Richard.Elling@RichardElling.com>\r\nCloses #8057","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/6644e5bb6e1a6c25c5006c819abf93c7bb662e80"},{"sha":"e89f1295d4faa88bb05a62c8dd5f781657db5955","author":{"name":"Don Brady","email":"b4a4b26374d158b4daa43c4a7aab3beb71db6862@delphix.com"},"message":"Add libzutil for libzfs or libzpool consumers\n\nAdds a libzutil for utility functions that are common to libzfs and\r\nlibzpool consumers (most of what was in libzfs_import.c).  This\r\nremoves the need for utilities to link against both libzpool and\r\nlibzfs.\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Don Brady <don.brady@delphix.com>\r\nCloses #8050","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/e89f1295d4faa88bb05a62c8dd5f781657db5955"},{"sha":"09b85f2dedfae2f15fed3140ffb3246bef36da97","author":{"name":"Brian Behlendorf","email":"32d50ad1e384dcb9a5bf7e40fa470df5dabaf348@llnl.gov"},"message":"ztest: reduce gangblock creation\n\nIn order to validate the gang block code ztest is configured to\r\nartificially force a fraction of large blocks to be written as\r\ngang blocks.  The default setting chosen for this was to\r\nwrite 25% of all blocks 32k or larger using gang blocks.\r\n\r\nThe confluence of an unrealistically large number of gang blocks,\r\nthe aggressive fault injection done by ztest, and the split\r\nsegment reconstruction logic introduced by device removal has\r\nresulted in the following type of failure:\r\n\r\n  zdb -bccsv -G -d ... exit code 3\r\n\r\nSpecifically, zdb was unable to open the pool because it was\r\nunable to reconstruct a damaged block.  Manual investigation\r\nof multiple failures clearly showed that the block could be\r\nreconstructed.  However, due to the large number of damaged\r\nsegments (>35) it could not be done in the allotted time.\r\n\r\nFurthermore, the large number of gang blocks was determined\r\nto be the reason for the unrealistically large number of\r\ndamaged segments.  In order to make this situation less\r\nlikely, this change both increases the forced gang block\r\nsize to 64k and reduces the frequency to 3% of blocks.\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Tom Caputi <tcaputi@datto.com>\r\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nCloses #8080","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/09b85f2dedfae2f15fed3140ffb3246bef36da97"},{"sha":"f0f978654581cea4cc042e7f185e1843d727acdb","author":{"name":"LOLi","email":"380e661881ad122dc4f68810db3326779727dc99@users.noreply.github.com"},"message":"zpool: bogus error for invalid dedupditto value\n\nWhen provided with an invalid 'dedupditto' value zpool prints\r\na misleading error message:\r\n\r\n    $ sudo zpool set dedupditto=99 pp\r\n    cannot set property for 'pp': property 'dedupditto'(14) not defined\r\n\r\nFix this by printing a meaningful error description for unsupported\r\n'dedupditto' values.\r\n\r\nReviewed-by: Olaf Faaland <faaland1@llnl.gov>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: loli10K <ezomori.nozomu@gmail.com>\r\nCloses #8079","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/f0f978654581cea4cc042e7f185e1843d727acdb"},{"sha":"95692927f28479c2d3cf0c1e60ad278c8748377c","author":{"name":"Don Brady","email":"b4a4b26374d158b4daa43c4a7aab3beb71db6862@delphix.com"},"message":"Fix libudev dependency in libzutil\n\nZFS should be able to build without libudev installed. The recent\r\nchange for libzutil inadvertently broke that.  Make the libudev code\r\nconditional in zutil_import.c to resolve the build failure.\r\n\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nReviewed-by: Tony Hutter <hutter2@llnl.gov>\r\nSigned-off-by: Don Brady <don.brady@delphix.com>\r\nCloses #8097","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/95692927f28479c2d3cf0c1e60ad278c8748377c"},{"sha":"4021ba4cfaa861602ae75d45713224d5a7866004","author":{"name":"Tom Caputi","email":"7c7cd52a3a65d08f1a75822ba72d5af779dc7c2f@datto.com"},"message":"Make vdev_set_deferred_resilver() recursive\n\nvdev_clear() can call vdev_set_deferred_resilver() with a\r\nnon-leaf vdev to setup a deferred resilver. However, this\r\nfunction is currently written to only handle leaf vdevs.\r\nThis bug was introduced with deferred resilvers in 80a91e74.\r\nThis patch makes this function recursive so that it can find\r\nappropriate vdevs to resilver and set vdev_resilver_deferred\r\non them.\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Tom Caputi <tcaputi@datto.com>\r\nIssue #7732\r\nCloses #8082","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/4021ba4cfaa861602ae75d45713224d5a7866004"},{"sha":"ac53e50f799b39d9cd04e5ecc0a6552c9490c84a","author":{"name":"Tom Caputi","email":"7c7cd52a3a65d08f1a75822ba72d5af779dc7c2f@datto.com"},"message":"Fix vdev removal finishing race\n\nThis patch fixes a race condition where the end of\r\nvdev_remove_replace_with_indirect(), which holds\r\nsvr_lock, would race against spa_vdev_removal_destroy(),\r\nwhich destroys the same lock and is called asynchronously\r\nvia dsl_sync_task_nowait().\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Tom Caputi <tcaputi@datto.com>\r\nIssue #6900 \r\nCloses #8083","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/ac53e50f799b39d9cd04e5ecc0a6552c9490c84a"},{"sha":"f44ad9297da6e638482232636e9d63302b96f7e9","author":{"name":"Tom Caputi","email":"7c7cd52a3a65d08f1a75822ba72d5af779dc7c2f@datto.com"},"message":"Replay logs before starting ztest workers\n\nThis patch ensures that logs are replayed on all datasets prior\r\nto starting ztest workers. This ensures that the call to\r\nvdev_offline() a log device in ztest_fault_inject() will not fail\r\ndue to the log device being required for replay.\r\n\r\nThis patch also fixes a small issue found during testing where\r\nspa_keystore_load_wkey() does not check that the dataset specified\r\nis an encryption root. This check was present in libzfs, however.\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Tom Caputi <tcaputi@datto.com>\r\nCloses #8084","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/f44ad9297da6e638482232636e9d63302b96f7e9"},{"sha":"fde25c0a87ea7e2d4438485cd55646a91b99c4b4","author":{"name":"Tom Caputi","email":"7c7cd52a3a65d08f1a75822ba72d5af779dc7c2f@datto.com"},"message":"Fix dirtying vdev config on with RO spa\n\nThis patch simply corrects an issue where vdev_dtl_reassess()\r\ncould attempt to dirty the vdev config even when the spa was\r\nnot elligable for writing.\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Tom Caputi <tcaputi@datto.com>\r\nCloses #8085","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/fde25c0a87ea7e2d4438485cd55646a91b99c4b4"},{"sha":"20eb30d08e24c2240a5d66e2e62893e784f586af","author":{"name":"Tom Caputi","email":"7c7cd52a3a65d08f1a75822ba72d5af779dc7c2f@datto.com"},"message":"Fix divide by zero during indirect split damage\n\nThis patch simply ensures that vdev_indirect_splits_damage()\r\ncannot hit a divide by zero exception if a split has no\r\nchildren with valid data. The normal reconstruction code\r\npath in vdev_indirect_reconstruct_io_done() already has this\r\ncheck.\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Tom Caputi <tcaputi@datto.com>\r\nCloses #8086","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/20eb30d08e24c2240a5d66e2e62893e784f586af"},{"sha":"a2d88f778a31111fcf98ecf3a690888bab07055e","author":{"name":"Tom Caputi","email":"7c7cd52a3a65d08f1a75822ba72d5af779dc7c2f@datto.com"},"message":"Fix !zilog_is_dirty() assert during ztest\n\nztest occasionally hits an assert that !zilog_is_dirty() during\r\nzil_close(). This is caused by an interaction between 2 threads.\r\nFirst, ztest_run() waits for each test thread to complete and\r\ncloses the associated dataset as soon as the thread joins. At\r\nthe same time, the ztest_vdev_add_remove() test may attempt to\r\nremove the slog, which will open, dirty, and reset the logs on\r\nevery dataset in the pool (including those of other threads).\r\nThis patch simply ensures that we always join all of the test\r\nthreads before closing any datasets.\r\n\r\nReviewed-by: Matthew Ahrens <mahrens@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Tom Caputi <tcaputi@datto.com>\r\nCloses #8094","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/a2d88f778a31111fcf98ecf3a690888bab07055e"},{"sha":"d7bda38c762f7611f7f3588f762cd40dcd31a1c6","author":{"name":"Tony Hutter","email":"6daa96ee87b2609f2e9bb829e82343800b09fd00@llnl.gov"},"message":"Add BuildRequires gcc, make, elfutils-libelf-devel\n\nThis adds a BuildRequires for gcc, make, and elfutils-libelf-devel\r\ninto our spec files.  gcc has been a packaging requirement for\r\nawhile now:\r\n\r\nhttps://fedoraproject.org/wiki/Packaging:C_and_C%2B%2B\r\n\r\nThese additional BuildRequires allow us to mock build in\r\nFedora 29.\r\n\r\nReviewed-by: Neal Gompa <ngompa@datto.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by:  Tony Hutter <hutter2@llnl.gov>\r\nCloses #8095\r\nCloses #8102","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/d7bda38c762f7611f7f3588f762cd40dcd31a1c6"},{"sha":"c2bcfa71f40beb87d2a13f42193487e31f077137","author":{"name":"Paul Zuchowski","email":"1e078106fca44e97b3c018aaa947030e333ae9e2@users.noreply.github.com"},"message":"ZTS: Fix test zfs_mount_006_pos\n\nFor Linux, place a file in the mount point folder so it will be\r\nconsidered \"busy\".  Fix the while loop so it doesn't rm in\r\ndirectories above the testdir.  Add Linux-specific code to test\r\noverlay on|off.\r\n\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Paul Zuchowski <pzuchowski@datto.com>\r\nCloses #4990 \r\nCloses #8081","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/c2bcfa71f40beb87d2a13f42193487e31f077137"},{"sha":"d8244d34bdc3e7b864e0152b55477fa61dae3e90","author":{"name":"Tom Caputi","email":"7c7cd52a3a65d08f1a75822ba72d5af779dc7c2f@datto.com"},"message":"ZTS: Fix and reenable zfs_rename tests\n\nzfs_rename_006_pos has been flaky in the past because it was\r\nmissing a call to block_device_wait to ensure the zvols it creates\r\nare present before running dd. Whenever this this happened,\r\nzfs_rename_009_neg would also fail because the first test would\r\nleak a zvol clone that it did not know how to clean up. This patch\r\nfixes the root cause and reenables the test. It also fixes some\r\nminor grammar errors.\r\n\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: Tom Caputi <tcaputi@datto.com>\r\nCloses #5647 \r\nCloses #5648 \r\nCloses #8088","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/d8244d34bdc3e7b864e0152b55477fa61dae3e90"},{"sha":"877d925a9e816337bb62ee61d564118db0181477","author":{"name":"George Melikov","email":"1d6e1cf70ec6f9ab28d3ea4b27a49a77654d370e@gmelikov.ru"},"message":"Update zfs_admin_snapshot value (disabled)\n\nIt's disabled by default, update code and tests to reflect\r\nthe documentation.\r\n\r\nMinor cleanup in delegate_common.kshlib.\r\n\r\nReviewed-by: Gregor Kopka <gregor@kopka.net>\r\nReviewed-by: John Kennedy <john.kennedy@delphix.com>\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nSigned-off-by: George Melikov <mail@gmelikov.ru>\r\nCloses #7835 \r\nCloses #8045","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/877d925a9e816337bb62ee61d564118db0181477"},{"sha":"ad796b8a3b2565bcd9c7460b7bf9154e4850ca93","author":{"name":"Tony Hutter","email":"6daa96ee87b2609f2e9bb829e82343800b09fd00@llnl.gov"},"message":"Add zpool status -s (slow I/Os) and -p (parseable)\n\nThis patch adds a new slow I/Os (-s) column to zpool status to show the\r\nnumber of VDEV slow I/Os. This is the number of I/Os that didn't\r\ncomplete in zio_slow_io_ms milliseconds. It also adds a new parsable\r\n(-p) flag to display exact values.\r\n\r\n \tNAME         STATE     READ WRITE CKSUM  SLOW\r\n \ttestpool     ONLINE       0     0     0     -\r\n\t  mirror-0   ONLINE       0     0     0     -\r\n \t    loop0    ONLINE       0     0     0    20\r\n \t    loop1    ONLINE       0     0     0     0\r\n\r\nReviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>\r\nReviewed by: Matthew Ahrens <mahrens@delphix.com>\r\nSigned-off-by: Tony Hutter <hutter2@llnl.gov>\r\nCloses #7756\r\nCloses #6885","distinct":false,"url":"https://api.github.com/repos/datto/zfs/commits/ad796b8a3b2565bcd9c7460b7bf9154e4850ca93"}]}
{"actor":{"display_login":"alek-p"}}