diff options
| author | Ilya Maximets <i.maximets@ovn.org> | 2023-09-27 15:59:42 +0200 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2023-10-04 18:15:06 -0400 |
| commit | 70f88436aa5a8aeddd33e4d06270146af5d5bb52 (patch) | |
| tree | 6f3d8317224218c46c48eff0f5cb0b65bdffd9ad /hw/core/machine.c | |
| parent | d501f97d9607eff1750549e0270c034102786d33 (diff) | |
| download | focaccia-qemu-70f88436aa5a8aeddd33e4d06270146af5d5bb52.tar.gz focaccia-qemu-70f88436aa5a8aeddd33e4d06270146af5d5bb52.zip | |
virtio: remove unused next argument from virtqueue_split_read_next_desc()
The 'next' was converted from a local variable to an output parameter
in commit:
412e0e81b174 ("virtio: handle virtqueue_read_next_desc() errors")
But all the actual uses of the 'i/next' as an output were removed a few
months prior in commit:
aa570d6fb6bd ("virtio: combine the read of a descriptor")
Remove the unused argument to simplify the code.
Also, adding a comment to the function to describe what it is actually
doing, as it is not obvious that the 'desc' is both an input and an
output argument.
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Message-Id: <20230927140016.2317404-3-i.maximets@ovn.org>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Diffstat (limited to 'hw/core/machine.c')
0 files changed, 0 insertions, 0 deletions