On 11/10/2016 05:14 PM, Michael S. Tsirkin wrote: > From: Cao jin <[email protected]> > > msix_init() reports errors with error_report(), which is wrong when > it's used in realize(). The same issue was fixed for msi_init() in > commit 1108b2f. > > For some devices(like e1000e, vmxnet3) who won't fail because of > msix_init's failure, suppress the error report by passing NULL error object. > > Bonus: add comment for msix_init. > > CC: Jiri Pirko <[email protected]> > CC: Gerd Hoffmann <[email protected]> > CC: Dmitry Fleytman <[email protected]> > CC: Jason Wang <[email protected]> > CC: Michael S. Tsirkin <[email protected]> > CC: Hannes Reinecke <[email protected]> > CC: Paolo Bonzini <[email protected]> > CC: Alex Williamson <[email protected]> > CC: Markus Armbruster <[email protected]> > CC: Marcel Apfelbaum <[email protected]> > > Reviewed-by: Markus Armbruster <[email protected]> > Signed-off-by: Cao jin <[email protected]> > Acked-by: Marcel Apfelbaum <[email protected]> > Reviewed-by: Michael S. Tsirkin <[email protected]> > Signed-off-by: Michael S. Tsirkin <[email protected]> > --- > include/hw/pci/msix.h | 5 +++-- > hw/block/nvme.c | 5 ++++- > hw/misc/ivshmem.c | 8 ++++---- > hw/net/e1000e.c | 6 +++++- > hw/net/rocker/rocker.c | 7 ++++++- > hw/net/vmxnet3.c | 8 ++++++-- > hw/pci/msix.c | 34 +++++++++++++++++++++++++++++----- > hw/scsi/megasas.c | 5 ++++- > hw/usb/hcd-xhci.c | 13 ++++++++----- > hw/vfio/pci.c | 8 ++++++-- > hw/virtio/virtio-pci.c | 11 +++++------ > 11 files changed, 80 insertions(+), 30 deletions(-) > Reviewed-by: Hannes Reinecke <[email protected]>
Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking [email protected] +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg)
