• Xie Yongji's avatar
    vduse: Remove unnecessary spin lock protection · c32ee693
    Xie Yongji authored
    Now we use domain->iotlb_lock to protect two different
    variables: domain->bounce_maps->bounce_page and
    domain->iotlb. But for domain->bounce_maps->bounce_page,
    we actually don't need any synchronization between
    vduse_domain_get_bounce_page() and vduse_domain_free_bounce_pages()
    since vduse_domain_get_bounce_page() will only be called in
    page fault handler and vduse_domain_free_bounce_pages() will
    be called during file release.
    
    So let's remove the unnecessary spin lock protection in
    vduse_domain_get_bounce_page(). Then the usage of
    domain->iotlb_lock could be more clear: the lock will be
    only used to protect the domain->iotlb.
    Signed-off-by: default avatarXie Yongji <xieyongji@bytedance.com>
    Acked-by: default avatarJason Wang <jasowang@redhat.com>
    Message-Id: <20220803045523.23851-2-xieyongji@bytedance.com>
    Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
    c32ee693
iova_domain.c 12.8 KB