leandrof/linux
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

KVM: async_pf: use list_first_entry

Browse Source 
To make the intention clearer, use list_first_entry instead of
list_entry.

Signed-off-by: Geliang Tang <geliangtang@163.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
Geliang Tang 2016-01-01 19:47:15 +08:00 committed by Paolo Bonzini
parent d74c0e6b54
commit 433da86023
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
virt/kvm/async_pf.c
View File

@ -109,8 +109,8 @@ void kvm_clear_async_pf_completion_queue(struct kvm_vcpu *vcpu)
/* cancel outstanding work queue item */ /* cancel outstanding work queue item */
while (!list_empty(&vcpu->async_pf.queue)) { while (!list_empty(&vcpu->async_pf.queue)) {
struct kvm_async_pf *work = struct kvm_async_pf *work =
list_entry(vcpu->async_pf.queue.next, list_first_entry(&vcpu->async_pf.queue,
typeof(*work), queue); typeof(*work), queue);
list_del(&work->queue); list_del(&work->queue);
#ifdef CONFIG_KVM_ASYNC_PF_SYNC #ifdef CONFIG_KVM_ASYNC_PF_SYNC
@ -127,8 +127,8 @@ void kvm_clear_async_pf_completion_queue(struct kvm_vcpu *vcpu)
spin_lock(&vcpu->async_pf.lock); spin_lock(&vcpu->async_pf.lock);
while (!list_empty(&vcpu->async_pf.done)) { while (!list_empty(&vcpu->async_pf.done)) {
struct kvm_async_pf *work = struct kvm_async_pf *work =
list_entry(vcpu->async_pf.done.next, list_first_entry(&vcpu->async_pf.done,
typeof(*work), link); typeof(*work), link);
list_del(&work->link); list_del(&work->link);
kmem_cache_free(async_pf_cache, work); kmem_cache_free(async_pf_cache, work);
} }