From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Durrant Subject: [PATCH v3 2/3] x86/viridian: Make logging less verbose Date: Mon, 4 Aug 2014 18:37:46 +0100 Message-ID: <1407173867-10516-3-git-send-email-paul.durrant@citrix.com> References: <1407173867-10516-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: <1407173867-10516-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 The use of gdprintk() adds uninteresting prefixes to the log lines, and there's really too many lines. This patch reduces the verbosity. Signed-off-by: Paul Durrant Cc: Keir Fraser Cc: Jan Beulich --- xen/arch/x86/hvm/viridian.c | 36 +++++++++++++----------------------- 1 file changed, 13 insertions(+), 23 deletions(-) diff --git a/xen/arch/x86/hvm/viridian.c b/xen/arch/x86/hvm/viridian.c index 31c9656..6490a09 100644 --- a/xen/arch/x86/hvm/viridian.c +++ b/xen/arch/x86/hvm/viridian.c @@ -120,37 +120,27 @@ 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", - d->arch.hvm_domain.viridian.guest_os_id.fields.vendor); - gdprintk(XENLOG_INFO, "\tos: %x\n", - d->arch.hvm_domain.viridian.guest_os_id.fields.os); - gdprintk(XENLOG_INFO, "\tmajor: %x\n", - d->arch.hvm_domain.viridian.guest_os_id.fields.major); - gdprintk(XENLOG_INFO, "\tminor: %x\n", - d->arch.hvm_domain.viridian.guest_os_id.fields.minor); - gdprintk(XENLOG_INFO, "\tsp: %x\n", - d->arch.hvm_domain.viridian.guest_os_id.fields.service_pack); - gdprintk(XENLOG_INFO, "\tbuild: %x\n", - d->arch.hvm_domain.viridian.guest_os_id.fields.build_number); + printk(XENLOG_G_INFO "GUEST_OS_ID: vendor: %x os: %x major: %x minor: %x sp: %x build: %x\n", + d->arch.hvm_domain.viridian.guest_os_id.fields.vendor, + d->arch.hvm_domain.viridian.guest_os_id.fields.os, + d->arch.hvm_domain.viridian.guest_os_id.fields.major, + d->arch.hvm_domain.viridian.guest_os_id.fields.minor, + d->arch.hvm_domain.viridian.guest_os_id.fields.service_pack, + 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", - d->arch.hvm_domain.viridian.hypercall_gpa.fields.enabled); - gdprintk(XENLOG_INFO, "\tpfn: %lx\n", - (unsigned long)d->arch.hvm_domain.viridian.hypercall_gpa.fields.pfn); + printk(XENLOG_G_INFO "HYPERCALL: enabled: %x pfn: %lx\n", + d->arch.hvm_domain.viridian.hypercall_gpa.fields.enabled, + (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", - v->arch.hvm_vcpu.viridian.apic_assist.fields.enabled); - gdprintk(XENLOG_INFO, "\tpfn: %lx\n", - (unsigned long)v->arch.hvm_vcpu.viridian.apic_assist.fields.pfn); + printk(XENLOG_G_INFO "APIC_ASSIST[%d]: enabled: %x pfn: %lx\n", v->vcpu_id, + v->arch.hvm_vcpu.viridian.apic_assist.fields.enabled, + (unsigned long)v->arch.hvm_vcpu.viridian.apic_assist.fields.pfn); } static void enable_hypercall_page(struct domain *d) -- 1.7.10.4