linux-kernel - RE: [PATCH] ACPI / EC: Remove non-standard log emphasis

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1AE640813FDE7649BE1B193DEA596E88026D8036@SHSMSX101.ccr.corp.intel.com>
Date:	Wed, 25 Feb 2015 08:41:33 +0000
From:	"Zheng, Lv" <lv.zheng@...el.com>
To:	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Scot Doyle <lkml14@...tdoyle.com>
CC:	"linux-acpi@...r.kernel.org" <linux-acpi@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] ACPI / EC: Remove non-standard log emphasis

Hi,

> From: Rafael J. Wysocki [mailto:rjw@...ysocki.net]
> Sent: Wednesday, February 18, 2015 2:19 PM
> To: Scot Doyle
> Cc: Zheng, Lv; linux-acpi@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH] ACPI / EC: Remove non-standard log emphasis
> 
> On Sunday, February 15, 2015 07:43:08 PM Scot Doyle wrote:
> > Remove unusual pr_info() visual emphasis introduced in ad479e7f47ca
> > "ACPI / EC: Introduce STARTED/STOPPED flags to replace BLOCKED flag".
> >
> > Signed-off-by: Scot Doyle <lkml14@...tdoyle.com>
> 
> Applied, thanks!
> 
> > ---
> >  drivers/acpi/ec.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/acpi/ec.c b/drivers/acpi/ec.c
> > index 982b67f..857175e 100644
> > --- a/drivers/acpi/ec.c
> > +++ b/drivers/acpi/ec.c
> > @@ -680,7 +680,7 @@ static void acpi_ec_start(struct acpi_ec *ec, bool resuming)
> >  		/* Enable GPE for event processing (SCI_EVT=1) */
> >  		if (!resuming)
> >  			acpi_ec_submit_request(ec);
> > -		pr_info("+++++ EC started +++++\n");
> > +		pr_info("EC started\n");
> >  	}
> >  	spin_unlock_irqrestore(&ec->lock, flags);
> >  }
> > @@ -712,7 +712,7 @@ static void acpi_ec_stop(struct acpi_ec *ec, bool suspending)
> >  			acpi_ec_complete_request(ec);
> >  		clear_bit(EC_FLAGS_STARTED, &ec->flags);
> >  		clear_bit(EC_FLAGS_STOPPED, &ec->flags);
> > -		pr_info("+++++ EC stopped +++++\n");
> > +		pr_info("EC stopped\n");
> >  	}
> >  	spin_unlock_irqrestore(&ec->lock, flags);
> >  }
> >

I was using "+++++"/"#####"/"*****" to filter different EC log entries which makes debugging easier.
And, if we changed this from pr_info into pr_debug, then we will have nothing in the suspend/resume logs for the EC device.
While there are plenty of such log entries for each of other devices.
So I really doubt what's the meaning of such a change.
Could it improve anything or just make debugging more difficult for developers?

Thanks and best regards
-Lv


> 
> --
> I speak only for myself.
> Rafael J. Wysocki, Intel Open Source Technology Center.