{"push_id":1861172399,"size":115,"distinct_size":115,"ref":"refs/heads/master","head":"4871b51b9241b10f4fd8e04bbb21577886795e25","before":"49bcce4b9c11759678fd223aefb48691c4959d4f","commits":[{"sha":"a9859c90a5db200fd4f63ab2cdc973343348b9ef","author":{"name":"Eduardo Habkost","email":"1434a0c554a4542f3cf3c2e7be483467bba16117@redhat.com"},"message":"xilinx: Fix latent error handling bug\n\nAssigning directly to *errp is not valid, as errp may be null,\n&error_fatal, or &error_abort.  The !*errp conditional protects\nagainst the latter two, but we then leak @local_err.  Fortunately,\nthe qdev core always passes pointer to null, so this is \"merely\" a\nlatent bug.\n\nUse error_propagate() instead.\n\nCc: \"Edgar E. Iglesias\" <edgar.iglesias@gmail.com>\nCc: Alistair Francis <alistair.francis@xilinx.com>\nCc: Jason Wang <jasowang@redhat.com>\nCc: qemu-arm@nongnu.org\nSigned-off-by: Eduardo Habkost <ehabkost@redhat.com>\nMessage-Id: <20170608133906.12737-2-ehabkost@redhat.com>\nReviewed-by: Alistair Francis <alistair.francis@xilinx.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\n[Commit message clarified]\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/a9859c90a5db200fd4f63ab2cdc973343348b9ef"},{"sha":"57ef3f1278585fbac5f6102362d3c5e066976fdc","author":{"name":"Eduardo Habkost","email":"1434a0c554a4542f3cf3c2e7be483467bba16117@redhat.com"},"message":"block: Don't try to set *errp directly\n\nAssigning directly to *errp is not valid, as errp may be NULL,\n&error_fatal, or &error_abort.  Use error_propagate() instead.\n\nWith this, there's no need to check if errp is NULL anymore, as\nerror_propagate() and error_prepend() are able to handle that.\n\nCc: Kevin Wolf <kwolf@redhat.com>\nCc: Max Reitz <mreitz@redhat.com>\nCc: qemu-block@nongnu.org\nSigned-off-by: Eduardo Habkost <ehabkost@redhat.com>\nMessage-Id: <20170608133906.12737-3-ehabkost@redhat.com>\nReviewed-by: Alberto Garcia <berto@igalia.com>\nReviewed-by: Eric Blake <eblake@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/57ef3f1278585fbac5f6102362d3c5e066976fdc"},{"sha":"e79ea67a9785a5da4d1889b6e2bb71d03e916add","author":{"name":"Eduardo Habkost","email":"1434a0c554a4542f3cf3c2e7be483467bba16117@redhat.com"},"message":"websock: Don't try to set *errp directly\n\nAssigning directly to *errp is not valid, as errp may be NULL,\n&error_fatal, or &error_abort.  Use error_propagate() instead.\n\nCc: \"Daniel P. Berrange\" <berrange@redhat.com>\nSigned-off-by: Eduardo Habkost <ehabkost@redhat.com>\nMessage-Id: <20170608133906.12737-4-ehabkost@redhat.com>\nReviewed-by: Manos Pitsidianakis <el13635@mail.ntua.gr>\nReviewed-by: Eric Blake <eblake@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/e79ea67a9785a5da4d1889b6e2bb71d03e916add"},{"sha":"beeb175c0db29a4be13625c3e53cf674c3be9267","author":{"name":"Alistair Francis","email":"159fd3f0183a873ba8874443403d6ff506e71f24@xilinx.com"},"message":"util/qemu-error: Rename error_print_loc() to be more generic\n\nRename the error_print_loc() function in preparation for using it to\nprint warnings as well.\n\nSigned-off-by: Alistair Francis <alistair.francis@xilinx.com>\nReviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nReviewed-by: Thomas Huth <thuth@redhat.com>\nMessage-Id: <661b215695db878a0aef8401b506fb3da50e981a.1499866456.git.alistair.francis@xilinx.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/beeb175c0db29a4be13625c3e53cf674c3be9267"},{"sha":"97f40301f1dc3a3658427d1e9d964007b6f5673b","author":{"name":"Alistair Francis","email":"159fd3f0183a873ba8874443403d6ff506e71f24@xilinx.com"},"message":"error: Functions to report warnings and informational messages\n\nAdd warn_report(), warn_vreport() for reporting warnings, and\ninfo_report(), info_vreport() for informational messages.\n\nThese are implemented them with a helper function factored out of\nerror_vreport(), suitably generalized. This patch makes no changes\nto the output of the original error_report() function.\n\nSigned-off-by: Alistair Francis <alistair.francis@xilinx.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nMessage-Id: <c89e9980019f296ec9aa38d7689ac4d5c369296d.1499866456.git.alistair.francis@xilinx.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/97f40301f1dc3a3658427d1e9d964007b6f5673b"},{"sha":"3dc6f8693694a649a9c83f1e2746565b47683923","author":{"name":"Alistair Francis","email":"159fd3f0183a873ba8874443403d6ff506e71f24@xilinx.com"},"message":"Convert error_report() to warn_report()\n\nConvert all uses of error_report(\"warning:\"... to use warn_report()\ninstead. This helps standardise on a single method of printing warnings\nto the user.\n\nAll of the warnings were changed using these two commands:\n    find ./* -type f -exec sed -i \\\n      's|error_report(\".*warning[,:] |warn_report(\"|Ig' {} +\n\nIndentation fixed up manually afterwards.\n\nThe test-qdev-global-props test case was manually updated to ensure that\nthis patch passes make check (as the test cases are case sensitive).\n\nSigned-off-by: Alistair Francis <alistair.francis@xilinx.com>\nSuggested-by: Thomas Huth <thuth@redhat.com>\nCc: Jeff Cody <jcody@redhat.com>\nCc: Kevin Wolf <kwolf@redhat.com>\nCc: Max Reitz <mreitz@redhat.com>\nCc: Ronnie Sahlberg <ronniesahlberg@gmail.com>\nCc: Paolo Bonzini <pbonzini@redhat.com>\nCc: Peter Lieven <pl@kamp.de>\nCc: Josh Durgin <jdurgin@redhat.com>\nCc: \"Richard W.M. Jones\" <rjones@redhat.com>\nCc: Markus Armbruster <armbru@redhat.com>\nCc: Peter Crosthwaite <crosthwaite.peter@gmail.com>\nCc: Richard Henderson <rth@twiddle.net>\nCc: \"Aneesh Kumar K.V\" <aneesh.kumar@linux.vnet.ibm.com>\nCc: Greg Kurz <groug@kaod.org>\nCc: Rob Herring <robh@kernel.org>\nCc: Peter Maydell <peter.maydell@linaro.org>\nCc: Peter Chubb <peter.chubb@nicta.com.au>\nCc: Eduardo Habkost <ehabkost@redhat.com>\nCc: Marcel Apfelbaum <marcel@redhat.com>\nCc: \"Michael S. Tsirkin\" <mst@redhat.com>\nCc: Igor Mammedov <imammedo@redhat.com>\nCc: David Gibson <david@gibson.dropbear.id.au>\nCc: Alexander Graf <agraf@suse.de>\nCc: Gerd Hoffmann <kraxel@redhat.com>\nCc: Jason Wang <jasowang@redhat.com>\nCc: Marcelo Tosatti <mtosatti@redhat.com>\nCc: Christian Borntraeger <borntraeger@de.ibm.com>\nCc: Cornelia Huck <cohuck@redhat.com>\nCc: Stefan Hajnoczi <stefanha@redhat.com>\nAcked-by: David Gibson <david@gibson.dropbear.id.au>\nAcked-by: Greg Kurz <groug@kaod.org>\nAcked-by: Cornelia Huck <cohuck@redhat.com>\nReviewed-by: Stefan Hajnoczi <stefanha@redhat.com>\nReviewed by: Peter Chubb <peter.chubb@data61.csiro.au>\nAcked-by: Max Reitz <mreitz@redhat.com>\nAcked-by: Marcel Apfelbaum <marcel@redhat.com>\nMessage-Id: <e1cfa2cd47087c248dd24caca9c33d9af0c499b0.1499866456.git.alistair.francis@xilinx.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/3dc6f8693694a649a9c83f1e2746565b47683923"},{"sha":"c51c4f88072e7c598514a96d7e31b2a46520eca7","author":{"name":"Alistair Francis","email":"159fd3f0183a873ba8874443403d6ff506e71f24@xilinx.com"},"message":"char-socket: Report TCP socket waiting as information\n\nWhen QEMU is waiting for a TCP socket connection it reports that message as\nan error. This isn't an error it is just information so let's change the\nreport to use info_report() instead.\n\nSigned-off-by: Alistair Francis <alistair.francis@xilinx.com>\nReviewed-by: Thomas Huth <thuth@redhat.com>\nReviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nMessage-Id: <b3f3601c52843afca9a9b12c7a4fefd68e60de32.1499866456.git.alistair.francis@xilinx.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/c51c4f88072e7c598514a96d7e31b2a46520eca7"},{"sha":"e43ead1d0b9c467af4a2af8f5177316a0ccb5602","author":{"name":"Alistair Francis","email":"159fd3f0183a873ba8874443403d6ff506e71f24@xilinx.com"},"message":"error: Implement the warn and free Error functions\n\nImplement warn_report_err() and warn_reportf_err() functions which\nare the same as the error_report_err() and error_reportf_err()\nfunctions except report a warning instead of an error.\n\nSigned-off-by: Alistair Francis <alistair.francis@xilinx.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nMessage-Id: <276ff93eadc0b01b8243cc61ffc331f77922c0d0.1499866456.git.alistair.francis@xilinx.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/e43ead1d0b9c467af4a2af8f5177316a0ccb5602"},{"sha":"88f83f3539fdedd5f315c5fe434ffcb7a010cc73","author":{"name":"Alistair Francis","email":"159fd3f0183a873ba8874443403d6ff506e71f24@xilinx.com"},"message":"Convert error_report*_err() to warn_report*_err()\n\nConvert all uses of error_report*_err(\"Warning:\"... to use\nwarn_report*_err() instead. This helps standardise on a single\nmethod of printing warnings to the user.\n\nSigned-off-by: Alistair Francis <alistair.francis@xilinx.com>\nCc: Paolo Bonzini <pbonzini@redhat.com>\nCc: Richard Henderson <rth@twiddle.net>\nCc: Eduardo Habkost <ehabkost@redhat.com>\nCc: \"Michael S. Tsirkin\" <mst@redhat.com>\nReviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nReviewed-by: Eduardo Habkost <ehabkost@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nMessage-Id: <d8e088757186955f40f04ec4f4be7f640d3c8660.1499866456.git.alistair.francis@xilinx.com>\nSigned-off-by: Markus Armbruster <armbru@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/88f83f3539fdedd5f315c5fe434ffcb7a010cc73"},{"sha":"abcea034a05e617866d456bf6afc4c4d4ea26151","author":{"name":"Mark Cave-Ayland","email":"d8f5d512409aa32dd23a063424a40bf02a4e4dda@ilande.co.uk"},"message":"Update OpenBIOS images to fbc1b4a built from submodule.\n\nSigned-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/abcea034a05e617866d456bf6afc4c4d4ea26151"},{"sha":"03f47ee49e1478b5ffffb3a9b6203c672903196c","author":{"name":"Janosch Frank","email":"ed9db37428c3e0efa3c76bf823cee3f07bc3b207@linux.vnet.ibm.com"},"message":"s390x/kvm: Rework cmma management\n\nLet's keep track of cmma enablement and move the mem_path check into\nthe actual enablement. This now also warns users that do not use\ncpu-models about disabled cmma when using huge pages.\n\nSigned-off-by: Janosch Frank <frankja@linux.vnet.ibm.com>\nReviewed-by: Cornelia Huck <cohuck@redhat.com>\nSigned-off-by: Christian Borntraeger <borntraeger@de.ibm.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/03f47ee49e1478b5ffffb3a9b6203c672903196c"},{"sha":"a309b290aaa8ac55191a50e44bbd99b5fc586487","author":{"name":"Peter Maydell","email":"d4da0cae8bdd6af8ce5950fc7fec7c012df68229@linaro.org"},"message":"Merge remote-tracking branch 'remotes/armbru/tags/pull-error-2017-07-13' into staging\n\nError reporting patches for 2017-07-13\n\n# gpg: Signature made Thu 13 Jul 2017 12:55:45 BST\n# gpg:                using RSA key 0x3870B400EB918653\n# gpg: Good signature from \"Markus Armbruster <armbru@redhat.com>\"\n# gpg:                 aka \"Markus Armbruster <armbru@pond.sub.org>\"\n# Primary key fingerprint: 354B C8B3 D7EB 2A6B 6867  4E5F 3870 B400 EB91 8653\n\n* remotes/armbru/tags/pull-error-2017-07-13:\n  Convert error_report*_err() to warn_report*_err()\n  error: Implement the warn and free Error functions\n  char-socket: Report TCP socket waiting as information\n  Convert error_report() to warn_report()\n  error: Functions to report warnings and informational messages\n  util/qemu-error: Rename error_print_loc() to be more generic\n  websock: Don't try to set *errp directly\n  block: Don't try to set *errp directly\n  xilinx: Fix latent error handling bug\n\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/a309b290aaa8ac55191a50e44bbd99b5fc586487"},{"sha":"b98fcfd8840f290c406c32301340e96f00238a93","author":{"name":"Paolo Bonzini","email":"7438500e0207b21c4e5dfca8d2a7898734cd7706@redhat.com"},"message":"build: add -Wexpansion-to-defined\n\nThis warning is included in -Wall by clang, but not by GCC (which only\nenables it for -Wextra).  Include it in the list of warnings we enable\nto minimize the differences between the compilers:\n\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/b98fcfd8840f290c406c32301340e96f00238a93"},{"sha":"c6a88ddaac4aa89765313952d5702181423df1a4","author":{"name":"Philippe Mathieu-Daudé","email":"51f22ed298acb64cce73fdf98fc54cf485c01eda@amsat.org"},"message":"MAINTAINERS: update TCG entries\n\nmoved in a9ded601..244f1441 to accel/\n\nSigned-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nMessage-Id: <20170629150308.22766-2-f4bug@amsat.org>\nReviewed-by: Thomas Huth <thuth@redhat.com>\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/c6a88ddaac4aa89765313952d5702181423df1a4"},{"sha":"1962cb2001996528e4deda8476551eb838ea0758","author":{"name":"Philippe Mathieu-Daudé","email":"51f22ed298acb64cce73fdf98fc54cf485c01eda@amsat.org"},"message":"MAINTAINERS: update KVM entries\n\nmoved in 92229a57 to accel/\n\nSigned-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nMessage-Id: <20170629150308.22766-3-f4bug@amsat.org>\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/1962cb2001996528e4deda8476551eb838ea0758"},{"sha":"2c7ccb4e27bd980948720211641ea0812a70a328","author":{"name":"Philippe Mathieu-Daudé","email":"51f22ed298acb64cce73fdf98fc54cf485c01eda@amsat.org"},"message":"MAINTAINERS: update Xen entries\n\nmoved in 56e2cd24..28b99f47 to hw/xen/ and hw/i386/xen/\n\nSigned-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nAcked-by: Anthony PERARD <anthony.perard@citrix.com>\nMessage-Id: <20170629150308.22766-4-f4bug@amsat.org>\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/2c7ccb4e27bd980948720211641ea0812a70a328"},{"sha":"752a2ae2734feb3daa7e91f323aca6734437c64b","author":{"name":"Philippe Mathieu-Daudé","email":"51f22ed298acb64cce73fdf98fc54cf485c01eda@amsat.org"},"message":"MAINTAINERS: update TCI entry\n\nmoved in 244f1441 to tcg/\n\nSigned-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nMessage-Id: <20170629150308.22766-5-f4bug@amsat.org>\nReviewed-by: Stefan Weil <sw@weilnetz.de>\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/752a2ae2734feb3daa7e91f323aca6734437c64b"},{"sha":"747969db2a6cd6e93561f7edc50d824a7d67346b","author":{"name":"Philippe Mathieu-Daudé","email":"51f22ed298acb64cce73fdf98fc54cf485c01eda@amsat.org"},"message":"MAINTAINERS: add entry for \"Unimplemented\" device\n\nAlso voluntary myself as reviewer\n\nSigned-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>\nMessage-Id: <20170629150308.22766-6-f4bug@amsat.org>\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/747969db2a6cd6e93561f7edc50d824a7d67346b"},{"sha":"bcdeb9be566ded2eb35233aaccf38742a21e5daa","author":{"name":"Marc-André Lureau","email":"56471e898443ac4e80f8339c866febf4480627a0@redhat.com"},"message":"chardev: block during sync read\n\nA sync read should block until all requested data is\navailable (instead of retrying in qemu_chr_fe_read_all). Change the\nchannel to blocking during sync_read.\n\nSigned-off-by: Marc-AndrÃ© Lureau <marcandre.lureau@redhat.com>\nMessage-Id: <20170706170353.32601-1-marcandre.lureau@redhat.com>\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>\nSigned-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/bcdeb9be566ded2eb35233aaccf38742a21e5daa"},{"sha":"313e45b5fe45542602bfa801db7a13d485c29b04","author":{"name":"Anton Nefedov","email":"16b9c20d065bcadd90fb9a991df709dcf69035fb@virtuozzo.com"},"message":"char: move QemuOpts->ChardevBackend translation to a separate func\n\nparse function will be used by the following patch\n\nSigned-off-by: Anton Nefedov <anton.nefedov@virtuozzo.com>\nMessage-Id: <1499342940-56739-2-git-send-email-anton.nefedov@virtuozzo.com>\nReviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>\nSigned-off-by: Paolo Bonzini <pbonzini@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/313e45b5fe45542602bfa801db7a13d485c29b04"}]}
{"actor":{"display_login":"crobinso"}}