From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Don Slutz <dslutz@verizon.com>
Cc: Ian Campbell <ian.campbell@citrix.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
George Dunlap <george.dunlap@eu.citrix.com>,
Don Slutz <Don@CloudSwitch.com>,
Ian Jackson <ian.jackson@eu.citrix.com>,
xen-devel@lists.xen.org, Jan Beulich <JBeulich@suse.com>
Subject: Re: [PATCH v2 11/12] xenctx: Dump registers via hvm info if available.
Date: Wed, 6 Nov 2013 20:35:03 +0000 [thread overview]
Message-ID: <527AA7F7.7010901@citrix.com> (raw)
In-Reply-To: <1383768500-4245-12-git-send-email-dslutz@terremark.com>
On 06/11/13 20:08, Don Slutz wrote:
> From: Don Slutz <Don@CloudSwitch.com>
>
> This allows the output of registers that are not available in the
> not HVM view.
>
> Look up symbols for various registers as kernel data.
>
> Also a minor check that ip, sp, and cr3 are the same.
>
> Signed-off-by: Don Slutz <Don@CloudSwitch.com>
What about tools/misc/xen-hvmctx ?
This is a particularly awkward tool where we have two tools in different
subtrees which do about 80% the same thing.
I would certainly not object if someone were to merge these two tools
into one. It is particularly awkward as xen-hvmctx ends up being
installed into /usr/sbin/ and is therefore on the $PATH, while xenctx
ends up off the $PATH in /usr/lib/xen/bin/
I am also fairly sure we have other examples of tools like this with
sightly different variants living in different locations.
~Andrew
> ---
> tools/xentrace/xenctx.c | 284 ++++++++++++++++++++++++++++++++++++++++++++++-
> 1 files changed, 282 insertions(+), 2 deletions(-)
>
> diff --git a/tools/xentrace/xenctx.c b/tools/xentrace/xenctx.c
> index 6ec9c74..d3cbb22 100644
> --- a/tools/xentrace/xenctx.c
> +++ b/tools/xentrace/xenctx.c
> @@ -415,6 +415,247 @@ static void print_ctx(vcpu_guest_context_any_t *ctx)
> print_ctx_64(&ctx->x64);
> }
>
> +#if defined(__i386__) || defined(__x86_64__)
> +static void print_cpuctx_regs64(struct hvm_hw_cpu *ctx)
> +{
> + printf("rip: %016"PRIx64, ctx->rip);
> + print_symbol(ctx->rip, KERNEL_TEXT_ADDR);
> + print_flags(ctx->rflags);
> + printf("rsp: %016"PRIx64"\n", ctx->rsp);
> +
> + printf("rax: %016"PRIx64"\t", ctx->rax);
> + printf("rcx: %016"PRIx64"\t", ctx->rcx);
> + printf("rdx: %016"PRIx64"\n", ctx->rdx);
> +
> + printf("rbx: %016"PRIx64"\t", ctx->rbx);
> + printf("rsi: %016"PRIx64"\t", ctx->rsi);
> + printf("rdi: %016"PRIx64"\n", ctx->rdi);
> +
> + printf("rbp: %016"PRIx64"\t", ctx->rbp);
> + printf(" r8: %016"PRIx64"\t", ctx->r8);
> + printf(" r9: %016"PRIx64"\n", ctx->r9);
> +
> + printf("r10: %016"PRIx64"\t", ctx->r10);
> + printf("r11: %016"PRIx64"\t", ctx->r11);
> + printf("r12: %016"PRIx64"\n", ctx->r12);
> +
> + printf("r13: %016"PRIx64"\t", ctx->r13);
> + printf("r14: %016"PRIx64"\t", ctx->r14);
> + printf("r15: %016"PRIx64"\n", ctx->r15);
> +
> + printf(" cs: %04x @ %016"PRIx64, ctx->cs_sel, ctx->cs_base);
> + if (ctx->cs_base)
> + print_symbol(ctx->cs_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->cs_limit, ctx->cs_arbytes);
> + printf(" ss: %04x @ %016"PRIx64, ctx->ss_sel, ctx->ss_base);
> + if (ctx->ss_base)
> + print_symbol(ctx->ss_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->ss_limit, ctx->ss_arbytes);
> + printf(" ds: %04x @ %016"PRIx64, ctx->ds_sel, ctx->ds_base);
> + if (ctx->ds_base)
> + print_symbol(ctx->ds_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->ds_limit, ctx->ds_arbytes);
> + printf(" es: %04x @ %016"PRIx64, ctx->es_sel, ctx->es_base);
> + if (ctx->es_base)
> + print_symbol(ctx->es_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->es_limit, ctx->es_arbytes);
> + printf(" fs: %04x @ %016"PRIx64, ctx->fs_sel, ctx->fs_base);
> + if (ctx->fs_base)
> + print_symbol(ctx->fs_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->fs_limit, ctx->fs_arbytes);
> + printf(" gs: %04x @ %016"PRIx64"\\%016"PRIx64, ctx->gs_sel,
> + ctx->gs_base, ctx->shadow_gs);
> + if (ctx->gs_base)
> + print_symbol(ctx->gs_base, KERNEL_DATA_ADDR);
> + printf("\\");
> + if (ctx->shadow_gs)
> + print_symbol(ctx->shadow_gs, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->gs_limit, ctx->gs_arbytes);
> +
> + if (xenctx.disp_all) {
> + printf(" tr: %04x @ %016"PRIx64, ctx->tr_sel, ctx->tr_base);
> + if (ctx->tr_base)
> + print_symbol(ctx->tr_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->tr_limit, ctx->tr_arbytes);
> + printf(" ldt: %04x @ %016"PRIx64, ctx->ldtr_sel, ctx->ldtr_base);
> + if (ctx->ldtr_base)
> + print_symbol(ctx->ldtr_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->ldtr_limit, ctx->ldtr_arbytes);
> + printf("\n");
> + printf("cr0: %08"PRIx64"\n", ctx->cr0);
> + printf("cr2: %08"PRIx64, ctx->cr2);
> + print_symbol(ctx->cr2, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("cr3: %08"PRIx64"\n", ctx->cr3);
> + printf("cr4: %08"PRIx64"\n", ctx->cr4);
> + printf("\n");
> + printf("dr0: %08"PRIx64, ctx->dr0);
> + print_symbol(ctx->dr0, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr1: %08"PRIx64, ctx->dr1);
> + print_symbol(ctx->dr1, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr2: %08"PRIx64, ctx->dr2);
> + print_symbol(ctx->dr2, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr3: %08"PRIx64, ctx->dr3);
> + print_symbol(ctx->dr3, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr6: %08"PRIx64"\n", ctx->dr6);
> + printf("dr7: %08"PRIx64"\n", ctx->dr7);
> + printf("\n");
> + printf("gdt: %016"PRIx64"/%lx", ctx->gdtr_base, (long)(ctx->gdtr_limit));
> + print_symbol(ctx->gdtr_base, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("idt: %016"PRIx64"/%lx", ctx->idtr_base, (long)(ctx->idtr_limit));
> + print_symbol(ctx->idtr_base, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("\n");
> + printf(" EFER: %08"PRIx64" flags: %08"PRIx64"\n",
> + ctx->msr_efer, ctx->msr_flags);
> + printf("SYSENTER cs: %08"PRIx64" esp: %08"PRIx64" eip: %08"PRIx64,
> + ctx->sysenter_cs, ctx->sysenter_esp, ctx->sysenter_eip);
> + print_symbol(ctx->sysenter_eip, KERNEL_TEXT_ADDR);
> + printf("\n");
> + printf(" STAR: %08"PRIx64" SFMASK: %08"PRIx64"\n",
> + ctx->msr_star, ctx->msr_syscall_mask);
> + printf("LSTAR: %08"PRIx64, ctx->msr_lstar);
> + print_symbol(ctx->msr_lstar, KERNEL_TEXT_ADDR);
> + printf("\n");
> + printf("CSTAR: %08"PRIx64, ctx->msr_cstar);
> + print_symbol(ctx->msr_cstar, KERNEL_TEXT_ADDR);
> + printf("\n");
> + printf("\n");
> + printf("\n");
> + printf(" tsc: %016"PRIx64"\n", ctx->tsc);
> + printf("\n");
> + printf("\n");
> + printf("pending_event: %lx pending_vector: %lx\n",
> + (long)(ctx->pending_event), (long)(ctx->pending_vector));
> + printf("pending error_code: %lx\n\n", (long)(ctx->error_code));
> + }
> +}
> +
> +static void print_cpuctx_regs32(struct hvm_hw_cpu *ctx)
> +{
> + printf("cs:eip: %04x:%08x", ctx->cs_sel, (uint32_t)ctx->rip);
> + print_symbol((uint32_t)ctx->rip, KERNEL_TEXT_ADDR);
> + print_flags((uint32_t)ctx->rflags);
> + printf("ss:esp: %04x:%08x\n", ctx->ss_sel, (uint32_t)ctx->rsp);
> + printf("\n");
> +
> + printf("eax: %08x\t", (uint32_t)ctx->rax);
> + printf("ebx: %08x\t", (uint32_t)ctx->rbx);
> + printf("ecx: %08x\t", (uint32_t)ctx->rcx);
> + printf("edx: %08x\n", (uint32_t)ctx->rdx);
> +
> + printf("esi: %08x\t", (uint32_t)ctx->rsi);
> + printf("edi: %08x\t", (uint32_t)ctx->rdi);
> + printf("ebp: %08x\n", (uint32_t)ctx->rbp);
> + printf("\n");
> +
> + printf(" cs: %04x @ %08"PRIx64, ctx->cs_sel, ctx->cs_base);
> + if (ctx->cs_base)
> + print_symbol(ctx->cs_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->cs_limit, ctx->cs_arbytes);
> + printf(" ss: %04x @ %08"PRIx64, ctx->ss_sel, ctx->ss_base);
> + if (ctx->ss_base)
> + print_symbol(ctx->ss_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->ss_limit, ctx->ss_arbytes);
> + printf(" ds: %04x @ %08"PRIx64, ctx->ds_sel, ctx->ds_base);
> + if (ctx->ds_base)
> + print_symbol(ctx->ds_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->ds_limit, ctx->ds_arbytes);
> + printf(" es: %04x @ %08"PRIx64, ctx->es_sel, ctx->es_base);
> + if (ctx->es_base)
> + print_symbol(ctx->es_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->es_limit, ctx->es_arbytes);
> + printf(" fs: %04x @ %08"PRIx64, ctx->fs_sel, ctx->fs_base);
> + if (ctx->fs_base)
> + print_symbol(ctx->fs_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->fs_limit, ctx->fs_arbytes);
> + printf(" gs: %04x @ %08"PRIx64"\\%08"PRIx64, ctx->gs_sel,
> + ctx->gs_base, ctx->shadow_gs);
> + if (ctx->gs_base)
> + print_symbol(ctx->gs_base, KERNEL_DATA_ADDR);
> + printf("\\");
> + if (ctx->shadow_gs)
> + print_symbol(ctx->shadow_gs, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->gs_limit, ctx->gs_arbytes);
> +
> + if (xenctx.disp_all) {
> + printf(" tr: %04x @ %08"PRIx64, ctx->tr_sel, ctx->tr_base);
> + if (ctx->tr_base)
> + print_symbol(ctx->tr_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->tr_limit, ctx->tr_arbytes);
> + printf(" ldt: %04x @ %08"PRIx64, ctx->ldtr_sel, ctx->ldtr_base);
> + if (ctx->ldtr_base)
> + print_symbol(ctx->ldtr_base, KERNEL_DATA_ADDR);
> + printf("\n /%08x(%x)\n", ctx->ldtr_limit, ctx->ldtr_arbytes);
> + printf("\n");
> + printf("cr0: %08"PRIx64"\n", ctx->cr0);
> + printf("cr2: %08"PRIx64, ctx->cr2);
> + print_symbol(ctx->cr2, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("cr3: %08"PRIx64"\n", ctx->cr3);
> + printf("cr4: %08"PRIx64"\n", ctx->cr4);
> + printf("\n");
> + printf("dr0: %08"PRIx64, ctx->dr0);
> + print_symbol(ctx->dr0, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr1: %08"PRIx64, ctx->dr1);
> + print_symbol(ctx->dr1, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr2: %08"PRIx64, ctx->dr2);
> + print_symbol(ctx->dr2, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr3: %08"PRIx64, ctx->dr3);
> + print_symbol(ctx->dr3, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("dr6: %08"PRIx64"\n", ctx->dr6);
> + printf("dr7: %08"PRIx64"\n", ctx->dr7);
> + printf("\n");
> + printf("gdt: %08"PRIx64"/%lx", ctx->gdtr_base, (long)(ctx->gdtr_limit));
> + print_symbol(ctx->gdtr_base, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("idt: %08"PRIx64"/%lx", ctx->idtr_base, (long)(ctx->idtr_limit));
> + print_symbol(ctx->idtr_base, KERNEL_DATA_ADDR);
> + printf("\n");
> + printf("\n");
> + printf(" EFER: %08"PRIx64" flags: %08"PRIx64"\n",
> + ctx->msr_efer, ctx->msr_flags);
> + printf("SYSENTER cs: %08"PRIx64" esp: %08"PRIx64" eip: %08"PRIx64,
> + ctx->sysenter_cs, ctx->sysenter_esp, ctx->sysenter_eip);
> + print_symbol(ctx->sysenter_eip, KERNEL_TEXT_ADDR);
> + printf("\n");
> + printf(" STAR: %08"PRIx64" SFMASK: %08"PRIx64"\n",
> + ctx->msr_star, ctx->msr_syscall_mask);
> + printf("LSTAR: %08"PRIx64, ctx->msr_lstar);
> + print_symbol(ctx->msr_lstar, KERNEL_TEXT_ADDR);
> + printf("\n");
> + printf("CSTAR: %08"PRIx64, ctx->msr_cstar);
> + print_symbol(ctx->msr_cstar, KERNEL_TEXT_ADDR);
> + printf("\n");
> + printf("\n");
> + printf(" tsc: %016"PRIx64"\n", ctx->tsc);
> + printf("\n");
> + printf("\n");
> + printf("pending_event: %lx pending_vector: %lx\n",
> + (long)(ctx->pending_event), (long)(ctx->pending_vector));
> + printf("pending error_code: %lx\n\n", (long)(ctx->error_code));
> + }
> +}
> +
> +static void print_cpuctx(struct hvm_hw_cpu *ctx)
> +{
> + if (guest_word_size == 8)
> + print_cpuctx_regs64(ctx);
> + else
> + print_cpuctx_regs32(ctx);
> +
> +}
> +#endif
> +
> #define NONPROT_MODE_SEGMENT_SHIFT 4
>
> static guest_word_t instr_pointer(vcpu_guest_context_any_t *ctx)
> @@ -887,6 +1128,9 @@ static int print_stack(vcpu_guest_context_any_t *ctx, int vcpu, int width, guest
> static void dump_ctx(int vcpu, guest_word_t mem_addr, guest_word_t stk_addr)
> {
> vcpu_guest_context_any_t ctx;
> +#if defined(__i386__) || defined(__x86_64__)
> + struct hvm_hw_cpu cpuctx;
> +#endif
>
> if (xc_vcpu_getcontext(xenctx.xc_handle, xenctx.domid, vcpu, &ctx) < 0) {
> perror("xc_vcpu_getcontext");
> @@ -896,7 +1140,6 @@ static void dump_ctx(int vcpu, guest_word_t mem_addr, guest_word_t stk_addr)
> #if defined(__i386__) || defined(__x86_64__)
> {
> if (xenctx.dominfo.hvm) {
> - struct hvm_hw_cpu cpuctx;
> xen_capabilities_info_t xen_caps = "";
> if (xc_domain_hvm_getcontext_partial(
> xenctx.xc_handle, xenctx.domid, HVM_SAVE_CODE(CPU),
> @@ -931,7 +1174,44 @@ static void dump_ctx(int vcpu, guest_word_t mem_addr, guest_word_t stk_addr)
> #endif
> } else {
> if (!stk_addr) {
> - print_ctx(&ctx);
> +#if defined(__i386__) || defined(__x86_64__)
> + if (xenctx.dominfo.hvm && ctxt_word_size == 8) {
> + if (guest_word_size == 4) {
> + if ((((uint32_t)ctx.x64.user_regs.eip) != cpuctx.rip) ||
> + (((uint32_t)ctx.x64.user_regs.esp) != cpuctx.rsp) ||
> + (((uint32_t)ctx.x64.ctrlreg[3]) != cpuctx.cr3)) {
> + fprintf(stderr, "Regs mismatch ip=%llx vs %llx sp=%llx vs %llx cr3=%llx vs %llx\n",
> + (long long)((uint32_t)ctx.x64.user_regs.eip),
> + (long long)cpuctx.rip,
> + (long long)((uint32_t)ctx.x64.user_regs.esp),
> + (long long)cpuctx.rsp,
> + (long long)((uint32_t)ctx.x64.ctrlreg[3]),
> + (long long)cpuctx.cr3);
> + fprintf(stdout, "=============Regs mismatch start=============\n");
> + print_ctx(&ctx);
> + fprintf(stdout, "=============Regs mismatch end=============\n");
> + }
> + } else {
> + if ((ctx.x64.user_regs.eip != cpuctx.rip) ||
> + (ctx.x64.user_regs.esp != cpuctx.rsp) ||
> + (ctx.x64.ctrlreg[3] != cpuctx.cr3)) {
> + fprintf(stderr, "Regs mismatch ip=%llx vs %llx sp=%llx vs %llx cr3=%llx vs %llx\n",
> + (long long)ctx.x64.user_regs.eip,
> + (long long)cpuctx.rip,
> + (long long)ctx.x64.user_regs.esp,
> + (long long)cpuctx.rsp,
> + (long long)ctx.x64.ctrlreg[3],
> + (long long)cpuctx.cr3);
> + fprintf(stdout, "=============Regs mismatch start=============\n");
> + print_ctx(&ctx);
> + fprintf(stdout, "=============Regs mismatch end=============\n");
> + }
> + }
> + print_cpuctx(&cpuctx);
> + }
> + else
> +#endif
> + print_ctx(&ctx);
> }
> #ifndef NO_TRANSLATION
> if (!stk_addr) {
next prev parent reply other threads:[~2013-11-06 20:35 UTC|newest]
Thread overview: 61+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-06 20:08 [PATCH v2 00/12] xenctx: Many changes Don Slutz
2013-11-06 20:08 ` [PATCH v2 01/12] xenctx: Clean up stack trace when hypercall_page not in symbol table Don Slutz
2013-11-07 8:07 ` Jan Beulich
2013-11-07 8:22 ` Ian Campbell
2013-11-07 8:47 ` Jan Beulich
2013-11-07 12:36 ` Ian Campbell
2013-11-07 13:28 ` Don Slutz
2013-11-07 12:38 ` Ian Campbell
2013-11-06 20:08 ` [PATCH v2 02/12] xenctx: Add -2 (--two-pages) option to switch stack size to 8KiB Don Slutz
2013-11-07 8:04 ` Jan Beulich
2013-11-07 12:40 ` Ian Campbell
2013-11-07 15:24 ` Don Slutz
2013-11-06 20:08 ` [PATCH v2 03/12] xenctx: Output ascii version of stack also Don Slutz
2013-11-07 8:09 ` Jan Beulich
2013-11-07 12:43 ` Ian Campbell
2013-11-07 13:17 ` Jan Beulich
2013-11-07 13:21 ` Jan Beulich
2013-11-08 0:12 ` Don Slutz
2013-11-06 20:08 ` [PATCH v2 04/12] xenctx: Add stack address to dump, switch to 16 bytes per line Don Slutz
2013-11-07 8:12 ` Jan Beulich
2013-11-07 12:46 ` Ian Campbell
2013-11-08 0:13 ` Don Slutz
2013-11-06 20:08 ` [PATCH v2 05/12] xenctx: Change print_symbol to do the space before Don Slutz
2013-11-07 8:13 ` Jan Beulich
2013-11-07 12:47 ` Ian Campbell
2013-11-06 20:08 ` [PATCH v2 06/12] xenctx: More info on failed to map page Don Slutz
2013-11-07 12:48 ` Ian Campbell
2013-11-06 20:08 ` [PATCH v2 07/12] xenctx: Add stack addr to call trace Don Slutz
2013-11-07 12:50 ` Ian Campbell
2013-11-07 14:34 ` Don Slutz
2013-11-07 14:44 ` Ian Campbell
2013-11-07 15:06 ` Don Slutz
2013-11-07 16:03 ` Ian Campbell
2013-11-07 18:13 ` Don Slutz
2013-11-08 10:15 ` Ian Campbell
2013-11-06 20:08 ` [PATCH v2 08/12] xenctx: Add -d <daddr> option to dump memory at daddr as a stack Don Slutz
2013-11-07 8:22 ` Jan Beulich
2013-11-08 0:21 ` Don Slutz
2013-11-06 20:08 ` [PATCH v2 09/12] xenctx: Add -m <maddr> option to dump memory at maddr Don Slutz
2013-11-07 8:25 ` Jan Beulich
2013-11-08 0:24 ` Don Slutz
2013-11-06 20:08 ` [PATCH v2 10/12] xenctx: change is_kernel_text() into is_kernel_addr() Don Slutz
2013-11-07 8:35 ` Jan Beulich
2013-11-08 0:51 ` Don Slutz
2013-11-08 8:40 ` Jan Beulich
2013-11-08 9:50 ` Ian Campbell
2013-11-08 13:50 ` Don Slutz
2013-11-06 20:08 ` [PATCH v2 11/12] xenctx: Dump registers via hvm info if available Don Slutz
2013-11-06 20:35 ` Andrew Cooper [this message]
2013-11-07 18:56 ` Don Slutz
2013-11-08 10:13 ` Ian Campbell
2013-11-08 10:29 ` Andrew Cooper
2013-11-08 10:32 ` Ian Campbell
2013-11-07 8:38 ` Jan Beulich
2013-11-07 21:19 ` Don Slutz
2013-11-06 20:08 ` [PATCH v2 12/12] xenctx: Add optional fCPU Don Slutz
2013-11-07 8:40 ` Jan Beulich
2013-11-07 12:53 ` Ian Campbell
2013-11-07 21:24 ` Don Slutz
2013-11-08 11:29 ` [PATCH v2 00/12] xenctx: Many changes George Dunlap
2013-11-08 12:54 ` Ian Campbell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=527AA7F7.7010901@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=Don@CloudSwitch.com \
--cc=JBeulich@suse.com \
--cc=dslutz@verizon.com \
--cc=george.dunlap@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=ian.jackson@eu.citrix.com \
--cc=stefano.stabellini@eu.citrix.com \
--cc=xen-devel@lists.xen.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).