[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH uq/master 1/2] Add qemu_ram_remap
From: |
Huang Ying |
Subject: |
Re: [Qemu-devel] [PATCH uq/master 1/2] Add qemu_ram_remap |
Date: |
Fri, 14 Jan 2011 09:23:23 +0800 |
On Fri, 2011-01-14 at 05:14 +0800, Blue Swirl wrote:
> On Thu, Jan 13, 2011 at 8:34 AM, Huang Ying <address@hidden> wrote:
> > qemu_ram_remap() unmaps the specified RAM pages, then re-maps these
> > pages again. This is used by KVM HWPoison support to clear HWPoisoned
> > page tables across guest rebooting, so that a new page may be
> > allocated later to recover the memory error.
> >
> > Signed-off-by: Huang Ying <address@hidden>
> > ---
> > cpu-all.h | 4 +++
> > cpu-common.h | 1
> > exec.c | 61
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> > 3 files changed, 65 insertions(+), 1 deletion(-)
> >
> > --- a/cpu-all.h
> > +++ b/cpu-all.h
> > @@ -863,10 +863,14 @@ target_phys_addr_t cpu_get_phys_page_deb
> > extern int phys_ram_fd;
> > extern ram_addr_t ram_size;
> >
> > +/* RAM is pre-allocated and passed into qemu_ram_alloc_from_ptr */
> > +#define RAM_PREALLOC_MASK (1 << 0)
> > +
> > typedef struct RAMBlock {
> > uint8_t *host;
> > ram_addr_t offset;
> > ram_addr_t length;
> > + uint32_t flags;
> > char idstr[256];
> > QLIST_ENTRY(RAMBlock) next;
> > #if defined(__linux__) && !defined(TARGET_S390X)
> > --- a/exec.c
> > +++ b/exec.c
> > @@ -2830,6 +2830,7 @@ ram_addr_t qemu_ram_alloc_from_ptr(Devic
> >
> > if (host) {
> > new_block->host = host;
> > + new_block->flags |= RAM_PREALLOC_MASK;
> > } else {
> > if (mem_path) {
> > #if defined (__linux__) && !defined(TARGET_S390X)
> > @@ -2883,7 +2884,9 @@ void qemu_ram_free(ram_addr_t addr)
> > QLIST_FOREACH(block, &ram_list.blocks, next) {
> > if (addr == block->offset) {
> > QLIST_REMOVE(block, next);
> > - if (mem_path) {
> > + if (block->flags & RAM_PREALLOC_MASK)
>
> Missing braces.
Sorry, forgot this one, will fix it.
Best Regards,
Huang Ying