{"push_id":1063160946,"size":15,"distinct_size":15,"ref":"refs/heads/master","head":"d44122ecd0fa62d20762bdd8f214f077cb8e011b","before":"42bb626f7ebc9197d2943b897a99e127315275ab","commits":[{"sha":"af74e865c40d00c12e5fd0c556d7812305930b24","author":{"email":"c267753bb5aaf784510feb7240567c0b0175c99a@virtuozzo.com","name":"Pavel Butsykin"},"message":"iotests: fix the broken 026.nocache output\n\nThis patch fixes longstanding issue with 026 iotest. Unfortunately,\nthis test contains 2 versions of the correct output, one for cached\nwrites and one for non-cached ones. People tends to fix only one\nversion of output of the test and thus noncached version becomes\nbroken. Unfortunately, it is default in tests/check-block.sh\n\nThe following problematic commits were made:\n    commit 3b5e14c76a6bb142bf250ddf99e24a0ac8c7bc12\n    Author: Max Reitz <mreitz@redhat.com>\n    Date:   Tue Dec 2 18:32:51 2014 +0100\n    qcow2: Flushing the caches in qcow2_close may fail\n\n    commit a069e2f1372a0a823ab506fc019852a2a652aa54\n    Author: John Snow <jsnow@redhat.com>\n    Date:   Fri Feb 6 16:26:17 2015 -0500\n    blkdebug: fix \"once\" rule\n\n    commit b106ad9185f35fc4ad669555ad0e79e276083bd7\n    Author: Kevin Wolf <kwolf@redhat.com>\n    Date:   Fri Mar 28 18:06:31 2014 +0100\n    qcow2: Don't rely on free_cluster_index in alloc_refcount_block()\n\nSigned-off-by: Pavel Butsykin <pbutsykin@virtuozzo.com>\nSigned-off-by: Denis V. Lunev <den@openvz.org>\nCC: Max Reitz <mreitz@redhat.com>\nCC: John Snow <jsnow@redhat.com>\nCC: Kevin Wolf <kwolf@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/af74e865c40d00c12e5fd0c556d7812305930b24"},{"sha":"143605a200f9d418d62d25d91b1477b1d91c0b18","author":{"email":"bb938cf255e055ff3507f2627d214e8e62118fcf@redhat.com","name":"Daniel P. Berrange"},"message":"qemu-img: fix formatting of error message\n\nThe error_reportf_err() will not automatically append a\n': ' before adding its suffix, so we must include that\nin the message we pass it, otherwise we get a badly\nformatted message lacking whitespace:\n\nqemu-img: Could not open 'driver=nbd,host=127.0.0.1,port=6666,tls-creds=tls0'Failed to connect socket: Connection refused\n\nSigned-off-by: Daniel P. Berrange <berrange@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/143605a200f9d418d62d25d91b1477b1d91c0b18"},{"sha":"c229708848cc77b0277e7260f7e739d441c1d086","author":{"email":"bb938cf255e055ff3507f2627d214e8e62118fcf@redhat.com","name":"Daniel P. Berrange"},"message":"block: initialize qcrypto API at startup\n\nAny programs which call the qcrypto APIs should ensure that\nqcrypto_init() has been called before anything else which\ncan use crypto. Essentially this means right at the start\nof the main method before initializing anything else.\n\nThis is important because some versions of gnutls/gcrypt\nrequire explicit initialization before use.\n\nSigned-off-by: Daniel P. Berrange <berrange@redhat.com>\nReviewed-by: Alex Bligh <alex@alex.org.uk>\nTested-by: Alex Bligh <alex@alex.org.uk>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/c229708848cc77b0277e7260f7e739d441c1d086"},{"sha":"1fd06db03da73db868e54b838bb5ea02c10eea6c","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"iotests: Make 150 use qemu-img map instead of du\n\nThe actual on-disk size of a file does not only depend on factors qemu\ncan control. Thus, we should not depend on this to determine whether a\nfile has indeed been fully allocated. Instead, use qemu-img map and hope\nthat if an area is referenced, it is indeed allocated, too.\n\nAlso, limit the supported image formats to raw and qcow2 because the\nactual qemu-img map output may depend on the image format.\n\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nTested-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/1fd06db03da73db868e54b838bb5ea02c10eea6c"},{"sha":"40a99aace3f9ffd65af07a89c381db3550b22316","author":{"email":"69d3ebcf5ee4e7f05a50dc23cd96655b730804b4@redhat.com","name":"Paolo Bonzini"},"message":"vpc: fix return value check for blk_pwrite\n\nbdrv_pwrite_sync used to return zero or negative error, while blk_pwrite returns\nthe number of written bytes when successful.  This caused VPC image creation\nto fail spectacularly: it wrote the first 512 bytes, and then exited immediately\nbecause of the non-zero answer from blk_pwrite.  But the truly spectacular part\nis that it returns a positive value (the 512 that blk_pwrite returned) causing\neveryone to believe that it succeeded.\n\nThis fixes qemu-iotests with vpc format.\n\nFixes: b8f45cdf7827e39f9a1e6cc446f5972cc6144237\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/40a99aace3f9ffd65af07a89c381db3550b22316"},{"sha":"4e876bcf2bdb3a7353df92d19bfec0afd1650bc4","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"qcow2: Prevent backing file names longer than 1023\n\nWe reject backing file names with a length of more than 1023 characters\nwhen opening a qcow2 file, so we should not produce such files\nourselves.\n\nCc: qemu-stable@nongnu.org\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/4e876bcf2bdb3a7353df92d19bfec0afd1650bc4"},{"sha":"c4189d85bcdaa27a9ec892146a652092fec704d0","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"MAINTAINERS: Block layer core, qcow2 and blkdebug\n\nAs agreed with Kevin and already practiced for a while, I am adding\nmyself as co-maintainer of the block layer core, qcow2 and blkdebug.\n\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/c4189d85bcdaa27a9ec892146a652092fec704d0"},{"sha":"0145b4e130994fab778b38a490e7d72ffb740589","author":{"email":"be4224c72adedf72d459320ee497e22a37089288@linux.vnet.ibm.com","name":"Sascha Silbe"},"message":"qemu-iotests: check: don't place files with predictable names in /tmp\n\nPlacing files with predictable or even hard-coded names in /tmp is a\nsecurity risk and can prevent or disturb operation on a multi-user\nmachine. Place them inside the \"scratch\" directory instead, as we\nalready do for most other test-related files.\n\nSigned-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nReviewed-by: Bo Tu <tubo@linux.vnet.ibm.com>\nMessage-id: 1459848109-29756-2-git-send-email-silbe@linux.vnet.ibm.com\nReviewed-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/0145b4e130994fab778b38a490e7d72ffb740589"},{"sha":"1759386b7cfdd36f11cb4228d2237468814b0989","author":{"email":"be4224c72adedf72d459320ee497e22a37089288@linux.vnet.ibm.com","name":"Sascha Silbe"},"message":"qemu-iotests: fix 051 on non-PC architectures\n\nCommit 61de4c68 [block: Remove BDRV_O_CACHE_WB] updated the reference\noutput for PCs, but neglected to do the same for the generic reference\noutput file. Fix 051 on all non-PC architectures by applying the same\nchange to the generic output file.\n\nFixes: 61de4c68 (\"block: Remove BDRV_O_CACHE_WB\")\nSigned-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nReviewed-by: Bo Tu <tubo@linux.vnet.ibm.com>\nMessage-id: 1459848109-29756-3-git-send-email-silbe@linux.vnet.ibm.com\nReviewed-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/1759386b7cfdd36f11cb4228d2237468814b0989"},{"sha":"c1c71e49bcf8a0e6f5ba2a41845678d4954c8106","author":{"email":"be4224c72adedf72d459320ee497e22a37089288@linux.vnet.ibm.com","name":"Sascha Silbe"},"message":"qemu-iotests: iotests.VM: remove qtest socket on error\n\nOn error, VM.launch() cleaned up the monitor unix socket, but left the\nqtest unix socket behind. This caused the remaining sub-tests to fail\nwith EADDRINUSE:\n\n+======================================================================\n+ERROR: testQuorum (__main__.TestFifoQuorumEvents)\n+----------------------------------------------------------------------\n+Traceback (most recent call last):\n+  File \"148\", line 63, in setUp\n+    self.vm.launch()\n+  File \"/home6/silbe/qemu/tests/qemu-iotests/iotests.py\", line 247, in launch\n+    self._qmp.accept()\n+  File \"/home6/silbe/qemu/tests/qemu-iotests/../../scripts/qmp/qmp.py\", line 141, in accept\n+    return self.__negotiate_capabilities()\n+  File \"/home6/silbe/qemu/tests/qemu-iotests/../../scripts/qmp/qmp.py\", line 57, in __negotiate_capabilities\n+    raise QMPConnectError\n+QMPConnectError\n+\n+======================================================================\n+ERROR: testQuorum (__main__.TestQuorumEvents)\n+----------------------------------------------------------------------\n+Traceback (most recent call last):\n+  File \"148\", line 63, in setUp\n+    self.vm.launch()\n+  File \"/home6/silbe/qemu/tests/qemu-iotests/iotests.py\", line 244, in launch\n+    self._qtest = qtest.QEMUQtestProtocol(self._qtest_path, server=True)\n+  File \"/home6/silbe/qemu/tests/qemu-iotests/../../scripts/qtest.py\", line 33, in __init__\n+    self._sock.bind(self._address)\n+  File \"/usr/lib64/python2.7/socket.py\", line 224, in meth\n+    return getattr(self._sock,name)(*args)\n+error: [Errno 98] Address already in use\n\nFix this by cleaning up both the monitor socket and the qtest socket iff\nthey exist.\n\nSigned-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nReviewed-by: Bo Tu <tubo@linux.vnet.ibm.com>\nMessage-id: 1459848109-29756-4-git-send-email-silbe@linux.vnet.ibm.com\nReviewed-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/c1c71e49bcf8a0e6f5ba2a41845678d4954c8106"},{"sha":"3f647b510f1f68e549425cd9671b1aa63c93ec3c","author":{"email":"be4224c72adedf72d459320ee497e22a37089288@linux.vnet.ibm.com","name":"Sascha Silbe"},"message":"qemu-iotests: 148: properly skip test if quorum support is missing\n\nqemu-iotests test case 148 already had some code for skipping the test\nif quorum support is missing, but it didn't work in all\ncases. TestQuorumEvents.setUp() gets run before the actual test class\n(which contains the skipping code) and tries to start qemu with a drive\nusing the quorum driver. For some reason this works fine when using\nqcow2, but fails for raw.\n\nAs the entire test case requires quorum, just check for availability\nbefore even starting the test suite. Introduce a verify_quorum()\nfunction in iotests.py for this purpose so future test cases can make\nuse of it.\n\nSigned-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nReviewed-by: Bo Tu <tubo@linux.vnet.ibm.com>\nMessage-id: 1459848109-29756-5-git-send-email-silbe@linux.vnet.ibm.com\nReviewed-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/3f647b510f1f68e549425cd9671b1aa63c93ec3c"},{"sha":"9bf8027dde31923b08a09d5cf2ddd47efe9146e8","author":{"email":"be4224c72adedf72d459320ee497e22a37089288@linux.vnet.ibm.com","name":"Sascha Silbe"},"message":"qemu-iotests: 068: don't require KVM\n\nNone of the other test cases explicitly enable KVM and there's no\nobvious reason for 068 to require it. Drop this so all test cases can be\nexecuted in environments where KVM is not available (e.g. because the\nuser doesn't have sufficient permissions to access /dev/kvm).\n\nSigned-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nReviewed-by: Bo Tu <tubo@linux.vnet.ibm.com>\nMessage-id: 1459848109-29756-6-git-send-email-silbe@linux.vnet.ibm.com\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/9bf8027dde31923b08a09d5cf2ddd47efe9146e8"},{"sha":"3ef3dcef563e231a13543264975b9ef75080f9b3","author":{"email":"be4224c72adedf72d459320ee497e22a37089288@linux.vnet.ibm.com","name":"Sascha Silbe"},"message":"qemu-iotests: iotests.py: get rid of __all__\n\nThe __all__ list contained a typo for as long as the iotests module\nexisted. That typo prevented \"from iotests import *\" (which is the\nonly case where iotests.__all__ is used at all) from ever working.\n\nThe names used by iotests are highly prone to name collisions, so\nimporting them all unconditionally is a bad idea anyway. Since __all__\nis not adding any value, let's just get rid of it.\n\nFixes: f345cfd0 (\"qemu-iotests: add iotests Python module\")\nSigned-off-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nReviewed-by: Bo Tu <tubo@linux.vnet.ibm.com>\nMessage-id: 1459848109-29756-8-git-send-email-silbe@linux.vnet.ibm.com\nReviewed-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/3ef3dcef563e231a13543264975b9ef75080f9b3"},{"sha":"5158ac58306b3e8644fc020730f651fe74aa6674","author":{"email":"b75d81c03bd01637475de8fdfe57bd6e8b41a08d@redhat.com","name":"Kevin Wolf"},"message":"Merge remote-tracking branch 'mreitz/tags/pull-block-for-kevin-2016-04-12' into queue-block\n\nBlock patches for 2.6-rc2.\n\n# gpg: Signature made Tue Apr 12 18:08:20 2016 CEST using RSA key ID E838ACAD\n# gpg: Good signature from \"Max Reitz <mreitz@redhat.com>\"\n\n* mreitz/tags/pull-block-for-kevin-2016-04-12:\n  qemu-iotests: iotests.py: get rid of __all__\n  qemu-iotests: 068: don't require KVM\n  qemu-iotests: 148: properly skip test if quorum support is missing\n  qemu-iotests: iotests.VM: remove qtest socket on error\n  qemu-iotests: fix 051 on non-PC architectures\n  qemu-iotests: check: don't place files with predictable names in /tmp\n\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/5158ac58306b3e8644fc020730f651fe74aa6674"},{"sha":"d44122ecd0fa62d20762bdd8f214f077cb8e011b","author":{"email":"313afa09ce093b19ae1dd74f57c1ebd94508584c@linaro.org","name":"Peter Maydell"},"message":"Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging\n\nBlock layer patches for 2.6\n\n# gpg: Signature made Tue 12 Apr 2016 17:10:29 BST using RSA key ID C88F2FD6\n# gpg: Good signature from \"Kevin Wolf <kwolf@redhat.com>\"\n\n* remotes/kevin/tags/for-upstream:\n  qemu-iotests: iotests.py: get rid of __all__\n  qemu-iotests: 068: don't require KVM\n  qemu-iotests: 148: properly skip test if quorum support is missing\n  qemu-iotests: iotests.VM: remove qtest socket on error\n  qemu-iotests: fix 051 on non-PC architectures\n  qemu-iotests: check: don't place files with predictable names in /tmp\n  MAINTAINERS: Block layer core, qcow2 and blkdebug\n  qcow2: Prevent backing file names longer than 1023\n  vpc: fix return value check for blk_pwrite\n  iotests: Make 150 use qemu-img map instead of du\n  block: initialize qcrypto API at startup\n  qemu-img: fix formatting of error message\n  iotests: fix the broken 026.nocache output\n\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>","distinct":true,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/d44122ecd0fa62d20762bdd8f214f077cb8e011b"}]}