{"push_id":783996391,"size":77,"distinct_size":0,"ref":"refs/heads/<1441612292-14845-1-git-send-email-pbonzini@redhat.com>","head":"53a23ba4bb9d135fda96246227abfe10371c9d5f","before":"1c905f96b88f67a5e2b0714e52f06175e4c458e2","commits":[{"sha":"e166b4148208656635ea2fe39df8b1e875a34fb8","author":{"email":"7fe71e81f2d2cc030abec5d7fec63059ccd2f2eb@linux.vnet.ibm.com","name":"Bo Tu"},"message":"qemu-iotests: qemu machine type support\n\nThis patch adds qemu machine type support to the io test suite.\nBased on the qemu default machine type and alias of the default machine\ntype the reference output file can now vary from the default to a\nmachine specific output file if necessary. When using a machine specific\nreference file if the default machine has an alias then use the alias as the output\nfile name otherwise use the default machine name as the output file name.\n\nReviewed-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Michael Mueller <mimu@linux.vnet.ibm.com>\nReviewed-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nSigned-off-by: Xiao Guang Chen <chenxg@linux.vnet.ibm.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/e166b4148208656635ea2fe39df8b1e875a34fb8"},{"sha":"2711fd33a4b18c5e35a6f7efe57b5d868def829e","author":{"email":"7fe71e81f2d2cc030abec5d7fec63059ccd2f2eb@linux.vnet.ibm.com","name":"Bo Tu"},"message":"qemu-iotests: disable default qemu devices for cross-platform compatibility\n\nThis patch fixes an io test suite issue that was introduced with the\ncommit c88930a6866e74953e931ae749781e98e486e5c8 'qemu-char: Permit only\na single \"stdio\" character device'. The option supresses the creation of\ndefault devices such as the floopy and cdrom. Output files for test case\n067, 071, 081 and 087 need to be updated to accommodate this change.\nUse virtio-blk instead of virtio-blk-pci as the device driver for test\ncase 067. For virtio-blk-pci is the same with virtio-blk as device\ndriver but other platform such as s390 may not recognize the virtio-blk-pci.\n\nThe default devices differ across machines. As the qemu output often\ncontains these devices (or events for them, like opening a CD tray on\nreset), the reference output currently is rather machine-specific.\n\nAll existing qemu tests explicitly configure the devices they're working\nwith, so just pass -nodefaults to qemu by default to disable the default\ndevices. Update the reference outputs accordingly.\n\nReviewed-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Michael Mueller <mimu@linux.vnet.ibm.com>\nReviewed-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nSigned-off-by: Xiao Guang Chen <chenxg@linux.vnet.ibm.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/2711fd33a4b18c5e35a6f7efe57b5d868def829e"},{"sha":"d8683155fa76cabff112271771e43e21034ff2ba","author":{"email":"7fe71e81f2d2cc030abec5d7fec63059ccd2f2eb@linux.vnet.ibm.com","name":"Bo Tu"},"message":"qemu-iotests: s390x: fix test 041 and 055\n\nThere is no 'ide-cd' device defined on non-pc platform, so\ntest_medium_not_found() test should be skipped.\n\nReviewed-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Michael Mueller <mimu@linux.vnet.ibm.com>\nReviewed-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nSigned-off-by: Xiao Guang Chen <chenxg@linux.vnet.ibm.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/d8683155fa76cabff112271771e43e21034ff2ba"},{"sha":"212789925efffe1c552b114321ee74081a7efb03","author":{"email":"7fe71e81f2d2cc030abec5d7fec63059ccd2f2eb@linux.vnet.ibm.com","name":"Bo Tu"},"message":"qemu-iotests: s390x: fix test 049, reject negative sizes in QemuOpts\n\nwhen creating an image qemu-img enable us specifying the size of the\nimage using -o size=xx options. But when we specify an invalid size\nsuch as a negtive size then different platform gives different result.\n\nparse_option_size() function in util/qemu-option.c will be called to\nparse the size, a cast was called in the function to cast the input\n(saved as a double in the function) size to an unsigned int64 value,\nwhen the input is a negtive value or exceeds the maximum of uint64, then\nthe result is undefined.\n\nAccording to C99 6.3.1.4, the result of converting a floating point\nnumber to an integer that cannot represent the (integer part of) number\nis undefined.  And sure enough the results are different on x86 and\ns390.\n\nC99 Language spec 6.3.1.4 Real floating and integers:\nthe result of this assignment/cast is undefined if the float is not\nin the open interval (-1, U<type>_MAX+1).\n\nReviewed-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Sascha Silbe <silbe@linux.vnet.ibm.com>\nSigned-off-by: Bo Tu <tubo@linux.vnet.ibm.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/212789925efffe1c552b114321ee74081a7efb03"},{"sha":"137a905fdf43880c077438d57ef2d319569da9eb","author":{"email":"7fe71e81f2d2cc030abec5d7fec63059ccd2f2eb@linux.vnet.ibm.com","name":"Bo Tu"},"message":"qemu-iotests: s390x: fix test 130\n\nThe default device id of hard disk on the s390 platform is \"virtio0\"\nwhich differs to the \"ide0-hd0\" for the x86 platform. Setting id in\nthe drive definition, ie:\"qemu -drive id=testdisk\", will be the same\non all platforms.\n\nReviewed-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Bo Tu <tubo@linux.vnet.ibm.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/137a905fdf43880c077438d57ef2d319569da9eb"},{"sha":"bdd03cdf5dc3176bc7169a1d5709303e9279fffb","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"block/raw-posix: Use raw_normalize_devicepath()\n\nThe filename given to qemu_open() in block/raw-posix.c should generally\nhave been processed by raw_normalize_devicepath(); unless we are only\nprobing (in which case the caller often checks whether the file is a\nblock device or not, and this property will be changed by\nraw_normalize_devicepath() on NetBSD) or it is about a deprecated device\n(i.e. floppy).\n\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/bdd03cdf5dc3176bc7169a1d5709303e9279fffb"},{"sha":"e814dffcc9810ed77fe99081be9751b620a894c4","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"qemu-img: Fix crash in amend invocation\n\nExample:\n$ ./qemu-img create -f qcow2 /tmp/t.qcow2 64M\n$ ./qemu-img amend -f qcow2 -o backing_file=/tmp/t.qcow2, -o help \\\n    /tmp/t.qcow2\n\nThis should not crash. This actually is tested by iotest 082, but not\ncaught due to the segmentation fault being silent (which is something\nthat needs to be fixed, too).\n\nReported-by: Dr. David Alan Gilbert <dgilbert@redhat.com>\nCc: qemu-stable <qemu-stable@nongnu.org>\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/e814dffcc9810ed77fe99081be9751b620a894c4"},{"sha":"8e4922535b6479c7a2fa6b14b0148c6ae4fcc003","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"iotests: More options for VM.add_drive()\n\nThis patch allows specifying the interface to be used for the drive, and\nmakes specifying a path optional (if the path is None, the \"file\" option\nwill be omitted, thus creating an empty drive).\n\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/8e4922535b6479c7a2fa6b14b0148c6ae4fcc003"},{"sha":"0ed82f7a096537923ef3705946f254d2f61eaf93","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"iotests: Respect -nodefaults in tests 41 and 55\n\nWhile -nodefaults is set in $QEMU_OPTIONS, this is currently (wrongly)\nignored for Python iotests. In order to be prepared for when this is\nfixed, we should explicitly add an IDE CD-ROM drive instead of relying\non it being created automatically.\n\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/0ed82f7a096537923ef3705946f254d2f61eaf93"},{"sha":"934659c460d46c948cf348822fda1d38556ed9a4","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"iotests: Do not suppress segfaults in bash tests\n\nCurrently, if a qemu/qemu-io/qemu-img/qemu-nbd invocation receives a\nsegmentation fault, that message is invisible in most cases since the\noutput is generally filtered and bash suppresses the segmentation fault\nnotice for any but the last element of a pipe.\n\nMost of the time, the test will then fail anyway because of missing\noutput, but not necessarily (as happened with test 82 recently).\n\nFix this by making the corresponding environment variables point to\nwrapper functions which execute the respective command in a subshell.\n\nGiving options to qemu/qemu-io/qemu-img and path names with spaces were\nbroken for the Python tests; this patch \"accidentally\" fixes that.\n\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/934659c460d46c948cf348822fda1d38556ed9a4"},{"sha":"2ef6093cd6990314304f2d3b18eb476ee418d73c","author":{"email":"6779b6f8ab957befe3afd965a2c13eebbdd5a946@redhat.com","name":"Max Reitz"},"message":"iotests: Warn if python subprocess is killed\n\nCurrently, if a subprocess of a python test (i.e. qemu-io, qemu-img, or\nqemu) receives a signal and is subsequently aborted, this is not logged.\n\nThis patch makes python tests always check the exit code of these\nsubprocesses, and emit a message if they have been killed.\n\nSigned-off-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/2ef6093cd6990314304f2d3b18eb476ee418d73c"},{"sha":"355ee2d0e8ca536a6278c9c763ddd2f136eace3f","author":{"email":"e9db499e13ac90573163837d2fb1fc9f85402d6d@igalia.com","name":"Alberto Garcia"},"message":"qcow2: mark the memory as no longer needed after qcow2_cache_empty()\n\nAfter having emptied the cache, the data in the cache tables is no\nlonger useful, so we can tell the kernel that we are done with it. In\nLinux this frees the resources associated with it.\n\nThe effect of this can be seen in the HMP commit operation: it moves\ndata from the top to the base image (and fills both caches), then it\nempties the top image. At this point the data in that cache is no\nlonger needed so it's just wasting memory.\n\nSigned-off-by: Alberto Garcia <berto@igalia.com>\nReviewed-by: Max Reitz <mreitz@redhat.com>\nMessage-id: 08538b098e1faf6c92496477cf9b47a20e5aacea.1438690126.git.berto@igalia.com\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/355ee2d0e8ca536a6278c9c763ddd2f136eace3f"},{"sha":"279621c046ce57de0af9e3c00663b48d3a7835ae","author":{"email":"e9db499e13ac90573163837d2fb1fc9f85402d6d@igalia.com","name":"Alberto Garcia"},"message":"qcow2: add option to clean unused cache entries after some time\n\nThis adds a new 'cache-clean-interval' option that cleans all qcow2\ncache entries that haven't been used in a certain interval, given in\nseconds.\n\nThis allows setting a large L2 cache size so it can handle scenarios\nwith lots of I/O and at the same time use little memory during periods\nof inactivity.\n\nThis feature currently relies on MADV_DONTNEED to free that memory, so\nit is not useful in systems that don't follow that behavior.\n\nSigned-off-by: Alberto Garcia <berto@igalia.com>\nReviewed-by: Max Reitz <mreitz@redhat.com>\nMessage-id: a70d12da60433df9360ada648b3f34b8f6f354ce.1438690126.git.berto@igalia.com\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/279621c046ce57de0af9e3c00663b48d3a7835ae"},{"sha":"7f65ce834accce0b7e4bc79313bacf229b957783","author":{"email":"e9db499e13ac90573163837d2fb1fc9f85402d6d@igalia.com","name":"Alberto Garcia"},"message":"docs: document how to configure the qcow2 L2/refcount caches\n\nQEMU has options to configure the size of the L2 and refcount\ncaches for the qcow2 format. However, choosing the right sizes for\na particular disk image is not a straightforward operation since\nthe ratio between the cache size and the allocated disk space is\nnot obvious and depends on the size of the cluster and the refcount\nentries.\n\nThis document attempts to give an overview of both caches and how to\nconfigure their sizes.\n\nSigned-off-by: Alberto Garcia <berto@igalia.com>\nReviewed-by: Max Reitz <mreitz@redhat.com>\nMessage-id: 55de928e139b1ba3f3d40fe9c6c88f30b1f36410.1438690126.git.berto@igalia.com\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/7f65ce834accce0b7e4bc79313bacf229b957783"},{"sha":"909c260c71d1bee7018e17034580ffd0743508db","author":{"email":"e9db499e13ac90573163837d2fb1fc9f85402d6d@igalia.com","name":"Alberto Garcia"},"message":"qcow2: reorder fields in Qcow2CachedTable to reduce padding\n\nChanging the current ordering saves 8 bytes per cache entry in x86_64.\n\nSigned-off-by: Alberto Garcia <berto@igalia.com>\nReviewed-by: Max Reitz <mreitz@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nReviewed-by: Kevin Wolf <kwolf@redhat.com>\nMessage-id: 0bd55291211df3dfb514d0e7d2031dd5c4f9f807.1438690126.git.berto@igalia.com\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/909c260c71d1bee7018e17034580ffd0743508db"},{"sha":"834cb2ada5db197a11c99142d50222945d196fc0","author":{"email":"59c7438a9275f5dd0b96a91958b493a9d005ad46@cn.fujitsu.com","name":"Wen Congyang"},"message":"quorum: validate vote threshold against num_children even if read-pattern is fifo\n\nWe need to use threshold to check if too many write operation fails.\nIf threshold is larger than num children, we always get write error\nevent even if all write operations success.\n\nSigned-off-by: Wen Congyang <wency@cn.fujitsu.com>\nMessage-id: 55962F72.3060003@cn.fujitsu.com\nReviewed-by: Alberto Garcia <berto@igalia.com>\nReviewed-by: Max Reitz <mreitz@redhat.com>\nSigned-off-by: Max Reitz <mreitz@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/834cb2ada5db197a11c99142d50222945d196fc0"},{"sha":"c804b5791d51608c0e12e4cc2b40b3d763ce796c","author":{"email":"b75d81c03bd01637475de8fdfe57bd6e8b41a08d@redhat.com","name":"Kevin Wolf"},"message":"Merge remote-tracking branch 'mreitz/tags/pull-block-for-kevin-2015-09-04' into queue-block\n\nBlock patches from 2015-08-24 until 2015-09-04.\n\n# gpg: Signature made Fri Sep  4 21:02:10 2015 CEST using RSA key ID E838ACAD\n# gpg: Good signature from \"Max Reitz <mreitz@redhat.com>\"\n\n* mreitz/tags/pull-block-for-kevin-2015-09-04:\n  quorum: validate vote threshold against num_children even if read-pattern is fifo\n  qcow2: reorder fields in Qcow2CachedTable to reduce padding\n  docs: document how to configure the qcow2 L2/refcount caches\n  qcow2: add option to clean unused cache entries after some time\n  qcow2: mark the memory as no longer needed after qcow2_cache_empty()\n\nSigned-off-by: Kevin Wolf <kwolf@redhat.com>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/c804b5791d51608c0e12e4cc2b40b3d763ce796c"},{"sha":"b45c03f585ea9bb1af76c73e82195418c294919d","author":{"email":"5d00b6b660f20fb41305186c34c40973ec5cb35b@redhat.com","name":"Markus Armbruster"},"message":"arm: Use g_new() & friends where that makes obvious sense\n\ng_new(T, n) is neater than g_malloc(sizeof(T) * n).  It's also safer,\nfor two reasons.  One, it catches multiplication overflowing size_t.\nTwo, it returns T * rather than void *, which lets the compiler catch\nmore type errors.\n\nThis commit only touches allocations with size arguments of the form\nsizeof(T).\n\nCoccinelle semantic patch:\n\n    @@\n    type T;\n    @@\n    -g_malloc(sizeof(T))\n    +g_new(T, 1)\n    @@\n    type T;\n    @@\n    -g_try_malloc(sizeof(T))\n    +g_try_new(T, 1)\n    @@\n    type T;\n    @@\n    -g_malloc0(sizeof(T))\n    +g_new0(T, 1)\n    @@\n    type T;\n    @@\n    -g_try_malloc0(sizeof(T))\n    +g_try_new0(T, 1)\n    @@\n    type T;\n    expression n;\n    @@\n    -g_malloc(sizeof(T) * (n))\n    +g_new(T, n)\n    @@\n    type T;\n    expression n;\n    @@\n    -g_try_malloc(sizeof(T) * (n))\n    +g_try_new(T, n)\n    @@\n    type T;\n    expression n;\n    @@\n    -g_malloc0(sizeof(T) * (n))\n    +g_new0(T, n)\n    @@\n    type T;\n    expression n;\n    @@\n    -g_try_malloc0(sizeof(T) * (n))\n    +g_try_new0(T, n)\n    @@\n    type T;\n    expression p, n;\n    @@\n    -g_realloc(p, sizeof(T) * (n))\n    +g_renew(T, p, n)\n    @@\n    type T;\n    expression p, n;\n    @@\n    -g_try_realloc(p, sizeof(T) * (n))\n    +g_try_renew(T, p, n)\n    @@\n    type T;\n    expression n;\n    @@\n    -(T *)g_new(T, n)\n    +g_new(T, n)\n    @@\n    type T;\n    expression n;\n    @@\n    -(T *)g_new0(T, n)\n    +g_new0(T, n)\n    @@\n    type T;\n    expression p, n;\n    @@\n    -(T *)g_renew(T, p, n)\n    +g_renew(T, p, n)\n\nSigned-off-by: Markus Armbruster <armbru@redhat.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nMessage-id: 1440524394-15640-1-git-send-email-armbru@redhat.com\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/b45c03f585ea9bb1af76c73e82195418c294919d"},{"sha":"857b55adb77004d9ec9202078b7f1f3a1a076112","author":{"email":"313afa09ce093b19ae1dd74f57c1ebd94508584c@linaro.org","name":"Peter Maydell"},"message":"target-arm/arm-semi.c: Fix broken SYS_WRITE0 via gdb\n\nA spurious trailing \"\\n\" in the gdb syscall format string used\nfor SYS_WRITE0 meant that gdb would reject the remote syscall,\nwith the effect that the output from the guest was silently dropped.\nRemove the newline so that gdb accepts the packet.\n\nCc: qemu-stable@nongnu.org\n\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/857b55adb77004d9ec9202078b7f1f3a1a076112"},{"sha":"205ace55ffff77964e50af08c99639ec47db53f6","author":{"email":"743530263aad94f95d2346cd889fa87f26863b07@linaro.org","name":"Christopher Covington"},"message":"target-arm: Improve semihosting debug prints\n\nPrint semihosting debugging information before the\ndo_arm_semihosting() call so that angel_SWIreason_ReportException,\nwhich causes the function to not return, gets the same debug prints as\nother semihosting calls. Also print out the semihosting call number.\n\nSigned-off-by: Christopher Covington <christopher.covington@linaro.org>\nReviewed-by: Peter Maydell <peter.maydell@linaro.org>\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>\nTested-by: Christopher Covington <cov@codeaurora.org>\nMessage-id: 1439483745-28752-3-git-send-email-peter.maydell@linaro.org","distinct":false,"url":"https://api.github.com/repos/famz/qemu-patchew/commits/205ace55ffff77964e50af08c99639ec47db53f6"}]}