On 26/07/2017 10:41, Mao Zhongyi wrote:
> qemu_bh_new() is a wrapper around aio_bh_new(), which returns
> null only when g_new() does. It doesn't. So remove the dead
> error handling.
>
> Cc: Dmitry Fleytman <dmi...@daynix.com>
> Cc: Paolo Bonzini <pbonz...@redhat.com>
> Cc: Markus Armbruster <arm...@redhat.com>
> Signed-off-by: Mao Zhongyi <maozy.f...@cn.fujitsu.com>
> ---
> hw/scsi/vmw_pvscsi.c | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/hw/scsi/vmw_pvscsi.c b/hw/scsi/vmw_pvscsi.c
> index 4a106da..d92973e 100644
> --- a/hw/scsi/vmw_pvscsi.c
> +++ b/hw/scsi/vmw_pvscsi.c
> @@ -1138,10 +1138,6 @@ pvscsi_init(PCIDevice *pci_dev)
> }
>
> s->completion_worker = qemu_bh_new(pvscsi_process_completion_queue, s);
> - if (!s->completion_worker) {
> - pvscsi_cleanup_msi(s);
> - return -ENOMEM;
> - }
>
> scsi_bus_new(&s->bus, sizeof(s->bus), DEVICE(pci_dev),
> &pvscsi_scsi_info, NULL);
>
Queued both patches, thanks.
Paolo
