[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 05/18] vhost: Return number of free memslots
From: |
David Hildenbrand |
Subject: |
[PATCH v4 05/18] vhost: Return number of free memslots |
Date: |
Tue, 26 Sep 2023 20:57:25 +0200 |
Let's return the number of free slots instead of only checking if there
is a free slot. Required to support memory devices that consume multiple
memslots.
This is a preparation for memory devices that consume multiple memslots.
Reviewed-by: Maciej S. Szmigiero <maciej.szmigiero@oracle.com>
Signed-off-by: David Hildenbrand <david@redhat.com>
---
hw/mem/memory-device.c | 2 +-
hw/virtio/vhost-stub.c | 4 ++--
hw/virtio/vhost.c | 4 ++--
include/hw/virtio/vhost.h | 2 +-
4 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/hw/mem/memory-device.c b/hw/mem/memory-device.c
index 98e355c960..e09960744d 100644
--- a/hw/mem/memory-device.c
+++ b/hw/mem/memory-device.c
@@ -63,7 +63,7 @@ static void memory_device_check_addable(MachineState *ms,
MemoryRegion *mr,
error_setg(errp, "hypervisor has no free memory slots left");
return;
}
- if (!vhost_has_free_slot()) {
+ if (!vhost_get_free_memslots()) {
error_setg(errp, "a used vhost backend has no free memory slots left");
return;
}
diff --git a/hw/virtio/vhost-stub.c b/hw/virtio/vhost-stub.c
index aa858ef3fb..d53dd9d288 100644
--- a/hw/virtio/vhost-stub.c
+++ b/hw/virtio/vhost-stub.c
@@ -2,9 +2,9 @@
#include "hw/virtio/vhost.h"
#include "hw/virtio/vhost-user.h"
-bool vhost_has_free_slot(void)
+unsigned int vhost_get_free_memslots(void)
{
- return true;
+ return UINT_MAX;
}
bool vhost_user_init(VhostUserState *user, CharBackend *chr, Error **errp)
diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
index c16ad14535..8e84dca246 100644
--- a/hw/virtio/vhost.c
+++ b/hw/virtio/vhost.c
@@ -54,7 +54,7 @@ static unsigned int used_shared_memslots;
static QLIST_HEAD(, vhost_dev) vhost_devices =
QLIST_HEAD_INITIALIZER(vhost_devices);
-bool vhost_has_free_slot(void)
+unsigned int vhost_get_free_memslots(void)
{
unsigned int free = UINT_MAX;
struct vhost_dev *hdev;
@@ -71,7 +71,7 @@ bool vhost_has_free_slot(void)
}
free = MIN(free, cur_free);
}
- return free > 0;
+ return free;
}
static void vhost_dev_sync_region(struct vhost_dev *dev,
diff --git a/include/hw/virtio/vhost.h b/include/hw/virtio/vhost.h
index 6a173cb9fa..603bf834be 100644
--- a/include/hw/virtio/vhost.h
+++ b/include/hw/virtio/vhost.h
@@ -315,7 +315,7 @@ uint64_t vhost_get_features(struct vhost_dev *hdev, const
int *feature_bits,
*/
void vhost_ack_features(struct vhost_dev *hdev, const int *feature_bits,
uint64_t features);
-bool vhost_has_free_slot(void);
+unsigned int vhost_get_free_memslots(void);
int vhost_net_set_backend(struct vhost_dev *hdev,
struct vhost_vring_file *file);
--
2.41.0
- [PATCH v4 00/18] virtio-mem: Expose device memory through multiple memslots, David Hildenbrand, 2023/09/26
- [PATCH v4 01/18] vhost: Rework memslot filtering and fix "used_memslot" tracking, David Hildenbrand, 2023/09/26
- [PATCH v4 02/18] vhost: Remove vhost_backend_can_merge() callback, David Hildenbrand, 2023/09/26
- [PATCH v4 03/18] softmmu/physmem: Fixup qemu_ram_block_from_host() documentation, David Hildenbrand, 2023/09/26
- [PATCH v4 04/18] kvm: Return number of free memslots, David Hildenbrand, 2023/09/26
- [PATCH v4 05/18] vhost: Return number of free memslots,
David Hildenbrand <=
- [PATCH v4 06/18] memory-device: Support memory devices with multiple memslots, David Hildenbrand, 2023/09/26
- [PATCH v4 07/18] stubs: Rename qmp_memory_device.c to memory_device.c, David Hildenbrand, 2023/09/26
- [PATCH v4 08/18] memory-device: Track required and actually used memslots in DeviceMemoryState, David Hildenbrand, 2023/09/26
- [PATCH v4 09/18] memory-device, vhost: Support memory devices that dynamically consume memslots, David Hildenbrand, 2023/09/26
- [PATCH v4 10/18] kvm: Add stub for kvm_get_max_memslots(), David Hildenbrand, 2023/09/26
- [PATCH v4 11/18] vhost: Add vhost_get_max_memslots(), David Hildenbrand, 2023/09/26
- [PATCH v4 12/18] memory-device, vhost: Support automatic decision on the number of memslots, David Hildenbrand, 2023/09/26
- [PATCH v4 17/18] memory, vhost: Allow for marking memory device memory regions unmergeable, David Hildenbrand, 2023/09/26
- [PATCH v4 13/18] memory: Clarify mapping requirements for RamDiscardManager, David Hildenbrand, 2023/09/26
- [PATCH v4 14/18] virtio-mem: Pass non-const VirtIOMEM via virtio_mem_range_cb, David Hildenbrand, 2023/09/26