LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] x86: xen: platform-pci-unplug: use pr_err() and pr_warn() instead of raw printk()
@ 2021-09-06 7:55 zhaoxiao
2021-09-06 7:58 ` Juergen Gross
0 siblings, 1 reply; 5+ messages in thread
From: zhaoxiao @ 2021-09-06 7:55 UTC (permalink / raw)
To: tglx, mingo, bp, x86
Cc: hpa, xen-devel, linux-kernel, boris.ostrovsky, jgross,
sstabellini, zhaoxiao
Since we have the nice helpers pr_err() and pr_warn(), use them instead
of raw printk().
Signed-off-by: zhaoxiao <long870912@gmail.com>
---
arch/x86/xen/platform-pci-unplug.c | 16 +++++++++-------
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/arch/x86/xen/platform-pci-unplug.c b/arch/x86/xen/platform-pci-unplug.c
index 96d7f7d39cb9..6586eaf59c00 100644
--- a/arch/x86/xen/platform-pci-unplug.c
+++ b/arch/x86/xen/platform-pci-unplug.c
@@ -19,6 +19,8 @@
#define XEN_PLATFORM_ERR_PROTOCOL -2
#define XEN_PLATFORM_ERR_BLACKLIST -3
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
/* store the value of xen_emul_unplug after the unplug is done */
static int xen_platform_pci_unplug;
static int xen_emul_unplug;
@@ -30,13 +32,13 @@ static int check_platform_magic(void)
magic = inw(XEN_IOPORT_MAGIC);
if (magic != XEN_IOPORT_MAGIC_VAL) {
- printk(KERN_ERR "Xen Platform PCI: unrecognised magic value\n");
+ pr_err("Xen Platform PCI: unrecognised magic value\n");
return XEN_PLATFORM_ERR_MAGIC;
}
protocol = inb(XEN_IOPORT_PROTOVER);
- printk(KERN_DEBUG "Xen Platform PCI: I/O protocol version %d\n",
+ pr_debug("Xen Platform PCI: I/O protocol version %d\n",
protocol);
switch (protocol) {
@@ -44,12 +46,12 @@ static int check_platform_magic(void)
outw(XEN_IOPORT_LINUX_PRODNUM, XEN_IOPORT_PRODNUM);
outl(XEN_IOPORT_LINUX_DRVVER, XEN_IOPORT_DRVVER);
if (inw(XEN_IOPORT_MAGIC) != XEN_IOPORT_MAGIC_VAL) {
- printk(KERN_ERR "Xen Platform: blacklisted by host\n");
+ pr_err("Xen Platform: blacklisted by host\n");
return XEN_PLATFORM_ERR_BLACKLIST;
}
break;
default:
- printk(KERN_WARNING "Xen Platform PCI: unknown I/O protocol version\n");
+ pr_warn("Xen Platform PCI: unknown I/O protocol version\n");
return XEN_PLATFORM_ERR_PROTOCOL;
}
@@ -155,12 +157,12 @@ void xen_unplug_emulated_devices(void)
* been compiled for this kernel (modules or built-in are both OK). */
if (!xen_emul_unplug) {
if (xen_must_unplug_nics()) {
- printk(KERN_INFO "Netfront and the Xen platform PCI driver have "
+ pr_info("Netfront and the Xen platform PCI driver have "
"been compiled for this kernel: unplug emulated NICs.\n");
xen_emul_unplug |= XEN_UNPLUG_ALL_NICS;
}
if (xen_must_unplug_disks()) {
- printk(KERN_INFO "Blkfront and the Xen platform PCI driver have "
+ pr_info("Blkfront and the Xen platform PCI driver have "
"been compiled for this kernel: unplug emulated disks.\n"
"You might have to change the root device\n"
"from /dev/hd[a-d] to /dev/xvd[a-d]\n"
@@ -200,7 +202,7 @@ static int __init parse_xen_emul_unplug(char *arg)
else if (!strncmp(p, "never", l))
xen_emul_unplug |= XEN_UNPLUG_NEVER;
else
- printk(KERN_WARNING "unrecognised option '%s' "
+ pr_warn("unrecognised option '%s' "
"in parameter 'xen_emul_unplug'\n", p);
}
return 0;
--
2.20.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86: xen: platform-pci-unplug: use pr_err() and pr_warn() instead of raw printk()
2021-09-06 7:55 [PATCH] x86: xen: platform-pci-unplug: use pr_err() and pr_warn() instead of raw printk() zhaoxiao
@ 2021-09-06 7:58 ` Juergen Gross
0 siblings, 0 replies; 5+ messages in thread
From: Juergen Gross @ 2021-09-06 7:58 UTC (permalink / raw)
To: zhaoxiao, tglx, mingo, bp, x86
Cc: hpa, xen-devel, linux-kernel, boris.ostrovsky, sstabellini
[-- Attachment #1.1.1: Type: text/plain, Size: 297 bytes --]
On 06.09.21 09:55, zhaoxiao wrote:
> Since we have the nice helpers pr_err() and pr_warn(), use them instead
> of raw printk().
>
> Signed-off-by: zhaoxiao <long870912@gmail.com>
Any reason you are sending this patch again? It has already been applied
through the Xen tree.
Juergen
[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3135 bytes --]
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86: xen: platform-pci-unplug: use pr_err() and pr_warn() instead of raw printk()
2021-08-25 11:41 zhaoxiao
2021-08-25 13:12 ` Juergen Gross
@ 2021-08-30 10:15 ` Juergen Gross
1 sibling, 0 replies; 5+ messages in thread
From: Juergen Gross @ 2021-08-30 10:15 UTC (permalink / raw)
To: zhaoxiao, tglx, mingo, bp, x86
Cc: hpa, xen-devel, linux-kernel, boris.ostrovsky, sstabellini
[-- Attachment #1.1.1: Type: text/plain, Size: 368 bytes --]
On 25.08.21 13:41, zhaoxiao wrote:
> Since we have the nice helpers pr_err() and pr_warn(), use them instead
> of raw printk().
>
> Signed-off-by: zhaoxiao <zhaoxiao@uniontech.com>
Pushed to xen/tip.git for-linus-5.15 with a small fix (moved the
"#define pr_fmt" before the first #include in order to avoid
build warnings due to redefinition).
Juergen
[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3135 bytes --]
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] x86: xen: platform-pci-unplug: use pr_err() and pr_warn() instead of raw printk()
2021-08-25 11:41 zhaoxiao
@ 2021-08-25 13:12 ` Juergen Gross
2021-08-30 10:15 ` Juergen Gross
1 sibling, 0 replies; 5+ messages in thread
From: Juergen Gross @ 2021-08-25 13:12 UTC (permalink / raw)
To: zhaoxiao, tglx, mingo, bp, x86
Cc: hpa, xen-devel, linux-kernel, boris.ostrovsky, sstabellini
[-- Attachment #1.1.1: Type: text/plain, Size: 248 bytes --]
On 25.08.21 13:41, zhaoxiao wrote:
> Since we have the nice helpers pr_err() and pr_warn(), use them instead
> of raw printk().
>
> Signed-off-by: zhaoxiao <zhaoxiao@uniontech.com>
Reviewed-by: Juergen Gross <jgross@suse.com>
Juergen
[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3135 bytes --]
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] x86: xen: platform-pci-unplug: use pr_err() and pr_warn() instead of raw printk()
@ 2021-08-25 11:41 zhaoxiao
2021-08-25 13:12 ` Juergen Gross
2021-08-30 10:15 ` Juergen Gross
0 siblings, 2 replies; 5+ messages in thread
From: zhaoxiao @ 2021-08-25 11:41 UTC (permalink / raw)
To: tglx, mingo, bp, x86
Cc: hpa, xen-devel, linux-kernel, boris.ostrovsky, jgross,
sstabellini, zhaoxiao
Since we have the nice helpers pr_err() and pr_warn(), use them instead
of raw printk().
Signed-off-by: zhaoxiao <zhaoxiao@uniontech.com>
---
arch/x86/xen/platform-pci-unplug.c | 17 ++++++++++-------
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/arch/x86/xen/platform-pci-unplug.c b/arch/x86/xen/platform-pci-unplug.c
index 96d7f7d39cb9..e84bff542539 100644
--- a/arch/x86/xen/platform-pci-unplug.c
+++ b/arch/x86/xen/platform-pci-unplug.c
@@ -19,6 +19,9 @@
#define XEN_PLATFORM_ERR_PROTOCOL -2
#define XEN_PLATFORM_ERR_BLACKLIST -3
+#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
+
+
/* store the value of xen_emul_unplug after the unplug is done */
static int xen_platform_pci_unplug;
static int xen_emul_unplug;
@@ -30,13 +33,13 @@ static int check_platform_magic(void)
magic = inw(XEN_IOPORT_MAGIC);
if (magic != XEN_IOPORT_MAGIC_VAL) {
- printk(KERN_ERR "Xen Platform PCI: unrecognised magic value\n");
+ pr_err("Xen Platform PCI: unrecognised magic value\n");
return XEN_PLATFORM_ERR_MAGIC;
}
protocol = inb(XEN_IOPORT_PROTOVER);
- printk(KERN_DEBUG "Xen Platform PCI: I/O protocol version %d\n",
+ pr_debug("Xen Platform PCI: I/O protocol version %d\n",
protocol);
switch (protocol) {
@@ -44,12 +47,12 @@ static int check_platform_magic(void)
outw(XEN_IOPORT_LINUX_PRODNUM, XEN_IOPORT_PRODNUM);
outl(XEN_IOPORT_LINUX_DRVVER, XEN_IOPORT_DRVVER);
if (inw(XEN_IOPORT_MAGIC) != XEN_IOPORT_MAGIC_VAL) {
- printk(KERN_ERR "Xen Platform: blacklisted by host\n");
+ pr_err("Xen Platform: blacklisted by host\n");
return XEN_PLATFORM_ERR_BLACKLIST;
}
break;
default:
- printk(KERN_WARNING "Xen Platform PCI: unknown I/O protocol version\n");
+ pr_warn("Xen Platform PCI: unknown I/O protocol version\n");
return XEN_PLATFORM_ERR_PROTOCOL;
}
@@ -155,12 +158,12 @@ void xen_unplug_emulated_devices(void)
* been compiled for this kernel (modules or built-in are both OK). */
if (!xen_emul_unplug) {
if (xen_must_unplug_nics()) {
- printk(KERN_INFO "Netfront and the Xen platform PCI driver have "
+ pr_info("Netfront and the Xen platform PCI driver have "
"been compiled for this kernel: unplug emulated NICs.\n");
xen_emul_unplug |= XEN_UNPLUG_ALL_NICS;
}
if (xen_must_unplug_disks()) {
- printk(KERN_INFO "Blkfront and the Xen platform PCI driver have "
+ pr_info("Blkfront and the Xen platform PCI driver have "
"been compiled for this kernel: unplug emulated disks.\n"
"You might have to change the root device\n"
"from /dev/hd[a-d] to /dev/xvd[a-d]\n"
@@ -200,7 +203,7 @@ static int __init parse_xen_emul_unplug(char *arg)
else if (!strncmp(p, "never", l))
xen_emul_unplug |= XEN_UNPLUG_NEVER;
else
- printk(KERN_WARNING "unrecognised option '%s' "
+ pr_warn("unrecognised option '%s' "
"in parameter 'xen_emul_unplug'\n", p);
}
return 0;
--
2.20.1
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-09-06 7:58 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-06 7:55 [PATCH] x86: xen: platform-pci-unplug: use pr_err() and pr_warn() instead of raw printk() zhaoxiao
2021-09-06 7:58 ` Juergen Gross
-- strict thread matches above, loose matches on Subject: below --
2021-08-25 11:41 zhaoxiao
2021-08-25 13:12 ` Juergen Gross
2021-08-30 10:15 ` Juergen Gross
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).