qemu_ram_free: Implement it
Now that we can support a ram_addr_t space with holes, we can implement qemu_ram_free(). Signed-off-by: Alex Williamson <alex.williamson@redhat.com> Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
This commit is contained in:
parent
a55bbe3187
commit
04b1665372
@ -867,6 +867,9 @@ typedef struct RAMBlock {
|
||||
ram_addr_t length;
|
||||
char idstr[256];
|
||||
QLIST_ENTRY(RAMBlock) next;
|
||||
#if defined(__linux__) && !defined(TARGET_S390X)
|
||||
int fd;
|
||||
#endif
|
||||
} RAMBlock;
|
||||
|
||||
typedef struct RAMList {
|
||||
|
62
exec.c
62
exec.c
@ -2701,7 +2701,9 @@ static long gethugepagesize(const char *path)
|
||||
return fs.f_bsize;
|
||||
}
|
||||
|
||||
static void *file_ram_alloc(ram_addr_t memory, const char *path)
|
||||
static void *file_ram_alloc(RAMBlock *block,
|
||||
ram_addr_t memory,
|
||||
const char *path)
|
||||
{
|
||||
char *filename;
|
||||
void *area;
|
||||
@ -2764,11 +2766,38 @@ static void *file_ram_alloc(ram_addr_t memory, const char *path)
|
||||
close(fd);
|
||||
return (NULL);
|
||||
}
|
||||
block->fd = fd;
|
||||
return area;
|
||||
}
|
||||
#endif
|
||||
|
||||
static ram_addr_t find_ram_offset(ram_addr_t size)
|
||||
{
|
||||
RAMBlock *block, *next_block;
|
||||
ram_addr_t offset, mingap = ULONG_MAX;
|
||||
|
||||
if (QLIST_EMPTY(&ram_list.blocks))
|
||||
return 0;
|
||||
|
||||
QLIST_FOREACH(block, &ram_list.blocks, next) {
|
||||
ram_addr_t end, next = ULONG_MAX;
|
||||
|
||||
end = block->offset + block->length;
|
||||
|
||||
QLIST_FOREACH(next_block, &ram_list.blocks, next) {
|
||||
if (next_block->offset >= end) {
|
||||
next = MIN(next, next_block->offset);
|
||||
}
|
||||
}
|
||||
if (next - end >= size && next - end < mingap) {
|
||||
offset = end;
|
||||
mingap = next - end;
|
||||
}
|
||||
}
|
||||
return offset;
|
||||
}
|
||||
|
||||
static ram_addr_t last_ram_offset(void)
|
||||
{
|
||||
RAMBlock *block;
|
||||
ram_addr_t last = 0;
|
||||
@ -2812,7 +2841,7 @@ ram_addr_t qemu_ram_alloc(DeviceState *dev, const char *name, ram_addr_t size)
|
||||
|
||||
if (mem_path) {
|
||||
#if defined (__linux__) && !defined(TARGET_S390X)
|
||||
new_block->host = file_ram_alloc(size, mem_path);
|
||||
new_block->host = file_ram_alloc(new_block, size, mem_path);
|
||||
if (!new_block->host) {
|
||||
new_block->host = qemu_vmalloc(size);
|
||||
#ifdef MADV_MERGEABLE
|
||||
@ -2842,7 +2871,7 @@ ram_addr_t qemu_ram_alloc(DeviceState *dev, const char *name, ram_addr_t size)
|
||||
QLIST_INSERT_HEAD(&ram_list.blocks, new_block, next);
|
||||
|
||||
ram_list.phys_dirty = qemu_realloc(ram_list.phys_dirty,
|
||||
(new_block->offset + size) >> TARGET_PAGE_BITS);
|
||||
last_ram_offset() >> TARGET_PAGE_BITS);
|
||||
memset(ram_list.phys_dirty + (new_block->offset >> TARGET_PAGE_BITS),
|
||||
0xff, size >> TARGET_PAGE_BITS);
|
||||
|
||||
@ -2854,7 +2883,32 @@ ram_addr_t qemu_ram_alloc(DeviceState *dev, const char *name, ram_addr_t size)
|
||||
|
||||
void qemu_ram_free(ram_addr_t addr)
|
||||
{
|
||||
/* TODO: implement this. */
|
||||
RAMBlock *block;
|
||||
|
||||
QLIST_FOREACH(block, &ram_list.blocks, next) {
|
||||
if (addr == block->offset) {
|
||||
QLIST_REMOVE(block, next);
|
||||
if (mem_path) {
|
||||
#if defined (__linux__) && !defined(TARGET_S390X)
|
||||
if (block->fd) {
|
||||
munmap(block->host, block->length);
|
||||
close(block->fd);
|
||||
} else {
|
||||
qemu_vfree(block->host);
|
||||
}
|
||||
#endif
|
||||
} else {
|
||||
#if defined(TARGET_S390X) && defined(CONFIG_KVM)
|
||||
munmap(block->host, block->length);
|
||||
#else
|
||||
qemu_vfree(block->host);
|
||||
#endif
|
||||
}
|
||||
qemu_free(block);
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
/* Return a host pointer to ram allocated with qemu_ram_alloc.
|
||||
|
Loading…
Reference in New Issue
Block a user