All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 05/06][RESEND] Fix compilation warning for arch/x86/kernel/apm_32.c
@ 2009-07-21  2:32 Subrata Modak
  2009-08-04 15:32 ` [tip:x86/urgent] x86: Work around compilation warning in arch/x86/kernel/apm_32.c tip-bot for Subrata Modak
  0 siblings, 1 reply; 2+ messages in thread
From: Subrata Modak @ 2009-07-21  2:32 UTC (permalink / raw)
  To: Ingo Molnar, linux-laptop, Stephen Rothwell
  Cc: Sachin P Sant, Ingo Molnar, David Howells, Subrata Modak,
	Balbir Singh, LKML

The following fix was initially inspired by David Howells fix few days back:
http://lkml.org/lkml/2009/7/9/109,

However, Ingo disapproves such fixes as it's dangerous - in something as
performance-uncritical. So, initialize 'err' to '0' to work around a GCC false 
positive warning:
http://lkml.org/lkml/2009/7/18/89,

Signed-off-by: Subrata Modak<subrata@linux.vnet.ibm.com>
---

--- a/arch/x86/kernel/apm_32.c	2009-06-15 07:50:51.000000000 +0530
+++ b/arch/x86/kernel/apm_32.c	2009-07-15 06:43:59.000000000 +0530
@@ -811,7 +811,7 @@ static int apm_do_idle(void)
 	u8 ret = 0;
 	int idled = 0;
 	int polling;
-	int err;
+	int err = 0;
 
 	polling = !!(current_thread_info()->status & TS_POLLING);
 	if (polling) {

---
Regards--
Subrata


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [tip:x86/urgent] x86: Work around compilation warning in arch/x86/kernel/apm_32.c
  2009-07-21  2:32 [PATCH 05/06][RESEND] Fix compilation warning for arch/x86/kernel/apm_32.c Subrata Modak
@ 2009-08-04 15:32 ` tip-bot for Subrata Modak
  0 siblings, 0 replies; 2+ messages in thread
From: tip-bot for Subrata Modak @ 2009-08-04 15:32 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: linux-kernel, hpa, mingo, sachinp, subrata, balbir, dhowells,
	tglx, sfr, mingo

Commit-ID:  dc731fbbadf5d65c98fcd6c86472aa286c16458a
Gitweb:     http://git.kernel.org/tip/dc731fbbadf5d65c98fcd6c86472aa286c16458a
Author:     Subrata Modak <subrata@linux.vnet.ibm.com>
AuthorDate: Tue, 21 Jul 2009 08:02:27 +0530
Committer:  Ingo Molnar <mingo@elte.hu>
CommitDate: Tue, 4 Aug 2009 16:34:23 +0200

x86: Work around compilation warning in arch/x86/kernel/apm_32.c

The following fix was initially inspired by David Howells fix
few days back:

  http://lkml.org/lkml/2009/7/9/109

However, Ingo disapproves such fixes as it's dangerous (it can
hide future, relevant warnings) - in something as
performance-uncritical.

So, initialize 'err' to '0' to work around a GCC false positive
warning:

  http://lkml.org/lkml/2009/7/18/89

Signed-off-by: Subrata Modak<subrata@linux.vnet.ibm.com>
Cc: Sachin P Sant <sachinp@linux.vnet.ibm.com>
Cc: David Howells <dhowells@redhat.com>
Cc: Balbir Singh <balbir@linux.vnet.ibm.com>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>
LKML-Reference: <20090721023226.31855.67236.sendpatchset@subratamodak.linux.ibm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>


---
 arch/x86/kernel/apm_32.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/x86/kernel/apm_32.c b/arch/x86/kernel/apm_32.c
index 79302e9..442b550 100644
--- a/arch/x86/kernel/apm_32.c
+++ b/arch/x86/kernel/apm_32.c
@@ -811,7 +811,7 @@ static int apm_do_idle(void)
 	u8 ret = 0;
 	int idled = 0;
 	int polling;
-	int err;
+	int err = 0;
 
 	polling = !!(current_thread_info()->status & TS_POLLING);
 	if (polling) {

^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-08-04 15:33 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-07-21  2:32 [PATCH 05/06][RESEND] Fix compilation warning for arch/x86/kernel/apm_32.c Subrata Modak
2009-08-04 15:32 ` [tip:x86/urgent] x86: Work around compilation warning in arch/x86/kernel/apm_32.c tip-bot for Subrata Modak

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.