lkml.org 
[lkml]   [2014]   [Jul]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 2/4] KVM: use pci device flag operation helper functions
    Il 22/07/2014 18:19, Ethan Zhao ha scritto:
    > Use helper function instead of direct operation to pci device
    > flag when set device to assigned or deassigned.
    >
    > Signed-off-by: Ethan Zhao <ethan.zhao@oracle.com>
    > ---
    > virt/kvm/assigned-dev.c | 2 +-
    > virt/kvm/iommu.c | 4 ++--
    > 2 files changed, 3 insertions(+), 3 deletions(-)
    >
    > diff --git a/virt/kvm/assigned-dev.c b/virt/kvm/assigned-dev.c
    > index bf06577..d122bda 100644
    > --- a/virt/kvm/assigned-dev.c
    > +++ b/virt/kvm/assigned-dev.c
    > @@ -302,7 +302,7 @@ static void kvm_free_assigned_device(struct kvm *kvm,
    > else
    > pci_restore_state(assigned_dev->dev);
    >
    > - assigned_dev->dev->dev_flags &= ~PCI_DEV_FLAGS_ASSIGNED;
    > + pci_set_dev_deassigned(assigned_dev->dev);
    >
    > pci_release_regions(assigned_dev->dev);
    > pci_disable_device(assigned_dev->dev);
    > diff --git a/virt/kvm/iommu.c b/virt/kvm/iommu.c
    > index 0df7d4b..8cfe021 100644
    > --- a/virt/kvm/iommu.c
    > +++ b/virt/kvm/iommu.c
    > @@ -194,7 +194,7 @@ int kvm_assign_device(struct kvm *kvm,
    > goto out_unmap;
    > }
    >
    > - pdev->dev_flags |= PCI_DEV_FLAGS_ASSIGNED;
    > + pci_set_dev_assigned(pdev);
    >
    > dev_info(&pdev->dev, "kvm assign device\n");
    >
    > @@ -220,7 +220,7 @@ int kvm_deassign_device(struct kvm *kvm,
    >
    > iommu_detach_device(domain, &pdev->dev);
    >
    > - pdev->dev_flags &= ~PCI_DEV_FLAGS_ASSIGNED;
    > + pci_set_dev_deassigned(pdev);
    >
    > dev_info(&pdev->dev, "kvm deassign device\n");
    >
    >

    Acked-by: Paolo Bonzini <pbonzini@redhat.com>


    \
     
     \ /
      Last update: 2014-07-24 13:41    [W:5.701 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site