Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1365616 > unrolled thread
| Started by | Daniel Bristot de Oliveira <bristot@redhat.com> |
|---|---|
| First post | 2016-03-28 20:10 +0200 |
| Last post | 2016-03-31 06:30 +0200 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message Daniel Bristot de Oliveira <bristot@redhat.com> - 2016-03-28 20:10 +0200
RE: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message "Moore, Robert" <robert.moore@intel.com> - 2016-03-29 21:20 +0200
Re: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message Daniel Bristot de Oliveira <bristot@redhat.com> - 2016-03-31 03:20 +0200
Re: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message Joe Perches <joe@perches.com> - 2016-03-31 06:30 +0200
| From | Daniel Bristot de Oliveira <bristot@redhat.com> |
|---|---|
| Date | 2016-03-28 20:10 +0200 |
| Subject | [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message |
| Message-ID | <rhJLY-2kc-9@gated-at.bofh.it> |
A "\n" at the end of bellow ACPI_INFO message is causing a blank line
in the kernel log:
ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n", tables_loaded));
This patch removes the "\n".
Kernel log before this patch:
ACPI: Core revision 20160108
ACPI: 2 ACPI AML tables successfully acquired and loaded
Security Framework initialized
Kernel log after this patch:
ACPI: Core revision 20160108
ACPI: 2 ACPI AML tables successfully acquired and loaded
Security Framework initialized
Signed-off-by: Daniel Bristot de Oliveira <bristot@redhat.com>
diff --git a/drivers/acpi/acpica/tbxfload.c b/drivers/acpi/acpica/tbxfload.c
index 3151968..e435b84 100644
--- a/drivers/acpi/acpica/tbxfload.c
+++ b/drivers/acpi/acpica/tbxfload.c
@@ -240,7 +240,8 @@ acpi_status acpi_tb_load_namespace(void)
}
if (!tables_failed) {
- ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n", tables_loaded));
+ ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded",
+ tables_loaded));
} else {
ACPI_ERROR((AE_INFO,
"%u table load failures, %u successful",
--
2.5.0
[toc] | [next] | [standalone]
| From | "Moore, Robert" <robert.moore@intel.com> |
|---|---|
| Date | 2016-03-29 21:20 +0200 |
| Subject | RE: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message |
| Message-ID | <ri7lg-2gu-7@gated-at.bofh.it> |
| In reply to | #1365616 |
Actually, I did in fact put that there to break up the output after the tables are loaded. Is this a problem?
> -----Original Message-----
> From: Daniel Bristot de Oliveira [mailto:bristot@redhat.com]
> Sent: Monday, March 28, 2016 10:59 AM
> To: Moore, Robert; Zheng, Lv; Wysocki, Rafael J; Len Brown
> Cc: linux-acpi@vger.kernel.org; devel@acpica.org; linux-
> kernel@vger.kernel.org
> Subject: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot
> message
>
> A "\n" at the end of bellow ACPI_INFO message is causing a blank line in
> the kernel log:
>
> ACPI_INFO(("%u ACPI AML tables successfully acquired and loaded\n",
> tables_loaded));
>
> This patch removes the "\n".
>
> Kernel log before this patch:
> ACPI: Core revision 20160108
> ACPI: 2 ACPI AML tables successfully acquired and loaded
>
> Security Framework initialized
>
> Kernel log after this patch:
> ACPI: Core revision 20160108
> ACPI: 2 ACPI AML tables successfully acquired and loaded
> Security Framework initialized
>
> Signed-off-by: Daniel Bristot de Oliveira <bristot@redhat.com>
>
> diff --git a/drivers/acpi/acpica/tbxfload.c
> b/drivers/acpi/acpica/tbxfload.c index 3151968..e435b84 100644
> --- a/drivers/acpi/acpica/tbxfload.c
> +++ b/drivers/acpi/acpica/tbxfload.c
> @@ -240,7 +240,8 @@ acpi_status acpi_tb_load_namespace(void)
> }
>
> if (!tables_failed) {
> - ACPI_INFO(("%u ACPI AML tables successfully acquired and
> loaded\n", tables_loaded));
> + ACPI_INFO(("%u ACPI AML tables successfully acquired and
> loaded",
> + tables_loaded));
> } else {
> ACPI_ERROR((AE_INFO,
> "%u table load failures, %u successful",
> --
> 2.5.0
[toc] | [prev] | [next] | [standalone]
| From | Daniel Bristot de Oliveira <bristot@redhat.com> |
|---|---|
| Date | 2016-03-31 03:20 +0200 |
| Subject | Re: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message |
| Message-ID | <rizrc-5wZ-3@gated-at.bofh.it> |
| In reply to | #1366559 |
On 03/29/2016 04:09 PM, Moore, Robert wrote: > Actually, I did in fact put that there to break up the output after the tables are loaded. Is this a problem? Well, I do not believe that there is a real problem on it. On the other hand, it does not seem to be common to have blank lines in the kernel log, and as there is no info about from where the black line comes from, it does not even seems to be connected to the previous message. So although my patch is about "cosmetics", IMHO it is worth as pattern or best practices. Feel free to reject if it is not the case, np :-). -- Daniel
[toc] | [prev] | [next] | [standalone]
| From | Joe Perches <joe@perches.com> |
|---|---|
| Date | 2016-03-31 06:30 +0200 |
| Subject | Re: [PATCH] ACPICA: Remove unnecessary "\n" from an ACPI_INFO boot message |
| Message-ID | <riCp5-7TE-3@gated-at.bofh.it> |
| In reply to | #1367732 |
On Wed, 2016-03-30 at 22:11 -0300, Daniel Bristot de Oliveira wrote: > On 03/29/2016 04:09 PM, Moore, Robert wrote: > > Actually, I did in fact put that there to break up the output after the tables are loaded. Is this a problem? > Well, I do not believe that there is a real problem on it. > > On the other hand, it does not seem to be common to have blank lines in > the kernel log, and as there is no info about from where the black line > comes from, it does not even seems to be connected to the previous > message. So although my patch is about "cosmetics", IMHO it is worth as > pattern or best practices. FWIW: I agree with Daniel.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web