Laurent Vivier <lviv...@redhat.com> 于2020年9月20日周日 下午11:21写道: > > in vhost_vdpa_listener_region_del(), try_unmap is always true and so, > vhost_vdpa_dma_unmap() is always called. We can remove the variable > > Signed-off-by: Laurent Vivier <lviv...@redhat.com>
Reviewed-by: Li Qiang <liq...@gmail.com> > --- > hw/virtio/vhost-vdpa.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/hw/virtio/vhost-vdpa.c b/hw/virtio/vhost-vdpa.c > index 4580f3efd8a2..c6cae6a6cabf 100644 > --- a/hw/virtio/vhost-vdpa.c > +++ b/hw/virtio/vhost-vdpa.c > @@ -153,7 +153,6 @@ static void vhost_vdpa_listener_region_del(MemoryListener > *listener, > hwaddr iova; > Int128 llend, llsize; > int ret; > - bool try_unmap = true; > > if (vhost_vdpa_listener_skipped_section(section)) { > return; > @@ -176,11 +175,9 @@ static void > vhost_vdpa_listener_region_del(MemoryListener *listener, > > llsize = int128_sub(llend, int128_make64(iova)); > > - if (try_unmap) { > - ret = vhost_vdpa_dma_unmap(v, iova, int128_get64(llsize)); > - if (ret) { > - error_report("vhost_vdpa dma unmap error!"); > - } > + ret = vhost_vdpa_dma_unmap(v, iova, int128_get64(llsize)); > + if (ret) { > + error_report("vhost_vdpa dma unmap error!"); > } > > memory_region_unref(section->mr); > -- > 2.26.2 > >