GitHub PushEvent at 2014-03-20 22:08:50
Repository: zfs
Actor: behlendorf
Payload: {"shas":[["00fcdee1f88f5f1413913f26ebd1cc43107e82e0","3b55b1b4fdd6c57a0901678680641ec226477e5a@gmail.com","Fix regression introduced in port of Illumos #3744\n\nRemove the redundant call to zfs_unmount_snap() which was being\ndone after char array was freed,\n\nThis fixes an upstream regression that was introduced in commit\nzfsonlinux/zfs@d09f25dc66774959499a89bf3680d09c6e541ce8, which\nported the Illumos 3744 changes.\n\nSigned-off-by: Andrey Vesnovaty <andrey.vesnovaty@gmail.com>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nSigned-off-by: Richard Yao <ryao@gentoo.org>\nCloses #2156","Andrey Vesnovaty",true],["d9119bd66df789cbd43d0f6aef4cd43b6baba58c","32d50ad1e384dcb9a5bf7e40fa470df5dabaf348@llnl.gov","Revert \"sighandler() should take 2 arguments\"\n\nThis reverts commit 0bb89b6c594259829556f6dea5a89e722f214fd3 in\nfavor of a cleaner implementation.\n\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nIssue #2182","Brian Behlendorf",true],["312f82ce65f37ed60f440194623fe38cdb1f601d","98cf26a406474b361b5ebeb54c5068a192c4d45f@intel.com","sighandler() should take 2 arguments\n\nStopping arcstat.py with ^C always ends up with error:\nTypeError: sighandler() takes no arguments (2 given)\n\nSince no special signal handling was done in sighandler(),\nit's simpler to just set SIGINT handler to SIG_DFL, which\nterminates the script.\n\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nSigned-off-by: Isaac Huang <he.huang@intel.com>\nCloses #2179","Isaac Huang",true],["8959b29e31975a281acc411e9f60203f09716e79","c64f7f1e6060e851acfe9b5bdea00a17cb81871c@gentoo.org","Assert alignment in umem_alloc_aligned\n\nValgrind suggests that the address we are returning is not properly\naligned, so lets add an assertion.\n\n==87740==  Address 0x1012a22a is 554 bytes inside a block of size 4,096\nalloc'd\n==87740==    at 0x4C2BBA0: memalign (in\n/usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)\n==87740==    by 0x4C2BCC7: posix_memalign (in\n/usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)\n==87740==    by 0x52FA845: zio_buf_alloc (umem.h:101)\n==87740==    by 0x52F6226: zil_alloc_lwb (zil.c:463)\n==87740==    by 0x52F8559: zil_commit (zil.c:566)\n==87740==    by 0x40611D: ztest_freeze (ztest.c:5909)\n==87740==    by 0x4066A7: ztest_init (ztest.c:6048)\n==87740==    by 0x407AF4: main (ztest.c:6226)\n\nSigned-off-by: Richard Yao <ryao@gentoo.org>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2174","Richard Yao",true],["26b42f3f9d03f85cc7966dc2fe4dfe9216601b0e","c64f7f1e6060e851acfe9b5bdea00a17cb81871c@gentoo.org","Implement -t option to zpool import for temporary pool names\n\nOriginally, users had to handle spa namespace collisions by either\nexporting the already imported pool or by specifying a new name for the\npool with a conflicting name. In the case of root pools from virtual\nguests, neither approach to collision resolution is reasonable. This is\naddressed by extending the new name syntax with a -t option to specify\nthat the new name is temporary. When specified, this sets an internal\nflag that is passed into the kernel to tell it that all label updates\nshould refer to the name used in the original label. Consequently, the\noriginal pool name will be retained on export.\n\nSigned-off-by: Richard Yao <ryao@gentoo.org>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2189","Richard Yao",true],["a15dac42df202bf9e6825a62b81a36ad449ef53f","1d34d2e6c70ee806ecb1ebb590a2bf31fd995e1f@gmail.com","config: compile test rather than run test\n\nWhen testing compiler flags, we only need to do compile test. Otherwise,\nconfigure will fail with \"configure: error: cannot run test program while\ncross compiling\" when cross compiling.\n\nSigned-off-by: Chunwei Chen <tuxoko@gmail.com>\nSigned-off-by: Brian Behlendorf <behlendorf1@llnl.gov>\nCloses #2191","Chunwei Chen",true]],"size":6,"ref":"refs/heads/master","head":"a15dac42df202bf9e6825a62b81a36ad449ef53f"}