From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Durrant Subject: [PATCH v2 2/3] x86/viridian: Note that logging is under control of the guest Date: Mon, 4 Aug 2014 14:12:40 +0100 Message-ID: <1407157961-7239-3-git-send-email-paul.durrant@citrix.com> References: <1407157961-7239-1-git-send-email-paul.durrant@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1407157961-7239-1-git-send-email-paul.durrant@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xen.org Cc: Paul Durrant , Keir Fraser , Jan Beulich List-Id: xen-devel@lists.xenproject.org All viridian logging is actually in response to an action (MSR read/write, etc.) in the guest. As such XENLOG_G_ log levels should be used. Signed-off-by: Paul Durrant Cc: Keir Fraser Cc: Jan Beulich --- xen/arch/x86/hvm/viridian.c | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/xen/arch/x86/hvm/viridian.c b/xen/arch/x86/hvm/viridian.c index 31c9656..2d79403 100644 --- a/xen/arch/x86/hvm/viridian.c +++ b/xen/arch/x86/hvm/viridian.c @@ -120,36 +120,36 @@ int cpuid_viridian_leaves(unsigned int leaf, unsigned int *eax, static void dump_guest_os_id(const struct domain *d) { - gdprintk(XENLOG_INFO, "GUEST_OS_ID:\n"); - gdprintk(XENLOG_INFO, "\tvendor: %x\n", + gdprintk(XENLOG_G_INFO, "GUEST_OS_ID:\n"); + gdprintk(XENLOG_G_INFO, "\tvendor: %x\n", d->arch.hvm_domain.viridian.guest_os_id.fields.vendor); - gdprintk(XENLOG_INFO, "\tos: %x\n", + gdprintk(XENLOG_G_INFO, "\tos: %x\n", d->arch.hvm_domain.viridian.guest_os_id.fields.os); - gdprintk(XENLOG_INFO, "\tmajor: %x\n", + gdprintk(XENLOG_G_INFO, "\tmajor: %x\n", d->arch.hvm_domain.viridian.guest_os_id.fields.major); - gdprintk(XENLOG_INFO, "\tminor: %x\n", + gdprintk(XENLOG_G_INFO, "\tminor: %x\n", d->arch.hvm_domain.viridian.guest_os_id.fields.minor); - gdprintk(XENLOG_INFO, "\tsp: %x\n", + gdprintk(XENLOG_G_INFO, "\tsp: %x\n", d->arch.hvm_domain.viridian.guest_os_id.fields.service_pack); - gdprintk(XENLOG_INFO, "\tbuild: %x\n", + gdprintk(XENLOG_G_INFO, "\tbuild: %x\n", d->arch.hvm_domain.viridian.guest_os_id.fields.build_number); } static void dump_hypercall(const struct domain *d) { - gdprintk(XENLOG_INFO, "HYPERCALL:\n"); - gdprintk(XENLOG_INFO, "\tenabled: %x\n", + gdprintk(XENLOG_G_INFO, "HYPERCALL:\n"); + gdprintk(XENLOG_G_INFO, "\tenabled: %x\n", d->arch.hvm_domain.viridian.hypercall_gpa.fields.enabled); - gdprintk(XENLOG_INFO, "\tpfn: %lx\n", + gdprintk(XENLOG_G_INFO, "\tpfn: %lx\n", (unsigned long)d->arch.hvm_domain.viridian.hypercall_gpa.fields.pfn); } static void dump_apic_assist(const struct vcpu *v) { - gdprintk(XENLOG_INFO, "APIC_ASSIST[%d]:\n", v->vcpu_id); - gdprintk(XENLOG_INFO, "\tenabled: %x\n", + gdprintk(XENLOG_G_INFO, "APIC_ASSIST[%d]:\n", v->vcpu_id); + gdprintk(XENLOG_G_INFO, "\tenabled: %x\n", v->arch.hvm_vcpu.viridian.apic_assist.fields.enabled); - gdprintk(XENLOG_INFO, "\tpfn: %lx\n", + gdprintk(XENLOG_G_INFO, "\tpfn: %lx\n", (unsigned long)v->arch.hvm_vcpu.viridian.apic_assist.fields.pfn); } -- 1.7.10.4