aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEugenio Pérez <eperezma@redhat.com>2022-04-27 17:49:31 +0200
committerLaurent Vivier <laurent@vivier.eu>2022-06-28 10:56:42 +0200
commit832fef7cc14d65f99d523f883ef384014e6476a7 (patch)
tree9adf0294081b78569a6a3a8823334ba3b88dc1bf
parent5bba9bcfbb42e7c016626420e148a1bf1b080835 (diff)
downloadqemu-832fef7cc14d65f99d523f883ef384014e6476a7.zip
qemu-832fef7cc14d65f99d523f883ef384014e6476a7.tar.gz
qemu-832fef7cc14d65f99d523f883ef384014e6476a7.tar.bz2
util: Return void on iova_tree_remove
It always returns IOVA_OK so nobody uses it. Acked-by: Jason Wang <jasowang@redhat.com> Reviewed-by: Peter Xu <peterx@redhat.com> Signed-off-by: Eugenio Pérez <eperezma@redhat.com> Message-Id: <20220427154931.3166388-1-eperezma@redhat.com> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
-rw-r--r--include/qemu/iova-tree.h4
-rw-r--r--util/iova-tree.c4
2 files changed, 2 insertions, 6 deletions
diff --git a/include/qemu/iova-tree.h b/include/qemu/iova-tree.h
index c938fb0..16bbfdf 100644
--- a/include/qemu/iova-tree.h
+++ b/include/qemu/iova-tree.h
@@ -72,10 +72,8 @@ int iova_tree_insert(IOVATree *tree, const DMAMap *map);
* provided. The range does not need to be exactly what has inserted,
* all the mappings that are included in the provided range will be
* removed from the tree. Here map->translated_addr is meaningless.
- *
- * Return: 0 if succeeded, or <0 if error.
*/
-int iova_tree_remove(IOVATree *tree, const DMAMap *map);
+void iova_tree_remove(IOVATree *tree, const DMAMap *map);
/**
* iova_tree_find:
diff --git a/util/iova-tree.c b/util/iova-tree.c
index 6dff29c..fee530a 100644
--- a/util/iova-tree.c
+++ b/util/iova-tree.c
@@ -164,15 +164,13 @@ void iova_tree_foreach(IOVATree *tree, iova_tree_iterator iterator)
g_tree_foreach(tree->tree, iova_tree_traverse, iterator);
}
-int iova_tree_remove(IOVATree *tree, const DMAMap *map)
+void iova_tree_remove(IOVATree *tree, const DMAMap *map)
{
const DMAMap *overlap;
while ((overlap = iova_tree_find(tree, map))) {
g_tree_remove(tree->tree, overlap);
}
-
- return IOVA_OK;
}
/**