SYS-OSS/FRET-qemu
4
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

aio-posix: remove confusing QLIST_SAFE_REMOVE()

Browse Source 
QLIST_SAFE_REMOVE() is confusing here because the node must be on the
list.  We actually just wanted to clear the linked list pointers when
removing it from the list.  QLIST_REMOVE() now does this, so switch to
it.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Link: https://lore.kernel.org/r/20200224103406.1894923-3-stefanha@redhat.com
Message-Id: <20200224103406.1894923-3-stefanha@redhat.com>
This commit is contained in:
Stefan Hajnoczi 2020-02-24 10:34:06 +00:00
parent a31ca6801c
commit c39cbedb54
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
util/aio-posix.c
View File

@ -493,7 +493,7 @@ static bool aio_dispatch_ready_handlers(AioContext *ctx,
AioHandler *node; AioHandler *node;
while ((node = QLIST_FIRST(ready_list))) { while ((node = QLIST_FIRST(ready_list))) {
QLIST_SAFE_REMOVE(node, node_ready); QLIST_REMOVE(node, node_ready);
progress = aio_dispatch_handler(ctx, node) || progress; progress = aio_dispatch_handler(ctx, node) || progress;
} }