@@ -20,6 +20,7 @@
#include <xen/symbols.h>
#include <xen/lib.h>
#include <xen/sched.h>
+#include <xen/xsplice.h>
#include <asm/div64.h>
#include <asm/page.h>
@@ -331,16 +332,23 @@ static char *pointer(char *str, char *end, const char **fmt_ptr,
{
unsigned long sym_size, sym_offset;
char namebuf[KSYM_NAME_LEN+1];
+ bool_t payload = 0;
/* Advance parents fmt string, as we have consumed 's' or 'S' */
++*fmt_ptr;
s = symbols_lookup((unsigned long)arg, &sym_size, &sym_offset, namebuf);
-
- /* If the symbol is not found, fall back to printing the address */
+ /* If the symbol is not found, fall back to printing the address. */
if ( !s )
break;
+ /*
+ * namebuf contents and s for core hypervisor are same but for xSplice
+ * payloads they differ (namebuf contains the name of the payload).
+ */
+ if ( strncmp(namebuf, s, KSYM_NAME_LEN) )
+ payload = 1;
+
/* Print symbol name */
str = string(str, end, s, -1, -1, 0);
@@ -354,6 +362,13 @@ static char *pointer(char *str, char *end, const char **fmt_ptr,
str = number(str, end, sym_size, 16, -1, -1, SPECIAL);
}
+ if ( payload )
+ {
+ str = string(str, end, " [", -1, -1, 0);
+ str = string(str, end, namebuf, -1, -1, 0);
+ str = string(str, end, "]", -1, -1, 0);
+ }
+
return str;
}
@@ -14,7 +14,9 @@
#include <xen/smp.h>
#include <xen/softirq.h>
#include <xen/spinlock.h>
+#include <xen/string.h>
#include <xen/symbols.h>
+#include <xen/virtual_region.h>
#include <xen/vmap.h>
#include <xen/wait.h>
#include <xen/xsplice_elf.h>
@@ -54,6 +56,8 @@ struct payload {
struct list_head applied_list; /* Linked to 'applied_list'. */
struct xsplice_patch_func_internal *funcs; /* The array of functions to patch. */
unsigned int nfuncs; /* Nr of functions to patch. */
+ struct virtual_region region; /* symbol, bug.frame patching and
+ exception table (x86). */
struct xsplice_symbol *symtab; /* All symbols. */
char *strtab; /* Pointer to .strtab. */
unsigned int nsyms; /* Nr of entries in .strtab and symbols. */
@@ -142,6 +146,51 @@ unsigned long xsplice_symbols_lookup_by_name(const char *symname)
return 0;
}
+static const char *xsplice_symbols_lookup(unsigned long addr,
+ unsigned long *symbolsize,
+ unsigned long *offset,
+ char *namebuf)
+{
+ struct payload *data;
+ unsigned int i;
+ int best;
+
+ /*
+ * No locking since this list is only ever changed during apply or revert
+ * context.
+ */
+ list_for_each_entry ( data, &applied_list, applied_list )
+ {
+ if ( (void *)addr < data->text_addr &&
+ (void *)addr >= (data->text_addr + data->pages * PAGE_SIZE) )
+ continue;
+
+ best = -1;
+
+ for ( i = 0; i < data->nsyms; i++ )
+ {
+ if ( data->symtab[i].value <= addr &&
+ (best == -1 ||
+ data->symtab[best].value < data->symtab[i].value) )
+ best = i;
+ }
+
+ if ( best == -1 )
+ return NULL;
+
+ if ( symbolsize )
+ *symbolsize = data->symtab[best].size;
+ if ( offset )
+ *offset = addr - data->symtab[best].value;
+ if ( namebuf )
+ strlcpy(namebuf, data->name, KSYM_NAME_LEN);
+
+ return data->symtab[best].name;
+ }
+
+ return NULL;
+}
+
static struct payload *find_payload(const char *name)
{
struct payload *data, *found = NULL;
@@ -361,6 +410,7 @@ static int prepare_payload(struct payload *payload,
const struct xsplice_elf_sec *sec;
unsigned int i;
struct xsplice_patch_func_internal *f;
+ struct virtual_region *region;
sec = xsplice_elf_sec_by_name(elf, ".xsplice.funcs");
ASSERT(sec);
@@ -422,6 +472,13 @@ static int prepare_payload(struct payload *payload,
}
}
+ /* Setup the virtual region with proper data. */
+ region = &payload->region;
+
+ region->symbols_lookup = xsplice_symbols_lookup;
+ region->start = (unsigned long)payload->text_addr;
+ region->end = (unsigned long)(payload->text_addr + payload->text_size);
+
return 0;
}
@@ -768,6 +825,7 @@ static int apply_payload(struct payload *data)
arch_xsplice_patching_leave();
list_add_tail(&data->applied_list, &applied_list);
+ register_virtual_region(&data->region);
return 0;
}
@@ -786,6 +844,7 @@ static int revert_payload(struct payload *data)
arch_xsplice_patching_leave();
list_del_init(&data->applied_list);
+ unregister_virtual_region(&data->region);
return 0;
}