diff options
author | Joe Perches <joe@perches.com> | 2013-06-28 06:21:41 -0400 |
---|---|---|
committer | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2013-06-28 11:19:58 -0400 |
commit | 283c0972d53769ee44750cad4c27e3f5fa26ec1f (patch) | |
tree | 598a36146515f114e21047daa9e19ea8ac8e5fdc /drivers/xen/events.c | |
parent | 65e053a7038c608f89272d4fb1f899890c579f0a (diff) |
xen: Convert printks to pr_<level>
Convert printks to pr_<level> (excludes printk(KERN_DEBUG...)
to be more consistent throughout the xen subsystem.
Add pr_fmt with KBUILD_MODNAME or "xen:" KBUILD_MODNAME
Coalesce formats and add missing word spaces
Add missing newlines
Align arguments and reflow to 80 columns
Remove DRV_NAME from formats as pr_fmt adds the same content
This does change some of the prefixes of these messages
but it also does make them more consistent.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Diffstat (limited to 'drivers/xen/events.c')
-rw-r--r-- | drivers/xen/events.c | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/drivers/xen/events.c b/drivers/xen/events.c index 6a6bbe4ede92..a58ac435a9a4 100644 --- a/drivers/xen/events.c +++ b/drivers/xen/events.c | |||
@@ -21,6 +21,8 @@ | |||
21 | * Jeremy Fitzhardinge <jeremy@xensource.com>, XenSource Inc, 2007 | 21 | * Jeremy Fitzhardinge <jeremy@xensource.com>, XenSource Inc, 2007 |
22 | */ | 22 | */ |
23 | 23 | ||
24 | #define pr_fmt(fmt) "xen:" KBUILD_MODNAME ": " fmt | ||
25 | |||
24 | #include <linux/linkage.h> | 26 | #include <linux/linkage.h> |
25 | #include <linux/interrupt.h> | 27 | #include <linux/interrupt.h> |
26 | #include <linux/irq.h> | 28 | #include <linux/irq.h> |
@@ -600,8 +602,7 @@ static unsigned int __startup_pirq(unsigned int irq) | |||
600 | rc = HYPERVISOR_event_channel_op(EVTCHNOP_bind_pirq, &bind_pirq); | 602 | rc = HYPERVISOR_event_channel_op(EVTCHNOP_bind_pirq, &bind_pirq); |
601 | if (rc != 0) { | 603 | if (rc != 0) { |
602 | if (!probing_irq(irq)) | 604 | if (!probing_irq(irq)) |
603 | printk(KERN_INFO "Failed to obtain physical IRQ %d\n", | 605 | pr_info("Failed to obtain physical IRQ %d\n", irq); |
604 | irq); | ||
605 | return 0; | 606 | return 0; |
606 | } | 607 | } |
607 | evtchn = bind_pirq.port; | 608 | evtchn = bind_pirq.port; |
@@ -693,8 +694,8 @@ int xen_bind_pirq_gsi_to_irq(unsigned gsi, | |||
693 | 694 | ||
694 | irq = xen_irq_from_gsi(gsi); | 695 | irq = xen_irq_from_gsi(gsi); |
695 | if (irq != -1) { | 696 | if (irq != -1) { |
696 | printk(KERN_INFO "xen_map_pirq_gsi: returning irq %d for gsi %u\n", | 697 | pr_info("%s: returning irq %d for gsi %u\n", |
697 | irq, gsi); | 698 | __func__, irq, gsi); |
698 | goto out; | 699 | goto out; |
699 | } | 700 | } |
700 | 701 | ||
@@ -812,10 +813,10 @@ int xen_destroy_irq(int irq) | |||
812 | * (free_domain_pirqs). | 813 | * (free_domain_pirqs). |
813 | */ | 814 | */ |
814 | if ((rc == -ESRCH && info->u.pirq.domid != DOMID_SELF)) | 815 | if ((rc == -ESRCH && info->u.pirq.domid != DOMID_SELF)) |
815 | printk(KERN_INFO "domain %d does not have %d anymore\n", | 816 | pr_info("domain %d does not have %d anymore\n", |
816 | info->u.pirq.domid, info->u.pirq.pirq); | 817 | info->u.pirq.domid, info->u.pirq.pirq); |
817 | else if (rc) { | 818 | else if (rc) { |
818 | printk(KERN_WARNING "unmap irq failed %d\n", rc); | 819 | pr_warn("unmap irq failed %d\n", rc); |
819 | goto out; | 820 | goto out; |
820 | } | 821 | } |
821 | } | 822 | } |
@@ -1621,8 +1622,8 @@ static void restore_pirqs(void) | |||
1621 | 1622 | ||
1622 | rc = HYPERVISOR_physdev_op(PHYSDEVOP_map_pirq, &map_irq); | 1623 | rc = HYPERVISOR_physdev_op(PHYSDEVOP_map_pirq, &map_irq); |
1623 | if (rc) { | 1624 | if (rc) { |
1624 | printk(KERN_WARNING "xen map irq failed gsi=%d irq=%d pirq=%d rc=%d\n", | 1625 | pr_warn("xen map irq failed gsi=%d irq=%d pirq=%d rc=%d\n", |
1625 | gsi, irq, pirq, rc); | 1626 | gsi, irq, pirq, rc); |
1626 | xen_free_irq(irq); | 1627 | xen_free_irq(irq); |
1627 | continue; | 1628 | continue; |
1628 | } | 1629 | } |
@@ -1844,13 +1845,11 @@ void xen_callback_vector(void) | |||
1844 | callback_via = HVM_CALLBACK_VECTOR(HYPERVISOR_CALLBACK_VECTOR); | 1845 | callback_via = HVM_CALLBACK_VECTOR(HYPERVISOR_CALLBACK_VECTOR); |
1845 | rc = xen_set_callback_via(callback_via); | 1846 | rc = xen_set_callback_via(callback_via); |
1846 | if (rc) { | 1847 | if (rc) { |
1847 | printk(KERN_ERR "Request for Xen HVM callback vector" | 1848 | pr_err("Request for Xen HVM callback vector failed\n"); |
1848 | " failed.\n"); | ||
1849 | xen_have_vector_callback = 0; | 1849 | xen_have_vector_callback = 0; |
1850 | return; | 1850 | return; |
1851 | } | 1851 | } |
1852 | printk(KERN_INFO "Xen HVM callback vector for event delivery is " | 1852 | pr_info("Xen HVM callback vector for event delivery is enabled\n"); |
1853 | "enabled\n"); | ||
1854 | /* in the restore case the vector has already been allocated */ | 1853 | /* in the restore case the vector has already been allocated */ |
1855 | if (!test_bit(HYPERVISOR_CALLBACK_VECTOR, used_vectors)) | 1854 | if (!test_bit(HYPERVISOR_CALLBACK_VECTOR, used_vectors)) |
1856 | alloc_intr_gate(HYPERVISOR_CALLBACK_VECTOR, | 1855 | alloc_intr_gate(HYPERVISOR_CALLBACK_VECTOR, |