{"push_id":1356200777,"size":74,"distinct_size":74,"ref":"refs/heads/master","head":"1b0d3845b454eaaac0b2064c78926ca4d739a080","before":"0975b8b823a888d474fa33821dfe84e6904db197","commits":[{"sha":"426ec9049e29397af95f8c28e020590bd1fe57dd","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Use local error object in vfio_initfn\n\nTo prepare for migration to realize, let's use a local error\nobject in vfio_initfn. Also let's use the same error prefix for all\nerror messages.\n\nOn top of the 1-1 conversion, we start using a common error prefix for\nall error messages. We also introduce a similar warning prefix which will\nbe used later on.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/426ec9049e29397af95f8c28e020590bd1fe57dd"},{"sha":"cde4279baa246fdfadb06d021e3b8d0a0234879f","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Pass an error object to vfio_populate_vga\n\nPass an error object to prepare for the same operation in\nvfio_populate_device. Eventually this contributes to the migration\nto VFIO-PCI realize.\n\nWe now report an error on vfio_get_region_info failure.\n\nvfio_probe_igd_bar4_quirk is not involved in the migration to realize\nand simply calls error_reportf_err.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/cde4279baa246fdfadb06d021e3b8d0a0234879f"},{"sha":"2312d907ddcdeffa45350e5960be46d6105b8ec2","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Pass an error object to vfio_populate_device\n\nPass an error object to prepare for migration to VFIO-PCI realize.\nThe returned value will be removed later on.\n\nThe case where error recovery cannot be enabled is not converted into\nan error object but directly reported through error_report, as before.\nPopulating an error instead would cause the future realize function to\nfail, which is not wanted.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/2312d907ddcdeffa45350e5960be46d6105b8ec2"},{"sha":"008d0e2d7bb1bb8c0393d5cc7501a05c2bc4ed81","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Pass an error object to vfio_msix_early_setup\n\nPass an error object to prepare for migration to VFIO-PCI realize.\nThe returned value will be removed later on.\n\nWe now format an error in case of reading failure for\n- the MSIX flags\n- the MSIX table,\n- the MSIX PBA.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/008d0e2d7bb1bb8c0393d5cc7501a05c2bc4ed81"},{"sha":"7dfb34247e231e1f317e3558af96c90f06fef8a5","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Pass an error object to vfio_intx_enable\n\nPass an error object to prepare for migration to VFIO-PCI realize.\n\nThe error object is propagated down to vfio_intx_enable_kvm().\n\nThe three other callers, vfio_intx_enable_kvm(), vfio_msi_disable_common()\nand vfio_pci_post_reset() do not propagate the error and simply call\nerror_reportf_err() with the ERR_PREFIX formatting.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/7dfb34247e231e1f317e3558af96c90f06fef8a5"},{"sha":"7ef165b9a8d903babddaf9c61c54ae9b6e739604","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Pass an error object to vfio_add_capabilities\n\nPass an error object to prepare for migration to VFIO-PCI realize.\nThe error is cascaded downto vfio_add_std_cap and then vfio_msi(x)_setup,\nvfio_setup_pcie_cap.\n\nvfio_add_ext_cap does not return anything else than 0 so let's transform\nit into a void function.\n\nAlso use pci_add_capability2 which takes an error object.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/7ef165b9a8d903babddaf9c61c54ae9b6e739604"},{"sha":"7237011d0596e57f42d2a4e0556034b8d13533fe","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Pass an error object to vfio_pci_igd_opregion_init\n\nPass an error object to prepare for migration to VFIO-PCI realize.\n\nIn vfio_probe_igd_bar4_quirk, simply report the error.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/7237011d0596e57f42d2a4e0556034b8d13533fe"},{"sha":"01905f58f166646619c35a2ebfc3ca3ed4cad62d","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio: Pass an Error object to vfio_connect_container\n\nThe error is currently simply reported in vfio_get_group. Don't\nbother too much with the prefix which will be handled at upper level,\nlater on.\n\nAlso return an error value in case container->error is not 0 and\nthe container is teared down.\n\nOn vfio_spapr_remove_window failure, we also report an error whereas\nit was silent before.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/01905f58f166646619c35a2ebfc3ca3ed4cad62d"},{"sha":"1b808d5be070e9d07e5d0e5b825a31a0cf87001d","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio: Pass an error object to vfio_get_group\n\nPass an error object to prepare for migration to VFIO-PCI realize.\n\nFor the time being let's just simply report the error in\nvfio platform's vfio_base_device_init(). A subsequent patch will\nduly propagate the error up to vfio_platform_realize.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/1b808d5be070e9d07e5d0e5b825a31a0cf87001d"},{"sha":"59f7d6743ccbe17587e491dc5d79cad8bf31f76a","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio: Pass an error object to vfio_get_device\n\nPass an error object to prepare for migration to VFIO-PCI realize.\n\nIn vfio platform vfio_base_device_init we currently just report the\nerror. Subsequent patches will propagate the error up to the realize\nfunction.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/59f7d6743ccbe17587e491dc5d79cad8bf31f76a"},{"sha":"5ff7419d4c7706fc3f625c0fd48cae191d2cecbc","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/platform: Pass an error object to vfio_populate_device\n\nPropagate the vfio_populate_device errors up to vfio_base_device_init.\nThe error object also is passed to vfio_init_intp. At the moment we\nonly report the error. Subsequent patches will propagate the error\nup to the realize function.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/5ff7419d4c7706fc3f625c0fd48cae191d2cecbc"},{"sha":"0d84f47bff2f98d0cea5fb27e9b62225ccebd70a","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/platform: fix a wrong returned value in vfio_populate_device\n\nIn case the vfio_init_intp fails we currently do not return an\nerror value. This patch fixes the bug. The returned value is not\nexplicit but in practice the error object is the one used to\nreport the error to the end-user and the actual returned error\nvalue is not used.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/0d84f47bff2f98d0cea5fb27e9b62225ccebd70a"},{"sha":"9bdbfbd50dfcc23252224112e8cbb720a8535d40","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/platform: Pass an error object to vfio_base_device_init\n\nThis patch propagates errors encountered during vfio_base_device_init\nup to the realize function.\n\nIn case the host value is not set or badly formed we now report an\nerror.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/9bdbfbd50dfcc23252224112e8cbb720a8535d40"},{"sha":"1a22aca1d0071bdc4abc1a72894a13f0076b2404","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Conversion to realize\n\nThis patch converts VFIO PCI to realize function.\n\nAlso original initfn errors now are propagated using QEMU\nerror objects. All errors are formatted with the same pattern:\n\"vfio: %s: the error description\"\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/1a22aca1d0071bdc4abc1a72894a13f0076b2404"},{"sha":"ec3bcf424edb76e00b6ff545360329c4cedf9953","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Remove vfio_msix_early_setup returned value\n\nThe returned value is not used anymore by the caller, vfio_realize,\nsince the error now is stored in the error object. So let's remove it.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/ec3bcf424edb76e00b6ff545360329c4cedf9953"},{"sha":"e04cff9d975389b83d59b89cc3904b20ae0b4ce1","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Remove vfio_populate_device returned value\n\nThe returned value (either -errno or -1) is not used anymore by the caller,\nvfio_realize, since the error now is stored in the error object. So let's\nremove it.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/e04cff9d975389b83d59b89cc3904b20ae0b4ce1"},{"sha":"4a946268504e72fe0c547b9dda97adbe277a585f","author":{"name":"Eric Auger","email":"2a198c874b601f75a3d0b1f09e1746f2d6685c4e@redhat.com"},"message":"vfio/pci: Handle host oversight\n\nIn case the end-user calls qemu with -vfio-pci option without passing\neither sysfsdev or host property value, the device is interpreted as\n0000:00:00.0. Let's create a specific error message to guide the end-user.\n\nSigned-off-by: Eric Auger <eric.auger@redhat.com>\nReviewed-by: Markus Armbruster <armbru@redhat.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/4a946268504e72fe0c547b9dda97adbe277a585f"},{"sha":"31e6a7b17b35711eb44f0e686b5ba68d15bfe4c1","author":{"name":"Thorsten Kohfeldt","email":"e2f272abee20c6be3cf68f8b1c55980712031306@gmx.de"},"message":"vfio/pci: Fix vfio_rtl8168_quirk_data_read address offset\n\nIntroductory comment for rtl8168 VFIO MSI-X quirk states:\nAt BAR2 offset 0x70 there is a dword data register,\n         offset 0x74 is a dword address register.\nvfio: vfio_bar_read(0000:05:00.0:BAR2+0x70, 4) = 0xfee00398 // read data\n\nThus, correct offset for data read is 0x70,\nbut function vfio_rtl8168_quirk_data_read() wrongfully uses offset 0x74.\n\nSigned-off-by: Thorsten Kohfeldt <thorsten.kohfeldt@gmx.de>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/31e6a7b17b35711eb44f0e686b5ba68d15bfe4c1"},{"sha":"893bfc3cc893ed36cedc364e99cf483e9b08c294","author":{"name":"Cao jin","email":"24fd75c154449c69743d8c21314f41324ec1720a@cn.fujitsu.com"},"message":"vfio: fix duplicate function call\n\nWhen vfio device is reset(encounter FLR, or bus reset), if need to do\nbus reset(vfio_pci_hot_reset_one is called), vfio_pci_pre_reset &\nvfio_pci_post_reset will be called twice.\n\nSigned-off-by: Cao jin <caoj.fnst@cn.fujitsu.com>\nSigned-off-by: Alex Williamson <alex.williamson@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/893bfc3cc893ed36cedc364e99cf483e9b08c294"},{"sha":"a2f9976ea83c50b452d46b8b1189242617e4ac73","author":{"name":"Eduardo Habkost","email":"1434a0c554a4542f3cf3c2e7be483467bba16117@redhat.com"},"message":"tests: Add test case for x86 feature parsing compatibility\n\nAdd a new test case to ensure the existing behavior of the\nfeature parsing code will be kept.\n\nSigned-off-by: Eduardo Habkost <ehabkost@redhat.com>","distinct":true,"url":"https://api.github.com/repos/crobinso/qemu/commits/a2f9976ea83c50b452d46b8b1189242617e4ac73"}]}
{"actor":{"display_login":"crobinso"}}